ComplexNamespaceInputWidget::__construct() - Code Metrics - Inspection of "Daily Inspection: Merge "Add GENDER support to pro..." - wikimedia/mediawiki - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — master (939199)

unknown

created 2016-11-04 17:19 UTC

ComplexNamespaceInputWidget::__construct() A

↳ Parent: ComplexNamespaceInputWidget

Complexity

Conditions	3
Paths	4

Size

Total Lines	55
Code Lines	32

Duplication

Lines	28
Ratio	50.91 %

Importance

Changes

Metric	Value
cc	3
eloc	32
nc	4
nop	1
dl	28
loc	55
rs	9.7692
c	0
b	0
f	0

How to fix Long Method

<?php
/**
 * MediaWiki Widgets – ComplexNamespaceInputWidget class.
 *
 * @copyright 2011-2015 MediaWiki Widgets Team and others; see AUTHORS.txt
 * @license The MIT License (MIT); see LICENSE.txt
 */
namespace MediaWiki\Widget;

/**
 * Namespace input widget. Displays a dropdown box with the choice of available namespaces, plus two
 * checkboxes to include associated namespace or to invert selection.
 */
class ComplexNamespaceInputWidget extends \OOUI\Widget {

	protected $config;
	protected $namespace;
	protected $associated = null;
	protected $associatedLabel = null;
	protected $invert = null;
	protected $invertLabel = null;

	/**
	 * @param array $config Configuration options
	 * @param array $config['namespace'] Configuration for the NamespaceInputWidget
/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}
	 *  dropdown with list of namespaces
	 * @param string $config['namespace']['includeAllValue'] If specified,
/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}
	 *  add an "all namespaces" option to the dropdown, and use this as the input value for it
	 * @param array|null $config['invert'] Configuration for the "invert selection"
/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}
	 *  CheckboxInputWidget. If null, the checkbox will not be generated.
	 * @param array|null $config['associated'] Configuration for the "include associated namespace"
/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}
	 *  CheckboxInputWidget. If null, the checkbox will not be generated.
	 * @param array $config['invertLabel'] Configuration for the FieldLayout with label
/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}
	 *  wrapping the "invert selection" checkbox
	 * @param string $config['invertLabel']['label'] Label text for the label
/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}
	 * @param array $config['associatedLabel'] Configuration for the FieldLayout with label
/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}
	 *  wrapping the "include associated namespace" checkbox
	 * @param string $config['associatedLabel']['label'] Label text for the label
/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}
	 */
	public function __construct( array $config = [] ) {
		// Configuration initialization
		$config = array_merge(
			[
				// Config options for nested widgets
				'namespace' => [],
				'invert' => [],
				'invertLabel' => [],
				'associated' => [],
				'associatedLabel' => [],
			],
			$config
		);

		// Parent constructor
		parent::__construct( $config );

		// Properties
		$this->config = $config;

		$this->namespace = new NamespaceInputWidget( $config['namespace'] );
		if ( $config['associated'] !== null ) {
			$this->associated = new \OOUI\CheckboxInputWidget( array_merge(
				[ 'value' => '1' ],
				$config['associated']
			) );
			// TODO Should use a LabelWidget? But they don't work like HTML <label>s yet
			$this->associatedLabel = new \OOUI\FieldLayout(
				$this->associated,
				array_merge(
					[ 'align' => 'inline' ],
					$config['associatedLabel']
				)
			);
		}
		if ( $config['invert'] !== null ) {
			$this->invert = new \OOUI\CheckboxInputWidget( array_merge(
				[ 'value' => '1' ],
				$config['invert']
			) );
			// TODO Should use a LabelWidget? But they don't work like HTML <label>s yet
			$this->invertLabel = new \OOUI\FieldLayout(
				$this->invert,
				array_merge(
					[ 'align' => 'inline' ],
					$config['invertLabel']
				)
			);
		}

		// Initialization
		$this
			->addClasses( [ 'mw-widget-complexNamespaceInputWidget' ] )
			->appendContent( $this->namespace, $this->associatedLabel, $this->invertLabel );
	}

	protected function getJavaScriptClassName() {
		return 'mw.widgets.ComplexNamespaceInputWidget';
	}

	public function getConfig( &$config ) {
		$config = array_merge(
			$config,
			array_intersect_key(
				$this->config,
				array_fill_keys(
					[
						'namespace',
						'invert',
						'invertLabel',
						'associated',
						'associatedLabel'
					],
					true
				)
			)
		);
		return parent::getConfig( $config );
	}
}


1		<?php
2		/**
3		* MediaWiki Widgets – ComplexNamespaceInputWidget class.
4		*
5		* @copyright 2011-2015 MediaWiki Widgets Team and others; see AUTHORS.txt
6		* @license The MIT License (MIT); see LICENSE.txt
7		*/
8		namespace MediaWiki\Widget;
9
10		/**
11		* Namespace input widget. Displays a dropdown box with the choice of available namespaces, plus two
12		* checkboxes to include associated namespace or to invert selection.
13		*/
14		class ComplexNamespaceInputWidget extends \OOUI\Widget {
15
16		protected $config;
17		protected $namespace;
18		protected $associated = null;
19		protected $associatedLabel = null;
20		protected $invert = null;
21		protected $invertLabel = null;
22
23		/**
24		* @param array $config Configuration options
25		* @param array $config['namespace'] Configuration for the NamespaceInputWidget
		0 ignored issues – show Documentation introduced 2016-01-16 18:00 UTC by Report Bug Copy Issue Report There is no parameter named `$config['namespace']`. Did you maybe mean `$config`? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. It has, however, found a similar but not annotated parameter which might be a good fit. Consider the following example. The parameter `$ireland` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $ireland */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was changed, but the annotation was not. Loading history...
26		* dropdown with list of namespaces
27		* @param string $config['namespace']['includeAllValue'] If specified,
		0 ignored issues – show Documentation introduced 2016-01-16 18:00 UTC by Report Bug Copy Issue Report There is no parameter named `$config['namespace']['includeAllValue']`. Did you maybe mean `$config`? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. It has, however, found a similar but not annotated parameter which might be a good fit. Consider the following example. The parameter `$ireland` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $ireland */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was changed, but the annotation was not. Loading history...
28		* add an "all namespaces" option to the dropdown, and use this as the input value for it
29		* @param array\|null $config['invert'] Configuration for the "invert selection"
		0 ignored issues – show Documentation introduced 2016-01-16 18:00 UTC by Report Bug Copy Issue Report There is no parameter named `$config['invert']`. Did you maybe mean `$config`? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. It has, however, found a similar but not annotated parameter which might be a good fit. Consider the following example. The parameter `$ireland` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $ireland */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was changed, but the annotation was not. Loading history...
30		* CheckboxInputWidget. If null, the checkbox will not be generated.
31		* @param array\|null $config['associated'] Configuration for the "include associated namespace"
		0 ignored issues – show Documentation introduced 2016-01-16 18:00 UTC by Report Bug Copy Issue Report There is no parameter named `$config['associated']`. Did you maybe mean `$config`? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. It has, however, found a similar but not annotated parameter which might be a good fit. Consider the following example. The parameter `$ireland` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $ireland */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was changed, but the annotation was not. Loading history...
32		* CheckboxInputWidget. If null, the checkbox will not be generated.
33		* @param array $config['invertLabel'] Configuration for the FieldLayout with label
		0 ignored issues – show Documentation introduced 2016-01-16 18:00 UTC by Report Bug Copy Issue Report There is no parameter named `$config['invertLabel']`. Did you maybe mean `$config`? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. It has, however, found a similar but not annotated parameter which might be a good fit. Consider the following example. The parameter `$ireland` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $ireland */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was changed, but the annotation was not. Loading history...
34		* wrapping the "invert selection" checkbox
35		* @param string $config['invertLabel']['label'] Label text for the label
		0 ignored issues – show Documentation introduced 2016-01-16 18:00 UTC by Report Bug Copy Issue Report There is no parameter named `$config['invertLabel']['label']`. Did you maybe mean `$config`? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. It has, however, found a similar but not annotated parameter which might be a good fit. Consider the following example. The parameter `$ireland` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $ireland */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was changed, but the annotation was not. Loading history...
36		* @param array $config['associatedLabel'] Configuration for the FieldLayout with label
		0 ignored issues – show Documentation introduced 2016-01-16 18:00 UTC by Report Bug Copy Issue Report There is no parameter named `$config['associatedLabel']`. Did you maybe mean `$config`? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. It has, however, found a similar but not annotated parameter which might be a good fit. Consider the following example. The parameter `$ireland` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $ireland */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was changed, but the annotation was not. Loading history...
37		* wrapping the "include associated namespace" checkbox
38		* @param string $config['associatedLabel']['label'] Label text for the label
		0 ignored issues – show Documentation introduced 2016-01-16 18:00 UTC by Report Bug Copy Issue Report There is no parameter named `$config['associatedLabel']['label']`. Did you maybe mean `$config`? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. It has, however, found a similar but not annotated parameter which might be a good fit. Consider the following example. The parameter `$ireland` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $ireland */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was changed, but the annotation was not. Loading history...
39		*/
40		public function __construct( array $config = [] ) {
41		// Configuration initialization
42		$config = array_merge(
43		[
44		// Config options for nested widgets
45		'namespace' => [],
46		'invert' => [],
47		'invertLabel' => [],
48		'associated' => [],
49		'associatedLabel' => [],
50		],
51		$config
52		);
53
54		// Parent constructor
55		parent::__construct( $config );
56
57		// Properties
58		$this->config = $config;
59
60		$this->namespace = new NamespaceInputWidget( $config['namespace'] );
61	View Code Duplication	if ( $config['associated'] !== null ) {
62		$this->associated = new \OOUI\CheckboxInputWidget( array_merge(
63		[ 'value' => '1' ],
64		$config['associated']
65		) );
66		// TODO Should use a LabelWidget? But they don't work like HTML <label>s yet
67		$this->associatedLabel = new \OOUI\FieldLayout(
68		$this->associated,
69		array_merge(
70		[ 'align' => 'inline' ],
71		$config['associatedLabel']
72		)
73		);
74		}
75	View Code Duplication	if ( $config['invert'] !== null ) {
76		$this->invert = new \OOUI\CheckboxInputWidget( array_merge(
77		[ 'value' => '1' ],
78		$config['invert']
79		) );
80		// TODO Should use a LabelWidget? But they don't work like HTML <label>s yet
81		$this->invertLabel = new \OOUI\FieldLayout(
82		$this->invert,
83		array_merge(
84		[ 'align' => 'inline' ],
85		$config['invertLabel']
86		)
87		);
88		}
89
90		// Initialization
91		$this
92		->addClasses( [ 'mw-widget-complexNamespaceInputWidget' ] )
93		->appendContent( $this->namespace, $this->associatedLabel, $this->invertLabel );
94		}
95
96		protected function getJavaScriptClassName() {
97		return 'mw.widgets.ComplexNamespaceInputWidget';
98		}
99
100		public function getConfig( &$config ) {
101		$config = array_merge(
102		$config,
103		array_intersect_key(
104		$this->config,
105		array_fill_keys(
106		[
107		'namespace',
108		'invert',
109		'invertLabel',
110		'associated',
111		'associatedLabel'
112		],
113		true
114		)
115		)
116		);
117		return parent::getConfig( $config );
118		}
119		}
120

wikimedia / mediawiki

Branch — master (939199)

ComplexNamespaceInputWidget::__construct() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like