nWidart / laravel-menus

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/MenuBuilder.php

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

Doc Comments +13 added lines, -11 removed lines

@@ -220,7 +220,7 @@ 
     /**
      * Determine if the given name in the presenter style.
      *
-     * @param $name
+     * @param string|null $name
      *
      * @return bool
      */
@@ -244,7 +244,7 @@ 
      *
      * @param $name
      *
-     * @return mixed
+     * @return string
      */
     public function getStyle($name)
     {
@@ -256,7 +256,7 @@ 
     /**
      * Set new presenter class from given alias name.
      *
-     * @param $name
+     * @param string|null $name
      */
     public function setPresenterFromStyle($name)
     {
@@ -339,10 +339,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())
     {
@@ -374,7 +374,7 @@ 
      * @param array $parameters
      * @param array $attributes
      *
-     * @return static
+     * @return MenuItem
      */
     public function route($route, $title, $parameters = array(), $order = null, $attributes = array())
     {
@@ -416,11 +416,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())
     {
@@ -447,7 +447,7 @@ 
      * Add new divider item.
      *
      * @param int $order
-     * @return \Nwidart\Menus\MenuItem
+     * @return MenuBuilder
      */
     public function addDivider($order = null)
     {
@@ -459,7 +459,8 @@ 
     /**
      * Add new header item.
      *
-     * @return \Nwidart\Menus\MenuItem
+     * @param string $title
+     * @return MenuBuilder
      */
     public function addHeader($title, $order = null)
     {
@@ -543,6 +544,7 @@ 
     /**
      * Render menu via view presenter.
      *
+     * @param string $presenter
      * @return \Illuminate\View\View
      */
     public function renderView($presenter = null)

src/Menu.php

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/Presenters/Bootstrap/SidebarMenuPresenter.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/MenuItem.php

Doc Comments +10 added lines, -10 removed lines

@@ -148,8 +148,8 @@ 
     /**
      * Register new child menu with dropdown.
      *
-     * @param $title
-     * @param callable $callback
+     * @param string $title
+     * @param Closure $callback
      *
      * @return $this
      */
@@ -178,8 +178,8 @@ 
     /**
      * Create new menu item and set the action to route.
      *
-     * @param $route
-     * @param $title
+     * @param string $route
+     * @param string $title
      * @param array $parameters
      * @param array $attributes
      *
@@ -205,8 +205,8 @@ 
     /**
      * Create new menu item  and set the action to url.
      *
-     * @param $url
-     * @param $title
+     * @param string $url
+     * @param string $title
      * @param array $attributes
      *
      * @return MenuItem
@@ -290,7 +290,7 @@ 
     /**
      * Same with "addHeader" method.
      *
-     * @param $title
+     * @param string $title
      *
      * @return $this
      */
@@ -403,7 +403,7 @@ 
     /**
      * Check is the current item divider.
      *
-     * @param $name
+     * @param string $name
      *
      * @return bool
      */
@@ -435,7 +435,7 @@ 
     /**
      * Check the active state for current menu.
      *
-     * @return mixed
+     * @return boolean
      */
     public function hasActiveOnChild()
     {
@@ -587,7 +587,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

@@ -326,7 +326,7 @@
             return route($this->route[0], $this->route[1]);
         }
         
-        if(empty($this->url))
+        if (empty($this->url))
             return url("/#");
         
         return url($this->url);

src/MenusServiceProvider.php

Spacing +1 added lines, -1 removed lines

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

Tests/MenuTest.php

Spacing +13 added lines, -13 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
@@ -37,7 +37,7 @@ 
     /** @test */
     public function it_makes_is_an_alias_for_create()
     {
-        $this->menu->make('test', function (MenuBuilder $menu) {
+        $this->menu->make('test', function(MenuBuilder $menu) {
         });
 
         $expected = <<<TEXT
@@ -54,7 +54,7 @@ 
     /** @test */
     public function it_render_is_an_alias_of_get()
     {
-        $this->menu->make('test', function (MenuBuilder $menu) {
+        $this->menu->make('test', function(MenuBuilder $menu) {
         });
 
         $expected = <<<TEXT
@@ -71,7 +71,7 @@ 
     /** @test */
     public function it_can_get_the_instance_of_a_menu()
     {
-        $this->menu->create('test', function (MenuBuilder $menu) {
+        $this->menu->create('test', function(MenuBuilder $menu) {
         });
 
         $this->assertInstanceOf(MenuBuilder::class, $this->menu->instance('test'));
@@ -80,10 +80,10 @@ 
     /** @test */
     public function it_can_modify_a_menu_instance()
     {
-        $this->menu->create('test', function (MenuBuilder $menu) {
+        $this->menu->create('test', function(MenuBuilder $menu) {
         });
 
-        $this->menu->modify('test', function (MenuBuilder $builder) {
+        $this->menu->modify('test', function(MenuBuilder $builder) {
             $builder->url('hello', 'world');
         });
 
@@ -93,7 +93,7 @@ 
     /** @test */
     public function it_gets_a_partial_for_dropdown_styles()
     {
-        $this->menu->create('test', function (MenuBuilder $menu) {
+        $this->menu->create('test', function(MenuBuilder $menu) {
         });
 
         $this->assertContains('.dropdown-submenu', $this->menu->style());
@@ -102,9 +102,9 @@ 
     /** @test */
     public function it_can_get_all_menus()
     {
-        $this->menu->create('main', function (MenuBuilder $menu) {
+        $this->menu->create('main', function(MenuBuilder $menu) {
         });
-        $this->menu->create('footer', function (MenuBuilder $menu) {
+        $this->menu->create('footer', function(MenuBuilder $menu) {
         });
 
         $this->assertCount(2, $this->menu->all());
@@ -113,9 +113,9 @@ 
     /** @test */
     public function it_can_count_menus()
     {
-        $this->menu->create('main', function (MenuBuilder $menu) {
+        $this->menu->create('main', function(MenuBuilder $menu) {
         });
-        $this->menu->create('footer', function (MenuBuilder $menu) {
+        $this->menu->create('footer', function(MenuBuilder $menu) {
         });
 
         $this->assertEquals(2, $this->menu->count());
@@ -124,9 +124,9 @@ 
     /** @test */
     public function it_can_destroy_all_menus()
     {
-        $this->menu->create('main', function (MenuBuilder $menu) {
+        $this->menu->create('main', function(MenuBuilder $menu) {
         });
-        $this->menu->create('footer', function (MenuBuilder $menu) {
+        $this->menu->create('footer', function(MenuBuilder $menu) {
         });
 
         $this->assertCount(2, $this->menu->all());

src/Presenters/Admin/AdminltePresenter.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/Foundation/ZurbMenuPresenter.php

Spacing +3 added lines, -3 removed lines

@@ -28,7 +28,7 @@ 
      */
     public function getMenuWithoutDropdownWrapper($item)
     {
-        return '<li'.$this->getActiveState($item).'><a href="'. $item->getUrl() .'">'.$item->title.'</a></li>';
+        return '<li' . $this->getActiveState($item) . '><a href="' . $item->getUrl() . '">' . $item->title . '</a></li>';
     }
 
     /**
@@ -53,9 +53,9 @@ 
     public function getMenuWithDropDownWrapper($item)
     {
         return '<li class="dropdown dropdown-primary">
-                    <a class="dropdown-toggle" href="#">'.$item->title.'</a>
+                    <a class="dropdown-toggle" href="#">'.$item->title . '</a>
                     <ul class="menu">
-                      '.$this->getChildMenuItems($item).'
+                      '.$this->getChildMenuItems($item) . '
                     </ul>
                 </li>' . PHP_EOL;
     }