stable
Clone or download
Read-only
Part of story #38631 @ notifications in trackers The mass change case will be in a dedicted contribution How to test: - In the follow-up comment of your artifact, mention some users (e.g @username_1, @other_2) - Submit the artifact => A notification is sent to the mentioned users, the permission should be also taken in account - Add some comment in the PUT /artifacts/:id REST route: ``` { "values": [], "comment": { "body": "@user review asap plz", "format": "commonmark" } } ``` => A notification is sent to the mentioned users, the permission should be also taken in account Change-Id: Ic7dec45367bea59337da86dd56f730670d643188
Modified Files
| Name | ||||
|---|---|---|---|---|
| A | plugins/tracker/include/Notifications/Recipient/MentionedUserInCommentRetriever.php | +46 | −0 | Go to diff View file |
| A | plugins/tracker/include/Notifications/Recipient/MentionedUsernameCollection.php | +35 | −0 | Go to diff View file |
| M | plugins/tracker/include/Tracker/Artifact/Changeset/PostCreation/ActionsRunner.php | +3 | −1 | Go to diff View file |
| M | plugins/tracker/include/Tracker/Notifications/RecipientsManager.php | +11 | −3 | Go to diff View file |
| M | plugins/tracker/include/Tracker/Notifications/Settings/Administration/NotificationsAdminSettingsControllerCommon.php | +3 | −1 | Go to diff View file |
| M | plugins/tracker/include/Tracker/Tracker.class.php | +4 | −2 | Go to diff View file |
| M | plugins/tracker/include/trackerPlugin.php | +3 | −1 | Go to diff View file |
| A | plugins/tracker/tests/unit/Notifications/Recipient/MentionedUserInCommentRetrieverTest.php | +107 | −0 | Go to diff View file |
| M | plugins/tracker/tests/unit/Notifications/RecipientsManagerTest.php | +45 | −29 | Go to diff View file |
| A | plugins/tracker/tests/unit/Test/Builders/ChangesetCommentTestBuilder.php | +59 | −0 | Go to diff View file |