comparison vendor/consolidation/annotated-command/src/Parser/Internal/BespokeDocBlockParser.php @ 17:129ea1e6d783

Update, including to Drupal core 8.6.10

      */
     protected $tagProcessors = [
         'command' => 'processCommandTag',
         'name' => 'processCommandTag',
         'arg' => 'processArgumentTag',
-        'param' => 'processArgumentTag',
+        'param' => 'processParamTag',
         'return' => 'processReturnTag',
         'option' => 'processOptionTag',
         'default' => 'processDefaultTag',
         'aliases' => 'processAliases',
         'usage' => 'processUsageTag',

      * @deprecated
      */
     protected function processAlternateDescriptionTag($tag)
     {
         $this->commandInfo->setDescription($tag->getContent());
+    }
+
+    /**
+     * Store the data from a @param annotation in our argument descriptions.
+     */
+    protected function processParamTag($tag)
+    {
+        if ($tag->hasTypeVariableAndDescription($matches)) {
+            if ($this->ignoredParamType($matches['type'])) {
+                return;
+            }
+        }
+        return $this->processArgumentTag($tag);
+    }
+
+    protected function ignoredParamType($paramType)
+    {
+        // TODO: We should really only allow a couple of types here,
+        // e.g. 'string', 'array', 'bool'. Blacklist things we do not
+        // want for now to avoid breaking commands with weird types.
+        // Fix in the next major version.
+        //
+        // This works:
+        //   return !in_array($paramType, ['string', 'array', 'integer', 'bool']);
+        return preg_match('#(InputInterface|OutputInterface)$#', $paramType);
     }
 
     /**
      * Store the data from a @arg annotation in our argument descriptions.
      */