php-school / cli-menu

src/CliMenu.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace PhpSchool\CliMenu;
 
-use PhpSchool\CliMenu\Exception\InvalidInstantiationException;
 use PhpSchool\CliMenu\Exception\InvalidTerminalException;
 use PhpSchool\CliMenu\Exception\MenuNotOpenException;
 use PhpSchool\CliMenu\MenuItem\LineBreakItem;

Spacing +3 added lines, -3 removed lines

@@ -215,7 +215,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()) {
@@ -275,7 +275,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));
 
@@ -309,7 +309,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\r",
                 str_repeat(' ', $this->style->getMargin()),

src/CliMenuBuilder.php

Doc Comments +2 added lines, -2 removed lines

@@ -375,7 +375,7 @@ 
     }
 
     /**
-     * @return array
+     * @return SelectableItem[]
      */
     private function getDefaultItems()
     {
@@ -463,7 +463,7 @@ 
 
     /**
      * @param string $id
-     * @return CliMenu
+     * @return CliMenuBuilder
      * @throws RuntimeException
      */
     public function getSubMenu($id)

Spacing +2 added lines, -2 removed lines

@@ -404,7 +404,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();
         }));
     }
@@ -481,7 +481,7 @@ 
      */
     private function buildSubMenus(array $items)
     {
-        return array_map(function ($item) {
+        return array_map(function($item) {
             if (!is_string($item)) {
                 return $item;
             }

src/MenuStyle.php

Doc Comments +2 added lines, -2 removed lines

@@ -376,8 +376,8 @@
     /**
      * Get padding for right had side of content
      *
-     * @param $contentLength
-     * @return int
+     * @param integer $contentLength
+     * @return double
      */
     public function getRightHandPadding($contentLength)
     {

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

examples/confirm.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();
 };
 

examples/toggle-item-extra.php

Spacing +3 added lines, -3 removed lines

@@ -4,13 +4,13 @@
 use PhpSchool\CliMenu\CliMenuBuilder;
 use PhpSchool\CliMenu\MenuItem\MenuItemInterface;
 
-require_once(__DIR__ . '/../vendor/autoload.php');
+require_once(__DIR__.'/../vendor/autoload.php');
 
-$itemCallable = function (CliMenu $menu) {
+$itemCallable = function(CliMenu $menu) {
     static $i = 1;
 
     foreach ($menu->getItems() as $item) {
-        $i % 2 === 0
+        $i%2 === 0
             ? $item->showItemExtra()
             : $item->hideItemExtra();
 

examples/flash.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();
 };
 

examples/crazy-redraw.php

Spacing +7 added lines, -7 removed lines

@@ -5,13 +5,13 @@ 
 use PhpSchool\CliMenu\MenuItem\LineBreakItem;
 use PhpSchool\CliMenu\MenuItem\MenuItemInterface;
 
-require_once(__DIR__ . '/../vendor/autoload.php');
+require_once(__DIR__.'/../vendor/autoload.php');
 
-$itemCallable = function (CliMenu $menu) {
-    $colour = function () {
+$itemCallable = function(CliMenu $menu) {
+    $colour = function() {
         return array_rand(array_flip(['blue', 'green', 'red', 'yellow']));
     };
-    $int = function () {
+    $int = function() {
         return rand(1, 20);
     };
 
@@ -26,11 +26,11 @@ 
 
     $items = $menu->getItems();
     
-    array_walk($items, function (MenuItemInterface $item) use ($menu) {
+    array_walk($items, function(MenuItemInterface $item) use ($menu) {
         $menu->removeItem($item);
     });
 
-    $items = array_filter($items, function (MenuItemInterface $item) {
+    $items = array_filter($items, function(MenuItemInterface $item) {
         return !$item instanceof LineBreakItem;
     });
 
@@ -41,7 +41,7 @@ 
 
     array_walk(
         $items,
-        function (MenuItemInterface $item) use ($menu) {
+        function(MenuItemInterface $item) use ($menu) {
             $menu->addItem($item);
         }
     );

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

Spacing +3 added lines, -3 removed lines

@@ -75,12 +75,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);
+        $dialogueHalfLength = (mb_strlen($this->text) + ($this->style->getPadding()*2))/2;
+        $widthHalfLength    = ceil($parentStyle->getWidth()/2);
         $this->x            = $widthHalfLength - $dialogueHalfLength;
     }