Matthias Mullie has uploaded a new change for review.
https://gerrit.wikimedia.org/r/269405
Change subject: Don't trigger new-topic notification when user is also
mentioned in first post
......................................................................
Don't trigger new-topic notification when user is also mentioned in first post
Bug: T124803
Change-Id: I621bff4701603240bc05891915d9a9740684be0f
---
M includes/Notifications/Controller.php
1 file changed, 8 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Flow
refs/changes/05/269405/1
diff --git a/includes/Notifications/Controller.php
b/includes/Notifications/Controller.php
index 50e3edf..fbb5979 100644
--- a/includes/Notifications/Controller.php
+++ b/includes/Notifications/Controller.php
@@ -186,9 +186,16 @@
}
$mentionEvent = $this->generateMentionEvent( $topicTitle,
$topicTitle, $topicWorkflow, $user );
+ $mentionedUsers = array();
if ( $mentionEvent ) {
$events[] = $mentionEvent;
+ $mentionedUsers = $mentionEvent->getExtraParam(
'mentioned-users' );
}
+
+ // also look at users mentioned in first post: if there are
any, this
+ // notification shouldn't go through (because they'll already
receive
+ // the mention notification)
+ $mentionedUsers += $this->getMentionedUsers( $firstPost,
$topicWorkflow->getArticleTitle() );
$events = array();
$events[] = EchoEvent::create( array(
@@ -209,7 +216,7 @@
$topicWorkflow->getArticleTitle()->getArticleID( Title::GAID_FOR_UPDATE ),
),
// pass along mentioned users to other
notification, so it knows who to ignore
- 'mentioned-users' => $mentionEvent ?
$mentionEvent->getExtraParam( 'mentioned-users' ) : array(),
+ 'mentioned-users' => $mentionedUsers,
)
) );
--
To view, visit https://gerrit.wikimedia.org/r/269405
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I621bff4701603240bc05891915d9a9740684be0f
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Matthias Mullie <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
