Gerrit Patch Uploader has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/93508


Change subject: Remove namespace restriction for mention notifications
......................................................................

Remove namespace restriction for mention notifications

This is unlikely to cause any issues since a signature is
still required.

Bug: 55491
Change-Id: Ice51ae05b581b98f1d8a90104e1707c3a590b39d
---
M Hooks.php
1 file changed, 2 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Echo 
refs/changes/08/93508/1

diff --git a/Hooks.php b/Hooks.php
index 758a1c5..3d35630 100644
--- a/Hooks.php
+++ b/Hooks.php
@@ -450,11 +450,8 @@
                global $wgEchoNotifications, $wgRequest;
                if ( $revision ) {
                        $title = $article->getTitle();
-                       // If the edit is to a talk page or a project page, 
send it to the
-                       // discussion parser.
-                       if ( $title->isTalkPage() || $title->inNamespace( 
NS_PROJECT ) ) {
-                               
EchoDiscussionParser::generateEventsForRevision( $revision );
-                       }
+
+                       EchoDiscussionParser::generateEventsForRevision( 
$revision );
 
                        // Handle the case of someone undoing an edit, either 
through the
                        // 'undo' link in the article history or via the API.

-- 
To view, visit https://gerrit.wikimedia.org/r/93508
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ice51ae05b581b98f1d8a90104e1707c3a590b39d
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Echo
Gerrit-Branch: master
Gerrit-Owner: Gerrit Patch Uploader <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to