Code style fixes.

Author: corpsee

src/CheckerApplication.php

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace PhpDocChecker;
 
@@ -20,9 +20,6 @@ class CheckerApplication extends Application
 {
     /**
      * Override the default command name logic and return our check command.
-     *
-     * @param InputInterface $input
-     * @return string
      */
     protected function getCommandName(InputInterface $input): string
     {

src/CheckerCommand.php

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace PhpDocChecker;
 
@@ -29,7 +29,7 @@ class CheckerCommand extends Command
     protected array $warnings = [];
 
     protected array $exclude = [];
-    
+
     protected array $files = [];
 
     protected OutputInterface $output;
@@ -60,11 +60,6 @@ protected function configure(): void
 
     /**
      * Execute the actual docblock checker.
-     *
-     * @param InputInterface $input
-     * @param OutputInterface $output
-     *
-     * @return int
      */
     protected function execute(InputInterface $input, OutputInterface $output): int
     {
@@ -92,7 +87,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         if (!\is_null($exclude)) {
             $this->exclude = \array_map('trim', \explode(',', $exclude));
         }
-        
+
         // Set up files:
         if (!\is_null($files)) {
             $this->files = \array_map('trim', \explode(',', $files));
@@ -218,7 +213,6 @@ protected function execute(InputInterface $input, OutputInterface $output): int
     /**
      * Iterate through a directory and check all of the PHP files within it.
      *
-     * @param string $path
      * @param string[] $workList
      */
     protected function processDirectory(string $path = '', array &$workList = []): void
@@ -232,7 +226,7 @@ protected function processDirectory(string $path = '', array &$workList = []): v
 
             $itemPath = $path . $item->getFilename();
 
-            if (\in_array($itemPath, $this->exclude)) {
+            if (\in_array($itemPath, $this->exclude, true)) {
                 continue;
             }
 
@@ -245,11 +239,10 @@ protected function processDirectory(string $path = '', array &$workList = []): v
             }
         }
     }
-    
+
     /**
      * Iterate through the files and check them out
      *
-     * @param string $path
      * @param string[] $files
      * @param string[] $workList
      */
@@ -258,7 +251,7 @@ protected function processFiles(string $path = '', array $files = [], array &$wo
         foreach ($files as $item) {
             $itemPath = $path . $item;
 
-            if (\in_array($itemPath, $this->exclude)) {
+            if (\in_array($itemPath, $this->exclude, true)) {
                 continue;
             }
 
@@ -271,16 +264,14 @@ protected function processFiles(string $path = '', array $files = [], array &$wo
     /**
      * Check a specific PHP file for errors.
      *
-     * @param string $file
      *
-     * @return array
      */
     protected function processFile(string $file): array
     {
         $result = $this->checkerFileProcessor->processFile($file);
 
         $this->errors   = \array_merge($this->errors, $result['errors']);
-        $this->warnings = \array_merge($this->warnings, $result['warnings']);;
+        $this->warnings = \array_merge($this->warnings, $result['warnings']);
 
         if (0 === \count($result['errors'])) {
             $this->passed += 1;

src/CheckerFileProcessor.php

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace PhpDocChecker;
 
@@ -48,10 +48,6 @@ public function __construct(
 
     /**
      * Check a specific PHP file for errors.
-     *
-     * @param string $file
-     *
-     * @return array
      */
     public function processFile(string $file): array
     {

src/DocBlockParser.php

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace PhpDocChecker;
 
@@ -95,7 +95,7 @@ protected function parseComment(string $comment): void
         $comment = \preg_split('/\r?\n\r?/', $comment);
 
         // Trim asterisks and whitespace from the beginning and whitespace from the end of lines
-        $comment = \array_map(function($line) {
+        $comment = \array_map(function ($line) {
             return \ltrim(\rtrim($line), "* \t\n\r\0\x0B");
         }, $comment);
 
@@ -106,7 +106,7 @@ protected function parseComment(string $comment): void
             if (self::isTagged($line)) {
                 $b++;
                 $blocks[] = [];
-            } else if($b == -1) {
+            } elseif ($b == -1) {
                 $b = 0;
                 $blocks[] = [];
             }
@@ -141,7 +141,7 @@ protected function parseComment(string $comment): void
                         $parts
                     );
 
-                    if (isset($mapped['var']) && \substr( $mapped['var'], 0, 3) === '...') {
+                    if (isset($mapped['var']) && \substr($mapped['var'], 0, 3) === '...') {
                         $mapped['var'] = substr($mapped['var'], 3);
                     }
                     // Store as a mapped array
@@ -158,8 +158,6 @@ protected function parseComment(string $comment): void
      * Whether or not a docblock contains a given @tag.
      *
      * @param string $tag The name of the @tag to check for
-     *
-     * @return bool
      */
     public function hasTag(string $tag): bool
     {
@@ -168,10 +166,6 @@ public function hasTag(string $tag): bool
 
     /**
      * The value of a tag
-     *
-     * @param String $tag
-     *
-     * @return array|null
      */
     public function tag(string $tag): ?array
     {
@@ -181,10 +175,7 @@ public function tag(string $tag): ?array
     /**
      * The value of a tag (concatenated for multiple values)
      *
-     * @param string $tag
      * @param string $sep The separator for concatenating
-     *
-     * @return string|null
      */
     public function tagImplode(string $tag, string $sep = ' '): ?string
     {
@@ -194,8 +185,6 @@ public function tagImplode(string $tag, string $sep = ' '): ?string
     /**
      * The value of a tag (merged recursively)
      *
-     * @param string $tag
-     *
      * @return array
      */
     public function tagMerge(string $tag): ?array
@@ -205,10 +194,6 @@ public function tagMerge(string $tag): ?array
 
     /**
      * Whether or not a string begins with a @tag
-     *
-     * @param string $str
-     *
-     * @return bool
      */
     public static function isTagged(string $str): bool
     {
@@ -217,10 +202,6 @@ public static function isTagged(string $str): bool
 
     /**
      * The tag at the beginning of a string
-     *
-     * @param string $str
-     *
-     * @return string
      */
     public static function strTag(string $str): string
     {

src/FileProcessor.php

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace PhpDocChecker;
 
@@ -32,8 +32,6 @@ class FileProcessor
 
     /**
      * Load and parse a PHP file.
-     *
-     * @param string $file
      */
     public function __construct(string $file)
     {
@@ -43,13 +41,12 @@ public function __construct(string $file)
             $parser = (new ParserFactory())->create(ParserFactory::PREFER_PHP7);
             $stmts = $parser->parse(file_get_contents($file));
             $this->processStatements($stmts);
-        } catch (\Exception $ex) {}
+        } catch (\Exception $ex) {
+        }
     }
 
     /**
      * Return a list of class details from the given PHP file.
-     *
-     * @return array
      */
     public function getClasses(): array
     {
@@ -58,8 +55,6 @@ public function getClasses(): array
 
     /**
      * Return a list of method details from the given PHP file.
-     *
-     * @return array
      */
     public function getMethods(): array
     {
@@ -69,9 +64,6 @@ public function getMethods(): array
     /**
      * Looks for class definitions, and then within them method definitions, docblocks, etc.
      *
-     * @param array  $statements
-     * @param string $prefix
-     *
      * @return mixed
      */
     protected function processStatements(array $statements, string $prefix = '')
@@ -197,11 +189,6 @@ protected function processStatements(array $statements, string $prefix = '')
 
     /**
      * Find and parse a docblock for a given class or method.
-     *
-     * @param Stmt  $stmt
-     * @param array $uses
-     *
-     * @return array|null
      */
     protected function getDocblock(Stmt $stmt, array $uses = []): ?array
     {
@@ -220,11 +207,6 @@ protected function getDocblock(Stmt $stmt, array $uses = []): ?array
 
     /**
      * Use Paul Scott's docblock parser to parse a docblock, then return the relevant parts.
-     *
-     * @param string $text
-     * @param array  $uses
-     *
-     * @return array
      */
     protected function processDocblock(string $text, array $uses = []): array
     {

tests/src/CheckerFileProcessorTest.php

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Tests\PhpDocChecker;