Actions
Defect #44650
closedNoMethodError in issue update notification when bulk deleting a parent issue and its subtask
Resolution:
Fixed
Affected version:
Description
An issue update notification can raise NoMethodError if the issue is deleted after its journal is loaded. This patch skips email generation in that case to prevent the error.
For example, this can happen when bulk deleting a parent issue and its child, with the child deleted first:
- Deleting the child creates a journal for the parent and queues an update notification.
- The notification job loads the journal. The bulk deletion then deletes the parent.
- Email generation cannot find the parent through the journal and raises
NoMethodError.
Details:
- Return from
Mailer#issue_editwhenjournal.journalizedisnil. - Reproduced this order of loading and deletion and confirmed that the patched code raises no error and sends no email.
- Confirmed that all tests pass.
Files
Updated by Go MAEDA 19 days ago
- Tracker changed from Patch to Defect
- Subject changed from Skip issue update notifications for deleted issues to NoMethodError in issue update notification when bulk deleting a parent issue and its subtask
- Category set to Email notifications
- Status changed from New to Resolved
- Assignee set to Go MAEDA
Committed the patch in r25300. Thank you for your contribution.
Actions