consolidation / output-formatters

src/FormatterManager.php

Doc Comments +8 added lines

@@ -182,6 +182,9 @@ 
         return $this->isValidFormatForReflectionClass($formatter, $dataType);
     }
 
+    /**
+     * @param \ReflectionClass $dataType
+     */
     public function isValidFormatForReflectionClass(FormatterInterface $formatter, $dataType)
     {
         if ($this->canSimplifyToArray($dataType)) {
@@ -196,6 +199,10 @@ 
         }
         return array_reduce(
             $formatter->validDataTypes(),
+
+            /**
+             * @param \ReflectionClass $supportedType
+             */
             function ($carry, $supportedType) use ($dataType) {
                 return
                     $carry ||
@@ -266,6 +273,7 @@ 
 
     /**
      * Test to see if the stipulated format exists
+     * @param string $format
      */
     public function hasFormatter($format)
     {

Spacing +7 added lines, -7 removed lines

@@ -92,13 +92,13 @@ 
 
         if (count($validFormats) > 1) {
             // Make an input option for --format
-            $description = 'Format the result data. Available formats: ' . implode(',', $validFormats);
+            $description = 'Format the result data. Available formats: '.implode(',', $validFormats);
             $automaticOptions[FormatterOptions::FORMAT] = new InputOption(FormatterOptions::FORMAT, '', InputOption::VALUE_OPTIONAL, $description, $defaultFormat);
         }
 
         if ($availableFields) {
             $defaultFields = $options->get(FormatterOptions::DEFAULT_FIELDS, [], implode(',', $availableFields));
-            $description = 'Available fields: ' . implode(', ', $this->availableFieldsList($availableFields));
+            $description = 'Available fields: '.implode(', ', $this->availableFieldsList($availableFields));
             $automaticOptions[FormatterOptions::FIELDS] = new InputOption(FormatterOptions::FIELDS, '', InputOption::VALUE_OPTIONAL, $description, $defaultFields);
         }
 
@@ -112,8 +112,8 @@ 
     protected function availableFieldsList($availableFields)
     {
         return array_map(
-            function ($key) use ($availableFields) {
-                return $availableFields[$key] . " ($key)";
+            function($key) use ($availableFields) {
+                return $availableFields[$key]." ($key)";
             },
             array_keys($availableFields)
         );
@@ -196,7 +196,7 @@ 
         }
         return array_reduce(
             $formatter->validDataTypes(),
-            function ($carry, $supportedType) use ($dataType) {
+            function($carry, $supportedType) use ($dataType) {
                 return
                     $carry ||
                     ($dataType->getName() == $supportedType->getName()) ||
@@ -216,10 +216,10 @@ 
      */
     public function write(OutputInterface $output, $format, $structuredOutput, FormatterOptions $options)
     {
-        $formatter = $this->getFormatter((string)$format);
+        $formatter = $this->getFormatter((string) $format);
         if (!is_string($structuredOutput) && !$this->isValidFormat($formatter, $structuredOutput)) {
             $validFormats = $this->validFormats($structuredOutput);
-            throw new InvalidFormatException((string)$format, $structuredOutput, $validFormats);
+            throw new InvalidFormatException((string) $format, $structuredOutput, $validFormats);
         }
         // Give the formatter a chance to override the options
         $options = $this->overrideOptions($formatter, $structuredOutput, $options);