php-school / cli-menu

src/CliMenu.php

Doc Comments +2 added lines, -2 removed lines

@@ -61,7 +61,7 @@ 
     protected $parent;
 
     /**
-     * @param $title
+     * @param string|false $title
      * @param array $items
      * @param TerminalInterface|null $terminal
      * @param MenuStyle|null $style
@@ -204,7 +204,7 @@ 
     /**
      * Move the selection ina  given direction, up / down
      *
-     * @param $direction
+     * @param string $direction
      */
     protected function moveSelection($direction)
     {

Unused Use Statements -2 removed lines

@@ -2,11 +2,9 @@
 
 namespace PhpSchool\CliMenu;
 
-use Assert\Assertion;
 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;

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));
 

src/CliMenuBuilder.php

Doc Comments +1 added lines, -1 removed lines

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

Spacing +5 added lines, -5 removed lines

@@ -367,7 +367,7 @@ 
     {
         $actions = [];
         if ($this->parent) {
-            $actions[] = new SelectableItem($this->goBackButtonText, function (CliMenu $child) {
+            $actions[] = new SelectableItem($this->goBackButtonText, function(CliMenu $child) {
                 if ($parent = $child->getParent()) {
                     $child->closeThis();
                     $parent->open();
@@ -375,7 +375,7 @@ 
             });
         }
         
-        $actions[] = new SelectableItem($this->exitButtonText, function (CliMenu $menu) {
+        $actions[] = new SelectableItem($this->exitButtonText, function(CliMenu $menu) {
             $menu->close();
         });
         return $actions;
@@ -397,7 +397,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();
         }));
     }
@@ -410,7 +410,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;
             }
@@ -460,7 +460,7 @@ 
      */
     private function buildSubMenus(array $items)
     {
-        return array_map(function ($item) {
+        return array_map(function($item) {
             if (!is_string($item)) {
                 return $item;
             }

src/Terminal/WindowsTerminal.php

Doc Comments +2 added lines, -2 removed lines

@@ -53,7 +53,7 @@ 
     /**
      * Check if TTY is in raw mode
      *
-     * @return bool
+     * @return boolean|null
      */
     public function isRaw()
     {
@@ -63,7 +63,7 @@ 
     /**
      * Test whether terminal is valid TTY
      *
-     * @return bool
+     * @return boolean|null
      */
     public function isTTY()
     {

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/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;