diff options
author | Dave Lawrence <dlawrence@mozilla.com> | 2013-07-10 19:37:55 +0200 |
---|---|---|
committer | Dave Lawrence <dlawrence@mozilla.com> | 2013-07-10 19:37:55 +0200 |
commit | 41cbe22c70d6e0299eacc0a09b68b8d1a52c17a4 (patch) | |
tree | c8bb5a5c0b12293eaa00fcdf60515fe33ccc39cd /Bugzilla/BugMail.pm | |
parent | b1a8bc55f2b5822405866bf4419f6b9cb49dd503 (diff) | |
download | bugzilla-41cbe22c70d6e0299eacc0a09b68b8d1a52c17a4.tar.gz bugzilla-41cbe22c70d6e0299eacc0a09b68b8d1a52c17a4.tar.xz |
Bug 822980 - backport bug 815532 to bmo (Bugzilla::User objects should be cached where appropriate)
r=glob
Diffstat (limited to 'Bugzilla/BugMail.pm')
-rw-r--r-- | Bugzilla/BugMail.pm | 6 |
1 files changed, 3 insertions, 3 deletions
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( |