nWidart / laravel-menus

src/Menu.php

Doc Comments +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
      * Create new menu.
      *
      * @param string   $name
-     * @param Callable $resolver
+     * @param Closure $resolver
      *
      * @return \Nwidart\Menus\MenuBuilder
      */

Spacing +1 added lines, -1 removed lines

@@ -93,7 +93,7 @@
      */
     public function modify($name, Closure $callback)
     {
-        $menu = collect($this->menus)->filter(function ($menu) use ($name) {
+        $menu = collect($this->menus)->filter(function($menu) use ($name) {
             return $menu->getName() == $name;
         })->first();
 

src/MenuBuilder.php

Doc Comments +13 added lines, -12 removed lines

@@ -215,7 +215,7 @@ 
     /**
      * Determine if the given name in the presenter style.
      *
-     * @param $name
+     * @param string|null $name
      *
      * @return bool
      */
@@ -239,7 +239,7 @@ 
      *
      * @param $name
      *
-     * @return mixed
+     * @return string
      */
     public function getStyle($name)
     {
@@ -251,7 +251,7 @@ 
     /**
      * Set new presenter class from given alias name.
      *
-     * @param $name
+     * @param string|null $name
      */
     public function setPresenterFromStyle($name)
     {
@@ -261,7 +261,6 @@ 
     /**
      * Set the resolved item bindings
      *
-     * @param array $arr
      */
     public function setBindings(array $bindings)
     {
@@ -332,10 +331,10 @@ 
      * Create new menu with dropdown.
      *
      * @param $title
-     * @param callable $callback
+     * @param \Closure $callback
      * @param array    $attributes
      *
-     * @return $this
+     * @return MenuItem
      */
     public function dropdown($title, \Closure $callback, $order = null, array $attributes = array())
     {
@@ -367,7 +366,7 @@ 
      * @param array $parameters
      * @param array $attributes
      *
-     * @return static
+     * @return MenuItem
      */
     public function route($route, $title, $parameters = array(), $order = null, $attributes = array())
     {
@@ -409,11 +408,11 @@ 
     /**
      * Register new menu item using url.
      *
-     * @param $url
-     * @param $title
+     * @param string $url
+     * @param string $title
      * @param array $attributes
      *
-     * @return static
+     * @return MenuItem
      */
     public function url($url, $title, $order = 0, $attributes = array())
     {
@@ -440,7 +439,7 @@ 
      * Add new divider item.
      *
      * @param int $order
-     * @return \Nwidart\Menus\MenuItem
+     * @return MenuBuilder
      */
     public function addDivider($order = null)
     {
@@ -452,7 +451,8 @@ 
     /**
      * Add new header item.
      *
-     * @return \Nwidart\Menus\MenuItem
+     * @param string $title
+     * @return MenuBuilder
      */
     public function addHeader($title, $order = null)
     {
@@ -536,6 +536,7 @@ 
     /**
      * Render menu via view presenter.
      *
+     * @param string $presenter
      * @return \Illuminate\View\View
      */
     public function renderView($presenter = null)

Spacing +3 added lines, -3 removed lines

@@ -119,7 +119,7 @@ 
      */
     public function findBy($key, $value)
     {
-        return collect($this->items)->filter(function ($item) use ($key, $value) {
+        return collect($this->items)->filter(function($item) use ($key, $value) {
             return $item->{$key} == $value;
         })->first();
     }
@@ -303,7 +303,7 @@ 
      */
     protected function resolveItems(array &$items)
     {
-        $resolver = function ($property) {
+        $resolver = function($property) {
             return $this->resolve($property) ?: $property;
         };
 
@@ -607,7 +607,7 @@ 
     public function getOrderedItems()
     {
         if (config('menus.ordering') || $this->ordering) {
-            return $this->toCollection()->sortBy(function ($item) {
+            return $this->toCollection()->sortBy(function($item) {
                 return $item->order;
             })->all();
         }

src/MenuItem.php

Doc Comments +6 added lines, -6 removed lines

@@ -138,7 +138,7 @@ 
      * Register new child menu with dropdown.
      *
      * @param $title
-     * @param callable $callback
+     * @param Closure $callback
      *
      * @return $this
      */
@@ -172,7 +172,7 @@ 
      * @param array $parameters
      * @param array $attributes
      *
-     * @return array
+     * @return MenuItem
      */
     public function route($route, $title, $parameters = array(), $order = 0, $attributes = array())
     {
@@ -198,7 +198,7 @@ 
      * @param $title
      * @param array $attributes
      *
-     * @return array
+     * @return MenuItem
      */
     public function url($url, $title, $order = 0, $attributes = array())
     {
@@ -383,7 +383,7 @@ 
     /**
      * Check is the current item divider.
      *
-     * @param $name
+     * @param string $name
      *
      * @return bool
      */
@@ -415,7 +415,7 @@ 
     /**
      * Check the active state for current menu.
      *
-     * @return mixed
+     * @return boolean
      */
     public function hasActiveOnChild()
     {
@@ -567,7 +567,7 @@ 
      * Set hide condition for current menu item.
      *
      * @param  Closure
-     * @return boolean
+     * @return MenuItem
      */
     public function hideWhen(Closure $callback)
     {

Spacing +1 added lines, -1 removed lines

@@ -296,7 +296,7 @@
     public function getChilds()
     {
         if (config('menus.ordering')) {
-            return collect($this->childs)->sortBy(function ($child) {
+            return collect($this->childs)->sortBy(function($child) {
                 return $child->order;
             })->all();
         }

src/Presenters/Bootstrap/NavbarPresenter.php

Doc Comments +2 added lines, -1 removed lines

@@ -32,6 +32,7 @@ 
 
     /**
      * {@inheritdoc }.
+     * @param \Nwidart\Menus\MenuItem $item
      */
     public function getActiveState($item, $state = ' class="active"')
     {
@@ -41,7 +42,7 @@ 
     /**
      * Get active state on child items.
      *
-     * @param $item
+     * @param \Nwidart\Menus\MenuItem $item
      * @param string $state
      *
      * @return null|string

src/Presenters/Bootstrap/SidebarMenuPresenter.php

Doc Comments +2 added lines, -1 removed lines

@@ -42,6 +42,7 @@ 
 
     /**
      * {@inheritdoc }.
+     * @param \Nwidart\Menus\MenuItem $item
      */
     public function getActiveState($item, $state = ' class="active"')
     {
@@ -51,7 +52,7 @@ 
     /**
      * Get active state on child items.
      *
-     * @param $item
+     * @param \Nwidart\Menus\MenuItem $item
      * @param string $state
      *
      * @return null|string

Tests/MenuTest.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
     /** @test */
     public function it_generates_an_empty_menu()
     {
-        $this->menu->create('test', function (MenuBuilder $menu) {
+        $this->menu->create('test', function(MenuBuilder $menu) {
         });
 
         $expected = <<<TEXT

src/MenusServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     {
         $this->registerHtmlPackage();
 
-        $this->app['menus'] = $this->app->share(function ($app) {
+        $this->app['menus'] = $this->app->share(function($app) {
             return new Menu($app['view'], $app['config']);
         });
     }