This converts user options management to a separate service for use in DI context. User options are accessed quite early on in installation process and full-on options management depends on the database. Prior we have protected from accessing the DB by setting a hacky $wgUser with 0 id, and relying on the implementation that it doesn't go into the database to get the default user options. Now we can't really do that since DBLoadBalancer is required to instantiate the options manager. Instead, we redefine the options manager with a DefaultOptionsManager, that only provides access to default options and doesn't require DB access. UserOptionsManager uses PreferencesFactory, however injecting it will produce a cyclic dependency. The problem is that we separate options to different kinds, which are inferred from the PreferencesFactory declaration for those options (e.g. if it's a radio button in the UI declaration, the option is of multiselect kind). This is plain wrong, the dependency should be wise versa. This will be addressed separately, since it's requires larger refactoring. For now the PreferencesFactory is obtained on demand. This will be addressed in a followup. Bug: T248527 Change-Id: I74917c5eaec184d188911a319895b941ed55ee87
149 lines
5 KiB
PHP
149 lines
5 KiB
PHP
<?php
|
|
|
|
use MediaWiki\Config\ServiceOptions;
|
|
use MediaWiki\MediaWikiServices;
|
|
use MediaWiki\User\UserOptionsLookup;
|
|
use MediaWiki\User\UserOptionsManager;
|
|
use Psr\Log\NullLogger;
|
|
|
|
/**
|
|
* @group Database
|
|
* @covers MediaWiki\User\UserOptionsManager
|
|
*/
|
|
class UserOptionsManagerTest extends UserOptionsLookupTest {
|
|
|
|
private function getManager(
|
|
string $langCode = 'qqq',
|
|
array $defaultOptionsOverrides = []
|
|
) {
|
|
$services = MediaWikiServices::getInstance();
|
|
return new UserOptionsManager(
|
|
new ServiceOptions(
|
|
UserOptionsManager::CONSTRUCTOR_OPTIONS,
|
|
new HashConfig( [ 'HiddenPrefs' => [ 'hidden_user_option' ] ] )
|
|
),
|
|
$this->getDefaultManager( $langCode, $defaultOptionsOverrides ),
|
|
$services->getLanguageConverterFactory(),
|
|
$services->getDBLoadBalancer(),
|
|
new NullLogger()
|
|
);
|
|
}
|
|
|
|
protected function getLookup(
|
|
string $langCode = 'qqq',
|
|
array $defaultOptionsOverrides = []
|
|
) : UserOptionsLookup {
|
|
return $this->getManager( $langCode, $defaultOptionsOverrides );
|
|
}
|
|
|
|
/**
|
|
* @covers MediaWiki\User\UserOptionsManager::getOption
|
|
*/
|
|
public function testGetOptionsExcludeDefaults() {
|
|
$manager = $this->getManager();
|
|
$manager->setOption( $this->getAnon( __METHOD__ ), 'new_option', 'new_value' );
|
|
$this->assertSame( [
|
|
'language' => 'en',
|
|
'variant' => 'en',
|
|
'new_option' => 'new_value'
|
|
], $manager->getOptions( $this->getAnon( __METHOD__ ), UserOptionsManager::EXCLUDE_DEFAULTS ) );
|
|
}
|
|
|
|
/**
|
|
* @covers MediaWiki\User\UserOptionsManager::getOption
|
|
*/
|
|
public function testGetOptionHiddenPref() {
|
|
$user = $this->getAnon( __METHOD__ );
|
|
$manager = $this->getManager();
|
|
$manager->setOption( $user, 'hidden_user_option', 'hidden_value' );
|
|
$this->assertNull( $manager->getOption( $user, 'hidden_user_option' ) );
|
|
$this->assertSame( 'hidden_value',
|
|
$manager->getOption( $user, 'hidden_user_option', null, true ) );
|
|
}
|
|
|
|
/**
|
|
* @covers MediaWiki\User\UserOptionsManager::setOption
|
|
*/
|
|
public function testSetOptionNullIsDefault() {
|
|
$user = $this->getAnon( __METHOD__ );
|
|
$manager = $this->getManager();
|
|
$manager->setOption( $user, 'default_string_option', 'override_value' );
|
|
$this->assertSame( 'override_value', $manager->getOption( $user, 'default_string_option' ) );
|
|
$manager->setOption( $user, 'default_string_option', null );
|
|
$this->assertSame( 'string_value', $manager->getOption( $user, 'default_string_option' ) );
|
|
}
|
|
|
|
/**
|
|
* @covers MediaWiki\User\UserOptionsManager::getOption
|
|
* @covers MediaWiki\User\UserOptionsManager::setOption
|
|
* @covers MediaWiki\User\UserOptionsManager::saveOptions
|
|
*/
|
|
public function testGetSetSave() {
|
|
$user = $this->getTestUser()->getUser();
|
|
$manager = $this->getManager();
|
|
$this->assertSame( [], $manager->getOptions( $user, UserOptionsManager::EXCLUDE_DEFAULTS ) );
|
|
$manager->setOption( $user, 'string_option', 'user_value' );
|
|
$manager->setOption( $user, 'int_option', 42 );
|
|
$manager->setOption( $user, 'bool_option', true );
|
|
$this->assertSame( 'user_value', $manager->getOption( $user, 'string_option' ) );
|
|
$this->assertSame( 42, $manager->getIntOption( $user, 'int_option' ) );
|
|
$this->assertSame( true, $manager->getBoolOption( $user, 'bool_option' ) );
|
|
$manager->saveOptions( $user );
|
|
$manager = $this->getManager();
|
|
$this->assertSame( 'user_value', $manager->getOption( $user, 'string_option' ) );
|
|
$this->assertSame( 42, $manager->getIntOption( $user, 'int_option' ) );
|
|
$this->assertSame( true, $manager->getBoolOption( $user, 'bool_option' ) );
|
|
}
|
|
|
|
/**
|
|
* @covers MediaWiki\User\UserOptionsManager::loadUserOptions
|
|
*/
|
|
public function testLoadUserOptionsHook() {
|
|
$user = $this->getTestUser()->getUser();
|
|
$this->setTemporaryHook(
|
|
'UserLoadOptions',
|
|
function ( User $hookUser, &$options ) use ( $user ) {
|
|
if ( $hookUser->equals( $user ) ) {
|
|
$options['from_hook'] = 'value_from_hook';
|
|
}
|
|
}
|
|
);
|
|
$this->assertSame( 'value_from_hook', $this->getManager()->getOption( $user, 'from_hook' ) );
|
|
}
|
|
|
|
/**
|
|
* @covers MediaWiki\User\UserOptionsManager::saveOptions
|
|
*/
|
|
public function testSaveUserOptionsHookAbort() {
|
|
$user = $this->getTestUser()->getUser();
|
|
$this->setTemporaryHook(
|
|
'UserSaveOptions',
|
|
function () {
|
|
return false;
|
|
}
|
|
);
|
|
$manager = $this->getManager();
|
|
$manager->setOption( $user, 'will_be_aborted_by_hook', 'value' );
|
|
$manager->saveOptions( $user );
|
|
$this->assertNull( $this->getManager()->getOption( $user, 'will_be_aborted_by_hook' ) );
|
|
}
|
|
|
|
/**
|
|
* @covers MediaWiki\User\UserOptionsManager::saveOptions
|
|
*/
|
|
public function testSaveUserOptionsHookModify() {
|
|
$user = $this->getTestUser()->getUser();
|
|
$this->setTemporaryHook(
|
|
'UserLoadOptions',
|
|
function ( User $hookUser, &$options ) use ( $user ) {
|
|
if ( $hookUser->equals( $user ) ) {
|
|
$options['from_hook'] = 'value_from_hook';
|
|
}
|
|
}
|
|
);
|
|
$manager = $this->getManager();
|
|
$manager->saveOptions( $user );
|
|
$this->assertSame( 'value_from_hook', $manager->getOption( $user, 'from_hook' ) );
|
|
$this->assertSame( 'value_from_hook', $this->getManager()->getOption( $user, 'from_hook' ) );
|
|
}
|
|
}
|