Comment 1 for bug 419057

Revision history for this message
Jonathan Lange (jml) wrote :

I was going to say that this fix is trivial, but it actually involves changing the parameters to a bunch of functions.

I don't have a clear picture of how the stack fits together, but here's what I've found:
  * IRevisionMailJobSource.create
  * RevisionMailJob.create
  * BranchMailer.forRevision
  * BranchMailer.__init__

and possibly others.

AIUI, these all would need to be changed to take the revision ID. Also, whatever is at the top of the stack (a scanner event?) would have to pass the revision ID in.

Not difficult per se, just quite a few places to change, and a bit of exploration to do.