Since &process_bug returns the last comment object from old Bugzilla::Bug->check object, attachment will be considered to one for 'previous' comment than the actual new comment added via email_in script. process_bug should renew Bugzilla::Bug object before getting latest comment object from it.
Attachment #631280 - Flags: review?
Comment on attachment 631280 [details] [diff] [review] patch v.1 Looks good and works fine. A comment should be added explaining why we recreate the bug object instead of reusing the existing one. This can be done on checkin. r=LpSolit
Attachment #631280 - Flags: review?(LpSolit) → review+
Assignee: incoming.email → shimono
Severity: normal → minor
Status: NEW → ASSIGNED
Target Milestone: --- → Bugzilla 4.2
Committing to: bzr+ssh://firstname.lastname@example.org/bugzilla/trunk/ modified email_in.pl Committed revision 8267. Committing to: bzr+ssh://email@example.com/bugzilla/4.2/ modified email_in.pl Committed revision 8097.
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.