consolidation / annotated-command

src/Parser/Internal/TagFactory.php

Doc Comments +3 added lines

@@ -21,6 +21,9 @@
         $this->tags = [];
     }
 
+    /**
+     * @param string $line
+     */
     public function parseLine($line)
     {
         if (DocblockTag::isTag($line)) {

src/Parser/Internal/BespokeDocBlockParser.php

Doc Comments +3 added lines

@@ -230,6 +230,9 @@
         return !empty($nextLine);
     }
 
+    /**
+     * @param DocblockTag[] $tags
+     */
     protected function processAllTags($tags)
     {
         // Iterate over all of the tags, and process them as necessary.

Spacing +8 added lines, -8 removed lines

@@ -62,7 +62,7 @@ 
     protected function processCommandTag($tag)
     {
         if (!$tag->hasWordAndDescription($matches)) {
-            throw new \Exception('Could not determine command name from tag ' . (string)$tag);
+            throw new \Exception('Could not determine command name from tag '.(string)$tag);
         }
         $commandName = $matches['word'];
         $this->commandInfo->setName($commandName);
@@ -89,7 +89,7 @@ 
     protected function processArgumentTag($tag)
     {
         if (!$tag->hasVariable($matches)) {
-            throw new \Exception('Could not determine argument name from tag ' . (string)$tag);
+            throw new \Exception('Could not determine argument name from tag '.(string)$tag);
         }
         if ($matches['variable'] == $this->optionParamName()) {
             return;
@@ -103,7 +103,7 @@ 
     protected function processOptionTag($tag)
     {
         if (!$tag->hasVariable($matches)) {
-            throw new \Exception('Could not determine option name from tag ' . (string)$tag);
+            throw new \Exception('Could not determine option name from tag '.(string)$tag);
         }
         $this->addOptionOrArgumentTag($tag, $this->commandInfo->options(), $matches['variable'], $matches['description']);
     }
@@ -122,7 +122,7 @@ 
     protected function processDefaultTag($tag)
     {
         if (!$tag->hasVariable($matches)) {
-            throw new \Exception('Could not determine parameter name for default value from tag ' . (string)$tag);
+            throw new \Exception('Could not determine parameter name for default value from tag '.(string)$tag);
         }
         $variableName = $matches['variable'];
         $defaultValue = $this->interpretDefaultValue($matches['description']);
@@ -143,7 +143,7 @@ 
     {
         $lines = explode("\n", $tag->getContent());
         $usage = trim(array_shift($lines));
-        $description = static::removeLineBreaks(implode("\n", array_map(function ($line) {
+        $description = static::removeLineBreaks(implode("\n", array_map(function($line) {
             return trim($line);
         }, $lines)));
 
@@ -166,7 +166,7 @@ 
         // The return type might be a variable -- '$this'. It will
         // usually be a type, like RowsOfFields, or \Namespace\RowsOfFields.
         if (!$tag->hasVariableAndDescription($matches)) {
-            throw new \Exception('Could not determine return type from tag ' . (string)$tag);
+            throw new \Exception('Could not determine return type from tag '.(string)$tag);
         }
         // Look at namespace and `use` statments to make returnType a fqdn
         $returnType = $matches['variable'];
@@ -216,7 +216,7 @@ 
         // Trim all of the lines individually.
         $lines =
             array_map(
-                function ($line) {
+                function($line) {
                     return trim($line);
                 },
                 $lines
@@ -225,7 +225,7 @@ 
         // Everything up to the first blank line goes in the description.
         $description = array_shift($lines);
         while ($this->nextLineIsNotEmpty($lines)) {
-            $description .= ' ' . array_shift($lines);
+            $description .= ' '.array_shift($lines);
         }
 
         // Everything else goes in the help.

src/Parser/Internal/FullyQualifiedClassCache.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
         // If there is no 'use' statement for the className, then
         // generate an effective classname from the namespace
         if (!isset($this->classCache[$filename][$className])) {
-            $this->classCache[$filename][$className] = $namespaceName . '\\' . $className;
+            $this->classCache[$filename][$className] = $namespaceName.'\\'.$className;
         }
 
         fclose($handle);

src/CommandFileDiscovery.php

Spacing +7 added lines, -7 removed lines

@@ -151,8 +151,8 @@ 
      */
     public function ignoreNamespacePart($base, $ignore)
     {
-        $replacementPart = '\\' . $base . '\\';
-        $ignoredPart = $replacementPart . $ignore . '\\';
+        $replacementPart = '\\'.$base.'\\';
+        $ignoredPart = $replacementPart.$ignore.'\\';
         $this->strippedNamespaces[$ignoredPart] = $replacementPart;
 
         return $this;
@@ -255,7 +255,7 @@ 
         // Strip out any part of the namespace the client did not want.
         // @see CommandFileDiscovery::ignoreNamespacePart
         return array_map(
-            function ($fqcn) {
+            function($fqcn) {
                 return str_replace(
                     array_keys($this->strippedNamespaces),
                     array_values($this->strippedNamespaces),
@@ -317,7 +317,7 @@ 
      */
     protected function getSearchDepth()
     {
-        return $this->searchDepth <= 0 ? '== 0' : '<= ' . $this->searchDepth;
+        return $this->searchDepth <= 0 ? '== 0' : '<= '.$this->searchDepth;
     }
 
     /**
@@ -415,7 +415,7 @@ 
         return $this->joinParts(
             '\\',
             $namespaceParts,
-            function ($item) {
+            function($item) {
                 return !is_numeric($item) && !empty($item);
             }
         );
@@ -434,7 +434,7 @@ 
         $path = $this->joinParts(
             '/',
             $pathParts,
-            function ($item) {
+            function($item) {
                 return !empty($item);
             }
         );
@@ -451,7 +451,7 @@ 
     protected function joinParts($delimiter, $parts, $filterFunction)
     {
         $parts = array_map(
-            function ($item) use ($delimiter) {
+            function($item) use ($delimiter) {
                 return rtrim($item, $delimiter);
             },
             $parts