Completed
Pull Request — master (#125)
by
unknown
04:54
created
src/MenuItem/SplitItem.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -2,7 +2,6 @@
 block discarded – undo
2 2
 
3 3
 namespace PhpSchool\CliMenu\MenuItem;
4 4
 
5
-use Assert\Assertion;
6 5
 use PhpSchool\CliMenu\CliMenu;
7 6
 use PhpSchool\CliMenu\CliMenuBuilder;
8 7
 use PhpSchool\CliMenu\MenuStyle;
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
96 96
 
97 97
     public function end() : CliMenuBuilder
98 98
     {
99
-        $this->items = array_map(function ($item) {
99
+        $this->items = array_map(function($item) {
100 100
             if (!is_string($item) || empty($this->subMenuBuilders[$item])) {
101 101
                 return $item;
102 102
             }
@@ -125,9 +125,9 @@  discard block
 block discarded – undo
125 125
         }
126 126
 
127 127
         $length = $style->getDisplaysExtra()
128
-            ? floor(($style->getContentWidth() - mb_strlen($style->getItemExtra()) + 2) / $numberOfItems) - $this->margin
129
-            : floor($style->getContentWidth() / $numberOfItems) - $this->margin;
130
-        $missingLength = $style->getContentWidth() % $numberOfItems;
128
+            ? floor(($style->getContentWidth() - mb_strlen($style->getItemExtra()) + 2)/$numberOfItems) - $this->margin
129
+            : floor($style->getContentWidth()/$numberOfItems) - $this->margin;
130
+        $missingLength = $style->getContentWidth()%$numberOfItems;
131 131
 
132 132
         $lines = 0;
133 133
         $cells = [];
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
                 sprintf('%s%s', $marker, $item->getText()),
139 139
                 $length
140 140
             );
141
-            $cell = array_map(function ($row) use ($index, $length, $style, $isSelected) {
141
+            $cell = array_map(function($row) use ($index, $length, $style, $isSelected) {
142 142
                 $invertedColoursSetCode = $isSelected
143 143
                     ? $style->getInvertedColoursSetCode()
144 144
                     : '';
Please login to merge, or discard this patch.
src/CliMenu.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -283,7 +283,7 @@  discard block
 block discarded – undo
283 283
                     : $this->selectedItem++;
284 284
 
285 285
                 if (!array_key_exists($this->selectedItem, $this->items)) {
286
-                    $this->selectedItem  = $direction === 'UP'
286
+                    $this->selectedItem = $direction === 'UP'
287 287
                         ? end($itemKeys)
288 288
                         : reset($itemKeys);
289 289
                 } elseif ($this->getSelectedItem()->canSelect()) {
@@ -387,7 +387,7 @@  discard block
 block discarded – undo
387 387
             $frame->addRows($this->drawMenuItem(new LineBreakItem($this->style->getTitleSeparator())));
388 388
         }
389 389
 
390
-        array_map(function ($item, $index) use ($frame) {
390
+        array_map(function($item, $index) use ($frame) {
391 391
             $frame->addRows($this->drawMenuItem($item, $index === $this->selectedItem));
392 392
         }, $this->items, array_keys($this->items));
393 393
 
@@ -438,7 +438,7 @@  discard block
 block discarded – undo
438 438
             $borderColour = '';
439 439
         }
440 440
 
441
-        return array_map(function ($row) use ($invertedColoursSetCode, $invertedColoursUnsetCode, $borderColour) {
441
+        return array_map(function($row) use ($invertedColoursSetCode, $invertedColoursUnsetCode, $borderColour) {
442 442
             return sprintf(
443 443
                 "%s%s%s%s%s%s%s%s%s%s%s%s\n",
444 444
                 str_repeat(' ', $this->style->getMargin()),
Please login to merge, or discard this patch.
src/CliMenuBuilder.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -84,6 +84,9 @@
 block discarded – undo
84 84
      */
85 85
     private $disabled = false;
86 86
 
87
+    /**
88
+     * @param SplitItem $previousBuilder
89
+     */
87 90
     public function __construct(CliMenuBuilder $parent = null, $previousBuilder = null)
88 91
     {
89 92
         $this->parent   = $parent;
Please login to merge, or discard this patch.