Extend Phan analysis

Author: mmoll

Diff for: .phan/config.php

@@ -22,6 +22,7 @@
     'directory_list' => [
         'MO4',
         'tests',
+        'vendor/squizlabs/php_codesniffer',
     ],
 
     // A directory list that defines files that will be excluded

Diff for: .travis.yml

@@ -31,7 +31,7 @@ script:
   - vendor/bin/phpcs # Stylecheck against PHPCS's ruleset
   - vendor/bin/phpstan analyse --no-progress # Run PHPStan
   - vendor/bin/psalm --show-info=false # run psalm
-  - if [ "${DEPENDENCIES}" = "highest" ]; then vendor/bin/phan -i; fi; # Run phan
+  - if [ "${DEPENDENCIES}" = "highest" ]; then vendor/bin/phan; fi; # Run phan
   - if [ "${TRAVIS_PHP_VERSION}" != "7.3" ]; then php -d zend_extension=xdebug.so vendor/bin/phpunit --coverage-clover=coverage.xml; fi; # Generate Code coverage report
 
 jobs:

Diff for: MO4/Library/PregLibrary.php

@@ -34,6 +34,8 @@ class PregLibrary
      * @return string[]|array[]
      *
      * @throws RuntimeException
+     *
+     * @psalm-suppress ArgumentTypeCoercion
      */
     public static function mo4_preg_split($pattern, $subject, $limit=-1, $flags=0): array
     {

Diff for: MO4/Sniffs/Arrays/ArrayDoubleArrowAlignmentSniff.php

@@ -34,8 +34,9 @@ class ArrayDoubleArrowAlignmentSniff implements Sniff
      * @var array
      */
     protected  $arrayTokens = [
-        T_ARRAY,
+        // @phan-suppress-next-line PhanUndeclaredConstant
         T_OPEN_SHORT_ARRAY,
+        T_ARRAY,
     ];
 
 

Diff for: MO4/Sniffs/Arrays/MultiLineArraySniff.php

@@ -31,8 +31,9 @@ class MultiLineArraySniff implements Sniff
      * @var array
      */
     protected  $arrayTokens = [
-        T_ARRAY,
+        // @phan-suppress-next-line PhanUndeclaredConstant
         T_OPEN_SHORT_ARRAY,
+        T_ARRAY,
     ];
 
 

Diff for: MO4/Sniffs/Commenting/PropertyCommentSniff.php

@@ -218,7 +218,7 @@ protected function processTokenWithinScope(
                 $commentStart = $phpcsFile->findPrevious(
                     T_COMMENT,
                     $commentEnd,
-                    null,
+                    0,
                     true
                 );
                 $phpcsFile->addError(

Diff for: MO4/Sniffs/Formatting/AlphabeticalUseStatementsSniff.php

@@ -234,7 +234,7 @@ private function checkIsNonImportUse(File $phpcsFile, int $stackPtr): bool
         $prev = $phpcsFile->findPrevious(
             PHP_CodeSniffer_Tokens::$emptyTokens,
             ($stackPtr - 1),
-            null,
+            0,
             true,
             null,
             true

Diff for: MO4/Sniffs/Formatting/UnnecessaryNamespaceUsageSniff.php

@@ -245,7 +245,7 @@ protected function getUseStatements(
             $aliasNamePtr = $phpcsFile->findPrevious(
                 PHP_CodeSniffer_Tokens::$emptyTokens,
                 ($useEnd - 1),
-                null,
+                0,
                 true
             );
 

Diff for: composer.json

@@ -33,6 +33,6 @@
         "dealerdirect/phpcodesniffer-composer-installer": "~0.7",
         "escapestudios/symfony2-coding-standard": "^3.10.0",
         "slevomat/coding-standard": "^6.2.0",
-        "squizlabs/php_codesniffer": "^3.5.4"
+        "squizlabs/php_codesniffer": "^3.5.8"
     }
 }