Umherirrender has uploaded a new change for review.
https://gerrit.wikimedia.org/r/65275
Change subject: Set context when using UserrightsPage
......................................................................
Set context when using UserrightsPage
SpecialPage::getContext called and $mContext is null. Return
RequestContext::getMain(); for sanity
Change-Id: Ie0ed360835164e8a4eb1c675ec8edd944ab95898
---
M includes/Article.php
M includes/SkinTemplate.php
M includes/api/ApiQueryUsers.php
M includes/api/ApiUserrights.php
4 files changed, 6 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/75/65275/1
diff --git a/includes/Article.php b/includes/Article.php
index a0d4438..dd880ff 100644
--- a/includes/Article.php
+++ b/includes/Article.php
@@ -860,6 +860,7 @@
* TODO: actions other than 'view'
*/
public function getRobotPolicy( $action, $pOutput ) {
+ wfDebug( __METHOD__ );
global $wgArticleRobotPolicies, $wgNamespaceRobotPolicies,
$wgDefaultRobotPolicy;
$ns = $this->getTitle()->getNamespace();
diff --git a/includes/SkinTemplate.php b/includes/SkinTemplate.php
index e3a6ccf..fa90954 100644
--- a/includes/SkinTemplate.php
+++ b/includes/SkinTemplate.php
@@ -1253,6 +1253,7 @@
}
$sur = new UserrightsPage;
+ $sur->setContext( $this->getContext() );
if ( $sur->userCanExecute( $this->getUser() ) ) {
$nav_urls['userrights'] = array(
'href' => self::makeSpecialUrlSubpage(
'Userrights', $rootUser )
diff --git a/includes/api/ApiQueryUsers.php b/includes/api/ApiQueryUsers.php
index 0a0aaee..aec57a0 100644
--- a/includes/api/ApiQueryUsers.php
+++ b/includes/api/ApiQueryUsers.php
@@ -204,11 +204,13 @@
}
}
+ $context = $this->getContext();
// Second pass: add result data to $retval
foreach ( $goodNames as $u ) {
if ( !isset( $data[$u] ) ) {
$data[$u] = array( 'name' => $u );
$urPage = new UserrightsPage;
+ $urPage->setContext( $context );
$iwUser = $urPage->fetchUser( $u );
if ( $iwUser instanceof UserRightsProxy ) {
diff --git a/includes/api/ApiUserrights.php b/includes/api/ApiUserrights.php
index 80db29b..870201e 100644
--- a/includes/api/ApiUserrights.php
+++ b/includes/api/ApiUserrights.php
@@ -38,6 +38,7 @@
$user = $this->getUrUser();
$form = new UserrightsPage;
+ $form->setContext( $this->getContext() );
$r['user'] = $user->getName();
$r['userid'] = $user->getId();
list( $r['added'], $r['removed'] ) =
@@ -62,6 +63,7 @@
$params = $this->extractRequestParams();
$form = new UserrightsPage;
+ $form->setContext( $this->getContext() );
$status = $form->fetchUser( $params['user'] );
if ( !$status->isOK() ) {
$errors = $status->getErrorsArray();
--
To view, visit https://gerrit.wikimedia.org/r/65275
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie0ed360835164e8a4eb1c675ec8edd944ab95898
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Umherirrender <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
