consolidation / annotated-command

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/Parser/Internal/BespokeDocBlockParser.php

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.

Doc Comments +6 added lines

@@ -144,6 +144,9 @@ 
         }
     }
 
+    /**
+     * @param string $description
+     */
     protected function splitOutDefault($description)
     {
         if (!preg_match('#(.*)(Default: *)(.*)#', trim($description), $matches)) {
@@ -283,6 +286,9 @@ 
         return !empty($nextLine);
     }
 
+    /**
+     * @param DocblockTag[] $tags
+     */
     protected function processAllTags($tags)
     {
         // Iterate over all of the tags, and process them as necessary.

src/CommandFileDiscovery.php

Spacing +7 added lines, -7 removed lines

@@ -154,9 +154,9 @@ 
     {
         $replacementPart = '\\';
         if (!empty($base)) {
-            $replacementPart .= $base . '\\';
+            $replacementPart .= $base.'\\';
         }
-        $ignoredPart = $replacementPart . $ignore . '\\';
+        $ignoredPart = $replacementPart.$ignore.'\\';
         $this->strippedNamespaces[$ignoredPart] = $replacementPart;
 
         return $this;
@@ -259,7 +259,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),
@@ -321,7 +321,7 @@ 
      */
     protected function getSearchDepth()
     {
-        return $this->searchDepth <= 0 ? '== 0' : '<= ' . $this->searchDepth;
+        return $this->searchDepth <= 0 ? '== 0' : '<= '.$this->searchDepth;
     }
 
     /**
@@ -419,7 +419,7 @@ 
         return $this->joinParts(
             '\\',
             $namespaceParts,
-            function ($item) {
+            function($item) {
                 return !is_numeric($item) && !empty($item);
             }
         );
@@ -438,7 +438,7 @@ 
         $path = $this->joinParts(
             '/',
             $pathParts,
-            function ($item) {
+            function($item) {
                 return !empty($item);
             }
         );
@@ -455,7 +455,7 @@ 
     protected function joinParts($delimiter, $parts, $filterFunction)
     {
         $parts = array_map(
-            function ($item) use ($delimiter) {
+            function($item) use ($delimiter) {
                 return rtrim($item, $delimiter);
             },
             $parts

src/Options/AlterOptionsCommandEvent.php

Doc Comments +3 added lines

@@ -64,6 +64,9 @@
         }
     }
 
+    /**
+     * @param Command $command
+     */
     public function findAndAddHookOptions($command)
     {
         if (!$command instanceof AnnotatedCommand) {

src/Hooks/Dispatchers/StatusDeterminerHookDispatcher.php

Doc Comments +3 added lines

@@ -39,6 +39,9 @@
         }
     }
 
+    /**
+     * @param callable $determiner
+     */
     protected function callDeterminer($determiner, $result)
     {
         if ($determiner instanceof StatusDeterminerInterface) {

src/Input/StdinHandler.php

Doc Comments +3 added lines

@@ -210,6 +210,9 @@
         return !empty($path) && ($path != '-');
     }
 
+    /**
+     * @param string $optionOrArg
+     */
     protected function getOptionOrArg(InputInterface $input, $optionOrArg)
     {
         if ($input->hasOption($optionOrArg)) {

src/AnnotationData.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
         if (is_array($data)) {
             $this->offsetSet($key, array_merge($data, $value));
         } elseif (is_scalar($data)) {
-            $this->offsetSet($key, $data . $value);
+            $this->offsetSet($key, $data.$value);
         }
         return $this;
     }

src/ParameterInjection.php

Doc Comments +6 added lines

@@ -13,11 +13,17 @@
         $this->register('Symfony\Component\Console\Output\OutputInterface', $this);
     }
 
+    /**
+     * @param string $interfaceName
+     */
     public function register($interfaceName, ParameterInjector $injector)
     {
         $this->injectors[$interfaceName] = $injector;
     }
 
+    /**
+     * @param CommandData $commandData
+     */
     public function args($commandData)
     {
         return array_merge(