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

@@ -204,7 +204,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()) {
@@ -265,7 +265,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));
 
@@ -295,7 +295,7 @@ 
             ? $this->style->getSelectedUnsetCode()
             : $this->style->getUnselectedUnsetCode();
 
-        return array_map(function ($row) use ($setColour, $unsetColour) {
+        return array_map(function($row) use ($setColour, $unsetColour) {
             return sprintf(
                 "%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,
@@ -498,6 +501,9 @@ 
         return new Password(new InputIO($this, $this->terminal), $style);
     }
 
+    /**
+     * @param string $text
+     */
     private function guardSingleLine($text)
     {
         if (strpos($text, "\n") !== false) {

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\Terminal\Terminal;
 

Spacing +2 added lines, -2 removed lines

@@ -239,7 +239,7 @@ 
      */
     protected function calculateContentWidth() : void
     {
-        $this->contentWidth = $this->width - ($this->padding * 2);
+        $this->contentWidth = $this->width - ($this->padding*2);
     }
 
     public function getFg() : string
@@ -308,7 +308,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);
         
         return $this;
     }

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

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%s%s\n",

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/Dialogue.php

Spacing +3 added lines, -3 removed lines

@@ -69,12 +69,12 @@
     {
         //y
         $textLines          = count(explode("\n", $this->text)) + 2;
-        $this->y            = ceil($this->parentMenu->getCurrentFrame()->count() / 2) - ceil($textLines / 2) + 1;
+        $this->y            = ceil($this->parentMenu->getCurrentFrame()->count()/2) - ceil($textLines/2) + 1;
 
         //x
         $parentStyle        = $this->parentMenu->getStyle();
-        $dialogueHalfLength = (mb_strlen($this->text) + ($this->style->getPadding() * 2)) / 2;
-        $widthHalfLength    = ceil($parentStyle->getWidth() / 2 + $parentStyle->getMargin());
+        $dialogueHalfLength = (mb_strlen($this->text) + ($this->style->getPadding()*2))/2;
+        $widthHalfLength    = ceil($parentStyle->getWidth()/2 + $parentStyle->getMargin());
         $this->x            = $widthHalfLength - $dialogueHalfLength;
     }