Completed
Pull Request — master (#127)
by Aydin
01:52
created
src/CliMenuBuilder.php 2 patches
Unused Use Statements   -3 removed lines patch added patch discarded remove patch
@@ -5,12 +5,9 @@
 block discarded – undo
5 5
 use PhpSchool\CliMenu\Action\ExitAction;
6 6
 use PhpSchool\CliMenu\Action\GoBackAction;
7 7
 use PhpSchool\CliMenu\MenuItem\AsciiArtItem;
8
-use PhpSchool\CliMenu\MenuItem\LineBreakItem;
9 8
 use PhpSchool\CliMenu\MenuItem\MenuItemInterface;
10
-use PhpSchool\CliMenu\MenuItem\MenuMenuItem;
11 9
 use PhpSchool\CliMenu\MenuItem\SelectableItem;
12 10
 use PhpSchool\CliMenu\MenuItem\SplitItem;
13
-use PhpSchool\CliMenu\MenuItem\StaticItem;
14 11
 use PhpSchool\CliMenu\Terminal\TerminalFactory;
15 12
 use PhpSchool\CliMenu\Util\ColourUtil;
16 13
 use PhpSchool\Terminal\Terminal;
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -369,7 +369,7 @@  discard block
 block discarded – undo
369 369
 
370 370
     private function itemsHaveExtra(array $items) : bool
371 371
     {
372
-        return !empty(array_filter($items, function (MenuItemInterface $item) {
372
+        return !empty(array_filter($items, function(MenuItemInterface $item) {
373 373
             return $item->showsItemExtra();
374 374
         }));
375 375
     }
@@ -424,12 +424,12 @@  discard block
 block discarded – undo
424 424
             throw new RuntimeException(sprintf('Menu: "%s" cannot be retrieved until menu has been built', $id));
425 425
         }
426 426
 
427
-        return $this->subMenus['submenu-placeholder-' . $id];
427
+        return $this->subMenus['submenu-placeholder-'.$id];
428 428
     }
429 429
     
430 430
     private function buildSplitItems(array $items) : array
431 431
     {
432
-        return array_map(function ($item) {
432
+        return array_map(function($item) {
433 433
             if (!is_string($item) || 0 !== strpos($item, 'splititem-placeholder-')) {
434 434
                 return $item;
435 435
             }
Please login to merge, or discard this patch.
src/MenuItem/SplitItem.php 2 patches
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -2,9 +2,7 @@
 block discarded – undo
2 2
 
3 3
 namespace PhpSchool\CliMenu\MenuItem;
4 4
 
5
-use Assert\Assertion;
6 5
 use PhpSchool\CliMenu\CliMenu;
7
-use PhpSchool\CliMenu\CliMenuBuilder;
8 6
 use PhpSchool\CliMenu\MenuStyle;
9 7
 use PhpSchool\CliMenu\Util\StringUtil;
10 8
 
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -105,14 +105,14 @@  discard block
 block discarded – undo
105 105
         }
106 106
 
107 107
         $length = $style->getDisplaysExtra()
108
-            ? floor(($style->getContentWidth() - mb_strlen($style->getItemExtra()) + 2) / $numberOfItems)
109
-            : floor($style->getContentWidth() / $numberOfItems);
108
+            ? floor(($style->getContentWidth() - mb_strlen($style->getItemExtra()) + 2)/$numberOfItems)
109
+            : floor($style->getContentWidth()/$numberOfItems);
110 110
             
111 111
         $length -= $this->margin;
112 112
         
113
-        $missingLength = $style->getContentWidth() % $numberOfItems;
113
+        $missingLength = $style->getContentWidth()%$numberOfItems;
114 114
 
115
-        $cells = array_map(function ($index, $item) use ($selected, $length, $style) {
115
+        $cells = array_map(function($index, $item) use ($selected, $length, $style) {
116 116
             $isSelected = $selected && $index === $this->selectedItemIndex;
117 117
             $marker = $item->canSelect()
118 118
                 ? sprintf("%s ", $style->getMarker($isSelected))
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
                 sprintf('%s%s', $marker, $item->getText()),
122 122
                 $length
123 123
             );
124
-            return array_map(function ($row) use ($length, $style, $isSelected) {
124
+            return array_map(function($row) use ($length, $style, $isSelected) {
125 125
                 $invertedColoursSetCode = $isSelected
126 126
                     ? $style->getInvertedColoursSetCode()
127 127
                     : '';
Please login to merge, or discard this patch.
examples/split-item.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -3,9 +3,9 @@
 block discarded – undo
3 3
 use PhpSchool\CliMenu\CliMenu;
4 4
 use PhpSchool\CliMenu\CliMenuBuilder;
5 5
 
6
-require_once(__DIR__ . '/../vendor/autoload.php');
6
+require_once(__DIR__.'/../vendor/autoload.php');
7 7
 
8
-$itemCallable = function (CliMenu $menu) {
8
+$itemCallable = function(CliMenu $menu) {
9 9
     echo $menu->getSelectedItem()->getText();
10 10
 };
11 11
 
Please login to merge, or discard this patch.
src/BuilderUtils.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
      */
65 65
     public function addSubMenu(string $name, CliMenuBuilder $subMenuBuilder = null) : Builder
66 66
     {
67
-        $this->menuItems[]  = $id = 'submenu-placeholder-' . $name;
67
+        $this->menuItems[] = $id = 'submenu-placeholder-'.$name;
68 68
 
69 69
         if (null === $subMenuBuilder) {
70 70
             $this->subMenuBuilders[$id] = new CliMenuBuilder($this);
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
 
78 78
     private function buildSubMenus(array $items) : array
79 79
     {
80
-        return array_map(function ($item) {
80
+        return array_map(function($item) {
81 81
             if (!is_string($item) || 0 !== strpos($item, 'submenu-placeholder-')) {
82 82
                 return $item;
83 83
             }
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
@@ -285,7 +285,7 @@  discard block
 block discarded – undo
285 285
                 : $this->selectedItem++;
286 286
 
287 287
             if (!array_key_exists($this->selectedItem, $this->items)) {
288
-                $this->selectedItem  = $direction === 'UP'
288
+                $this->selectedItem = $direction === 'UP'
289 289
                     ? end($itemKeys)
290 290
                     : reset($itemKeys);
291 291
             }
@@ -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.