Closed Bug 544026 Opened 14 years ago Closed 14 years ago

Separate out BugMail::Send()'s recipient logic.

Categories

(Bugzilla :: Creating/Changing Bugs, enhancement)

Product:
Bugzilla
Component:
Creating/Changing Bugs
Version:
3.4.3
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED DUPLICATE of bug 301447

People

(Reporter: docwhat, Unassigned)

Details

Currently, the recipient logic is huge, complicated and only inside BugMail::Send().

If this could be refactored out, then we could:
 * Test it separately, which would make debugging and troubleshooting easier.
 * Allow it to be used in extensions that want to send emails about bugs without sending it to people who shouldn't get emails.
 * Make BugMail::Send() smaller and easier to read.

In my case, I'm trying to write an extension that finds references to other bugs and sends an email out about it to the users on the referenced bugs.

This is really complicated to do because I would have to reproduce all the recipient logic from Send(), and I'm not entirely sure what it all does (i.e., what parts are unrelated to checking recipients).

Ciao!
This will be done when refactoring BugMail.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → DUPLICATE
Severity: normal → enhancement
By the way, I suspect that there is (or we could add) a hook inside of BugMail.pm right now that would help you with your extension, also.
You need to log in before you can comment on or make changes to this bug.