Completed
Push — master ( f84168...88ef34 )
by Greg
01:46
created
src/Formatters/TableFormatter.php 3 patches
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -123,6 +123,7 @@
 block discarded – undo
123 123
 
124 124
     /**
125 125
      * Add our custom table style(s) to the table.
126
+     * @param Table $table
126 127
      */
127 128
     protected static function addCustomTableStyles($table)
128 129
     {
Please login to merge, or discard this patch.
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -4,12 +4,10 @@
 block discarded – undo
4 4
 use Symfony\Component\Console\Output\OutputInterface;
5 5
 use Symfony\Component\Console\Helper\Table;
6 6
 use Symfony\Component\Console\Helper\TableStyle;
7
-
8 7
 use Consolidation\OutputFormatters\Validate\ValidDataTypesInterface;
9 8
 use Consolidation\OutputFormatters\Options\FormatterOptions;
10 9
 use Consolidation\OutputFormatters\Validate\ValidDataTypesTrait;
11 10
 use Consolidation\OutputFormatters\StructuredData\TableDataInterface;
12
-use Consolidation\OutputFormatters\Transformations\ReorderFields;
13 11
 use Consolidation\OutputFormatters\Exception\IncompatibleDataException;
14 12
 use Consolidation\OutputFormatters\Transformations\WordWrapper;
15 13
 
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
             if (!empty($headers)) {
90 90
                 array_splice($headers, 1, 0, ':');
91 91
             }
92
-            $data = array_map(function ($item) {
92
+            $data = array_map(function($item) {
93 93
                 array_splice($item, 1, 0, ':');
94 94
                 return $item;
95 95
             }, $data);
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
         $wrapper = new WordWrapper($options->get(FormatterOptions::TERMINAL_WIDTH));
117 117
         $wrapper->setPaddingFromStyle($tableStyle);
118 118
         if (!empty($headers)) {
119
-            $headerLengths = array_map(function ($item) {
119
+            $headerLengths = array_map(function($item) {
120 120
                 return strlen($item);
121 121
             }, $headers);
122 122
             $wrapper->setMinimumWidths($headerLengths);
Please login to merge, or discard this patch.
src/StructuredData/Xml/XmlSchema.php 2 patches
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -34,6 +34,9 @@  discard block
 block discarded – undo
34 34
         $this->addXmlChildren($dom, $element, $elementName, $structuredData);
35 35
     }
36 36
 
37
+    /**
38
+     * @param \DOMElement $xmlParent
39
+     */
37 40
     protected function addXmlChildren(\DOMDocument $dom, $xmlParent, $elementName, $structuredData)
38 41
     {
39 42
         foreach ($structuredData as $key => $value) {
@@ -52,6 +55,9 @@  discard block
 block discarded – undo
52 55
         $this->addXmlData($dom, $xmlParent, $elementName, $value);
53 56
     }
54 57
 
58
+    /**
59
+     * @param string $childElementName
60
+     */
55 61
     protected function determineElementName($key, $childElementName, $value)
56 62
     {
57 63
         if (is_numeric($key)) {
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -58,7 +58,7 @@
 block discarded – undo
58 58
             return $childElementName;
59 59
         }
60 60
         if (is_object($value)) {
61
-            $value = (array)$value;
61
+            $value = (array) $value;
62 62
         }
63 63
         if (!is_array($value)) {
64 64
             return $key;
Please login to merge, or discard this patch.
src/Transformations/Wrap/CalculateWidths.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
     {
53 53
         return $this->calculateColumnWidths(
54 54
             $rows,
55
-            function ($cell) {
55
+            function($cell) {
56 56
                 return strlen($cell);
57 57
             }
58 58
         );
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
     {
66 66
         return $this->calculateColumnWidths(
67 67
             $rows,
68
-            function ($cell) {
68
+            function($cell) {
69 69
                 return static::longestWordLength($cell);
70 70
             }
71 71
         );
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
     protected static function longestWordLength($str)
134 134
     {
135 135
         $words = preg_split('#[ /-]#', $str);
136
-        $lengths = array_map(function ($s) {
136
+        $lengths = array_map(function($s) {
137 137
             return strlen($s);
138 138
         }, $words);
139 139
         return max($lengths);
Please login to merge, or discard this patch.
src/Transformations/Wrap/ColumnWidths.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -188,7 +188,7 @@
 block discarded – undo
188 188
     {
189 189
         return array_reduce(
190 190
             $widths,
191
-            function ($carry, $item) {
191
+            function($carry, $item) {
192 192
                 return $carry + $item;
193 193
             }
194 194
         );
Please login to merge, or discard this patch.
src/FormatterManager.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -111,13 +111,13 @@  discard block
 block discarded – undo
111 111
 
112 112
         if (count($validFormats) > 1) {
113 113
             // Make an input option for --format
114
-            $description = 'Format the result data. Available formats: ' . implode(',', $validFormats);
114
+            $description = 'Format the result data. Available formats: '.implode(',', $validFormats);
115 115
             $automaticOptions[FormatterOptions::FORMAT] = new InputOption(FormatterOptions::FORMAT, '', InputOption::VALUE_REQUIRED, $description, $defaultFormat);
116 116
         }
117 117
 
118 118
         if ($availableFields) {
119 119
             $defaultFields = $options->get(FormatterOptions::DEFAULT_FIELDS, [], '');
120
-            $description = 'Available fields: ' . implode(', ', $this->availableFieldsList($availableFields));
120
+            $description = 'Available fields: '.implode(', ', $this->availableFieldsList($availableFields));
121 121
             $automaticOptions[FormatterOptions::FIELDS] = new InputOption(FormatterOptions::FIELDS, '', InputOption::VALUE_REQUIRED, $description, $defaultFields);
122 122
             $automaticOptions[FormatterOptions::FIELD] = new InputOption(FormatterOptions::FIELD, '', InputOption::VALUE_REQUIRED, "Select just one field, and force format to 'string'.", '');
123 123
         }
@@ -132,8 +132,8 @@  discard block
 block discarded – undo
132 132
     protected function availableFieldsList($availableFields)
133 133
     {
134 134
         return array_map(
135
-            function ($key) use ($availableFields) {
136
-                return $availableFields[$key] . " ($key)";
135
+            function($key) use ($availableFields) {
136
+                return $availableFields[$key]." ($key)";
137 137
             },
138 138
             array_keys($availableFields)
139 139
         );
@@ -197,10 +197,10 @@  discard block
 block discarded – undo
197 197
      */
198 198
     public function write(OutputInterface $output, $format, $structuredOutput, FormatterOptions $options)
199 199
     {
200
-        $formatter = $this->getFormatter((string)$format);
200
+        $formatter = $this->getFormatter((string) $format);
201 201
         if (!is_string($structuredOutput) && !$this->isValidFormat($formatter, $structuredOutput)) {
202 202
             $validFormats = $this->validFormats($structuredOutput);
203
-            throw new InvalidFormatException((string)$format, $structuredOutput, $validFormats);
203
+            throw new InvalidFormatException((string) $format, $structuredOutput, $validFormats);
204 204
         }
205 205
         // Give the formatter a chance to override the options
206 206
         $options = $this->overrideOptions($formatter, $structuredOutput, $options);
Please login to merge, or discard this patch.