php-school / cli-menu

src/CliMenuBuilder.php

Doc Comments +1 added lines, -1 removed lines

@@ -361,7 +361,7 @@
     }
 
     /**
-     * @return array
+     * @return SelectableItem[]
      */
     private function getDefaultItems()
     {

Spacing +3 added lines, -3 removed lines

@@ -391,7 +391,7 @@ 
      */
     private function itemsHaveExtra(array $items)
     {
-        return !empty(array_filter($items, function (MenuItemInterface $item) {
+        return !empty(array_filter($items, function(MenuItemInterface $item) {
             return $item->showsItemExtra();
         }));
     }
@@ -404,7 +404,7 @@ 
      */
     private function getMenuStyle()
     {
-        $diff = array_udiff_assoc($this->style, $this->getStyleClassDefaults(), function ($current, $default) {
+        $diff = array_udiff_assoc($this->style, $this->getStyleClassDefaults(), function($current, $default) {
             if ($current instanceof TerminalInterface) {
                 return 0;
             }
@@ -454,7 +454,7 @@ 
      */
     private function buildSubMenus(array $items)
     {
-        return array_map(function ($item) {
+        return array_map(function($item) {
             if (!is_string($item)) {
                 return $item;
             }

examples/remove-defaults.php

Spacing +3 added lines, -3 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();
 };
 
@@ -15,7 +15,7 @@ 
     ->addItem('Second Item', $itemCallable)
     ->addItem('Third Item', $itemCallable)
     ->disableDefaultItems()
-    ->addItem('CUSTOM CLOSE', function (CliMenu $menu) {
+    ->addItem('CUSTOM CLOSE', function(CliMenu $menu) {
         $menu->close();
     })
     ->build();

examples/submenu.php

Spacing +3 added lines, -3 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();
 };
 
@@ -16,7 +16,7 @@ 
     ->addLineBreak('-')
     ->addSubMenu('Options')
         ->setTitle('CLI Menu > Options')
-        ->addItem('First option', function (CliMenu $menu) {
+        ->addItem('First option', function(CliMenu $menu) {
             echo sprintf('Executing option: %s', $menu->getSelectedItem()->getText());
         })
         ->addLineBreak('-')

examples/useful-separation.php

Spacing +2 added lines, -2 removed lines

@@ -4,9 +4,9 @@
 use PhpSchool\CliMenu\CliMenuBuilder;
 use PhpSchool\CliMenu\MenuItem\MenuItem;
 
-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/CliMenu.php

Spacing +2 added lines, -2 removed lines

@@ -216,7 +216,7 @@ 
                 : $this->selectedItem++;
 
             if (!array_key_exists($this->selectedItem, $this->items)) {
-                $this->selectedItem  = $direction === 'up'
+                $this->selectedItem = $direction === 'up'
                     ? end($itemKeys)
                     : reset($itemKeys);
             } elseif ($this->getSelectedItem()->canSelect()) {
@@ -263,7 +263,7 @@ 
             $this->drawMenuItem(new LineBreakItem($this->style->getTitleSeparator()));
         }
 
-        array_map(function ($item, $index) {
+        array_map(function($item, $index) {
             $this->drawMenuItem($item, $index === $this->selectedItem);
         }, $this->items, array_keys($this->items));
 

Doc Comments +1 added lines, -1 removed lines

@@ -202,7 +202,7 @@
     /**
      * Move the selection in a given direction, up / down
      *
-     * @param $direction
+     * @param string $direction
      */
     protected function moveSelection($direction)
     {

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use PhpSchool\CliMenu\Exception\InvalidInstantiationException;
 use PhpSchool\CliMenu\Exception\InvalidTerminalException;
 use PhpSchool\CliMenu\MenuItem\LineBreakItem;
-use PhpSchool\CliMenu\MenuItem\MenuItem;
 use PhpSchool\CliMenu\MenuItem\MenuItemInterface;
 use PhpSchool\CliMenu\MenuItem\StaticItem;
 use PhpSchool\CliMenu\Terminal\TerminalFactory;

src/MenuItem/AsciiArtItem.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
      */
     public function getRows(MenuStyle $style, $selected = false)
     {
-        return array_map(function ($row) use ($style) {
+        return array_map(function($row) use ($style) {
             $length = mb_strlen($row);
 
             $padding = $style->getContentWidth() - $length;

src/MenuItem/SelectableTrait.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
             )
         );
 
-        return array_map(function ($row, $key) use ($style, $marker, $length) {
+        return array_map(function($row, $key) use ($style, $marker, $length) {
             if ($key === 0) {
                 return $this->showItemExtra
                     ? sprintf('%s%s  %s', $row, str_repeat(' ', $length - mb_strlen($row)), $style->getItemExtra())

src/MenuStyle.php

Spacing +1 added lines, -1 removed lines

@@ -305,7 +305,7 @@
      */
     public function setWidth($width)
     {
-        $availableWidth = $this->terminal->getWidth() - ($this->margin * 2) - ($this->padding * 2);
+        $availableWidth = $this->terminal->getWidth() - ($this->margin*2) - ($this->padding*2);
 
         if ($width >= $availableWidth) {
             $width = $availableWidth;

src/Terminal/UnixTerminal.php

Spacing +1 added lines, -1 removed lines

@@ -106,7 +106,7 @@
             exec('stty -icanon');
             $this->isCanonical = true;
         } else {
-            exec('stty ' . $this->getOriginalConfiguration());
+            exec('stty '.$this->getOriginalConfiguration());
             $this->isCanonical = false;
         }
     }