Skip to content
This repository was archived by the owner on Jun 30, 2024. It is now read-only.

Commit 1b527fd

Browse files
authored
User::getOption is deprecated (#346)
1 parent 7eaff36 commit 1b527fd

File tree

2 files changed

+17
-3
lines changed

2 files changed

+17
-3
lines changed

extension.json

+6-1
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,12 @@
4141
},
4242
"AchievementRegister": {
4343
"class": "MediaWiki\\Extension\\AchievementBadges\\HookHandler\\AchievementRegister",
44-
"services": ["MainConfig", "DBLoadBalancer", "RevisionStore"]
44+
"services": [
45+
"MainConfig",
46+
"DBLoadBalancer",
47+
"RevisionStore",
48+
"UserOptionsLookup"
49+
]
4550
}
4651
},
4752
"SpecialPages": {

includes/HookHandler/AchievementRegister.php

+11-2
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
use MediaWiki\MediaWikiServices;
1111
use MediaWiki\Revision\RevisionRecord;
1212
use MediaWiki\Revision\RevisionStore;
13+
use MediaWiki\User\UserOptionsLookup;
1314
use MWTimestamp;
1415
use User;
1516
use Wikimedia\Rdbms\ILoadBalancer;
@@ -40,19 +41,27 @@ class AchievementRegister implements
4041
*/
4142
private $revisionStore;
4243

44+
/**
45+
* @var UserOptionsLookup
46+
*/
47+
private $userOptionsLookup;
48+
4349
/**
4450
* @param Config $config
4551
* @param ILoadBalancer $DBLoadBalancer
4652
* @param RevisionStore $revisionStore
53+
* @param UserOptionsLookup $userOptionsLookup
4754
*/
4855
public function __construct(
4956
Config $config,
5057
ILoadBalancer $DBLoadBalancer,
51-
RevisionStore $revisionStore
58+
RevisionStore $revisionStore,
59+
UserOptionsLookup $userOptionsLookup
5260
) {
5361
$this->config = $config;
5462
$this->mDb = $DBLoadBalancer->getMaintenanceConnectionRef( DB_REPLICA );
5563
$this->revisionStore = $revisionStore;
64+
$this->userOptionsLookup = $userOptionsLookup;
5665
}
5766

5867
private const WEEKDAYS = [
@@ -177,7 +186,7 @@ public function onUserSaveSettings( $user ) {
177186
if ( !$this->config->get( Constants::CONFIG_KEY_ENABLE_BETA_FEATURE ) ) {
178187
return;
179188
}
180-
if ( $user->getOption( Constants::PREF_KEY_ACHIEVEMENT_ENABLE ) ) {
189+
if ( $this->userOptionsLookup->getOption( $user, Constants::PREF_KEY_ACHIEVEMENT_ENABLE ) ) {
181190
Achievement::achieve( [
182191
'key' => Constants::ACHV_KEY_ENABLE_ACHIEVEMENT_BADGES,
183192
'user' => $user,

0 commit comments

Comments
 (0)