Issues in Byline.php - All Issues - Inspection of "Transition to PHP8 testing part15" - redaxscript/redaxscript - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 4eb4a8...a52438 )

by Henry

created 2021-03-26 22:17 UTC

includes/View/Helper/Byline.php (4 issues)

Labels

Severity

Major 4

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php
namespace Redaxscript\View\Helper;

use Redaxscript\Dater;
use Redaxscript\Html;
use Redaxscript\Module;
use Redaxscript\View\ViewAbstract;
use function array_replace_recursive;

/**
 * helper class to create the byline
 *
 * @since 4.0.0
 *
 * @package Redaxscript
 * @category View
 * @author Henry Ruhs
 */

class Byline extends ViewAbstract
{
	/**
	 * options of the byline
	 *
	 * @var array
	 */

	protected $_optionArray =
	[
		'className' =>
		[
			'box' => 'rs-box-byline',
			'text' =>
			[
				'by' => 'rs-text-by',
				'author' => 'rs-text-author',
				'on' => 'rs-text-on',
				'date' => 'rs-text-date',
				'at' => 'rs-text-at',
				'time' => 'rs-text-time',
			]
		]
	];

	/**
	 * init the class
	 *
	 * @since 4.0.0
	 *
	 * @param array $optionArray options of the pagination
	 *
	 * @return self
	 */

	public function init(array $optionArray = []) : self
	{
		$this->_optionArray = array_replace_recursive($this->_optionArray, $optionArray);
		return $this;
	}

	/**
	 * render the view
	 *
	 * @since 4.0.0
	 *
	 * @param int $date timestamp of the date
	 * @param string $author name of the author
	 *
	 * @return string
	 */

	public function render(int $date = null, string $author = null) : string
	{
		$output = Module\Hook::trigger('bylineStart');
		$dater = new Dater();
		$dater->init($date);

		/* html element */

		$element = new Html\Element();
		$boxElement = $element
			->copy()
			->init('div',
			[
				'class' => $this->_optionArray['className']['box']
			]);
		$textElement = $element
			->copy()
			->init('span');

		/* collect output */

		if ($author)
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
		{
			$boxElement->html(
				$textElement
					->copy()
					->addClass($this->_optionArray['className']['text']['by'])
					->text($this->_language->get('posted_by')) .
				$textElement
					->copy()
					->addClass($this->_optionArray['className']['text']['author'])
					->text($author)
			);
		}
		if ($author && $date)
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
		{
			$boxElement->append(
				$textElement
					->copy()
					->addClass($this->_optionArray['className']['text']['on'])
					->text($this->_language->get('on'))
			);
		}
		if ($date)
0   == false // true
0   == null  // true
123 == false // false
123 == null  // false

// It is often better to use strict comparison
0 === false // false
0 === null  // false
		{
			$boxElement->append(
				$textElement
					->copy()
					->addClass($this->_optionArray['className']['text']['date'])
					->text($dater->formatDate()) .
				$textElement
					->copy()
					->addClass($this->_optionArray['className']['text']['at'])
					->text($this->_language->get('at')) .
				$textElement
					->copy()
					->addClass($this->_optionArray['className']['text']['time'])
					->text($dater->formatTime())
			);
		}
		$output .= $boxElement . Module\Hook::trigger('bylineEnd');
		return $output;
	}
}


1		<?php
2		namespace Redaxscript\View\Helper;
3
4		use Redaxscript\Dater;
5		use Redaxscript\Html;
6		use Redaxscript\Module;
7		use Redaxscript\View\ViewAbstract;
8		use function array_replace_recursive;
9
10		/**
11		* helper class to create the byline
12		*
13		* @since 4.0.0
14		*
15		* @package Redaxscript
16		* @category View
17		* @author Henry Ruhs
18		*/
19
20		class Byline extends ViewAbstract
21		{
22		/**
23		* options of the byline
24		*
25		* @var array
26		*/
27
28		protected $_optionArray =
29		[
30		'className' =>
31		[
32		'box' => 'rs-box-byline',
33		'text' =>
34		[
35		'by' => 'rs-text-by',
36		'author' => 'rs-text-author',
37		'on' => 'rs-text-on',
38		'date' => 'rs-text-date',
39		'at' => 'rs-text-at',
40		'time' => 'rs-text-time',
41		]
42		]
43		];
44
45		/**
46		* init the class
47		*
48		* @since 4.0.0
49		*
50		* @param array $optionArray options of the pagination
51		*
52		* @return self
53		*/
54
55	2	public function init(array $optionArray = []) : self
56		{
57	2	$this->_optionArray = array_replace_recursive($this->_optionArray, $optionArray);
58	2	return $this;
59		}
60
61		/**
62		* render the view
63		*
64		* @since 4.0.0
65		*
66		* @param int $date timestamp of the date
67		* @param string $author name of the author
68		*
69		* @return string
70		*/
71
72	2	public function render(int $date = null, string $author = null) : string
73		{
74	2	$output = Module\Hook::trigger('bylineStart');
75	2	$dater = new Dater();
76	2	$dater->init($date);
77
78		/* html element */
79
80	2	$element = new Html\Element();
81		$boxElement = $element
82	2	->copy()
83	2	->init('div',
84		[
85	2	'class' => $this->_optionArray['className']['box']
86		]);
87		$textElement = $element
88	2	->copy()
89	2	->init('span');
90
91		/* collect output */
92
93	2	if ($author)
		0 ignored issues – show Bug Best Practice introduced 2018-07-03 16:14 UTC by Report Bug Copy Issue Report Show Similar Issues like this The expression `$author` of type `null\|string` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
94		{
95	1	$boxElement->html(
96		$textElement
97	1	->copy()
98	1	->addClass($this->_optionArray['className']['text']['by'])
99	1	->text($this->_language->get('posted_by')) .
100		$textElement
101	1	->copy()
102	1	->addClass($this->_optionArray['className']['text']['author'])
103	1	->text($author)
104		);
105		}
106	2	if ($author && $date)
		0 ignored issues – show Bug Best Practice introduced 2018-07-03 16:14 UTC by Report Bug Copy Issue Report Show Similar Issues like this The expression `$author` of type `null\|string` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history... Bug Best Practice introduced 2018-07-26 21:58 UTC by Report Bug Copy Issue Report Show Similar Issues like this The expression `$date` of type `null\|integer` is loosely compared to `true`; this is ambiguous if the integer can be zero. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `integer` values, zero is a special case, in particular the following results might be unexpected: 0 == false // true 0 == null // true 123 == false // false 123 == null // false // It is often better to use strict comparison 0 === false // false 0 === null // false Loading history...
107		{
108	1	$boxElement->append(
109		$textElement
110	1	->copy()
111	1	->addClass($this->_optionArray['className']['text']['on'])
112	1	->text($this->_language->get('on'))
113		);
114		}
115	2	if ($date)
		0 ignored issues – show Bug Best Practice introduced 2018-07-26 21:58 UTC by Report Bug Copy Issue Report Show Similar Issues like this The expression `$date` of type `null\|integer` is loosely compared to `true`; this is ambiguous if the integer can be zero. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `integer` values, zero is a special case, in particular the following results might be unexpected: 0 == false // true 0 == null // true 123 == false // false 123 == null // false // It is often better to use strict comparison 0 === false // false 0 === null // false Loading history...
116		{
117	2	$boxElement->append(
118		$textElement
119	2	->copy()
120	2	->addClass($this->_optionArray['className']['text']['date'])
121	2	->text($dater->formatDate()) .
122		$textElement
123	2	->copy()
124	2	->addClass($this->_optionArray['className']['text']['at'])
125	2	->text($this->_language->get('at')) .
126		$textElement
127	2	->copy()
128	2	->addClass($this->_optionArray['className']['text']['time'])
129	2	->text($dater->formatTime())
130		);
131		}
132	2	$output .= $boxElement . Module\Hook::trigger('bylineEnd');
133	2	return $output;
134		}
135		}
136

redaxscript / redaxscript

Push — master ( 4eb4a8...a52438 )

includes/View/Helper/Byline.php (4 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like