From 41cbe22c70d6e0299eacc0a09b68b8d1a52c17a4 Mon Sep 17 00:00:00 2001 From: Dave Lawrence 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/BugMail.pm | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'Bugzilla/BugMail.pm') diff --git a/Bugzilla/BugMail.pm b/Bugzilla/BugMail.pm index a2501a99c..6e7fc7b80 100644 --- a/Bugzilla/BugMail.pm +++ b/Bugzilla/BugMail.pm @@ -210,7 +210,7 @@ sub Send { # Make sure %user_cache has every user in it so far referenced foreach my $user_id (keys %recipients) { - $user_cache{$user_id} ||= new Bugzilla::User($user_id); + $user_cache{$user_id} ||= new Bugzilla::User({ id => $user_id, cache => 1 }); } Bugzilla::Hook::process('bugmail_recipients', @@ -259,7 +259,7 @@ sub Send { foreach my $user_id (keys %recipients) { my %rels_which_want; - my $user = $user_cache{$user_id} ||= new Bugzilla::User($user_id); + my $user = $user_cache{$user_id} ||= new Bugzilla::User({ id => $user_id, cache => 1 }); # Deleted users must be excluded. next unless $user; @@ -505,7 +505,7 @@ sub _get_diffs { my $referenced_bugs = []; foreach my $diff (@$diffs) { - $user_cache->{$diff->{who}} ||= new Bugzilla::User($diff->{who}); + $user_cache->{$diff->{who}} ||= new Bugzilla::User({ id => $diff->{who}, cache => 1 }); $diff->{who} = $user_cache->{$diff->{who}}; if ($diff->{attach_id}) { $diff->{isprivate} = $dbh->selectrow_array( -- cgit v1.2.3-24-g4f1b