Merge tag 'v3.1' into fiksgatami-devfiksgatami-dev

1 files changed, 4 insertions, 4 deletions

diff --git a/perllib/FixMyStreet/Cobrand/Westminster.pm b/perllib/FixMyStreet/Cobrand/Westminster.pm
index c9f31f7f9..e00a7c092 100644
--- a/perllib/FixMyStreet/Cobrand/Westminster.pm
+++ b/perllib/FixMyStreet/Cobrand/Westminster.pm
@@ -78,15 +78,15 @@ sub open311_config {
     $h->{account_id} = $id || '0';
 }
 
-sub open311_extra_data {
-    my ($self, $row, $h, $extra) = @_;
+sub open311_extra_data_include {
+    my ($self, $row, $h) = @_;
 
     # Reports made via the app probably won't have a USRN because we don't
     # display the road layer. Instead we'll look up the closest asset from the
     # asset service at the point we're sending the report over Open311.
     if (!$row->get_extra_field_value('USRN')) {
         if (my $ref = $self->lookup_site_code($row, 'USRN')) {
-            push @$extra, { name => 'USRN', value => $ref };
+            $row->update_extra_field({ name => 'USRN', value => $ref });
         }
     }
 
@@ -96,7 +96,7 @@ sub open311_extra_data {
     my ($uprn_field) = grep { $_->{name} eq 'UPRN' } @$fields;
     if ( $uprn_field && !$uprn_field->{value} ) {
         if (my $ref = $self->lookup_site_code($row, 'UPRN')) {
-            push @$extra, { name => 'UPRN', value => $ref };
+            $row->update_extra_field({ name => 'UPRN', value => $ref });
         }
     }