edmondscommerce / phpqa

configDefaults/magento2/phpstan-magento2-bootstrap.php

Spacing +2 added lines, -2 removed lines

@@ -2,12 +2,12 @@
 
 /* Find bootstrap path */
 $rootPath = realpath(dirname(__FILE__));
-while (!file_exists($rootPath . '/app/bootstrap.php') && $rootPath !== '/') {
+while (!file_exists($rootPath.'/app/bootstrap.php') && $rootPath !== '/') {
     $rootPath = realpath(dirname($rootPath));
 }
 
 /* Include Magento bootstrap file */
-require_once $rootPath . '/app/bootstrap.php';
+require_once $rootPath.'/app/bootstrap.php';
 
 /* Create git hook class autoloader */
 $_git_hook_loaded_class = [];

src/Markdown/LinksChecker.php

Indentation +8 added lines, -8 removed lines

@@ -175,14 +175,14 @@
         #$start          = microtime(true);
         #fwrite(STDERR, "\n".'Validating link: '.$href);
         $context = stream_context_create([
-                                             'http' => [
-                                                 'method'           => 'HEAD',
-                                                 'protocol_version' => 1.1,
-                                                 'header'           => [
-                                                     'Connection: close',
-                                                 ],
-                                             ],
-                                         ]);
+                                                'http' => [
+                                                    'method'           => 'HEAD',
+                                                    'protocol_version' => 1.1,
+                                                    'header'           => [
+                                                        'Connection: close',
+                                                    ],
+                                                ],
+                                            ]);
         $result  = null;
         try {
             $headers = get_headers($href, 0, $context);

Spacing +10 added lines, -10 removed lines

@@ -28,7 +28,7 @@ 
      */
     private static function getMainReadme(string $projectRootDirectory): string
     {
-        $path = $projectRootDirectory . '/README.md';
+        $path = $projectRootDirectory.'/README.md';
         if (!is_file($path)) {
             throw new \RuntimeException(
                 "\n\nYou have no README.md file in your project"
@@ -48,7 +48,7 @@ 
     private static function getDocsFiles(string $projectRootDirectory): array
     {
         $files = [];
-        $dir   = $projectRootDirectory . '/docs';
+        $dir   = $projectRootDirectory.'/docs';
         if (!is_dir($dir)) {
             return $files;
         }
@@ -120,15 +120,15 @@ 
         $start = \rtrim($projectRootDirectory, '/');
         if ($path[0] !== '/' || 0 === \strpos($path, './')) {
             $relativeSubdirs = \preg_replace(
-                '%^' . $projectRootDirectory . '%',
+                '%^'.$projectRootDirectory.'%',
                 '',
                 \dirname($file)
             );
             if ($relativeSubdirs !== null) {
-                $start           .= '/' . \rtrim($relativeSubdirs, '/');
+                $start .= '/'.\rtrim($relativeSubdirs, '/');
             }
         }
-        $realpath = \realpath($start . '/' . $path);
+        $realpath = \realpath($start.'/'.$path);
         if (false === $realpath) {
             $errors[] = \sprintf("\nBad link for \"%s\" to \"%s\"\n", $link[1], $link[2]);
             $return   = 1;
@@ -150,14 +150,14 @@ 
         $files                = static::getFiles($projectRootDirectory);
         foreach ($files as $file) {
             $relativeFile = str_replace($projectRootDirectory, '', $file);
-            $title        = "\n$relativeFile\n" . str_repeat('-', strlen($relativeFile)) . "\n";
+            $title        = "\n$relativeFile\n".str_repeat('-', strlen($relativeFile))."\n";
             $errors       = [];
             $links        = static::getLinks($file);
             foreach ($links as $link) {
                 static::checkLink($projectRootDirectory, $link, $file, $errors, $return);
             }
             if ([] !== $errors) {
-                echo $title . implode('', $errors);
+                echo $title.implode('', $errors);
             }
         }
 
@@ -187,11 +187,11 @@ 
                                                  ],
                                              ],
                                          ]);
-        $result  = null;
+        $result = null;
         try {
             $headers = get_headers($href, 0, $context);
             if (false === $headers) {
-                throw new \RuntimeException('Failed getting headers for href ' . $href);
+                throw new \RuntimeException('Failed getting headers for href '.$href);
             }
             foreach ($headers as $header) {
                 if (false !== strpos($header, ' 200 ')) {
@@ -210,7 +210,7 @@ 
             $href,
             var_export($result, true)
         );
-        $return   = 1;
+        $return = 1;
         #$time     = round(microtime(true) - $start, 2);
         #fwrite(STDERR, "\n".'Failed ('.$time.' seconds): '.$href);
     }

src/Helper.php

Spacing +2 added lines, -2 removed lines

@@ -38,14 +38,14 @@
      */
     public static function getComposerJsonDecoded(string $path = null): array
     {
-        $path     = $path ?? self::getProjectRootDirectory() . '/composer.json';
+        $path     = $path ?? self::getProjectRootDirectory().'/composer.json';
         $contents = (string)\file_get_contents($path);
         if ('' === $contents) {
             throw new \RuntimeException('composer.json is empty');
         }
         $decoded = \json_decode($contents, true);
         if (JSON_ERROR_NONE !== \json_last_error()) {
-            throw new \RuntimeException('Failed loading composer.json: ' . \json_last_error_msg());
+            throw new \RuntimeException('Failed loading composer.json: '.\json_last_error_msg());
         }
 
         return $decoded;

src/PHPUnit/TestDox/TestResult.php

Spacing +3 added lines, -3 removed lines

@@ -74,7 +74,7 @@ 
     ): void {
         $this->testSuccesful                = false;
         $this->symbol                       = $symbol;
-        $this->additionalInformation        .= "\n" . $additionalInformation;
+        $this->additionalInformation .= "\n".$additionalInformation;
         $this->additionalInformationVerbose = $additionalInformationVerbose;
     }
 
@@ -94,7 +94,7 @@ 
             ),
             $this->symbol,
             $this->testMethod,
-            $verbose ? ' ' . $this->getFormattedRuntime() : '',
+            $verbose ? ' '.$this->getFormattedRuntime() : '',
             $this->getFormattedAdditionalInformation($verbose)
         );
     }
@@ -138,7 +138,7 @@ 
             \implode(
                 "\n",
                 \array_map(
-                    function (string $text) {
+                    function(string $text) {
                         return \sprintf('   │ %s', $text);
                     },
                     \explode("\n", $this->additionalInformation)

src/PHPUnit/CheckAnnotations.php

Spacing +8 added lines, -8 removed lines

@@ -43,12 +43,12 @@ 
     {
         if (!is_dir($pathToTestsDirectory)) {
             throw new \InvalidArgumentException(
-                '$pathToTestsDirectory "' . $pathToTestsDirectory . '" does not exist"'
+                '$pathToTestsDirectory "'.$pathToTestsDirectory.'" does not exist"'
             );
         }
-        $this->largePath  = $pathToTestsDirectory . '/Large';
-        $this->mediumPath = $pathToTestsDirectory . '/Medium';
-        $this->smallPath  = $pathToTestsDirectory . '/Small';
+        $this->largePath  = $pathToTestsDirectory.'/Large';
+        $this->mediumPath = $pathToTestsDirectory.'/Medium';
+        $this->smallPath  = $pathToTestsDirectory.'/Small';
         $this->checkLarge();
         $this->checkMedium();
         $this->checkSmall();
@@ -102,7 +102,7 @@ 
             return;
         }
 
-        $matches  = [];
+        $matches = [];
         preg_match_all(
             <<<REGEXP
 %(?<docblock>/\*(?:[^*]|\n|(?:\*(?:[^/]|\n)))*\*/|\n)\s+?public\s+?function\s+?(?<method>.+?)\(%
@@ -119,7 +119,7 @@ 
         foreach ($matches['method'] as $key => $method) {
             $docblock = $matches['docblock'][$key];
             /* Found the annotation - continue */
-            if (false !== \strpos($docblock, '@' . $annotation)) {
+            if (false !== \strpos($docblock, '@'.$annotation)) {
                 continue;
             }
             /* No @test annotation found & method not beginning test =  not a test, so continue */
@@ -127,7 +127,7 @@ 
                 continue;
             }
             $this->errors[$fileInfo->getFilename()][] =
-                'Failed finding @' . $annotation . ' for method: ' . $method;
+                'Failed finding @'.$annotation.' for method: '.$method;
         }
     }
 
@@ -156,7 +156,7 @@ 
             return false;
         }
         $docBlock = array_shift($matches['docblock']);
-        return strpos($docBlock, '@' . $annotation) !== false;
+        return strpos($docBlock, '@'.$annotation) !== false;
     }
 
 

src/Psr4Validator.php

Indentation +1 added lines, -1 removed lines

@@ -178,7 +178,7 @@
         $invalidPathMessage = "Namespace root '$namespaceRoot'\ncontains a path '$path'\nwhich doesn't exist\n";
         if (stripos($absPathRoot, "Magento") !== false) {
             $invalidPathMessage .= 'Magento\'s composer includes this by default, '
-                                   . 'it should be removed from the psr-4 section';
+                                    . 'it should be removed from the psr-4 section';
         }
         $this->missingPaths[$path] = $invalidPathMessage;
     }

Spacing +4 added lines, -4 removed lines

@@ -116,14 +116,14 @@ 
             );
         }
 
-        return rtrim($namespaceRoot . $relativeNs, '\\');
+        return rtrim($namespaceRoot.$relativeNs, '\\');
     }
 
     private function getActualNamespace(\SplFileInfo $fileInfo): string
     {
         $contents = \file_get_contents($fileInfo->getPathname());
         if (false === $contents) {
-            throw new \RuntimeException('Failed getting file contents for ' . $fileInfo->getPathname());
+            throw new \RuntimeException('Failed getting file contents for '.$fileInfo->getPathname());
         }
         \preg_match('%namespace\s+?([^;]+)%', $contents, $matches);
         if ([] === $matches) {
@@ -146,7 +146,7 @@ 
             $realPath,
             \RecursiveDirectoryIterator::SKIP_DOTS
         );
-        $iterator          = new \RecursiveIteratorIterator(
+        $iterator = new \RecursiveIteratorIterator(
             $directoryIterator,
             \RecursiveIteratorIterator::SELF_FIRST
         );
@@ -202,7 +202,7 @@ 
                     $paths = [$paths];
                 }
                 foreach ($paths as $path) {
-                    $absPathRoot     = $this->pathToProjectRoot . '/' . $path;
+                    $absPathRoot     = $this->pathToProjectRoot.'/'.$path;
                     $realAbsPathRoot = \realpath($absPathRoot);
                     if (false === $realAbsPathRoot) {
                         $this->addMissingPathError($path, $namespaceRoot, $absPathRoot);

src/PHPUnit/TestDox/CliTestDoxPrinter.php

Spacing +5 added lines, -5 removed lines

@@ -95,7 +95,7 @@ 
         }
 
         $this->currentTestResult = new \EdmondsCommerce\PHPQA\PHPUnit\TestDox\TestResult(
-            function (string $color, string $buffer) {
+            function(string $color, string $buffer) {
                 return $this->formatWithColor($color, $buffer);
             },
             $className,
@@ -128,8 +128,8 @@ 
     {
         if (null === $this->currentTestResult) {
             throw new \RuntimeException(
-                'Error in ' . __METHOD__ . ': '
-                . $t->getMessage() . "\n\n" . $t->getTraceAsString(),
+                'Error in '.__METHOD__.': '
+                . $t->getMessage()."\n\n".$t->getTraceAsString(),
                 $t->getCode(),
                 $t
             );
@@ -203,7 +203,7 @@ 
 
     protected function printHeader(): void
     {
-        $this->write("\n" . Timer::resourceUsage() . "\n\n");
+        $this->write("\n".Timer::resourceUsage()."\n\n");
     }
 
     private function printNonSuccessfulTestsSummary(int $numberOfExecutedTests): void
@@ -243,7 +243,7 @@ 
             $this->write(
                 $this->formatWithColor(
                     'fg-yellow',
-                    "\n" . self::SYMBOL_SKIP . ' Skipped Tests: ' . $skippedTests . "\n\n"
+                    "\n".self::SYMBOL_SKIP.' Skipped Tests: '.$skippedTests."\n\n"
                 )
             );
         }