consolidation / annotated-command

src/Hooks/HookManager.php

Doc Comments +26 added lines, -1 removed lines

@@ -56,7 +56,7 @@ 
     /**
      * Add a hook
      *
-     * @param mixed $callback The callback function to call
+     * @param callable $callback The callback function to call
      * @param string   $hook     The name of the hook to add
      * @param string   $name     The name of the command to hook
      *   ('*' for all)
@@ -329,6 +329,9 @@ 
         return $result;
     }
 
+    /**
+     * @param string[] $names
+     */
     protected function getCommandEventHooks($names)
     {
         return $this->getHooks($names, self::COMMAND_EVENT);
@@ -384,6 +387,7 @@ 
      * @param string|array $names The name of the function being hooked.
      * @param string $hook The specific hook name (e.g. alter)
      * @param string[] $stages The stages to apply hooks at (e.g. pre, post)
+     * @param AnnotationData $annotationData
      *
      * @return callable[]
      */
@@ -434,6 +438,10 @@ 
         return [];
     }
 
+    /**
+     * @param callable $provider
+     * @param InputInterface $input
+     */
     protected function callInjectConfigurationHook($provider, $input, AnnotationData $annotationData)
     {
         if ($provider instanceof InitializeHookInterface) {
@@ -444,6 +452,11 @@ 
         }
     }
 
+    /**
+     * @param callable $interactor
+     * @param InputInterface $input
+     * @param OutputInterface $output
+     */
     protected function callInteractor($interactor, $input, $output, AnnotationData $annotationData)
     {
         if ($interactor instanceof InteractorInterface) {
@@ -454,6 +467,9 @@ 
         }
     }
 
+    /**
+     * @param callable $validator
+     */
     protected function callValidator($validator, $args, AnnotationData $annotationData)
     {
         // TODO: Adding AnnotationData to ValidatorInterface would be
@@ -467,6 +483,9 @@ 
         }
     }
 
+    /**
+     * @param callable $processor
+     */
     protected function callProcessor($processor, $result, $args, AnnotationData $annotationData)
     {
         $processed = null;
@@ -485,6 +504,9 @@ 
         return $result;
     }
 
+    /**
+     * @param callable $determiner
+     */
     protected function callDeterminer($determiner, $result)
     {
         if ($determiner instanceof StatusDeterminerInterface) {
@@ -495,6 +517,9 @@ 
         }
     }
 
+    /**
+     * @param callable $extractor
+     */
     protected function callExtractor($extractor, $result)
     {
         if ($extractor instanceof ExtractOutputInterface) {

Unused Use Statements -2 removed lines

@@ -4,11 +4,9 @@
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
-
 use Symfony\Component\Console\ConsoleEvents;
 use Symfony\Component\Console\Event\ConsoleCommandEvent;
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
-
 use Consolidation\AnnotatedCommand\ExitCodeInterface;
 use Consolidation\AnnotatedCommand\OutputDataInterface;
 use Consolidation\AnnotatedCommand\AnnotationData;

Spacing +2 added lines, -2 removed lines

@@ -391,7 +391,7 @@ 
     {
         $names = array_merge(
             (array)$names,
-            ($annotationData == null) ? [] : array_map(function ($item) {
+            ($annotationData == null) ? [] : array_map(function($item) {
                 return "@$item";
             }, $annotationData->keys())
         );
@@ -412,7 +412,7 @@ 
         $hooks = [];
         foreach ($stages as $stage) {
             foreach ((array)$names as $name) {
-                $hooks = array_merge($hooks, $this->getHook($name, $stage . $hook));
+                $hooks = array_merge($hooks, $this->getHook($name, $stage.$hook));
             }
         }
         return $hooks;