From 41cbe22c70d6e0299eacc0a09b68b8d1a52c17a4 Mon Sep 17 00:00:00 2001
From: Dave Lawrence <dlawrence@mozilla.com>
Date: Wed, 10 Jul 2013 13:37:55 -0400
Subject: Bug 822980 - backport bug 815532 to bmo (Bugzilla::User objects
 should be cached where appropriate) r=glob

---
 Bugzilla/Bug.pm | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

(limited to 'Bugzilla/Bug.pm')

diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm
index 4dcb6fe6e..15272e542 100644
--- a/Bugzilla/Bug.pm
+++ b/Bugzilla/Bug.pm
@@ -3251,8 +3251,8 @@ sub assigned_to {
     my ($self) = @_;
     return $self->{'assigned_to_obj'} if exists $self->{'assigned_to_obj'};
     $self->{'assigned_to'} = 0 if $self->{'error'};
-    $self->{'assigned_to_obj'} ||= new Bugzilla::User($self->{'assigned_to'});
-    return $self->{'assigned_to_obj'};
+    return $self->{'assigned_to_obj'}
+        = new Bugzilla::User({ id => $self->{'assigned_to'}, cache => 1 });
 }
 
 sub blocked {
@@ -3512,7 +3512,8 @@ sub qa_contact {
     return undef if $self->{'error'};
 
     if (Bugzilla->params->{'useqacontact'} && $self->{'qa_contact'}) {
-        $self->{'qa_contact_obj'} = new Bugzilla::User($self->{'qa_contact'});
+        $self->{'qa_contact_obj'}
+            = new Bugzilla::User({ id => $self->{'qa_contact'}, cache => 1 });
     } else {
         # XXX - This is somewhat inconsistent with the assignee/reporter 
         # methods, which will return an empty User if they get a 0. 
@@ -3526,8 +3527,8 @@ sub reporter {
     my ($self) = @_;
     return $self->{'reporter'} if exists $self->{'reporter'};
     $self->{'reporter_id'} = 0 if $self->{'error'};
-    $self->{'reporter'} = new Bugzilla::User($self->{'reporter_id'});
-    return $self->{'reporter'};
+    return $self->{'reporter'}
+        = new Bugzilla::User({ id => $self->{'reporter_id'}, cache => 1 });
 }
 
 sub see_also {
-- 
cgit v1.2.3-24-g4f1b