stable
Clone or download
Read-only
story #9736 block notifications to non project Files, PhpWiki, Documents, Trackers,
Git & SVN Plugin members in private projects for non project member Given a private project with: - a registered user member of project - a registered user non memeber of project - a suspended user - a deleted user - an anonymous user Mails should be only sent to registered user members of project. For now filter applies for: - Tracker - Git - PHPWiki - Files - Docman - SVN Plugin Sorry for the size of review, I can't split it by services because changes are directly related to MailBuilder object, and my updates impacts all theses services. I also add Logger, so reviewer won't have to tests all services twice. Change-Id: I6f81e61745f6d6f5372739d1a738f7c81bbf7c28
Modified Files
Name | ||||
---|---|---|---|---|
M | plugins/docman/include/ApprovalTable/NotificationCycle.class.php | +7 | −7 | Go to diff View file |
M | plugins/docman/include/ApprovalTable/Reminder.class.php | +11 | −1 | Go to diff View file |
M | plugins/docman/include/ApprovalTable/Reviewer/ReviewerFactory.class.php | +16 | −5 | Go to diff View file |
M | plugins/docman/include/Docman_Controller.class.php | +10 | −3 | Go to diff View file |
M | plugins/docman/include/Docman_HTTPController.class.php | +15 | −1 | Go to diff View file |
M | plugins/docman/include/docmanPlugin.class.php | +13 | −7 | Go to diff View file |
M | plugins/docman/include/view/Docman_View_ItemDetailsSectionApprovalCreate.class.php | +16 | −7 | Go to diff View file |
M | plugins/docman/tests/NotificationManagerTest.php | +25 | −14 | Go to diff View file |
M | plugins/docman/tests/NotificationsManager_MoveTest.php | +14 | −8 | Go to diff View file |
M | plugins/git/hooks/post-receive.php | +8 | −2 | Go to diff View file |
M | plugins/git/include/gitPlugin.class.php | +9 | −2 | Go to diff View file |
M | plugins/svn/bin/svn_post_commit.php | +7 | −2 | Go to diff View file |
M | plugins/svn/include/Svn/Hooks/PostCommit.class.php | +5 | −2 | Go to diff View file |
M | plugins/tracker/include/Tracker/Artifact/Tracker_Artifact_Changeset.class.php | +13 | −5 | Go to diff View file |
M | plugins/tracker/include/Tracker/DateReminder/Tracker_DateReminderManager.class.php | +10 | −1 | Go to diff View file |
M | src/common/autoload.php | +4 | −2 | Go to diff View file |
M | src/common/frs/FRSReleaseFactory.class.php | +14 | −6 | Go to diff View file |
M | src/common/frs/FileModuleMonitorFactory.class.php | +16 | −6 | Go to diff View file |
M | src/common/mail/MailBuilder.php | +17 | −4 | Go to diff View file |
A | src/common/mail/MailFilter.php | +100 | −0 | Go to diff View file |
A | src/common/mail/MailLogger.php | +36 | −0 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/WikiNotification.php | +15 | −3 | Go to diff View file |
M | src/etc/logrotate.syslog.dist | +8 | −1 | Go to diff View file |
M | tests/simpletest/common/mail/MailBuilderTest.php | +10 | −1 | Go to diff View file |
A | tests/simpletest/common/mail/MailFilterTest.php | +224 | −0 | Go to diff View file |