Dialog::confirm() - Code Metrics - Inspection of "Develop (#589)" - redaxscript/redaxscript - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( e9b1ca...cbd317 )

by Henry

created 2018-12-15 13:52 UTC

Dialog::confirm() A

↳ Parent: Dialog

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	2

<?php
namespace Redaxscript\Modules\Dialog;

use Redaxscript\Head;
use Redaxscript\Html;
use Redaxscript\Module;

/**
 * javascript powered dialog
 *
 * @since 4.0.0
 *
 * @package Redaxscript
 * @category Modules
 * @author Henry Ruhs
 */

class Dialog extends Module\Module
{
	/**
	 * array of the module
	 *
	 * @var array
	 */

	protected static $_moduleArray =
	[
		'name' => 'Dialog',
		'alias' => 'Dialog',
		'author' => 'Redaxmedia',
		'description' => 'JavaScript powered dialog',
		'version' => '4.0.0'
	];

	/**
	 * array of the option
	 *
	 * @var array
	 */

	protected $_optionArray =
	[
		'className' =>
		[
			'overlay' => 'rs-overlay-dialog',
			'component' => 'rs-component-dialog',
			'title' => 'rs-title-dialog',
			'box' => 'rs-box-dialog',
			'field' => 'rs-field-default rs-field-text',
			'button' => 'rs-button-default',
			'buttonOk' => 'js-ok',
			'buttonCancel' => 'js-cancel'
		]
	];

	/**
	 * renderStart
	 *
	 * @since 4.0.0
	 */

	public function renderStart()
	{
		$firstParameter = $this->_registry->get('firstParameter');
		$secondParameter = $this->_registry->get('secondParameter');
		$thirdParameter = $this->_registry->get('thirdParameter');

		/* link */

		$link = Head\Link::getInstance();
		$link
			->init()
			->appendFile('modules/Dialog/dist/styles/dialog.min.css');

		/* script */

		$script = Head\Script::getInstance();
		$script
			->init('foot')
			->appendFile(
			[
				'modules/Dialog/assets/scripts/init.js',
				'modules/Dialog/dist/scripts/dialog.min.js'
			]);

		/* router */

		if ($firstParameter === 'module' && ($secondParameter === 'dialog' || $secondParameter === 'admin-dialog'))
		{
			$this->_registry->set('renderBreak', true);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			$dialog = $secondParameter === 'admin-dialog' ? new Admin\Dialog($this->_registry, $this->_request, $this->_language, $this->_config) : $this;
			if ($thirdParameter === 'alert')
			{
				echo $dialog->alert();
			}
			if ($thirdParameter === 'confirm')
			{
				echo $dialog->confirm();
			}
			if ($thirdParameter === 'prompt')
			{
				echo $dialog->confirm();
			}
		}
	}

	/**
	 * alert
	 *
	 * @since 4.0.0
	 *
	 * @param string $title
	 * @param string $message
	 *
	 * @return string
	 */

	public function alert(string $title = null, string $message = null) : string
	{
		return $this->_dialog('alert', $title, $message);
	}

	/**
	 * confirm
	 *
	 * @since 4.0.0
	 *
	 * @param string $title
	 * @param string $message
	 *
	 * @return string
	 */

	public function confirm(string $title = null, string $message = null) : string
	{
		return $this->_dialog('confirm', $title, $message);
	}

	/**
	 * prompt
	 *
	 * @since 4.0.0
	 *
	 * @param string $title
	 * @param string $message
	 *
	 * @return string
	 */

	public function prompt(string $title = null, string $message = null) : string
	{
		return $this->_dialog('prompt', $title, $message);
	}

	/**
	 * dialog
	 *
	 * @since 4.0.0
	 *
	 * @param string $type
	 * @param string $title
	 * @param string $message
	 *
	 * @return string
	 */

	protected function _dialog(string $type = null, string $title = null, string $message = null) : string
	{
		$output = null;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

		/* html elements */

		$element = new Html\Element();
		$overlayElement = $element
			->copy()
			->init('div',
			[
				'class' => $this->_optionArray['className']['overlay']
			]);
		$dialogElement = $element
			->copy()
			->init('div',
			[
				'class' => $this->_optionArray['className']['component']
			]);
		$titleElement = $element
			->copy()
			->init('h3',
			[
				'class' => $this->_optionArray['className']['title']
			])
			->text($title ? : $this->_language->get('confirm', '_dialog'));
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
		$boxElement = $element
			->copy()
			->init('div',
			[
				'class' => $this->_optionArray['className']['box']
			]);
		$textElement = $type === 'alert' || $type === 'confirm' ? $element
			->copy()
			->init('p')
			->text($message ? : $this->_language->get('continue_question', '_dialog') . $this->_language->get('question_mark')) : null;
		$fieldElement = $type === 'prompt' ? $element
			->copy()
			->init('input',
			[
				'class' => $this->_optionArray['className']['field'],
				'placeholder' => $message
			]) : null;
		$buttonElement = $element
			->copy()
			->init('button',
			[
				'class' => $this->_optionArray['className']['button']
			]);
		$buttonOkElement = $buttonElement
			->copy()
			->addClass($this->_optionArray['className']['buttonOk'])
			->text($this->_language->get('ok'));

		$buttonCancelElement = $type === 'confirm' || $type === 'prompt' ? $buttonElement
			->copy()
			->addClass($this->_optionArray['className']['buttonCancel'])
			->text($this->_language->get('cancel')) : null;


		/* collect output */

		$output = $overlayElement->html(
			$dialogElement->html(
				$titleElement .
				$boxElement->html(
					$textElement .
					$fieldElement .
					$buttonOkElement .
					$buttonCancelElement
				)
			)
		);
		return $output;
	}
}


1			<?php
2			namespace Redaxscript\Modules\Dialog;
3
4			use Redaxscript\Head;
5			use Redaxscript\Html;
6			use Redaxscript\Module;
7
8			/**
9			* javascript powered dialog
10			*
11			* @since 4.0.0
12			*
13			* @package Redaxscript
14			* @category Modules
15			* @author Henry Ruhs
16			*/
17
18			class Dialog extends Module\Module
19			{
20			/**
21			* array of the module
22			*
23			* @var array
24			*/
25
26			protected static $_moduleArray =
27			[
28			'name' => 'Dialog',
29			'alias' => 'Dialog',
30			'author' => 'Redaxmedia',
31			'description' => 'JavaScript powered dialog',
32			'version' => '4.0.0'
33			];
34
35			/**
36			* array of the option
37			*
38			* @var array
39			*/
40
41			protected $_optionArray =
42			[
43			'className' =>
44			[
45			'overlay' => 'rs-overlay-dialog',
46			'component' => 'rs-component-dialog',
47			'title' => 'rs-title-dialog',
48			'box' => 'rs-box-dialog',
49			'field' => 'rs-field-default rs-field-text',
50			'button' => 'rs-button-default',
51			'buttonOk' => 'js-ok',
52			'buttonCancel' => 'js-cancel'
53			]
54			];
55
56			/**
57			* renderStart
58			*
59			* @since 4.0.0
60			*/
61
62			public function renderStart()
63			{
64			$firstParameter = $this->_registry->get('firstParameter');
65			$secondParameter = $this->_registry->get('secondParameter');
66			$thirdParameter = $this->_registry->get('thirdParameter');
67
68			/* link */
69
70			$link = Head\Link::getInstance();
71			$link
72			->init()
73			->appendFile('modules/Dialog/dist/styles/dialog.min.css');
74
75			/* script */
76
77			$script = Head\Script::getInstance();
78			$script
79			->init('foot')
80			->appendFile(
81			[
82			'modules/Dialog/assets/scripts/init.js',
83			'modules/Dialog/dist/scripts/dialog.min.js'
84			]);
85
86			/* router */
87
88			if ($firstParameter === 'module' && ($secondParameter === 'dialog' \|\| $secondParameter === 'admin-dialog'))
89			{
90			$this->_registry->set('renderBreak', true);
			0 ignored issues – show Documentation introduced 2018-09-10 21:53 UTC by Report Bug Copy Issue Report `true` is of type `boolean`, but the function expects a `string\|array\|null`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
91			$dialog = $secondParameter === 'admin-dialog' ? new Admin\Dialog($this->_registry, $this->_request, $this->_language, $this->_config) : $this;
92			if ($thirdParameter === 'alert')
93			{
94			echo $dialog->alert();
95			}
96			if ($thirdParameter === 'confirm')
97			{
98			echo $dialog->confirm();
99			}
100			if ($thirdParameter === 'prompt')
101			{
102			echo $dialog->confirm();
103			}
104			}
105			}
106
107			/**
108			* alert
109			*
110			* @since 4.0.0
111			*
112			* @param string $title
113			* @param string $message
114			*
115			* @return string
116			*/
117
118			public function alert(string $title = null, string $message = null) : string
119			{
120			return $this->_dialog('alert', $title, $message);
121			}
122
123			/**
124			* confirm
125			*
126			* @since 4.0.0
127			*
128			* @param string $title
129			* @param string $message
130			*
131			* @return string
132			*/
133
134			public function confirm(string $title = null, string $message = null) : string
135			{
136			return $this->_dialog('confirm', $title, $message);
137			}
138
139			/**
140			* prompt
141			*
142			* @since 4.0.0
143			*
144			* @param string $title
145			* @param string $message
146			*
147			* @return string
148			*/
149
150			public function prompt(string $title = null, string $message = null) : string
151			{
152			return $this->_dialog('prompt', $title, $message);
153			}
154
155			/**
156			* dialog
157			*
158			* @since 4.0.0
159			*
160			* @param string $type
161			* @param string $title
162			* @param string $message
163			*
164			* @return string
165			*/
166
167			protected function _dialog(string $type = null, string $title = null, string $message = null) : string
168			{
169			$output = null;
			0 ignored issues – show Unused Code introduced 2018-12-15 13:46 UTC by Report Bug Copy Issue Report `$output` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
170
171			/* html elements */
172
173			$element = new Html\Element();
174			$overlayElement = $element
175			->copy()
176			->init('div',
177			[
178			'class' => $this->_optionArray['className']['overlay']
179			]);
180			$dialogElement = $element
181			->copy()
182			->init('div',
183			[
184			'class' => $this->_optionArray['className']['component']
185			]);
186			$titleElement = $element
187			->copy()
188			->init('h3',
189			[
190			'class' => $this->_optionArray['className']['title']
191			])
192			->text($title ? : $this->_language->get('confirm', '_dialog'));
			0 ignored issues – show Bug introduced 2018-12-15 13:46 UTC by Report Bug Copy Issue Report It seems like `$title ?: $this->_langua...t('confirm', '_dialog')` can also be of type `array`; however, `Redaxscript\Html\Element::text()` does only seem to accept `string\|integer\|null`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /** * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
193			$boxElement = $element
194			->copy()
195			->init('div',
196			[
197			'class' => $this->_optionArray['className']['box']
198			]);
199			$textElement = $type === 'alert' \|\| $type === 'confirm' ? $element
200			->copy()
201			->init('p')
202			->text($message ? : $this->_language->get('continue_question', '_dialog') . $this->_language->get('question_mark')) : null;
203			$fieldElement = $type === 'prompt' ? $element
204			->copy()
205			->init('input',
206			[
207			'class' => $this->_optionArray['className']['field'],
208			'placeholder' => $message
209			]) : null;
210			$buttonElement = $element
211			->copy()
212			->init('button',
213			[
214			'class' => $this->_optionArray['className']['button']
215			]);
216			$buttonOkElement = $buttonElement
217			->copy()
218			->addClass($this->_optionArray['className']['buttonOk'])
219			->text($this->_language->get('ok'));
			0 ignored issues – show Bug introduced 2018-12-15 13:46 UTC by Report Bug Copy Issue Report It seems like `$this->_language->get('ok')` targeting `Redaxscript\Language::get()` can also be of type `array`; however, `Redaxscript\Html\Element::text()` does only seem to accept `string\|integer\|null`, maybe add an additional type check? This check looks at variables that are passed out again to other methods. If the outgoing method call has stricter type requirements than the method itself, an issue is raised. An additional type check may prevent trouble. Loading history...
220			$buttonCancelElement = $type === 'confirm' \|\| $type === 'prompt' ? $buttonElement
221			->copy()
222			->addClass($this->_optionArray['className']['buttonCancel'])
223			->text($this->_language->get('cancel')) : null;
			0 ignored issues – show Bug introduced 2018-12-15 13:46 UTC by Report Bug Copy Issue Report It seems like `$this->_language->get('cancel')` targeting `Redaxscript\Language::get()` can also be of type `array`; however, `Redaxscript\Html\Element::text()` does only seem to accept `string\|integer\|null`, maybe add an additional type check? This check looks at variables that are passed out again to other methods. If the outgoing method call has stricter type requirements than the method itself, an issue is raised. An additional type check may prevent trouble. Loading history...
224
225			/* collect output */
226
227			$output = $overlayElement->html(
228			$dialogElement->html(
229			$titleElement .
230			$boxElement->html(
231			$textElement .
232			$fieldElement .
233			$buttonOkElement .
234			$buttonCancelElement
235			)
236			)
237			);
238			return $output;
239			}
240			}
241

redaxscript / redaxscript

Push — master ( e9b1ca...cbd317 )

Dialog::confirm() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like