Open
Bug 301447
Opened 19 years ago
Updated 8 years ago
Refactor Bugzilla::Bugmail into real objects
Categories
(Bugzilla :: Email Notifications, enhancement, P4)
Tracking
()
NEW
People
(Reporter: mkanat, Unassigned)
References
(Depends on 2 open bugs, Blocks 2 open bugs)
Details
Right now Bugmail.pm is a series of subroutines that deliver mail in a very
complex fashion.
I envision instead three objects:
Bugzilla::ChangeSet -- An object that desctibes what has changed about a bug in a
given change.
Bugzilla::Mailer -- An object that takes a ChangeSet and sends a bunch of mail.
Bugzilla::BugMail -- An individual bug email, sent to a single user. Generated
by Bugzilla::Mailer.
Reporter | ||
Updated•19 years ago
|
Assignee: email-notifications → mkanat
Priority: -- → P3
Target Milestone: --- → Bugzilla 3.0
Reporter | ||
Updated•18 years ago
|
Target Milestone: Bugzilla 3.0 → Bugzilla 3.2
Reporter | ||
Updated•18 years ago
|
Priority: P3 → P1
Reporter | ||
Comment 1•17 years ago
|
||
Bugzilla 3.2 is now frozen. Only enhancements blocking 3.2 or specifically approved for 3.2 may be checked in to the 3.2 branch. If you would like to nominate your enhancement for Bugzilla 3.2, set the "blocking3.2" flag to "?", and either the target milestone will be changed back, or the blocking3.2 flag will be granted, if we will accept this enhancement for Bugzilla 3.2.
Target Milestone: Bugzilla 3.2 → Bugzilla 4.0
Comment 2•17 years ago
|
||
I'ld like to see this happen for 4.0. Taking.
Status: NEW → ASSIGNED
Whiteboard: [roadmap: 4.0]
Updated•17 years ago
|
Assignee: mkanat → eseyman
Status: ASSIGNED → NEW
Reporter | ||
Updated•16 years ago
|
Whiteboard: [roadmap: 4.0] → [roadmap: 4.0][3.6 Focus]
Comment 4•15 years ago
|
||
In Bug 544026, I ask for the ability to send an email for a non-specific change to a bug.
This would be similar to calling Mailer() with a pre-determined ChangeSet but wanting a custom message to be used.
The use case is writing an extension that will send emails when a bug refers to another bug. The referred-to-bug would have a comment-type email sent containing the contents of the referring bug's comment.
Right now, getting the list of recipients out is impossible, short of re-writing Send().
Comment 5•13 years ago
|
||
I have a problem that is similar in nature - please let me know if I should file a separate bug.
I too think that Bugmail needs a refactoring, especially the send() function. I would really like to see a separation of the recipients logic from the calling of sendMail(). This would be useful for developers who wish to write extensions concerning Bugmail. They could call Bugzilla::Bugmail::getRecipients() for example, and just get a hash of the supposed recipients - without actually sending any e-mails.
I would be interested in helping write a patch if Emmanuel Seyman is no longer working on it.
Comment 6•13 years ago
|
||
There has been no activity from manu for the last 3 years, so you can safely start working on this bug. :)
Reporter | ||
Comment 7•13 years ago
|
||
However, note that the refactoring that should be done is the refactoring in comment 0, which will help make any other refactorings simpler.
Reporter | ||
Updated•13 years ago
|
Assignee: emmanuel.seyman → email-notifications
Comment 8•12 years ago
|
||
We are going to branch for Bugzilla 4.4 next week and this bug is either too invasive to be accepted for 4.4 at this point or shows no recent activity. The target milestone is reset and will be set again *only* when a patch is attached and approved.
I ask the assignee to reassign the bug to the default assignee if you don't plan to work on this bug in the near future, to make it clearer which bugs should be fixed by someone else.
Target Milestone: Bugzilla 4.4 → ---
Updated•8 years ago
|
Priority: P1 → P4
Whiteboard: [roadmap: 4.0][3.6 Focus]
You need to log in
before you can comment on or make changes to this bug.
Description
•