PHPCompatibility_Sniffs_PHP_NewInterfacesSniff::process() - Code Metrics - Inspection of "Merge pull request #144 from jrfnl/feature/new-int..." - wimg/PHPCompatibility - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a0c8a1...aee42a )

by Wim

created 2016-08-04 20:13 UTC

process() D

↳ Parent: PHPCompatibility_Sniffs_PHP_NewInterfacesSniff

Complexity

Conditions	10
Paths	19

Size

Total Lines	42
Code Lines	25

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
cc	10
eloc	25
c	1
b	0
f	1
nc	19
nop	2
dl	0
loc	42
rs	4.8196

How to fix Complexity

<?php
/**
 * PHPCompatibility_Sniffs_PHP_NewInterfacesSniff.
 *
 * PHP version 5.5
 *
 * @category  PHP
 * @package   PHPCompatibility
 * @author    Juliette Reinders Folmer <[email protected]>
 */

/**
 * PHPCompatibility_Sniffs_PHP_NewInterfacesSniff.
 *
 * @category  PHP
 * @package   PHPCompatibility
 * @author    Juliette Reinders Folmer <[email protected]>
 */
class PHPCompatibility_Sniffs_PHP_NewInterfacesSniff extends PHPCompatibility_Sniff
{

    /**
     * A list of new interfaces, not present in older versions.
     *
     * The array lists : version number with false (not present) or true (present).
     * If's sufficient to list the first version where the interface appears.
     *
     * @var array(string => array(string => int|string|null))
     */
    protected $newInterfaces = array(
                                'Countable' => array(
                                    '5.0' => false,
                                    '5.1' => true
                                ),
                                'OuterIterator' => array(
                                    '5.0' => false,
                                    '5.1' => true
                                ),
                                'RecursiveIterator' => array(
                                    '5.0' => false,
                                    '5.1' => true
                                ),
                                'SeekableIterator' => array(
                                    '5.0' => false,
                                    '5.1' => true
                                ),
                                'Serializable' => array(
                                    '5.0' => false,
                                    '5.1' => true,
                                ),
                                'SplObserver' => array(
                                    '5.0' => false,
                                    '5.1' => true
                                ),
                                'SplSubject' => array(
                                    '5.0' => false,
                                    '5.1' => true
                                ),

                                'JsonSerializable' => array(
                                    '5.3' => false,
                                    '5.4' => true
                                ),
                                'SessionHandlerInterface' => array(
                                    '5.3' => false,
                                    '5.4' => true
                                ),

                               );

    /**
     * A list of methods which cannot be used in combination with particular interfaces.
     *
     * @var array(string => array(string => string))
     */
    protected $unsupportedMethods = array(
                                     'Serializable' => array(
                                         '__sleep'  => 'http://php.net/serializable',
                                         '__wakeup' => 'http://php.net/serializable',
                                     ),
                                    );

    /**
     * Returns an array of tokens this test wants to listen for.
     *
     * @return array
     */
    public function register()
    {
        // Handle case-insensitivity of class names.
        $keys = array_keys( $this->newInterfaces );
        $keys = array_map( 'strtolower', $keys );
        $this->newInterfaces = array_combine( $keys, $this->newInterfaces );

        $keys = array_keys( $this->unsupportedMethods );
        $keys = array_map( 'strtolower', $keys );
        $this->unsupportedMethods = array_combine( $keys, $this->unsupportedMethods );

        return array(T_CLASS);

    }//end register()


    /**
     * Processes this test, when one of its tokens is encountered.
     *
     * @param PHP_CodeSniffer_File $phpcsFile The file being scanned.
     * @param int                  $stackPtr  The position of the current token in
     *                                        the stack passed in $tokens.
     *
     * @return void
     */
    public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
    {
        $interfaces = $this->findImplementedInterfaceNames($phpcsFile, $stackPtr);

        if (is_array($interfaces) === false || $interfaces === array()) {
            return;
        }

        $tokens       = $phpcsFile->getTokens();
        $checkMethods = false;

        if(isset($tokens[$stackPtr]['scope_closer'])) {
            $checkMethods = true;
            $scopeCloser = $tokens[$stackPtr]['scope_closer'];
        }

        foreach ($interfaces as $interface) {
            $lcInterface = strtolower($interface);
            if (isset($this->newInterfaces[$lcInterface]) === true) {
                $this->addError($phpcsFile, $stackPtr, $interface);
            }

            if ($checkMethods === true && isset($this->unsupportedMethods[$lcInterface]) === true) {
                $nextFunc = $stackPtr;
                while (($nextFunc = $phpcsFile->findNext(T_FUNCTION, ($nextFunc + 1), $scopeCloser)) !== false) {
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
                    $funcNamePos = $phpcsFile->findNext(T_STRING, $nextFunc);
                    $funcName    = strtolower($tokens[$funcNamePos]['content']);

                    if (isset($this->unsupportedMethods[$lcInterface][$funcName]) === true) {
                        $error = 'Classes that implement interface %s do not support the method %s(). See %s';
                        $data  = array(
                            $interface,
                            $tokens[$funcNamePos]['content'],
                            $this->unsupportedMethods[$lcInterface][$funcName],
                        );
                        $phpcsFile->addError($error, $funcNamePos, 'UnsupportedMethod', $data);
<?php

function getDate($date)
{
    if ($date !== null) {
        return new DateTime($date);
    }

    return false;
}
                    }
                }
            }
        }

    }//end process()


    /**
     * Generates the error or warning for this sniff.
     *
     * @param PHP_CodeSniffer_File $phpcsFile The file being scanned.
     * @param int                  $stackPtr  The position of the function
     *                                        in the token array.
     * @param string               $interface The name of the interface.
     * @param string               $pattern   The pattern used for the match.
     *
     * @return void
     */
    protected function addError($phpcsFile, $stackPtr, $interface, $pattern=null)
    {
        if ($pattern === null) {
            $pattern = strtolower($interface);
        }

        $error = '';

        $isError = false;
        foreach ($this->newInterfaces[$pattern] as $version => $present) {
            if ($this->supportsBelow($version)) {
                if ($present === false) {
                    $isError = true;
                    $error .= 'not present in PHP version ' . $version . ' or earlier';
                }
            }
        }

        if (strlen($error) > 0) {
            $error = 'The built-in interface ' . $interface . ' is ' . $error;

            if ($isError === true) {
                $phpcsFile->addError($error, $stackPtr);
            } else {
                $phpcsFile->addWarning($error, $stackPtr);
            }
        }

    }//end addError()

}//end class


Push — master ( a0c8a1...aee42a )

process() D

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Available Fixes

1			<?php
2			/**
3			* PHPCompatibility_Sniffs_PHP_NewInterfacesSniff.
4			*
5			* PHP version 5.5
6			*
7			* @category PHP
8			* @package PHPCompatibility
9			* @author Juliette Reinders Folmer <[email protected]>
10			*/
11
12			/**
13			* PHPCompatibility_Sniffs_PHP_NewInterfacesSniff.
14			*
15			* @category PHP
16			* @package PHPCompatibility
17			* @author Juliette Reinders Folmer <[email protected]>
18			*/
19			class PHPCompatibility_Sniffs_PHP_NewInterfacesSniff extends PHPCompatibility_Sniff
20			{
21
22			/**
23			* A list of new interfaces, not present in older versions.
24			*
25			* The array lists : version number with false (not present) or true (present).
26			* If's sufficient to list the first version where the interface appears.
27			*
28			* @var array(string => array(string => int\|string\|null))
29			*/
30			protected $newInterfaces = array(
31			'Countable' => array(
32			'5.0' => false,
33			'5.1' => true
34			),
35			'OuterIterator' => array(
36			'5.0' => false,
37			'5.1' => true
38			),
39			'RecursiveIterator' => array(
40			'5.0' => false,
41			'5.1' => true
42			),
43			'SeekableIterator' => array(
44			'5.0' => false,
45			'5.1' => true
46			),
47			'Serializable' => array(
48			'5.0' => false,
49			'5.1' => true,
50			),
51			'SplObserver' => array(
52			'5.0' => false,
53			'5.1' => true
54			),
55			'SplSubject' => array(
56			'5.0' => false,
57			'5.1' => true
58			),
59
60			'JsonSerializable' => array(
61			'5.3' => false,
62			'5.4' => true
63			),
64			'SessionHandlerInterface' => array(
65			'5.3' => false,
66			'5.4' => true
67			),
68
69			);
70
71			/**
72			* A list of methods which cannot be used in combination with particular interfaces.
73			*
74			* @var array(string => array(string => string))
75			*/
76			protected $unsupportedMethods = array(
77			'Serializable' => array(
78			'__sleep' => 'http://php.net/serializable',
79			'__wakeup' => 'http://php.net/serializable',
80			),
81			);
82
83			/**
84			* Returns an array of tokens this test wants to listen for.
85			*
86			* @return array
87			*/
88			public function register()
89			{
90			// Handle case-insensitivity of class names.
91			$keys = array_keys( $this->newInterfaces );
92			$keys = array_map( 'strtolower', $keys );
93			$this->newInterfaces = array_combine( $keys, $this->newInterfaces );
94
95			$keys = array_keys( $this->unsupportedMethods );
96			$keys = array_map( 'strtolower', $keys );
97			$this->unsupportedMethods = array_combine( $keys, $this->unsupportedMethods );
98
99			return array(T_CLASS);
100
101			}//end register()
102
103
104			/**
105			* Processes this test, when one of its tokens is encountered.
106			*
107			* @param PHP_CodeSniffer_File $phpcsFile The file being scanned.
108			* @param int $stackPtr The position of the current token in
109			* the stack passed in $tokens.
110			*
111			* @return void
112			*/
113			public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
114			{
115			$interfaces = $this->findImplementedInterfaceNames($phpcsFile, $stackPtr);
116
117			if (is_array($interfaces) === false \|\| $interfaces === array()) {
118			return;
119			}
120
121			$tokens = $phpcsFile->getTokens();
122			$checkMethods = false;
123
124			if(isset($tokens[$stackPtr]['scope_closer'])) {
125			$checkMethods = true;
126			$scopeCloser = $tokens[$stackPtr]['scope_closer'];
127			}
128
129			foreach ($interfaces as $interface) {
130			$lcInterface = strtolower($interface);
131			if (isset($this->newInterfaces[$lcInterface]) === true) {
132			$this->addError($phpcsFile, $stackPtr, $interface);
133			}
134
135			if ($checkMethods === true && isset($this->unsupportedMethods[$lcInterface]) === true) {
136			$nextFunc = $stackPtr;
137			while (($nextFunc = $phpcsFile->findNext(T_FUNCTION, ($nextFunc + 1), $scopeCloser)) !== false) {
			0 ignored issues – show Bug introduced 2016-08-04 08:07 UTC by Report Bug Copy Issue Report The variable `$scopeCloser` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
138			$funcNamePos = $phpcsFile->findNext(T_STRING, $nextFunc);
139			$funcName = strtolower($tokens[$funcNamePos]['content']);
140
141			if (isset($this->unsupportedMethods[$lcInterface][$funcName]) === true) {
142			$error = 'Classes that implement interface %s do not support the method %s(). See %s';
143			$data = array(
144			$interface,
145			$tokens[$funcNamePos]['content'],
146			$this->unsupportedMethods[$lcInterface][$funcName],
147			);
148			$phpcsFile->addError($error, $funcNamePos, 'UnsupportedMethod', $data);
			0 ignored issues – show Security Bug introduced 2016-08-04 08:07 UTC by Report Bug Copy Issue Report It seems like `$funcNamePos` defined by `$phpcsFile->findNext(T_STRING, $nextFunc)` on line `138` can also be of type `false`; however, `PHP_CodeSniffer_File::addError()` does only seem to accept `integer`, did you maybe forget to handle an error condition? This check looks for type mismatches where the missing type is `false`. This is usually indicative of an error condtion. Consider the follow example <?php function getDate($date) { if ($date !== null) { return new DateTime($date); } return false; } This function either returns a new `DateTime` object or false, if there was an error. This is a typical pattern in PHP programming to show that an error has occurred without raising an exception. The calling code should check for this returned `false` before passing on the value to another function or method that may not be able to handle a `false`. Loading history...
149			}
150			}
151			}
152			}
153
154			}//end process()
155
156
157			/**
158			* Generates the error or warning for this sniff.
159			*
160			* @param PHP_CodeSniffer_File $phpcsFile The file being scanned.
161			* @param int $stackPtr The position of the function
162			* in the token array.
163			* @param string $interface The name of the interface.
164			* @param string $pattern The pattern used for the match.
165			*
166			* @return void
167			*/
168			protected function addError($phpcsFile, $stackPtr, $interface, $pattern=null)
169			{
170			if ($pattern === null) {
171			$pattern = strtolower($interface);
172			}
173
174			$error = '';
175
176			$isError = false;
177			foreach ($this->newInterfaces[$pattern] as $version => $present) {
178			if ($this->supportsBelow($version)) {
179			if ($present === false) {
180			$isError = true;
181			$error .= 'not present in PHP version ' . $version . ' or earlier';
182			}
183			}
184			}
185
186			if (strlen($error) > 0) {
187			$error = 'The built-in interface ' . $interface . ' is ' . $error;
188
189			if ($isError === true) {
190			$phpcsFile->addError($error, $stackPtr);
191			} else {
192			$phpcsFile->addWarning($error, $stackPtr);
193			}
194			}
195
196			}//end addError()
197
198			}//end class
199

wimg / PHPCompatibility

Push — master ( a0c8a1...aee42a )

process() D

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Available Fixes

Duplication Side-by-Side

Filter issues like