Matthias Mullie has uploaded a new change for review.
https://gerrit.wikimedia.org/r/94912
Change subject: Don't send additional notifications if user already received
mention notification
......................................................................
Don't send additional notifications if user already received mention
notification
Change-Id: Id7b9d6f18b163765f01b205ed21039d626757f8f
---
M includes/Notifications/Controller.php
1 file changed, 7 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Flow
refs/changes/12/94912/1
diff --git a/includes/Notifications/Controller.php
b/includes/Notifications/Controller.php
index 6891779..0be32fb 100644
--- a/includes/Notifications/Controller.php
+++ b/includes/Notifications/Controller.php
@@ -362,6 +362,13 @@
$ids[] = $topic;
}
$users += self::getCreatorsFromPostIDs( $ids );
+
+ // ignore mentioned users, they'll get another
notification already
+ if ( isset( $extra['mentioned-users'] ) ) {
+ foreach ( $extra['mentioned-users'] as $uid =>
$user ) {
+ unset( $users[$uid] );
+ }
+ }
break;
default:
// Do nothing
--
To view, visit https://gerrit.wikimedia.org/r/94912
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id7b9d6f18b163765f01b205ed21039d626757f8f
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Matthias Mullie <mmul...@wikimedia.org>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
