php-school / cli-menu

src/CliMenu.php

Unused Use Statements -4 removed lines

@@ -2,8 +2,6 @@ 
 
 namespace PhpSchool\CliMenu;
 
-use PhpSchool\CliMenu\Dialogue\NumberInput;
-use PhpSchool\CliMenu\Exception\InvalidInstantiationException;
 use PhpSchool\CliMenu\Exception\InvalidTerminalException;
 use PhpSchool\CliMenu\Exception\MenuNotOpenException;
 use PhpSchool\CliMenu\Input\InputIO;
@@ -17,11 +15,9 @@ 
 use PhpSchool\CliMenu\Dialogue\Flash;
 use PhpSchool\CliMenu\Terminal\TerminalFactory;
 use PhpSchool\CliMenu\Util\StringUtil as s;
-use PhpSchool\Terminal\Exception\NotInteractiveTerminal;
 use PhpSchool\Terminal\InputCharacter;
 use PhpSchool\Terminal\NonCanonicalReader;
 use PhpSchool\Terminal\Terminal;
-use PhpSchool\Terminal\TerminalReader;
 
 /**
  * @author Michael Woodward <mikeymike.mw@gmail.com>

Spacing +3 added lines, -3 removed lines

@@ -279,7 +279,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()) {
@@ -349,7 +349,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));
 
@@ -393,7 +393,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()),

Doc Comments +6 added lines

@@ -87,6 +87,9 @@ 
      */
     private $currentFrame;
 
+    /**
+     * @param string $title
+     */
     public function __construct(
         ?string $title,
         array $items,
@@ -541,6 +544,9 @@ 
         return new Password(new InputIO($this, $this->terminal), $style);
     }
 
+    /**
+     * @param string $text
+     */
     private function guardSingleLine($text) : void
     {
         if (strpos($text, "\n") !== false) {

src/Input/Number.php

Spacing +2 added lines, -2 removed lines

@@ -79,11 +79,11 @@
 
     public function ask() : InputResult
     {
-        $this->inputIO->registerControlCallback(InputCharacter::UP, function (string $input) {
+        $this->inputIO->registerControlCallback(InputCharacter::UP, function(string $input) {
             return $this->validate($input) ? $input + 1 : $input;
         });
 
-        $this->inputIO->registerControlCallback(InputCharacter::DOWN, function (string $input) {
+        $this->inputIO->registerControlCallback(InputCharacter::DOWN, function(string $input) {
             return $this->validate($input) ? $input - 1 : $input;
         });
 

examples/input-password.php

Spacing +3 added lines, -3 removed lines

@@ -3,12 +3,12 @@
 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) {
     $result = $menu->askPassword()
         ->setPlaceholderText('')
-        ->setValidator(function ($password) {
+        ->setValidator(function($password) {
             if ($password === 'password') {
                 $this->setValidationFailedText('Password is too weak');
                 return false;

src/CliMenuBuilder.php

Spacing +3 added lines, -3 removed lines

@@ -152,7 +152,7 @@ 
      */
     public function addSubMenu(string $id, CliMenuBuilder $subMenuBuilder = null) : CliMenuBuilder
     {
-        $this->menuItems[]  = $id;
+        $this->menuItems[] = $id;
         
         if (null === $subMenuBuilder) {
             $this->subMenuBuilders[$id] = new static($this);
@@ -294,7 +294,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();
         }));
     }
@@ -359,7 +359,7 @@ 
 
     private function buildSubMenus(array $items) : array
     {
-        return array_map(function ($item) {
+        return array_map(function($item) {
             if (!is_string($item)) {
                 return $item;
             }

examples/custom-mapping.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');
 
-$exit = function (CliMenu $menu) {
+$exit = function(CliMenu $menu) {
     $menu->close();
 };
 

examples/nested-submenu.php

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

src/Dialogue/Confirm.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
         $this->emptyRow();
 
         $confirmText = sprintf(' < %s > ', $confirmText);
-        $leftFill    = ($promptWidth / 2) - (mb_strlen($confirmText) / 2);
+        $leftFill    = ($promptWidth/2) - (mb_strlen($confirmText)/2);
 
         $this->write(sprintf(
             "%s%s%s%s%s%s%s\n",

src/MenuItem/AsciiArtItem.php

Spacing +3 added lines, -3 removed lines

@@ -41,7 +41,7 @@ 
     {
         Assertion::inArray($position, [self::POSITION_CENTER, self::POSITION_RIGHT, self::POSITION_LEFT]);
         
-        $this->text = implode("\n", array_map(function (string $line) {
+        $this->text = implode("\n", array_map(function(string $line) {
             return rtrim($line, ' ');
         }, explode("\n", $text)));
         $this->position  = $position;
@@ -61,7 +61,7 @@ 
 
         $padding = $style->getContentWidth() - $this->artLength;
         
-        return array_map(function ($row) use ($padding) {
+        return array_map(function($row) use ($padding) {
             switch ($this->position) {
                 case self::POSITION_LEFT:
                     break;
@@ -70,7 +70,7 @@ 
                     break;
                 case self::POSITION_CENTER:
                 default:
-                    $left = ceil($padding / 2);
+                    $left = ceil($padding/2);
                     $row = sprintf('%s%s', str_repeat(' ', $left), $row);
                     break;
             }

src/MenuStyle.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace PhpSchool\CliMenu;
 
-use PhpSchool\CliMenu\Exception\InvalidInstantiationException;
 use PhpSchool\CliMenu\Terminal\TerminalFactory;
 use PhpSchool\CliMenu\Util\ColourUtil;
 use PhpSchool\Terminal\Terminal;

Spacing +2 added lines, -2 removed lines

@@ -313,7 +313,7 @@ 
     protected function calculateContentWidth() : void
     {
         $this->contentWidth = $this->width
-            - ($this->paddingLeftRight * 2)
+            - ($this->paddingLeftRight*2)
             - ($this->borderRightWidth + $this->borderLeftWidth);
 
         if ($this->contentWidth < 0) {
@@ -466,7 +466,7 @@ 
     public function setMarginAuto() : self
     {
         $this->marginAuto = true;
-        $this->margin = floor(($this->terminal->getWidth() - $this->width) / 2);
+        $this->margin = floor(($this->terminal->getWidth() - $this->width)/2);
 
         $this->generateBorderRows();
         $this->generatePaddingTopBottomRows();