Public

Project privacy set to public. By default, its content is available to everyone (authenticated or not). Please note that more restrictive permissions might exist on some items.

stable

Clone or download

Read-only

chore: extract strategy of status change only

Download Browse

Note: there was no tests for this scenario, I decided to write them at the strategy level. For strategy that are already covered by RecipientManager tests, I just keep them here with a @covers annotation. Part of: story #10469 choose if I receive a notification when I create/update something Change-Id: I3f5390134bc079728d69e7e2ad3c9da4d171834d

+290 −35
Manuel Vacelet (vaceletm) 2022-09-20 18:12
Manuel Vacelet (vaceletm) 2022-09-21 16:28
a780106aa137cd8d9ccd80ba125892e92f10c313
9cfd4b6691cc23930679a36d3ddb83245e03914a
git #tuleap/stable/a780106aa137cd8d9ccd80ba125892e92f10c313

Modified Files

Side by side diff
Inline diff
Name
M plugins/tracker/include/Tracker/Notifications/RecipientsManager.php +9 −32 Go to diff View file
A plugins/tracker/include/Tracker/Notifications/RemoveRecipient/ArtifactStatusChangeDetector.php +31 −0 Go to diff View file
A plugins/tracker/include/Tracker/Notifications/RemoveRecipient/ArtifactStatusChangeDetectorImpl.php +39 −0 Go to diff View file
A plugins/tracker/include/Tracker/Notifications/RemoveRecipient/RemoveRecipientWhenTheyAreInStatusUpdateOnlyMode.php +65 −0 Go to diff View file
M plugins/tracker/include/Tracker/Notifications/Settings/UserNotificationSettingsRetriever.php +1 −1 Go to diff View file
A plugins/tracker/include/Tracker/Notifications/UserNotificationOnlyStatusChange.php +27 −0 Go to diff View file
M plugins/tracker/include/Tracker/Notifications/UserNotificationOnlyStatusChangeDAO.php +2 −2 Go to diff View file
M plugins/tracker/tests/unit/Tracker/Notifications/RecipientsManagerTest.php +3 −0 Go to diff View file
A plugins/tracker/tests/unit/Tracker/Notifications/RemoveRecipient/RemoveRecipientWhenTheyAreInStatusUpdateOnlyModeTest.php +113 −0 Go to diff View file