Pwirth has submitted this change and it was merged. Change subject: fix for required options on api call ......................................................................
fix for required options on api call (need cherry pick to REL1_23) Change-Id: Iba469f364543c4f0fde8b786b5840474009d7ca9 --- M includes/CoreHooks.php 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Pwirth: Verified; Looks good to me, approved diff --git a/includes/CoreHooks.php b/includes/CoreHooks.php index 21b78cd..43b96ba 100755 --- a/includes/CoreHooks.php +++ b/includes/CoreHooks.php @@ -317,7 +317,8 @@ if (!$module instanceof ApiParse) { return true; } - if (Title::newFromText($module->getRequest()->getVal('page'))->userCan('read') == false){ + $oTitle = Title::newFromText( $module->getRequest()->getVal( 'page' ) ); + if ( !is_null( $oTitle ) && $oTitle->userCan( 'read' ) == false ) { $message = wfMessage('loginreqpagetext', wfMessage('loginreqlink')->plain())->plain(); return false; } -- To view, visit https://gerrit.wikimedia.org/r/206359 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iba469f364543c4f0fde8b786b5840474009d7ca9 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/BlueSpiceFoundation Gerrit-Branch: master Gerrit-Owner: Tweichart <weich...@hallowelt.biz> Gerrit-Reviewer: Mglaser <gla...@hallowelt.biz> Gerrit-Reviewer: Pwirth <wi...@hallowelt.biz> Gerrit-Reviewer: Robert Vogel <vo...@hallowelt.biz> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits