Completed
Pull Request — master (#66)
by Greg
02:23
created
src/Hooks/HookManager.php 1 patch
Doc Comments   +39 added lines, -1 removed lines patch added patch discarded remove patch
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
     /**
61 61
      * Add a hook
62 62
      *
63
-     * @param mixed $callback The callback function to call
63
+     * @param callable $callback The callback function to call
64 64
      * @param string   $hook     The name of the hook to add
65 65
      * @param string   $name     The name of the command to hook
66 66
      *   ('*' for all)
@@ -73,11 +73,17 @@  discard block
 block discarded – undo
73 73
         $this->hooks[$name][$hook][] = $callback;
74 74
     }
75 75
 
76
+    /**
77
+     * @param \Consolidation\AnnotatedCommand\Parser\CommandInfo $commandInfo
78
+     */
76 79
     public function recordHookOptions($commandInfo, $name)
77 80
     {
78 81
         $this->hookOptions[$name][] = $commandInfo;
79 82
     }
80 83
 
84
+    /**
85
+     * @param \Consolidation\AnnotatedCommand\AnnotatedCommand $command
86
+     */
81 87
     public static function getNames($command, $callback)
82 88
     {
83 89
         return array_filter(
@@ -289,6 +295,9 @@  discard block
 block discarded – undo
289 295
         $command->optionsHookForHookAnnotations($commandInfoList);
290 296
     }
291 297
 
298
+    /**
299
+     * @param \Consolidation\AnnotatedCommand\AnnotatedCommand $command
300
+     */
292 301
     protected function getHookOptionsForCommand($command)
293 302
     {
294 303
         if (isset($this->hookOptions[$command->getName()])) {
@@ -389,6 +398,9 @@  discard block
 block discarded – undo
389 398
         return $result;
390 399
     }
391 400
 
401
+    /**
402
+     * @param string[] $names
403
+     */
392 404
     protected function getCommandEventHooks($names)
393 405
     {
394 406
         return $this->getHooks($names, self::COMMAND_EVENT);
@@ -449,6 +461,7 @@  discard block
 block discarded – undo
449 461
      * @param string|array $names The name of the function being hooked.
450 462
      * @param string $hook The specific hook name (e.g. alter)
451 463
      * @param string[] $stages The stages to apply hooks at (e.g. pre, post)
464
+     * @param AnnotationData $annotationData
452 465
      *
453 466
      * @return callable[]
454 467
      */
@@ -499,6 +512,10 @@  discard block
 block discarded – undo
499 512
         return [];
500 513
     }
501 514
 
515
+    /**
516
+     * @param callable $provider
517
+     * @param InputInterface $input
518
+     */
502 519
     protected function callInjectConfigurationHook($provider, $input, AnnotationData $annotationData)
503 520
     {
504 521
         if ($provider instanceof InitializeHookInterface) {
@@ -509,6 +526,10 @@  discard block
 block discarded – undo
509 526
         }
510 527
     }
511 528
 
529
+    /**
530
+     * @param callable $optionHook
531
+     * @param \Consolidation\AnnotatedCommand\AnnotatedCommand $command
532
+     */
512 533
     protected function callOptionHook($optionHook, $command, AnnotationData $annotationData)
513 534
     {
514 535
         if ($optionHook instanceof OptionHookInterface) {
@@ -519,6 +540,11 @@  discard block
 block discarded – undo
519 540
         }
520 541
     }
521 542
 
543
+    /**
544
+     * @param callable $interactor
545
+     * @param InputInterface $input
546
+     * @param OutputInterface $output
547
+     */
522 548
     protected function callInteractor($interactor, $input, $output, AnnotationData $annotationData)
523 549
     {
524 550
         if ($interactor instanceof InteractorInterface) {
@@ -529,6 +555,9 @@  discard block
 block discarded – undo
529 555
         }
530 556
     }
531 557
 
558
+    /**
559
+     * @param callable $validator
560
+     */
532 561
     protected function callValidator($validator, $args, AnnotationData $annotationData)
533 562
     {
534 563
         // TODO: Adding AnnotationData to ValidatorInterface would be
@@ -542,6 +571,9 @@  discard block
 block discarded – undo
542 571
         }
543 572
     }
544 573
 
574
+    /**
575
+     * @param callable $processor
576
+     */
545 577
     protected function callProcessor($processor, $result, $args, AnnotationData $annotationData)
546 578
     {
547 579
         $processed = null;
@@ -560,6 +592,9 @@  discard block
 block discarded – undo
560 592
         return $result;
561 593
     }
562 594
 
595
+    /**
596
+     * @param callable $determiner
597
+     */
563 598
     protected function callDeterminer($determiner, $result)
564 599
     {
565 600
         if ($determiner instanceof StatusDeterminerInterface) {
@@ -570,6 +605,9 @@  discard block
 block discarded – undo
570 605
         }
571 606
     }
572 607
 
608
+    /**
609
+     * @param callable $extractor
610
+     */
573 611
     protected function callExtractor($extractor, $result)
574 612
     {
575 613
         if ($extractor instanceof ExtractOutputInterface) {
Please login to merge, or discard this patch.
src/Parser/Internal/AbstractCommandDocBlockParser.php 1 patch
Doc Comments   +2 added lines patch added patch discarded remove patch
@@ -207,6 +207,7 @@  discard block
 block discarded – undo
207 207
     /**
208 208
      * Given a docblock description in the form "$variable description",
209 209
      * return the variable name and description via the 'match' parameter.
210
+     * @param string $source
210 211
      */
211 212
     protected function pregMatchNameAndDescription($source, &$match)
212 213
     {
@@ -220,6 +221,7 @@  discard block
 block discarded – undo
220 221
     /**
221 222
      * Given a docblock description in the form "$variable description",
222 223
      * return the variable name and description via the 'match' parameter.
224
+     * @param string $source
223 225
      */
224 226
     protected function pregMatchOptionNameAndDescription($source, &$match)
225 227
     {
Please login to merge, or discard this patch.
src/CommandData.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@
 block discarded – undo
37 37
     /**
38 38
      * For internal use only; indicates that the function to be called
39 39
      * should be passed an InputInterface &/or an OutputInterface.
40
-     * @param booean $usesInputInterface
40
+     * @param boolean $usesInputInterface
41 41
      * @param boolean $usesOutputInterface
42 42
      * @return self
43 43
      */
Please login to merge, or discard this patch.
src/Options/PrepareTerminalWidthOption.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -43,6 +43,9 @@
 block discarded – undo
43 43
         $options->setWidth($width);
44 44
     }
45 45
 
46
+    /**
47
+     * @return CommandData
48
+     */
46 49
     protected function getTerminalWidth()
47 50
     {
48 51
         if (!$this->application) {
Please login to merge, or discard this patch.
src/AnnotatedCommand.php 1 patch
Doc Comments   +19 added lines patch added patch discarded remove patch
@@ -38,6 +38,9 @@  discard block
 block discarded – undo
38 38
     protected $usesOutputInterface;
39 39
     protected $returnType;
40 40
 
41
+    /**
42
+     * @param string $name
43
+     */
41 44
     public function __construct($name = null)
42 45
     {
43 46
         $commandInfo = false;
@@ -67,6 +70,9 @@  discard block
 block discarded – undo
67 70
         return $this;
68 71
     }
69 72
 
73
+    /**
74
+     * @param CommandProcessor $commandProcessor
75
+     */
70 76
     public function setCommandProcessor($commandProcessor)
71 77
     {
72 78
         $this->commandProcessor = $commandProcessor;
@@ -120,6 +126,9 @@  discard block
 block discarded – undo
120 126
         return $this;
121 127
     }
122 128
 
129
+    /**
130
+     * @param CommandInfo $commandInfo
131
+     */
123 132
     public function setCommandInfo($commandInfo)
124 133
     {
125 134
         $this->setDescription($commandInfo->getDescription());
@@ -198,6 +207,10 @@  discard block
 block discarded – undo
198 207
         return $dom;
199 208
     }
200 209
 
210
+    /**
211
+     * @param \DOMDocument $dom
212
+     * @param string $tagName
213
+     */
201 214
     protected function getSingleElementByTagName($dom, $parent, $tagName)
202 215
     {
203 216
         // There should always be exactly one '<command>' element.
@@ -275,6 +288,9 @@  discard block
 block discarded – undo
275 288
         return InputArgument::OPTIONAL;
276 289
     }
277 290
 
291
+    /**
292
+     * @param CommandInfo $commandInfo
293
+     */
278 294
     public function setCommandOptions($commandInfo, $automaticOptions = [])
279 295
     {
280 296
         $inputOptions = $commandInfo->inputOptions();
@@ -345,6 +361,9 @@  discard block
 block discarded – undo
345 361
         );
346 362
     }
347 363
 
364
+    /**
365
+     * @param Hooks\CommandInfo[] $commandInfoList
366
+     */
348 367
     public function optionsHookForHookAnnotations($commandInfoList)
349 368
     {
350 369
         foreach ($commandInfoList as $commandInfo) {
Please login to merge, or discard this patch.
src/Parser/CommandInfo.php 1 patch
Doc Comments   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -129,6 +129,9 @@  discard block
 block discarded – undo
129 129
         return new self($classNameOrInstance, $methodName, $cache);
130 130
     }
131 131
 
132
+    /**
133
+     * @param string $methodName
134
+     */
132 135
     protected function constructFromClassAndMethod($classNameOrInstance, $methodName)
133 136
     {
134 137
         $this->otherAnnotations = new AnnotationData();
@@ -423,7 +426,7 @@  discard block
 block discarded – undo
423 426
     /**
424 427
      * Set aliases that can be used in place of the command's primary name.
425 428
      *
426
-     * @param string|string[] $aliases
429
+     * @param string $aliases
427 430
      */
428 431
     public function setAliases($aliases)
429 432
     {
@@ -477,7 +480,7 @@  discard block
 block discarded – undo
477 480
     /**
478 481
      * Return the list of refleaction parameters.
479 482
      *
480
-     * @return ReflectionParameter[]
483
+     * @return \ReflectionParameter[]
481 484
      */
482 485
     public function getParameters()
483 486
     {
@@ -608,7 +611,7 @@  discard block
 block discarded – undo
608 611
      * Examine the parameters of the method for this command, and
609 612
      * build a list of commandline arguements for them.
610 613
      *
611
-     * @return array
614
+     * @return DefaultsWithDescriptions
612 615
      */
613 616
     protected function determineAgumentClassifications()
614 617
     {
@@ -628,6 +631,7 @@  discard block
 block discarded – undo
628 631
      * Examine the provided parameter, and determine whether it
629 632
      * is a parameter that will be filled in with a positional
630 633
      * commandline argument.
634
+     * @param DefaultsWithDescriptions $result
631 635
      */
632 636
     protected function addParameterToResult($result, $param)
633 637
     {
@@ -718,6 +722,8 @@  discard block
 block discarded – undo
718 722
     /**
719 723
      * Given a list that might be 'a b c' or 'a, b, c' or 'a,b,c',
720 724
      * convert the data into the last of these forms.
725
+     * @param string $text
726
+     * @return string
721 727
      */
722 728
     protected static function convertListToCommaSeparated($text)
723 729
     {
Please login to merge, or discard this patch.