New Hook: bugmail_recipients

RESOLVED FIXED in Bugzilla 3.6

Status

()

--
enhancement
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: mkanat, Assigned: mkanat)

Tracking

3.5.3
Bugzilla 3.6
Bug Flags:
approval +
approval3.6 +

Details

Attachments

(2 attachments)

(Assignee)

Description

9 years ago
Hooks need to be able to add new direct recipients to bugmails, and modify existing recipients.
(Assignee)

Comment 1

9 years ago
Created attachment 426534 [details] [diff] [review]
v1

Here we go! Right now it only passes "recipients", because that's all I need for the Voting extension, but if somebody needs $diffs in the future, we can pass that too. I'm just hesitant at the moment because $diffs is such a complex data structure that will probably eventually be replaced by a Bugzilla::ChangeSet.
Assignee: extensions → mkanat
Status: NEW → ASSIGNED
Attachment #426534 - Flags: review+
(Assignee)

Comment 2

9 years ago
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/                                                                                               
modified Bugzilla/BugMail.pm
modified Bugzilla/Hook.pm
modified extensions/Example/Extension.pm
Committed revision 6986.
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
(Assignee)

Comment 3

9 years ago
Oh, wait, I also need a $bug argument.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 4

9 years ago
Created attachment 426538 [details] [diff] [review]
Add the bug argument

Okay, this does it.
Attachment #426538 - Flags: review+
(Assignee)

Comment 5

9 years ago
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/                                                                                               
modified Bugzilla/BugMail.pm
modified Bugzilla/Hook.pm
modified extensions/Example/Extension.pm
Committed revision 6987.
Status: REOPENED → RESOLVED
Last Resolved: 9 years ago9 years ago
Resolution: --- → FIXED
(Assignee)

Comment 6

9 years ago
reed requested a backport to 3.6, and this hook is pretty non-invasive, so that seems reasonable.
Status: RESOLVED → REOPENED
Flags: approval3.6+
Flags: approval+
Resolution: FIXED → ---
Target Milestone: Bugzilla 3.8 → Bugzilla 3.6
(Assignee)

Comment 7

9 years ago
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/3.6/                                                                                                 
modified Bugzilla/BugMail.pm
modified Bugzilla/Hook.pm
modified extensions/Example/Extension.pm
Committed revision 6973.
Status: REOPENED → RESOLVED
Last Resolved: 9 years ago9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.