consolidation / annotated-command

src/Parser/CommandInfo.php

Indentation +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
     /**
      * @var boolean
      * @var string
-    */
+     */
     protected $docBlockIsParsed;
 
     /**

Spacing +2 added lines, -2 removed lines

@@ -254,8 +254,8 @@
 
     protected function convertName($camel)
     {
-        $splitter="-";
-        $camel=preg_replace('/(?!^)[[:upper:]][[:lower:]]/', '$0', preg_replace('/(?!^)[[:upper:]]+/', $splitter.'$0', $camel));
+        $splitter = "-";
+        $camel = preg_replace('/(?!^)[[:upper:]][[:lower:]]/', '$0', preg_replace('/(?!^)[[:upper:]]+/', $splitter.'$0', $camel));
         $camel = preg_replace("/$splitter/", ':', $camel, 1);
         return strtolower($camel);
     }

Doc Comments +6 added lines, -3 removed lines

@@ -245,7 +245,7 @@ 
     /**
      * Set aliases that can be used in place of the command's primary name.
      *
-     * @param string|string[] $aliases
+     * @param string $aliases
      */
     public function setAliases($aliases)
     {
@@ -283,7 +283,7 @@ 
     /**
      * Return the list of refleaction parameters.
      *
-     * @return ReflectionParameter[]
+     * @return \ReflectionParameter[]
      */
     public function getParameters()
     {
@@ -381,7 +381,7 @@ 
      * Examine the parameters of the method for this command, and
      * build a list of commandline arguements for them.
      *
-     * @return array
+     * @return DefaultsWithDescriptions
      */
     protected function determineAgumentClassifications()
     {
@@ -401,6 +401,7 @@ 
      * Examine the provided parameter, and determine whether it
      * is a parameter that will be filled in with a positional
      * commandline argument.
+     * @param DefaultsWithDescriptions $result
      */
     protected function addParameterToResult($result, $param)
     {
@@ -501,6 +502,8 @@ 
     /**
      * Given a list that might be 'a b c' or 'a, b, c' or 'a,b,c',
      * convert the data into the last of these forms.
+     * @param string $text
+     * @return string
      */
     protected static function convertListToCommaSeparated($text)
     {

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 namespace Consolidation\AnnotatedCommand\Parser;
 
-use Consolidation\AnnotatedCommand\Parser\Internal\CommandDocBlockParser;
 use Consolidation\AnnotatedCommand\Parser\Internal\CommandDocBlockParserFactory;
 use Consolidation\AnnotatedCommand\AnnotationData;
 

src/CommandProcessor.php

Unused Use Statements -3 removed lines

@@ -1,11 +1,8 @@
 <?php
 namespace Consolidation\AnnotatedCommand;
 
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Output\ConsoleOutputInterface;
-
 use Consolidation\OutputFormatters\FormatterManager;
 use Consolidation\OutputFormatters\FormatterOptions;
 use Consolidation\AnnotatedCommand\Hooks\HookManager;

Doc Comments +4 added lines, -1 removed lines

@@ -96,6 +96,9 @@ 
         return $this->processResults($names, $result, $args, $annotationData);
     }
 
+    /**
+     * @param AnnotationData $annotationData
+     */
     public function processResults($names, $result, $args, $annotationData)
     {
         return $this->hookManager()->alterResult($names, $result, $args, $annotationData);
@@ -152,7 +155,7 @@ 
      * should select the output format that is most appropriate
      * for use in scripts (e.g. to pipe to another command).
      *
-     * @return string
+     * @return boolean
      */
     protected function getFormat($options)
     {

src/AnnotatedCommandFactory.php

Spacing +2 added lines, -2 removed lines

@@ -75,7 +75,7 @@ 
         // set or setup.
         $commandMethodNames = array_filter(
             get_class_methods($classNameOrInstance) ?: [],
-            function ($m) {
+            function($m) {
                 return !preg_match('#^(_|get[A-Z]|set[A-Z])#', $m);
             }
         );
@@ -97,7 +97,7 @@ 
         return $this->createSelectedCommandsFromClassInfo(
             $commandInfoList,
             $commandFileInstance,
-            function ($commandInfo) use ($includeAllPublicMethods) {
+            function($commandInfo) use ($includeAllPublicMethods) {
                 return $includeAllPublicMethods || static::isCommandMethod($commandInfo);
             }
         );

src/Parser/Internal/CommandDocBlockParser2.php

Unused Use Statements -2 removed lines

@@ -4,8 +4,6 @@
 use phpDocumentor\Reflection\DocBlock;
 use phpDocumentor\Reflection\DocBlock\Tag\ParamTag;
 use phpDocumentor\Reflection\DocBlock\Tag\ReturnTag;
-use Consolidation\AnnotatedCommand\Parser\CommandInfo;
-use Consolidation\AnnotatedCommand\Parser\DefaultsWithDescriptions;
 
 /**
  * Given a class and method name, parse the annotations in the

src/CommandFileDiscovery.php

Spacing +3 added lines, -3 removed lines

@@ -247,7 +247,7 @@ 
      */
     protected function getSearchDepth()
     {
-        return $this->searchDepth <= 0 ? '== 0' : '<= ' . $this->searchDepth;
+        return $this->searchDepth <= 0 ? '== 0' : '<= '.$this->searchDepth;
     }
 
     /**
@@ -341,7 +341,7 @@ 
         return $this->joinParts(
             '\\',
             $namespaceParts,
-            function ($item) {
+            function($item) {
                 return !is_numeric($item) && !empty($item);
             }
         );
@@ -360,7 +360,7 @@ 
         return $this->joinParts(
             '/',
             $pathParts,
-            function ($item) {
+            function($item) {
                 return !empty($item);
             }
         );

src/Hooks/HookManager.php

Spacing +1 added lines, -1 removed lines

@@ -255,7 +255,7 @@
     {
         $names = array_merge(
             (array)$names,
-            ($annotationData == null) ? [] : array_map(function ($item) { return "@$item"; }, $annotationData->array_keys())
+            ($annotationData == null) ? [] : array_map(function($item) { return "@$item"; }, $annotationData->array_keys())
         );
         $names[] = '*';
         return array_merge(