Thijs/wiki.techinc.nl
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

* part of bug 23119: removed WikiError stuff for Special:Userrights, also updated CentralAuth extension

Browse source 
* fixed E_NOTICE in Special:GlobalGroupMembership when the user cannot change global permissions
This commit is contained in:
Alexandre Emsenhuber 2010-04-16 15:56:48 +00:00
parent 1b6fde57ed
commit ce2e172307
2 changed files with 24 additions and 18 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
includes/api/ApiUserrights.php
View file

@ -117,10 +117,12 @@ class ApiUserrights extends ApiBase {
}
$form = new UserrightsPage;
$user = $form->fetchUser( $params['user'] );
if ( $user instanceof WikiErrorMsg ) {
$this->dieUsageMsg( array_merge(
(array)$user->getMessageKey(), $user->getMessageArgs() ) );
$status = $form->fetchUser( $params['user'] );
if ( !$status->isOK() ) {
$errors = $status->getErrorsArray();
$this->dieUsageMsg( $errors[0] );
} else {
$user = $status->value;
}
$this->mUser = $user;

32
includes/specials/SpecialUserrights.php
View file

@ -141,10 +141,12 @@ class UserrightsPage extends SpecialPage {
function saveUserGroups( $username, $reason = '' ) {
global $wgRequest, $wgUser, $wgGroupsAddToSelf, $wgGroupsRemoveFromSelf;
$user = $this->fetchUser( $username );
if( $user instanceof WikiErrorMsg ) {
$wgOut->addWikiMsgArray( $user->getMessageKey(), $user->getMessageArgs() );
$status = $this->fetchUser( $username );
if( !$status->isOK() ) {
$wgOut->addWikiText( $status->getWikiText() );
return;
} else {
$user = $status->value;
}
$allgroups = $this->getAllGroups();
@ -247,10 +249,12 @@ class UserrightsPage extends SpecialPage {
function editUserGroupsForm( $username ) {
global $wgOut;
$user = $this->fetchUser( $username );
if( $user instanceof WikiErrorMsg ) {
$wgOut->addWikiMsgArray( $user->getMessageKey(), $user->getMessageArgs() );
$status = $this->fetchUser( $username );
if( !$status->isOK() ) {
$wgOut->addWikiText( $status->getWikiText() );
return;
} else {
$user = $status->value;
}
$groups = $user->getGroups();
@ -267,7 +271,7 @@ class UserrightsPage extends SpecialPage {
* return a user (or proxy) object for manipulating it.
*
* Side effects: error output for invalid access
* @return mixed User, UserRightsProxy, or WikiErrorMsg
* @return Status object
*/
public function fetchUser( $username ) {
global $wgUser, $wgUserrightsInterwikiDelimiter;
@ -283,16 +287,16 @@ class UserrightsPage extends SpecialPage {
$database = '';
} else {
if( !$wgUser->isAllowed( 'userrights-interwiki' ) ) {
return new WikiErrorMsg( 'userrights-no-interwiki' );
return Status::newFatal( 'userrights-no-interwiki' );
}
if( !UserRightsProxy::validDatabase( $database ) ) {
return new WikiErrorMsg( 'userrights-nodatabase', $database );
return Status::newFatal( 'userrights-nodatabase', $database );
}
}
}
if( $name === '' ) {
return new WikiErrorMsg( 'nouserspecified' );
return Status::newFatal( 'nouserspecified' );
}
if( $name{0} == '#' ) {
@ -307,13 +311,13 @@ class UserrightsPage extends SpecialPage {
}
if( !$name ) {
return new WikiErrorMsg( 'noname' );
return Status::newFatal( 'noname' );
}
} else {
$name = User::getCanonicalName( $name );
if( $name === false ) {
// invalid name
return new WikiErrorMsg( 'nosuchusershort', $username );
return Status::newFatal( 'nosuchusershort', $username );
}
}
@ -324,10 +328,10 @@ class UserrightsPage extends SpecialPage {
}
if( !$user || $user->isAnon() ) {
return new WikiErrorMsg( 'nosuchusershort', $username );
return Status::newFatal( 'nosuchusershort', $username );
}
return $user;
return Status::newGood( $user );
}
function makeGroupNameList( $ids ) {