-
Notifications
You must be signed in to change notification settings - Fork 24.4k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[REF] mail: clean code related to notification email generation
This commit cleans code about sending email notification to recipients. Purpose of this code is to prepare notification emails, group recipients and finally send emails. In this commit we clean and optimize a bit by * propagating some additional parameter to simplify code, like record on which the notification process runs; * inlining some code currently located in sub-methods. It eases code understanding as there are less code jumps. As those methods are not inherited inlining them can be done; * simplifying a bit notification emails preparation and computation; It allows to save a few query by avoiding to fetch and browse again some data that were already computed. This commit is linked to task ID 47934 and PR #24033. No functional change should occur with this commit.
- Loading branch information
1 parent
df8130b
commit 8905105
Showing
5 changed files
with
81 additions
and
122 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters