From 130c0bf25f3ef8a9dda89c2aa3febb497f23a2c2 Mon Sep 17 00:00:00 2001
From: Matthew Somerville <matthew-github@dracos.co.uk>
Date: Mon, 19 Mar 2018 18:04:23 +0000
Subject: Pass columns CSV needs into Problem->as_hashref.

Then it can skip working out some things as they are not needed.
---
 perllib/FixMyStreet/DB/Result/Problem.pm | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

(limited to 'perllib/FixMyStreet/DB/Result/Problem.pm')

diff --git a/perllib/FixMyStreet/DB/Result/Problem.pm b/perllib/FixMyStreet/DB/Result/Problem.pm
index 2deeb3084..9feb98c76 100644
--- a/perllib/FixMyStreet/DB/Result/Problem.pm
+++ b/perllib/FixMyStreet/DB/Result/Problem.pm
@@ -918,12 +918,11 @@ sub add_send_method {
 }
 
 sub as_hashref {
-    my $self = shift;
-    my $c    = shift;
+    my ($self, $c, $cols) = @_;
 
     my $state_t = FixMyStreet::DB->resultset("State")->display($self->state);
 
-    return {
+    my $out = {
         id        => $self->id,
         title     => $self->title,
         category  => $self->category,
@@ -935,16 +934,17 @@ sub as_hashref {
         state     => $self->state,
         state_t   => $state_t,
         used_map  => $self->used_map,
-        is_fixed  => $self->fixed_states->{ $self->state } ? 1 : 0,
-        photos    => [ map { $_->{url} } @{$self->photos} ],
-        meta      => $self->confirmed ? $self->meta_line( $c ) : '',
-        ($self->confirmed ? (
-            confirmed => $self->confirmed,
-            confirmed_pp => $c->cobrand->prettify_dt( $self->confirmed ),
-        ) : ()),
-        created => $self->created,
-        created_pp => $c->cobrand->prettify_dt( $self->created ),
+        created   => $self->created,
     };
+    $out->{is_fixed} = $self->fixed_states->{ $self->state } ? 1 : 0 if !$cols || $cols->{is_fixed};
+    $out->{photos} = [ map { $_->{url} } @{$self->photos} ] if !$cols || $cols->{photos};
+    $out->{meta} = $self->confirmed ? $self->meta_line( $c ) : '' if !$cols || $cols->{meta};
+    $out->{created_pp} = $c->cobrand->prettify_dt( $self->created ) if !$cols || $cols->{created_pp};
+    if ($self->confirmed) {
+        $out->{confirmed} = $self->confirmed if !$cols || $cols->{confirmed};
+        $out->{confirmed_pp} = $c->cobrand->prettify_dt( $self->confirmed ) if !$cols || $cols->{confirmed_pp};
+    }
+    return $out;
 }
 
 =head2 latest_moderation_log_entry
-- 
cgit v1.2.3