Pwirth has submitted this change and it was merged.
Change subject: TaskApi permissions: Fixed fatal
......................................................................
TaskApi permissions: Fixed fatal
A fatal error occured when a user has no 'read' permission.
Change-Id: Ifda3a970f1bd6efe33c8437800088ec2d84f30c3
---
M includes/CoreHooks.php
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Pwirth: Looks good to me, approved
jenkins-bot: Verified
diff --git a/includes/CoreHooks.php b/includes/CoreHooks.php
index 4ededc7..bf0fb30 100755
--- a/includes/CoreHooks.php
+++ b/includes/CoreHooks.php
@@ -147,7 +147,7 @@
$oRequest = $out->getRequest();
$oUser = $out->getUser();
- if( $oUser->getId() == 0) {
+ if( $oUser->getId() == 0 || !$oUser->isAllowed( 'read' ) ) {
return null; //do nothing for not logged in user,
prevent error with read permission for anon
}
--
To view, visit https://gerrit.wikimedia.org/r/310546
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ifda3a970f1bd6efe33c8437800088ec2d84f30c3
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/BlueSpiceFoundation
Gerrit-Branch: master
Gerrit-Owner: Robert Vogel <vo...@hallowelt.biz>
Gerrit-Reviewer: Dvogel hallowelt <daniel.vo...@hallowelt.com>
Gerrit-Reviewer: Ljonka <l.verhovs...@gmail.com>
Gerrit-Reviewer: Mglaser <gla...@hallowelt.biz>
Gerrit-Reviewer: Pwirth <wi...@hallowelt.biz>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
