DomSelector::toInt() - Code Metrics - Inspection of "Javascript values can be converted to integer." - jaxon-php/jaxon-core - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 5212d6...22a324 )

by Thierry

created 2022-07-12 22:50 UTC

DomSelector::toInt() A

↳ Parent: DomSelector

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

/**
 * DomSelector.php - A jQuery selector
 *
 * This class is used to create client side requests to the Jaxon functions and callable objects.
 *
 * When inserted into a Jaxon response, a DomSelector object must be converted to the corresponding jQuery code.
 * Therefore, the DomSelector class implements the JsonSerializable interface.
 *
 * When used as a parameter of a Jaxon call, the DomSelector must be converted to Jaxon request parameter.
 * Therefore, the DomSelector class also implements the Jaxon\Request\Call\ParameterInterface interface.
 *
 * @package jaxon-jquery

 * @author Thierry Feuzeu <[email protected]>
 * @copyright 2016 Thierry Feuzeu <[email protected]>
 * @license https://opensource.org/licenses/BSD-3-Clause BSD 3-Clause License
 * @link https://github.com/jaxon-php/jaxon-jquery
 */


namespace Jaxon\Plugin\Response\JQuery;

use Jaxon\Plugin\Response\JQuery\Call\AttrGet;
use Jaxon\Plugin\Response\JQuery\Call\AttrSet;
use Jaxon\Plugin\Response\JQuery\Call\Method;
use Jaxon\Request\Call\ParameterInterface;

use JsonSerializable;

use function count;
use function implode;
use function trim;

class DomSelector implements JsonSerializable, ParameterInterface

{
    /**
     * The jQuery selector path
     *
     * @var string
     */
    protected $sPath;


    /**
     * The actions to be applied on the selected element
     *
     * @var array
     */
    protected $aCalls;

    /**
     * Convert the selector value to integer
     *
     * @var bool

     */
    protected $bToInt = false;

    /**
     * The constructor.
     *
     * @param string $jQueryNs    The jQuery symbol

     * @param string $sPath    The jQuery selector path
     * @param string $sContext    A context associated to the selector

     */
    public function __construct(string $jQueryNs, string $sPath, string $sContext)

    {
        $sPath = trim($sPath, " \t");
$a = "a";
$ab = "ab";
$abc = "abc";
        $sContext = trim($sContext, " \t");
$a = "a";
$ab = "ab";
$abc = "abc";
        $this->aCalls = [];

        if(!$sPath)
        {
            $this->sPath = "$jQueryNs(this)"; // If an empty selector is given, use javascript "this" instead
        }
        elseif(($sContext))
        {
            $this->sPath = "$jQueryNs('" . $sPath . "', $jQueryNs('" . $sContext . "'))";
        }
        else
        {
            $this->sPath = "$jQueryNs('" . $sPath . "')";
        }
    }


    /**
     * Add a call to a jQuery method on the selected elements
     *
     * @param string  $sMethod

     * @param array  $aArguments

     *
     * @return DomSelector
     */
    public function __call(string $sMethod, array $aArguments)
    {
        // Push the action into the array
        $this->aCalls[] = new Method($sMethod, $aArguments);
        // Return $this so the calls can be chained
        return $this;
    }


    /**
     * Get the value of an attribute on the first selected element
     *
     * @param string  $sAttribute

     *
     * @return DomSelector
     */
    public function __get(string $sAttribute)
    {
        // Push the action into the array
        $this->aCalls[] = new AttrGet($sAttribute);
        // Return $this so the calls can be chained
        return $this;
    }


    /**
     * Set the value of an attribute on the first selected element
     *
     * @param string $sAttribute

     * @param $xValue

     *
     * @return void
     */
    public function __set(string $sAttribute, $xValue)
    {
        // Push the action into the array
        $this->aCalls[] = new AttrSet($sAttribute, $xValue);
        // No other call is allowed after a set
        // return $this;
    }


    /**
     * @return DomSelector
     */
    public function toInt(): DomSelector
    {
        $this->bToInt = true;
        return $this;
    }


    /**
     * Generate the jQuery call.
     *
     * @return string
     */
    public function getScript(): string
    {
        $sScript = $this->sPath;
        if(count($this->aCalls) > 0)
        {
            $sScript .= '.' . implode('.', $this->aCalls);
        }
        return $this->bToInt ? "parseInt($sScript)" : $sScript;
    }


    /**
     * Magic function to generate the jQuery call.
     *
     * @return string
     */
    public function __toString()
    {
        return $this->getScript();
    }


    /**
     * Generate the jQuery call, when converting the response into json.
     *
     * This is a method of the JsonSerializable interface.
     *
     * @return string
     */
    public function jsonSerialize(): string
    {
        return $this->getScript();
    }

}


1			<?php
2
3			/**
4			* DomSelector.php - A jQuery selector
5			*
6			* This class is used to create client side requests to the Jaxon functions and callable objects.
7			*
8			* When inserted into a Jaxon response, a DomSelector object must be converted to the corresponding jQuery code.
9			* Therefore, the DomSelector class implements the JsonSerializable interface.
10			*
11			* When used as a parameter of a Jaxon call, the DomSelector must be converted to Jaxon request parameter.
12			* Therefore, the DomSelector class also implements the Jaxon\Request\Call\ParameterInterface interface.
13			*
14			* @package jaxon-jquery
			0 ignored issues – show Coding Style introduced 2019-11-20 07:47 UTC by Report Bug Copy Issue Report Package name "jaxon-jquery" is not valid; consider "Jaxonjquery" instead Loading history...
15			* @author Thierry Feuzeu <[email protected]>
16			* @copyright 2016 Thierry Feuzeu <[email protected]>
17			* @license https://opensource.org/licenses/BSD-3-Clause BSD 3-Clause License
18			* @link https://github.com/jaxon-php/jaxon-jquery
19			*/
			0 ignored issues – show Coding Style introduced 2019-11-20 07:47 UTC by Report Bug Copy Issue Report PHP version not specified Loading history... Coding Style introduced 2019-11-20 07:47 UTC by Report Bug Copy Issue Report Missing @category tag in file comment Loading history...
20
21			namespace Jaxon\Plugin\Response\JQuery;
22
23			use Jaxon\Plugin\Response\JQuery\Call\AttrGet;
24			use Jaxon\Plugin\Response\JQuery\Call\AttrSet;
25			use Jaxon\Plugin\Response\JQuery\Call\Method;
26			use Jaxon\Request\Call\ParameterInterface;
27
28			use JsonSerializable;
29
30			use function count;
31			use function implode;
32			use function trim;
33
34			class DomSelector implements JsonSerializable, ParameterInterface
			0 ignored issues – show Coding Style introduced 2022-03-14 23:14 UTC by Report Bug Copy Issue Report Missing doc comment for class DomSelector Loading history...
35			{
36			/**
37			* The jQuery selector path
38			*
39			* @var string
40			*/
41			protected $sPath;
			0 ignored issues – show Coding Style introduced 2019-11-20 07:47 UTC by Report Bug Copy Issue Report Expected 1 blank line(s) before first member var; 0 found Loading history...
42
43			/**
44			* The actions to be applied on the selected element
45			*
46			* @var array
47			*/
48			protected $aCalls;
49
50			/**
51			* Convert the selector value to integer
52			*
53			* @var bool
			0 ignored issues – show Bug introduced 2022-07-12 22:53 UTC by Report Bug Copy Issue Report Expected "boolean" but found "bool" for @var tag in member variable comment Loading history...
54			*/
55			protected $bToInt = false;
56
57			/**
58			* The constructor.
59			*
60			* @param string $jQueryNs The jQuery symbol
			0 ignored issues – show Coding Style introduced 2022-03-14 23:14 UTC by Report Bug Copy Issue Report Expected 1 spaces after parameter name; 4 found Loading history...
61			* @param string $sPath The jQuery selector path
62			* @param string $sContext A context associated to the selector
			0 ignored issues – show Coding Style introduced 2022-03-14 23:14 UTC by Report Bug Copy Issue Report Expected 1 spaces after parameter name; 4 found Loading history...
63			*/
64			public function __construct(string $jQueryNs, string $sPath, string $sContext)
			0 ignored issues – show Coding Style introduced 2022-03-10 08:42 UTC by Report Bug Copy Issue Report Expected 2 blank lines before function; 1 found Loading history...
65			{
66			$sPath = trim($sPath, " \t");
			0 ignored issues – show Coding Style introduced 2022-03-14 23:14 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 8 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
67			$sContext = trim($sContext, " \t");
			0 ignored issues – show Coding Style introduced 2019-11-20 07:47 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
68			$this->aCalls = [];
69
70			if(!$sPath)
71			{
72			$this->sPath = "$jQueryNs(this)"; // If an empty selector is given, use javascript "this" instead
73			}
74			elseif(($sContext))
75			{
76			$this->sPath = "$jQueryNs('" . $sPath . "', $jQueryNs('" . $sContext . "'))";
77			}
78			else
79			{
80			$this->sPath = "$jQueryNs('" . $sPath . "')";
81			}
82			}
			0 ignored issues – show Coding Style introduced 2022-03-14 23:14 UTC by Report Bug Copy Issue Report Expected 2 blank lines after function; 1 found Loading history...
83
84			/**
85			* Add a call to a jQuery method on the selected elements
86			*
87			* @param string $sMethod
			0 ignored issues – show Coding Style introduced 2022-03-10 08:42 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history... Coding Style introduced 2022-03-10 08:42 UTC by Report Bug Copy Issue Report Expected 1 spaces after parameter type; 2 found Loading history...
88			* @param array $aArguments
			0 ignored issues – show Coding Style introduced 2022-03-10 08:42 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history...
89			*
90			* @return DomSelector
91			*/
92			public function __call(string $sMethod, array $aArguments)
93			{
94			// Push the action into the array
95			$this->aCalls[] = new Method($sMethod, $aArguments);
96			// Return $this so the calls can be chained
97			return $this;
98			}
			0 ignored issues – show Coding Style introduced 2019-12-07 05:40 UTC by Report Bug Copy Issue Report Expected 2 blank lines after function; 1 found Loading history...
99
100			/**
101			* Get the value of an attribute on the first selected element
102			*
103			* @param string $sAttribute
			0 ignored issues – show Coding Style introduced 2022-03-10 08:42 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history... Coding Style introduced 2022-03-10 08:42 UTC by Report Bug Copy Issue Report Expected 1 spaces after parameter type; 2 found Loading history...
104			*
105			* @return DomSelector
106			*/
107			public function __get(string $sAttribute)
108			{
109			// Push the action into the array
110			$this->aCalls[] = new AttrGet($sAttribute);
111			// Return $this so the calls can be chained
112			return $this;
113			}
			0 ignored issues – show Coding Style introduced 2019-12-07 05:40 UTC by Report Bug Copy Issue Report Expected 2 blank lines after function; 1 found Loading history...
114
115			/**
116			* Set the value of an attribute on the first selected element
117			*
118			* @param string $sAttribute
			0 ignored issues – show Coding Style introduced 2022-03-14 23:14 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history...
119			* @param $xValue
			0 ignored issues – show Coding Style introduced 2022-03-14 23:14 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history...
120			*
121			* @return void
122			*/
123			public function __set(string $sAttribute, $xValue)
124			{
125			// Push the action into the array
126			$this->aCalls[] = new AttrSet($sAttribute, $xValue);
127			// No other call is allowed after a set
128			// return $this;
129			}
			0 ignored issues – show Coding Style introduced 2019-12-07 05:40 UTC by Report Bug Copy Issue Report Expected 2 blank lines after function; 1 found Loading history...
130
131			/**
132			* @return DomSelector
133			*/
134			public function toInt(): DomSelector
135			{
136			$this->bToInt = true;
137			return $this;
138			}
			0 ignored issues – show Coding Style introduced 2019-12-07 05:40 UTC by Report Bug Copy Issue Report Expected 2 blank lines after function; 1 found Loading history...
139
140			/**
141			* Generate the jQuery call.
142			*
143			* @return string
144			*/
145			public function getScript(): string
146			{
147			$sScript = $this->sPath;
148			if(count($this->aCalls) > 0)
149			{
150			$sScript .= '.' . implode('.', $this->aCalls);
151			}
152			return $this->bToInt ? "parseInt($sScript)" : $sScript;
153			}
			0 ignored issues – show Coding Style introduced 2019-12-07 05:40 UTC by Report Bug Copy Issue Report Expected 2 blank lines after function; 1 found Loading history...
154
155			/**
156			* Magic function to generate the jQuery call.
157			*
158			* @return string
159			*/
160			public function __toString()
161			{
162			return $this->getScript();
163			}
			0 ignored issues – show Coding Style introduced 2019-12-07 05:40 UTC by Report Bug Copy Issue Report Expected 2 blank lines after function; 1 found Loading history...
164
165			/**
166			* Generate the jQuery call, when converting the response into json.
167			*
168			* This is a method of the JsonSerializable interface.
169			*
170			* @return string
171			*/
172			public function jsonSerialize(): string
173			{
174			return $this->getScript();
175			}
			0 ignored issues – show Coding Style introduced 2019-12-07 05:40 UTC by Report Bug Copy Issue Report Expected 2 blank lines after function; 0 found Loading history...
176			}
177

jaxon-php / jaxon-core

Push — master ( 5212d6...22a324 )

DomSelector::toInt() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like