Apply fixes from StyleCI

Author: StyleCIBot

src/Contracts/ItemRenderer.php

@@ -1,4 +1,6 @@
 <?php
+
+declare(strict_types=1);
 /**
  * Contains the ItemRenderer interface.
  *

src/Contracts/MenuRenderer.php

@@ -1,4 +1,6 @@
 <?php
+
+declare(strict_types=1);
 /**
  * Contains the MenuRenderer interface.
  *

src/Exceptions/DuplicateItemNameException.php

@@ -1,4 +1,6 @@
 <?php
+
+declare(strict_types=1);
 /**
  * Contains the DuplicateItemNameException class.
  *

src/Exceptions/InvalidMenuConfigurationException.php

@@ -1,4 +1,6 @@
 <?php
+
+declare(strict_types=1);
 /**
  * Contains the InvalidMenuConfigurationException class.
  *

src/Exceptions/MenuAlreadyExistsException.php

@@ -1,4 +1,6 @@
 <?php
+
+declare(strict_types=1);
 /**
  * Contains the MenuAlreadyExistsException class.
  *

src/Exceptions/MenuItemNotFoundException.php

@@ -1,4 +1,6 @@
 <?php
+
+declare(strict_types=1);
 /**
  * Contains the MenuItemNotFoundException class.
  *

src/Facades/Menu.php

@@ -1,4 +1,6 @@
 <?php
+
+declare(strict_types=1);
 /**
  * Contains the Menu facade class.
  *

src/Item.php

@@ -1,4 +1,6 @@
 <?php
+
+declare(strict_types=1);
 /**
  * Contains the Menu Item class.
  *
@@ -63,21 +65,41 @@ class Item
      */
     public function __construct(Menu $menu, $name, $title, $options)
     {
-        $this->menu       = $menu;
-        $this->name       = $name;
-        $this->title      = $title;
+        $this->menu = $menu;
+        $this->name = $name;
+        $this->title = $title;
         $this->attributes = Arr::except($options, $this->reserved);
-        $this->parent     = $this->resolveParent(Arr::get($options, 'parent', null));
-        $this->renderer   = Arr::get($options, 'renderer', null);
+        $this->parent = $this->resolveParent(Arr::get($options, 'parent', null));
+        $this->renderer = Arr::get($options, 'renderer', null);
 
-        $path       = Arr::only($options, array('url', 'route', 'action'));
+        $path = Arr::only($options, ['url', 'route', 'action']);
         if (!empty($path)) {
             $this->link = new Link($path, $this->menu->config->activeClass);
         }
 
         $this->checkActivation();
     }
 
+    /**
+     * Search in meta data if a property doesn't exist otherwise return the property
+     *
+     * @param  string
+     *
+     * @return string
+     */
+    public function __get($prop)
+    {
+        if (property_exists($this, $prop)) {
+            return $this->$prop;
+        }
+
+        if ($this->children()->has($prop)) {
+            return $this->children()->get($prop);
+        }
+
+        return $this->data($prop);
+    }
+
     /**
      * Creates a sub Item
      *
@@ -89,7 +111,7 @@ public function __construct(Menu $menu, $name, $title, $options)
      */
     public function addSubItem($name, $title, $options = [])
     {
-        $options           = is_array($options) ? $options : ['url' => $options];
+        $options = is_array($options) ? $options : ['url' => $options];
         $options['parent'] = $this;
 
         return $this->menu->addItem($name, $title, $options);
@@ -239,15 +261,15 @@ public function childrenAllowed(Authenticatable $user = null)
      */
     public function hasParent()
     {
-        return (bool)$this->parent;
+        return (bool) $this->parent;
     }
 
     /**
      * Sets the item as active
      */
     public function activate()
     {
-        if ($this->menu->config->activeElement == 'item') {
+        if ('item' == $this->menu->config->activeElement) {
             $this->setToActive();
         } else {
             if ($this->link) {
@@ -330,7 +352,7 @@ public function hasData($key)
      */
     public function hasLink()
     {
-        return (bool)$this->link;
+        return (bool) $this->link;
     }
 
     /**
@@ -362,26 +384,6 @@ public function hasProperty($property)
             $this->hasData($property);
     }
 
-    /**
-     * Search in meta data if a property doesn't exist otherwise return the property
-     *
-     * @param  string
-     *
-     * @return string
-     */
-    public function __get($prop)
-    {
-        if (property_exists($this, $prop)) {
-            return $this->$prop;
-        }
-
-        if ($this->children()->has($prop)) {
-            return $this->children()->get($prop);
-        }
-
-        return $this->data($prop);
-    }
-
     /**
      * Activate the item if it's enabled in menu config and item's url matches the request URI
      */
@@ -441,7 +443,7 @@ private function resolveParent($parent)
         throw new MenuItemNotFoundException(
             sprintf(
                 'Item named `%s` could not be found in the `%s` menu',
-                (string)$parent,
+                (string) $parent,
                 $this->menu->name
             )
         );

src/ItemCollection.php

@@ -1,4 +1,6 @@
 <?php
+
+declare(strict_types=1);
 /**
  * Contains the Menu Item Collection class.
  *
@@ -16,6 +18,27 @@
 
 class ItemCollection extends Collection
 {
+    /**
+     * Search the items based on an attribute
+     *
+     * @param string $method
+     * @param array  $args
+     *
+     * @return \Konekt\Menu\ItemCollection
+     */
+    public function __call($method, $args)
+    {
+        preg_match('/^[W|w]here([a-zA-Z0-9_]+)$/', $method, $matches);
+
+        if (!$matches) {
+            trigger_error('Call to undefined method ' . __CLASS__ . '::' . $method . '()', E_USER_ERROR);
+        }
+
+        $attribute = strtolower($matches[1]);
+        $value = $args ? $args[0] : null;
+
+        return $this->filterByProperty($attribute, $value);
+    }
     /**
      * Alias to addItem. Needed for Laravel 5.8 compatibility
      * @see https://github.com/artkonekt/menu/issues/3
@@ -183,28 +206,6 @@ public function haveParent()
         });
     }
 
-    /**
-     * Search the items based on an attribute
-     *
-     * @param string $method
-     * @param array  $args
-     *
-     * @return \Konekt\Menu\ItemCollection
-     */
-    public function __call($method, $args)
-    {
-        preg_match('/^[W|w]here([a-zA-Z0-9_]+)$/', $method, $matches);
-
-        if (!$matches) {
-            trigger_error('Call to undefined method ' . __CLASS__ . '::' . $method . '()', E_USER_ERROR);
-        }
-
-        $attribute = strtolower($matches[1]);
-        $value     = $args ? $args[0] : null;
-
-        return $this->filterByProperty($attribute, $value);
-    }
-
     /**
      * @param      $property
      * @param      $value

src/Link.php

@@ -1,4 +1,6 @@
 <?php
+
+declare(strict_types=1);
 /**
  * Contains the Menu Link class.
  *
@@ -18,6 +20,9 @@ class Link
 {
     use HasAttributes;
 
+    /** @var bool   Flag for active state */
+    public $isActive = false;
+
     /** @var array  Path Information */
     protected $path = [];
 
@@ -27,9 +32,6 @@ class Link
     /** @var string Explicit href for the link */
     protected $href;
 
-    /** @var bool   Flag for active state */
-    public $isActive = false;
-
     /**
      * Class constructor
      *
@@ -38,18 +40,35 @@ class Link
      */
     public function __construct($path = [], $activeClass = 'active')
     {
-        $this->path        = $path;
+        $this->path = $path;
         $this->activeClass = $activeClass;
     }
 
+    /**
+     * Check for a method of the same name if the attribute doesn't exist.
+     *
+     * @param  string $property
+     *
+     * @return mixed
+     */
+    public function __get($property)
+    {
+        return $this->attr($property);
+    }
+
+    public function __set($property, $value)
+    {
+        return $this->attr($property, $value);
+    }
+
     /**
      * Make the anchor active
      *
      * @return static
      */
     public function activate()
     {
-        $this->isActive            = true;
+        $this->isActive = true;
         $this->attributes['class'] = Utils::addHtmlClass(
             Arr::get($this->attributes, 'class', ''),
             $this->activeClass
@@ -92,23 +111,6 @@ public function url()
         return null;
     }
 
-    /**
-     * Check for a method of the same name if the attribute doesn't exist.
-     *
-     * @param  string $property
-     *
-     * @return mixed
-     */
-    public function __get($property)
-    {
-        return $this->attr($property);
-    }
-
-    public function __set($property, $value)
-    {
-        return $this->attr($property, $value);
-    }
-
     /**
      * Get the action for "url" option.
      *
@@ -118,7 +120,7 @@ protected function getUrl()
     {
         $url = $this->path['url'];
 
-        $uri    = is_array($url) ? $url[0] : $url;
+        $uri = is_array($url) ? $url[0] : $url;
         $params = is_array($url) ? array_slice($url, 1) : null;
 
         if (Utils::isAbsoluteUrl($uri)) {