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

Change subject: Check supposed revision being reverted is on the same page
......................................................................


Check supposed revision being reverted is on the same page

Bug: 57474
Change-Id: Idec054930dd20a01ec9d5132cce25378914629fc
---
M Hooks.php
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Bartosz Dziewoński: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/Hooks.php b/Hooks.php
index 758a1c5..69faf27 100644
--- a/Hooks.php
+++ b/Hooks.php
@@ -462,7 +462,7 @@
                                $undidRevId = $wgRequest->getVal( 
'wpUndidRevision' );
                                if ( $undidRevId ) {
                                        $undidRevision = Revision::newFromId( 
$undidRevId );
-                                       if ( $undidRevision ) {
+                                       if ( $undidRevision && 
$undidRevision->getTitle()->equals( $title ) ) {
                                                $victimId = 
$undidRevision->getUser();
                                                if ( $victimId ) { // No 
notifications for anonymous users
                                                        EchoEvent::create( 
array(

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Idec054930dd20a01ec9d5132cce25378914629fc
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Echo
Gerrit-Branch: master
Gerrit-Owner: Legoktm <[email protected]>
Gerrit-Reviewer: Bartosz Dziewoński <[email protected]>
Gerrit-Reviewer: jenkins-bot

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

Reply via email to