jenkins-bot has submitted this change and it was merged.

Change subject: Fix new topic notifications
......................................................................


Fix new topic notifications

New topic notifications were not being received.

Fix regression from partial rename of the change type in
9192d2fdf401b448e4ae530b1cbd265b54e9ee16 .

Bug: T131879
Change-Id: I2671b149e5ff832445a28526cbb2b5a08a68e698
---
M includes/Data/Listener/NotificationListener.php
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Legoktm: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/includes/Data/Listener/NotificationListener.php 
b/includes/Data/Listener/NotificationListener.php
index f747223..769dd40 100644
--- a/includes/Data/Listener/NotificationListener.php
+++ b/includes/Data/Listener/NotificationListener.php
@@ -33,7 +33,8 @@
                }
 
                switch( $row['rev_change_type'] ) {
-               case 'new-topic':
+               // Actually new-topic @todo rename
+               case 'new-post':
                        if ( !isset(
                                $metadata['board-workflow'],
                                $metadata['workflow'],

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2671b149e5ff832445a28526cbb2b5a08a68e698
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Mattflaschen <[email protected]>
Gerrit-Reviewer: Catrope <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: Sbisson <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to