From: Gervase Markham <gerv@gerv.net>
Date: Thu, 27 Jan 2011 10:35:32 +0000 (+0000)
Subject: Add diffs parameter to bugmail_recipients hook. r=mkanat.
X-Git-Tag: bugzilla-4.1.1~75
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=edfb8c95f0f19e0482b9c8c4eda4832787f7fd18;p=thirdparty%2Fbugzilla.git

Add diffs parameter to bugmail_recipients hook. r=mkanat.

https://bugzilla.mozilla.org/show_bug.cgi?id=616422
---

diff --git a/Bugzilla/BugMail.pm b/Bugzilla/BugMail.pm
index 41b7a3cf1b..9630ce0ede 100644
--- a/Bugzilla/BugMail.pm
+++ b/Bugzilla/BugMail.pm
@@ -191,7 +191,7 @@ sub Send {
     
     Bugzilla::Hook::process('bugmail_recipients',
                             { bug => $bug, recipients => \%recipients,
-                              users => \%user_cache });
+                              users => \%user_cache, diffs => $diffs });
 
     # Find all those user-watching anyone on the current list, who is not
     # on it already themselves.
diff --git a/Bugzilla/Hook.pm b/Bugzilla/Hook.pm
index abedfde4a8..de46f9a25c 100644
--- a/Bugzilla/Hook.pm
+++ b/Bugzilla/Hook.pm
@@ -502,8 +502,17 @@ Every id in the incoming C<recipients> hash will have an object in here.
 (But if you add additional recipients to the C<recipients> hash, you are 
 B<not> required to add them to this hash.)
 
-=back
+=item C<diffs>
+
+This is a list of hashes, each hash representing a change to the bug. Each 
+hash has the following members: C<field_name>, C<bug_when>, C<old>, C<new> 
+and C<who> (a L<Bugzilla::User>). If appropriate, there will also be 
+C<attach_id> or C<comment_id>; if either is present, there will be 
+C<isprivate>. See C<_get_diffs> in F<Bugzilla/BugMail.pm> to see exactly how 
+it is populated. Warning: the format and existence of the "diffs" parameter 
+is subject to change in future releases of Bugzilla.
 
+=back
 
 =head2 bugmail_relationships