UserSaveOptions::execute() - Code Metrics - SemanticMediaWiki/SemanticWatchlist - Measure and Improve Code Quality continuously with Scrutinizer

UserSaveOptions::execute() A
last analyzed 2021-05-31 10:35 UTC

↳ Parent: UserSaveOptions

Complexity

Conditions	4
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	4

Importance

Changes

Metric	Value
dl	0
loc	12
ccs	6
cts	6
cp	1
rs	9.8666
c	0
b	0
f	0
cc	4
nc	3
nop	0
crap	4

<?php

namespace SWL\MediaWiki\Hooks;

use SWL\tableUpdater;

use User;

/**
 * Called just before saving user preferences/options in order to find the
 * watchlist groups the user watches, and update the swl_users_per_group table.
 *
 * https://www.mediawiki.org/wiki/Manual:Hooks/UserSaveOptions
 *
 * @ingroup SWL
 *
 * @license GNU GPL v2+
 * @since 1.0
 *
 * @author Jeroen De Dauw < [email protected] >
 * @author mwjames
 */
class UserSaveOptions {

	/**
	 * @var TableUpdater
	 */
	private $tableUpdater;

	/**
	 * @var User
	 */
	private $user;

	/**
	 * @var array
	 */
	private $options;

	/**
	 * @since 1.0
	 *
	 * @param TableUpdater $tableUpdater
	 * @param User $user
	 * @param array &$options
	 */
	public function __construct( TableUpdater $tableUpdater, User $user, array &$options ) {
		$this->tableUpdater = $tableUpdater;

		$this->user = $user;
		$this->options =& $options;
	}

	/**
	 * @since 1.0
	 *
	 * @return boolean
	 */
	public function execute() {

		$groupIds = array();

		foreach ( $this->options as $name => $value ) {
			if ( strpos( $name, 'swl_watchgroup_' ) === 0 && $value ) {
				$groupIds[] = (int)substr( $name, strrpos( $name, '_' ) + 1 );
			}
		}

		return $this->performUpdate( $groupIds );
	}

	protected function performUpdate( array $groupIds ) {
		return $this->tableUpdater->updateGroupIdsForUser(
			$this->user->getId(),
			$groupIds
		);
	}

}


1		<?php
2
3		namespace SWL\MediaWiki\Hooks;
4
5		use SWL\tableUpdater;
6
7		use User;
8
9		/**
10		* Called just before saving user preferences/options in order to find the
11		* watchlist groups the user watches, and update the swl_users_per_group table.
12		*
13		* https://www.mediawiki.org/wiki/Manual:Hooks/UserSaveOptions
14		*
15		* @ingroup SWL
16		*
17		* @license GNU GPL v2+
18		* @since 1.0
19		*
20		* @author Jeroen De Dauw < [email protected] >
21		* @author mwjames
22		*/
23		class UserSaveOptions {
24
25		/**
26		* @var TableUpdater
27		*/
28		private $tableUpdater;
29
30		/**
31		* @var User
32		*/
33		private $user;
34
35		/**
36		* @var array
37		*/
38		private $options;
39
40		/**
41		* @since 1.0
42		*
43		* @param TableUpdater $tableUpdater
44		* @param User $user
45		* @param array &$options
46		*/
47	4	public function __construct( TableUpdater $tableUpdater, User $user, array &$options ) {
48	4	$this->tableUpdater = $tableUpdater;
		0 ignored issues – show Documentation Bug introduced 2015-12-04 20:47 UTC by Report Bug Copy Issue Report It seems like `$tableUpdater` of type `object<SWL\TableUpdater>` is incompatible with the declared type `object<SWL\MediaWiki\Hooks\TableUpdater>` of property `$tableUpdater`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
49	4	$this->user = $user;
50	4	$this->options =& $options;
51	4	}
52
53		/**
54		* @since 1.0
55		*
56		* @return boolean
57		*/
58	3	public function execute() {
59
60	3	$groupIds = array();
61
62	3	foreach ( $this->options as $name => $value ) {
63	2	if ( strpos( $name, 'swl_watchgroup_' ) === 0 && $value ) {
64	2	$groupIds[] = (int)substr( $name, strrpos( $name, '_' ) + 1 );
65		}
66		}
67
68	3	return $this->performUpdate( $groupIds );
69		}
70
71	3	protected function performUpdate( array $groupIds ) {
72	3	return $this->tableUpdater->updateGroupIdsForUser(
73	3	$this->user->getId(),
74	3	$groupIds
75		);
76		}
77
78		}
79

SemanticMediaWiki / SemanticWatchlist

UserSaveOptions::execute() A last analyzed 2021-05-31 10:35 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

UserSaveOptions::execute() A
last analyzed 2021-05-31 10:35 UTC