Bsitu has uploaded a new change for review.
https://gerrit.wikimedia.org/r/156466
Change subject: Fallback to master lookup for title when lookup in slave fails
......................................................................
Fallback to master lookup for title when lookup in slave fails
Change-Id: I5d6a4ec750dc8438c92c9e70c36554ff7dbc97f1
---
M model/Notification.php
1 file changed, 6 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Echo
refs/changes/66/156466/1
diff --git a/model/Notification.php b/model/Notification.php
index d502770..a0f808b 100644
--- a/model/Notification.php
+++ b/model/Notification.php
@@ -135,8 +135,13 @@
$user = $this->user;
if ( $event->getExtraParam( 'target-page' ) ) {
$notifMapper->attachListener( 'insert',
'add-target-page', function() use ( $event, $user, $eventIds ) {
+ $targetPageId = $event->getExtraParam(
'target-page' );
// Make sure the target-page id is a valid id
- $title = Title::newFromID(
$event->getExtraParam( 'target-page' ) );
+ $title = Title::newFromID( $targetPageId );
+ // Try master if there is no match
+ if ( !$title ) {
+ $title = Title::newFromID(
$targetPageId, Title::GAID_FOR_UPDATE );
+ }
if ( $title ) {
$targetMapper = new
EchoTargetPageMapper();
if ( $eventIds ) {
--
To view, visit https://gerrit.wikimedia.org/r/156466
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5d6a4ec750dc8438c92c9e70c36554ff7dbc97f1
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Echo
Gerrit-Branch: master
Gerrit-Owner: Bsitu <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
