php-school / cli-menu

src/MenuItem/SelectableTrait.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
             )
         );
 
-        return array_map(function ($row, $key) use ($style, $length) {
+        return array_map(function($row, $key) use ($style, $length) {
             $text = $this->disabled ? $style->getDisabledItemText($row) : $row;
 
             if ($key === 0) {

src/MenuStyle.php

Spacing +1 added lines, -1 removed lines

@@ -267,7 +267,7 @@
 
     public function setWidth(int $width) : self
     {
-        $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;

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;
 

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/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/Input/InputIO.php

Spacing +6 added lines, -6 removed lines

@@ -96,7 +96,7 @@ 
     {
         return max(
             array_map(
-                function (string $line) {
+                function(string $line) {
                     return mb_strlen($line);
                 },
                 $lines
@@ -108,14 +108,14 @@ 
     {
         $lines = 5; //1. empty 2. prompt text 3. empty 4. input 5. empty
 
-        return ceil($this->parentMenu->getCurrentFrame()->count() / 2) - ceil($lines /2) + 1;
+        return ceil($this->parentMenu->getCurrentFrame()->count()/2) - ceil($lines/2) + 1;
     }
 
     private function calculateYPositionWithError() : int
     {
         $lines = 7; //1. empty 2. prompt text 3. empty 4. input 5. empty 6. error 7. empty
 
-        return ceil($this->parentMenu->getCurrentFrame()->count() / 2) - ceil($lines /2) + 1;
+        return ceil($this->parentMenu->getCurrentFrame()->count()/2) - ceil($lines/2) + 1;
     }
 
     private function calculateXPosition(Input $input, string $userInput) : int
@@ -129,8 +129,8 @@ 
         );
 
         $parentStyle     = $this->parentMenu->getStyle();
-        $halfWidth       = ($width + ($input->getStyle()->getPadding() * 2)) / 2;
-        $parentHalfWidth = ceil($parentStyle->getWidth() / 2);
+        $halfWidth       = ($width + ($input->getStyle()->getPadding()*2))/2;
+        $parentHalfWidth = ceil($parentStyle->getWidth()/2);
 
         return $parentHalfWidth - $halfWidth;
     }
@@ -162,7 +162,7 @@ 
         );
 
         $textLength = mb_strlen(StringUtil::stripAnsiEscapeSequence($text));
-        $leftFill   = ($width / 2) - ($textLength / 2);
+        $leftFill   = ($width/2) - ($textLength/2);
         $rightFill  = ceil($width - $leftFill - $textLength);
 
         $this->drawLine(

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