consolidation / output-formatters

src/Transformations/WordWrapper.php

Spacing +1 added lines, -1 removed lines

@@ -260,7 +260,7 @@
     protected static function longestWordLength($str)
     {
         $words = preg_split('/[ -]/', $str);
-        $lengths = array_map(function ($s) {
+        $lengths = array_map(function($s) {
             return strlen($s);
         }, $words);
         return max($lengths);

src/Formatters/TableFormatter.php

Doc Comments +1 added lines

@@ -123,6 +123,7 @@
 
     /**
      * Add our custom table style(s) to the table.
+     * @param Table $table
      */
     protected static function addCustomTableStyles($table)
     {

Unused Use Statements -2 removed lines

@@ -4,12 +4,10 @@
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Helper\Table;
 use Symfony\Component\Console\Helper\TableStyle;
-
 use Consolidation\OutputFormatters\Validate\ValidDataTypesInterface;
 use Consolidation\OutputFormatters\Options\FormatterOptions;
 use Consolidation\OutputFormatters\Validate\ValidDataTypesTrait;
 use Consolidation\OutputFormatters\StructuredData\TableDataInterface;
-use Consolidation\OutputFormatters\Transformations\ReorderFields;
 use Consolidation\OutputFormatters\Exception\IncompatibleDataException;
 use Consolidation\OutputFormatters\Transformations\WordWrapper;
 

Spacing +2 added lines, -2 removed lines

@@ -89,7 +89,7 @@ 
             if (!empty($headers)) {
                 array_splice($headers, 1, 0, ':');
             }
-            $data = array_map(function ($item) {
+            $data = array_map(function($item) {
                 array_splice($item, 1, 0, ':');
                 return $item;
             }, $data);
@@ -116,7 +116,7 @@ 
         $wrapper = new WordWrapper($options->get(FormatterOptions::TERMINAL_WIDTH));
         $wrapper->setPaddingFromStyle($tableStyle);
         if (!empty($headers)) {
-            $headerLengths = array_map(function ($item) {
+            $headerLengths = array_map(function($item) {
                 return strlen($item);
             }, $headers);
             $wrapper->setMinimumWidths($headerLengths);

src/StructuredData/Xml/XmlSchema.php

Doc Comments +6 added lines

@@ -34,6 +34,9 @@ 
         $this->addXmlChildren($dom, $element, $elementName, $structuredData);
     }
 
+    /**
+     * @param \DOMElement $xmlParent
+     */
     protected function addXmlChildren(\DOMDocument $dom, $xmlParent, $elementName, $structuredData)
     {
         foreach ($structuredData as $key => $value) {
@@ -52,6 +55,9 @@ 
         $this->addXmlData($dom, $xmlParent, $elementName, $value);
     }
 
+    /**
+     * @param string $childElementName
+     */
     protected function determineElementName($key, $childElementName, $value)
     {
         if (is_numeric($key)) {

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
             return $childElementName;
         }
         if (is_object($value)) {
-            $value = (array)$value;
+            $value = (array) $value;
         }
         if (!is_array($value)) {
             return $key;

src/FormatterManager.php

Spacing +6 added lines, -6 removed lines

@@ -111,13 +111,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, [], '');
-            $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);
             $automaticOptions[FormatterOptions::FIELD] = new InputOption(FormatterOptions::FIELD, '', InputOption::VALUE_OPTIONAL, "Select just one field, and force format to 'string'.", '');
         }
@@ -132,8 +132,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)
         );
@@ -197,10 +197,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);

src/Formatters/HelpFormatter.php

Unused Use Statements -2 removed lines

@@ -2,8 +2,6 @@
 namespace Consolidation\OutputFormatters\Formatters;
 
 use Consolidation\OutputFormatters\Options\FormatterOptions;
-use Consolidation\OutputFormatters\Validate\ValidDataTypesInterface;
-use Consolidation\OutputFormatters\Validate\ValidDataTypesTrait;
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Helper\Table;
 use Symfony\Component\Console\Helper\TableCell;

Indentation +25 added lines, -25 removed lines

@@ -14,11 +14,11 @@ 
 class HelpFormatter implements FormatterInterface
 {
 
-  /**
-   * @inheritdoc
-   */
-  public function write(OutputInterface $output, $data, FormatterOptions $options)
-  {
+    /**
+     * @inheritdoc
+     */
+    public function write(OutputInterface $output, $data, FormatterOptions $options)
+    {
     $table = new Table($this->output());
     $table->setStyle('compact');
 
@@ -26,51 +26,51 @@ 
     $output->writeln($command->getDescription());
 
     if ($usages = $command->getExampleUsages()) {
-      $table->addRow(['','']);
-      $table->addRow([new TableCell('Examples:', array('colspan' => 2))]);
-      foreach ($usages as $key => $description) {
+        $table->addRow(['','']);
+        $table->addRow([new TableCell('Examples:', array('colspan' => 2))]);
+        foreach ($usages as $key => $description) {
         $table->addRow(['  ' . $key, $description]);
-      }
+        }
     }
 
     if ($arguments = $def->getArguments()) {
-      $table->addRow(['','']);
-      $table->addRow([new TableCell('Arguments:', array('colspan' => 2))]);
-      foreach ($arguments as $argument) {
+        $table->addRow(['','']);
+        $table->addRow([new TableCell('Arguments:', array('colspan' => 2))]);
+        foreach ($arguments as $argument) {
         $formatted = $this->formatArgumentName($argument);
         $description = $argument->getDescription();
         if ($argument->getDefault()) {
-          $description .= ' [default: ' . $argument->getDefault() . ']';
+            $description .= ' [default: ' . $argument->getDefault() . ']';
         }
         $table->addRow(['  ' . $formatted, $description]);
-      }
+        }
     }
 
     if ($options = $def->getOptions()) {
-      $table->addRow(['','']);
-      $table->addRow([new TableCell('Options:', array('colspan' => 2))]);
-      foreach ($options as $option) {
+        $table->addRow(['','']);
+        $table->addRow([new TableCell('Options:', array('colspan' => 2))]);
+        foreach ($options as $option) {
         $formatted = $this->formatOption($option);
         $table->addRow(['  ' . $formatted, $option->getDescription()]);
-      }
+        }
     }
 
     if ($topics = $command->getTopics()) {
-      $table->addRow(['','']);
-      $table->addRow([new TableCell('Topics:', array('colspan' => 2))]);
-      foreach ($topics as $topic) {
+        $table->addRow(['','']);
+        $table->addRow([new TableCell('Topics:', array('colspan' => 2))]);
+        foreach ($topics as $topic) {
         // @todo deal with long descriptions
         $table->addRow(['  ' . $topic, substr($allTopics[$topic]['description'], 0, 30)]);
-      }
+        }
     }
 
     if ($aliases = $command->getAliases()) {
-      $table->addRow(['','']);
-      $table->addRow([new TableCell('Aliases: '. implode(' ', $aliases), array('colspan' => 2))]);
+        $table->addRow(['','']);
+        $table->addRow([new TableCell('Aliases: '. implode(' ', $aliases), array('colspan' => 2))]);
     }
 
     $table->render();
 
     $output->writeln($help);
-  }
+    }
 }

Spacing +11 added lines, -11 removed lines

@@ -26,47 +26,47 @@
     $output->writeln($command->getDescription());
 
     if ($usages = $command->getExampleUsages()) {
-      $table->addRow(['','']);
+      $table->addRow(['', '']);
       $table->addRow([new TableCell('Examples:', array('colspan' => 2))]);
       foreach ($usages as $key => $description) {
-        $table->addRow(['  ' . $key, $description]);
+        $table->addRow(['  '.$key, $description]);
       }
     }
 
     if ($arguments = $def->getArguments()) {
-      $table->addRow(['','']);
+      $table->addRow(['', '']);
       $table->addRow([new TableCell('Arguments:', array('colspan' => 2))]);
       foreach ($arguments as $argument) {
         $formatted = $this->formatArgumentName($argument);
         $description = $argument->getDescription();
         if ($argument->getDefault()) {
-          $description .= ' [default: ' . $argument->getDefault() . ']';
+          $description .= ' [default: '.$argument->getDefault().']';
         }
-        $table->addRow(['  ' . $formatted, $description]);
+        $table->addRow(['  '.$formatted, $description]);
       }
     }
 
     if ($options = $def->getOptions()) {
-      $table->addRow(['','']);
+      $table->addRow(['', '']);
       $table->addRow([new TableCell('Options:', array('colspan' => 2))]);
       foreach ($options as $option) {
         $formatted = $this->formatOption($option);
-        $table->addRow(['  ' . $formatted, $option->getDescription()]);
+        $table->addRow(['  '.$formatted, $option->getDescription()]);
       }
     }
 
     if ($topics = $command->getTopics()) {
-      $table->addRow(['','']);
+      $table->addRow(['', '']);
       $table->addRow([new TableCell('Topics:', array('colspan' => 2))]);
       foreach ($topics as $topic) {
         // @todo deal with long descriptions
-        $table->addRow(['  ' . $topic, substr($allTopics[$topic]['description'], 0, 30)]);
+        $table->addRow(['  '.$topic, substr($allTopics[$topic]['description'], 0, 30)]);
       }
     }
 
     if ($aliases = $command->getAliases()) {
-      $table->addRow(['','']);
-      $table->addRow([new TableCell('Aliases: '. implode(' ', $aliases), array('colspan' => 2))]);
+      $table->addRow(['', '']);
+      $table->addRow([new TableCell('Aliases: '.implode(' ', $aliases), array('colspan' => 2))]);
     }
 
     $table->render();