Feature #2659
closed
Refactor Mailer delivery methods out of controller and into observers
100%
Description
There are a few Mailer method calls buried inside of controllers that would be better off as Observers (e.g. IssuesController#add). I'll do the refactoring, just wanted to open an issue so it appears in the release notes.
Updated by Jean-Philippe Lang almost 16 years ago
- how to do a single notification when several objects are added/updated (just how it occurs today when adding several files at the same time)? A notification should be also added when bulk editing issues. Sending a notification for each modified issue would be insane and pretty slow.
- maybe one day, we'd like to display the name of the users that were notified (eg. in the flash message). How to retrieve it from the observer?
Updated by Eric Davis almost 16 years ago
Jean-Philippe Lang wrote:
Before doing the refactoring, here are a few thoughts:
- how to do a single notification when several objects are added/updated (just how it occurs today when adding several files at the same time)?
I'll watch out for those. My main concern is the issue notifications. The add_files action isn't very complex so it won't need refactoring right now.
A notification should be also added when bulk editing issues. Sending a notification for each modified issue would be insane and pretty slow.
Bulk edits already send one email per issue:
# app/controllers/issues_controller.rb L252
Mailer.deliver_issue_edit(journal) if journal.details.any? && Setting.notified_events.include?('issue_updated')
- maybe one day, we'd like to display the name of the users that were notified (eg. in the flash message). How to retrieve it from the observer?
Would that be the responsibility of the Observer, the Mailer, or the Controller though?
Updated by Eric Davis over 15 years ago
- Status changed from 7 to Resolved
- Target version set to 0.8.3
- % Done changed from 0 to 100
- Resolution set to Fixed
I've added some observers to watch model objects for mail delivery instead of calling Mailer in r2637.
Updated by Jean-Philippe Lang over 15 years ago
- Status changed from Resolved to Closed
- Target version changed from 0.8.3 to 0.9.0
Thanks, I prefer to keep this refactoring for 0.9.