Denny Vrandecic has submitted this change and it was merged. Change subject: Log a warning if ItemChange::getSiteLinkDiff returns a non-array ......................................................................
Log a warning if ItemChange::getSiteLinkDiff returns a non-array Untested quick and dirty fix Change-Id: Idfab159d2077bd3c85dd4eee99e1b7bb11e1a4e2 --- M lib/maintenance/dispatchChanges.php 1 file changed, 7 insertions(+), 1 deletion(-) Approvals: Denny Vrandecic: Looks good to me, approved jenkins-bot: Verified diff --git a/lib/maintenance/dispatchChanges.php b/lib/maintenance/dispatchChanges.php index 2573d63..5ce791a 100644 --- a/lib/maintenance/dispatchChanges.php +++ b/lib/maintenance/dispatchChanges.php @@ -855,7 +855,13 @@ if ( $change instanceof ItemChange && !$change->isEmpty() ) { $siteLinkDiff = $change->getSiteLinkDiff(); - if ( is_array( $siteLinkDiff ) && isset( $siteLinkDiff[ $siteID ] ) ) { + if ( !is_array( $siteLinkDiff ) ) { + wfLogWarning( '$siteLinkDiff must be an array, ' . gettype( $siteLinkDiff ) . ' given' ); + wfProfileOut( __METHOD__ ); + return false; + } + + if ( isset( $siteLinkDiff[ $siteID ] ) ) { wfProfileOut( __METHOD__ ); return true; } -- To view, visit https://gerrit.wikimedia.org/r/73573 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idfab159d2077bd3c85dd4eee99e1b7bb11e1a4e2 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/Wikibase Gerrit-Branch: master Gerrit-Owner: Hoo man <h...@online.de> Gerrit-Reviewer: Denny Vrandecic <denny.vrande...@wikimedia.de> Gerrit-Reviewer: jenkins-bot _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits