Merge "ApiRollback: Properly deal with UserIdentity"
This commit is contained in:
commit
df59a848ca
1 changed files with 5 additions and 4 deletions
|
|
@ -21,6 +21,7 @@
|
||||||
*/
|
*/
|
||||||
|
|
||||||
use MediaWiki\ParamValidator\TypeDef\UserDef;
|
use MediaWiki\ParamValidator\TypeDef\UserDef;
|
||||||
|
use MediaWiki\User\UserIdentity;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @ingroup API
|
* @ingroup API
|
||||||
|
|
@ -33,7 +34,7 @@ class ApiRollback extends ApiBase {
|
||||||
private $mTitleObj = null;
|
private $mTitleObj = null;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @var User
|
* @var UserIdentity
|
||||||
*/
|
*/
|
||||||
private $mUser = null;
|
private $mUser = null;
|
||||||
|
|
||||||
|
|
@ -67,7 +68,7 @@ class ApiRollback extends ApiBase {
|
||||||
} );
|
} );
|
||||||
|
|
||||||
$retval = $pageObj->doRollback(
|
$retval = $pageObj->doRollback(
|
||||||
$this->getRbUser( $params ),
|
$this->getRbUser( $params )->getName(),
|
||||||
$summary,
|
$summary,
|
||||||
$params['token'],
|
$params['token'],
|
||||||
$params['markbot'],
|
$params['markbot'],
|
||||||
|
|
@ -148,9 +149,9 @@ class ApiRollback extends ApiBase {
|
||||||
/**
|
/**
|
||||||
* @param array $params
|
* @param array $params
|
||||||
*
|
*
|
||||||
* @return string
|
* @return UserIdentity
|
||||||
*/
|
*/
|
||||||
private function getRbUser( array $params ) {
|
private function getRbUser( array $params ) : UserIdentity {
|
||||||
if ( $this->mUser !== null ) {
|
if ( $this->mUser !== null ) {
|
||||||
return $this->mUser;
|
return $this->mUser;
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue