Refactor Bugzilla::Bugmail into real objects

NEW
Unassigned

Status

()

enhancement
P4
normal
14 years ago
3 years ago

People

(Reporter: mkanat, Unassigned)

Tracking

(Depends on 2 bugs, Blocks 2 bugs)

Dependency tree / graph

Details

Reporter

Description

14 years ago
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

14 years ago
Depends on: 281354
Reporter

Updated

14 years ago
Assignee: email-notifications → mkanat
Priority: -- → P3
Target Milestone: --- → Bugzilla 3.0

Updated

13 years ago
Depends on: 339862
Reporter

Updated

13 years ago
Target Milestone: Bugzilla 3.0 → Bugzilla 3.2
Reporter

Updated

13 years ago
Priority: P3 → P1
Reporter

Comment 1

12 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

11 years ago
I'ld like to see this happen for 4.0. Taking.
Status: NEW → ASSIGNED
Whiteboard: [roadmap: 4.0]

Updated

11 years ago
Assignee: mkanat → eseyman
Status: ASSIGNED → NEW
Reporter

Updated

10 years ago
Whiteboard: [roadmap: 4.0] → [roadmap: 4.0][3.6 Focus]
Reporter

Updated

10 years ago
Depends on: 492674
Reporter

Updated

10 years ago
Blocks: 23924
Reporter

Updated

10 years ago
Blocks: 26943

Updated

10 years ago
Duplicate of this bug: 544026

Comment 4

10 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

8 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

8 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

8 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

8 years ago
Assignee: emmanuel.seyman → email-notifications

Comment 8

7 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

3 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.