diff options
| author | Hakim Cassimally <hakim@mysociety.org> | 2015-02-13 17:12:21 +0000 |
|---|---|---|
| committer | Matthew Somerville <matthew@mysociety.org> | 2015-03-20 20:30:40 +0000 |
| commit | 6d9c719d25e2450ee2f31a5f305b603c2f451594 (patch) | |
| tree | afc3b450191ce752cbe3ee67450875dba73e052e /perllib/Open311 | |
| parent | 10b3a899af00205e46e177509a0b4c13bf746299 (diff) | |
Add Extra role to ease use of {extra} field.
Historically, the extra field has been used in two different ways by
different cobrands, both as a list (e.g. Open311 category fields) and a
hash (e.g. the Zurich cobrand).
This commit consolidates usage, adding an API to make use of the field
easier and always returning a hash for the code to use. Fixes #1018.
Diffstat (limited to 'perllib/Open311')
| -rw-r--r-- | perllib/Open311/PopulateServiceList.pm | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/perllib/Open311/PopulateServiceList.pm b/perllib/Open311/PopulateServiceList.pm index 0f6e32893..e4f0b8357 100644 --- a/perllib/Open311/PopulateServiceList.pm +++ b/perllib/Open311/PopulateServiceList.pm @@ -265,11 +265,7 @@ sub _add_meta_to_contact { @meta = grep { ! $ignore{ $_->{ code } } } @meta; } - if ( @meta ) { - $contact->extra( \@meta ); - } else { - $contact->extra( undef ); - } + $contact->set_extra_fields(@meta); $contact->update; } |