Reorganise methods

Author: AydinHassan

src/MenuItem/MenuMenuItem.php

@@ -12,29 +12,38 @@
  */
 class MenuMenuItem implements MenuItemInterface
 {
-    private $text = '';
+    /**
+     * @var string
+     */
+    private $text;
 
-    private $showItemExtra = false;
+    /**
+     * @var CliMenu
+     */
+    private $subMenu;
 
-    private $disabled = false;
+    /**
+     * @var bool
+     */
+    private $showItemExtra;
 
     /**
-     * @var SelectableStyle;
+     * @var bool
      */
-    private $style;
-    
+    private $disabled;
+
     /**
-     * @var CliMenu
+     * @var SelectableStyle;
      */
-    private $subMenu;
+    private $style;
 
     public function __construct(
         string $text,
         CliMenu $subMenu,
         bool $disabled = false
     ) {
-        $this->text     = $text;
-        $this->subMenu  = $subMenu;
+        $this->text = $text;
+        $this->subMenu = $subMenu;
         $this->disabled = $disabled;
 
         $this->style = new SelectableStyle();
@@ -79,41 +88,46 @@ public function getRows(MenuStyle $style, bool $selected = false) : array
     }
 
     /**
-     * Execute the items callable if required
+     * Return the raw string of text
      */
-    public function getSelectAction() : ?callable
+    public function getText() : string
     {
-        return function (CliMenu $menu) {
-            $this->showSubMenu($menu);
-        };
+        return $this->text;
     }
 
-    public function getStyle() : SelectableStyle
+    /**
+     * Set the raw string of text
+     */
+    public function setText(string $text) : void
     {
-        return $this->style;
+        $this->text = $text;
     }
 
-    public function setStyle(SelectableStyle $style) : self
+    /**
+     * Execute the items callable if required
+     */
+    public function getSelectAction() : ?callable
     {
-        $this->style = $style;
-
-        return $this;
+        return function (CliMenu $menu) {
+            $this->showSubMenu($menu);
+        };
     }
 
     /**
-     * Return the raw string of text
+     * Returns the sub menu
      */
-    public function getText() : string
+    public function getSubMenu() : CliMenu
     {
-        return $this->text;
+        return $this->subMenu;
     }
 
     /**
-     * Set the raw string of text
+     * Display the sub menu
      */
-    public function setText(string $text) : void
+    public function showSubMenu(CliMenu $parentMenu) : void
     {
-        $this->text = $text;
+        $parentMenu->closeThis();
+        $this->subMenu->open();
     }
 
     /**
@@ -124,11 +138,6 @@ public function canSelect() : bool
         return !$this->disabled;
     }
 
-    public function showsItemExtra() : bool
-    {
-        return $this->showItemExtra;
-    }
-
     /**
      * Enable showing item extra
      */
@@ -137,6 +146,14 @@ public function showItemExtra() : void
         $this->showItemExtra = true;
     }
 
+    /**
+     * Whether or not we are showing item extra
+     */
+    public function showsItemExtra() : bool
+    {
+        return $this->showItemExtra;
+    }
+
     /**
      * Disable showing item extra
      */
@@ -145,20 +162,15 @@ public function hideItemExtra() : void
         $this->showItemExtra = false;
     }
 
-    /**
-     * Returns the sub menu
-     */
-    public function getSubMenu() : CliMenu
+    public function getStyle() : SelectableStyle
     {
-        return $this->subMenu;
+        return $this->style;
     }
 
-    /**
-     * Display the sub menu
-     */
-    public function showSubMenu(CliMenu $parentMenu) : void
+    public function setStyle(SelectableStyle $style) : self
     {
-        $parentMenu->closeThis();
-        $this->subMenu->open();
+        $this->style = $style;
+
+        return $this;
     }
 }