php-school / cli-menu

src/CliMenuBuilder.php

Unused Use Statements -3 removed lines

@@ -5,12 +5,9 @@
 use PhpSchool\CliMenu\Action\ExitAction;
 use PhpSchool\CliMenu\Action\GoBackAction;
 use PhpSchool\CliMenu\MenuItem\AsciiArtItem;
-use PhpSchool\CliMenu\MenuItem\LineBreakItem;
 use PhpSchool\CliMenu\MenuItem\MenuItemInterface;
-use PhpSchool\CliMenu\MenuItem\MenuMenuItem;
 use PhpSchool\CliMenu\MenuItem\SelectableItem;
 use PhpSchool\CliMenu\MenuItem\SplitItem;
-use PhpSchool\CliMenu\MenuItem\StaticItem;
 use PhpSchool\CliMenu\Terminal\TerminalFactory;
 use PhpSchool\CliMenu\Util\ColourUtil;
 use PhpSchool\Terminal\Terminal;

Spacing +3 added lines, -3 removed lines

@@ -369,7 +369,7 @@ 
 
     private function itemsHaveExtra(array $items) : bool
     {
-        return !empty(array_filter($items, function (MenuItemInterface $item) {
+        return !empty(array_filter($items, function(MenuItemInterface $item) {
             return $item->showsItemExtra();
         }));
     }
@@ -424,12 +424,12 @@ 
             throw new RuntimeException(sprintf('Menu: "%s" cannot be retrieved until menu has been built', $id));
         }
 
-        return $this->subMenus['submenu-placeholder-' . $id];
+        return $this->subMenus['submenu-placeholder-'.$id];
     }
     
     private function buildSplitItems(array $items) : array
     {
-        return array_map(function ($item) {
+        return array_map(function($item) {
             if (!is_string($item) || 0 !== strpos($item, 'splititem-placeholder-')) {
                 return $item;
             }

src/MenuItem/SplitItem.php

Unused Use Statements -2 removed lines

@@ -2,9 +2,7 @@
 
 namespace PhpSchool\CliMenu\MenuItem;
 
-use Assert\Assertion;
 use PhpSchool\CliMenu\CliMenu;
-use PhpSchool\CliMenu\CliMenuBuilder;
 use PhpSchool\CliMenu\MenuStyle;
 use PhpSchool\CliMenu\Util\StringUtil;
 

Spacing +7 added lines, -7 removed lines

@@ -109,15 +109,15 @@ 
         }
 
         $length = $style->getDisplaysExtra()
-            ? floor($style->getContentWidth() / $numberOfItems) - (mb_strlen($style->getItemExtra()) + 2)
-            : floor($style->getContentWidth() / $numberOfItems);
+            ? floor($style->getContentWidth()/$numberOfItems) - (mb_strlen($style->getItemExtra()) + 2)
+            : floor($style->getContentWidth()/$numberOfItems);
         
         $length -= $this->margin;
         
-        $missingLength = $style->getContentWidth() % $numberOfItems;
+        $missingLength = $style->getContentWidth()%$numberOfItems;
         
         return $this->buildRows(
-            array_map(function ($index, $item) use ($selected, $length, $style) {
+            array_map(function($index, $item) use ($selected, $length, $style) {
                 $isSelected = $selected && $index === $this->selectedItemIndex;
                 $marker = $item->canSelect()
                     ? sprintf('%s ', $style->getMarker($isSelected))
@@ -149,7 +149,7 @@ 
         $extraPadLength = $style->getDisplaysExtra() ? 2 + mb_strlen($style->getItemExtra()) : 0;
         
         return array_map(
-            function ($i) use ($cells, $length, $missingLength, $extraPadLength) {
+            function($i) use ($cells, $length, $missingLength, $extraPadLength) {
                 return $this->buildRow($cells, $i, $length, $missingLength, $extraPadLength);
             },
             range(0, max(array_map('count', $cells)) - 1)
@@ -163,7 +163,7 @@ 
             implode(
                 '',
                 array_map(
-                    function ($cell) use ($index, $length, $extraPadLength) {
+                    function($cell) use ($index, $length, $extraPadLength) {
                         return $cell[$index] ?? str_repeat(' ', $length + $this->margin + $extraPadLength);
                     },
                     $cells
@@ -180,7 +180,7 @@ 
         bool $isSelected,
         string $itemExtra
     ) : array {
-        return array_map(function ($row, $index) use ($length, $style, $isSelected, $itemExtra) {
+        return array_map(function($row, $index) use ($length, $style, $isSelected, $itemExtra) {
             $invertedColoursSetCode = $isSelected
                 ? $style->getInvertedColoursSetCode()
                 : '';

examples/split-item.php

Spacing +2 added lines, -2 removed lines

@@ -3,9 +3,9 @@
 use PhpSchool\CliMenu\CliMenu;
 use PhpSchool\CliMenu\CliMenuBuilder;
 
-require_once(__DIR__ . '/../vendor/autoload.php');
+require_once(__DIR__.'/../vendor/autoload.php');
 
-$itemCallable = function (CliMenu $menu) {
+$itemCallable = function(CliMenu $menu) {
     echo $menu->getSelectedItem()->getText();
 };
 

src/BuilderUtils.php

Spacing +2 added lines, -2 removed lines

@@ -64,7 +64,7 @@ 
      */
     public function addSubMenu(string $name, CliMenuBuilder $subMenuBuilder = null) : Builder
     {
-        $this->menuItems[]  = $id = 'submenu-placeholder-' . $name;
+        $this->menuItems[] = $id = 'submenu-placeholder-'.$name;
 
         if (null === $subMenuBuilder) {
             $this->subMenuBuilders[$id] = new CliMenuBuilder($this);
@@ -77,7 +77,7 @@ 
 
     private function buildSubMenus(array $items) : array
     {
-        return array_map(function ($item) {
+        return array_map(function($item) {
             if (!is_string($item) || 0 !== strpos($item, 'submenu-placeholder-')) {
                 return $item;
             }

src/CliMenu.php

Spacing +3 added lines, -3 removed lines

@@ -285,7 +285,7 @@ 
                 : $this->selectedItem++;
 
             if (!array_key_exists($this->selectedItem, $this->items)) {
-                $this->selectedItem  = $direction === 'UP'
+                $this->selectedItem = $direction === 'UP'
                     ? end($itemKeys)
                     : reset($itemKeys);
             }
@@ -387,7 +387,7 @@ 
             $frame->addRows($this->drawMenuItem(new LineBreakItem($this->style->getTitleSeparator())));
         }
 
-        array_map(function ($item, $index) use ($frame) {
+        array_map(function($item, $index) use ($frame) {
             $frame->addRows($this->drawMenuItem($item, $index === $this->selectedItem));
         }, $this->items, array_keys($this->items));
 
@@ -438,7 +438,7 @@ 
             $borderColour = '';
         }
 
-        return array_map(function ($row) use ($invertedColoursSetCode, $invertedColoursUnsetCode, $borderColour) {
+        return array_map(function($row) use ($invertedColoursSetCode, $invertedColoursUnsetCode, $borderColour) {
             return sprintf(
                 "%s%s%s%s%s%s%s%s%s%s%s%s\n",
                 str_repeat(' ', $this->style->getMargin()),

src/Util/StringUtil.php

Spacing +6 added lines, -6 removed lines

@@ -15,7 +15,7 @@ 
     {
         return implode(
             $break,
-            array_map(function (string $line) use ($width, $break) {
+            array_map(function(string $line) use ($width, $break) {
                 $line = rtrim($line);
                 if (mb_strlen($line) <= $width) {
                     return $line;
@@ -25,16 +25,16 @@ 
                 $line   = '';
                 $actual = '';
                 foreach ($words as $word) {
-                    if (mb_strlen($actual . $word) <= $width) {
-                        $actual .= $word . ' ';
+                    if (mb_strlen($actual.$word) <= $width) {
+                        $actual .= $word.' ';
                     } else {
                         if ($actual !== '') {
-                            $line .= rtrim($actual) . $break;
+                            $line .= rtrim($actual).$break;
                         }
-                        $actual = $word . ' ';
+                        $actual = $word.' ';
                     }
                 }
-                return $line . trim($actual);
+                return $line.trim($actual);
             }, explode($break, $string))
         );
     }