spinzar / menu

src/MenuBuilder.php

Doc Comments +11 added lines, -9 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)
     {
@@ -341,10 +341,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())
     {
@@ -376,7 +376,7 @@ 
      * @param array $parameters
      * @param array $attributes
      *
-     * @return static
+     * @return MenuItem
      */
     public function route($route, $title, $parameters = array(), $order = null, $attributes = array())
     {
@@ -422,7 +422,7 @@ 
      * @param $title
      * @param array $attributes
      *
-     * @return static
+     * @return MenuItem
      */
     public function url($url, $title, $order = 0, $attributes = array())
     {
@@ -449,7 +449,7 @@ 
      * Add new divider item.
      *
      * @param int $order
-     * @return \Spinzar\Menu\MenuItem
+     * @return MenuBuilder
      */
     public function addDivider($order = null)
     {
@@ -461,7 +461,8 @@ 
     /**
      * Add new header item.
      *
-     * @return \Spinzar\Menu\MenuItem
+     * @param string $title
+     * @return MenuBuilder
      */
     public function addHeader($title, $order = null)
     {
@@ -545,6 +546,7 @@ 
     /**
      * Render menu via view presenter.
      *
+     * @param string $presenter
      * @return \Illuminate\View\View
      */
     public function renderView($presenter = null)

Spacing +5 added lines, -5 removed lines

@@ -124,7 +124,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();
     }
@@ -295,7 +295,7 @@ 
             
             foreach ($matches as $match) {
                 if (array_key_exists($match[1], $this->bindings)) {
-                    $key = preg_replace('/' . $match[0] . '/', $this->bindings[$match[1]], $key, 1);
+                    $key = preg_replace('/'.$match[0].'/', $this->bindings[$match[1]], $key, 1);
                 }
             }
         }
@@ -311,7 +311,7 @@ 
      */
     protected function resolveItems(array &$items)
     {
-        $resolver = function ($property) {
+        $resolver = function($property) {
             return $this->resolve($property) ?: $property;
         };
 
@@ -410,7 +410,7 @@ 
      */
     protected function formatUrl($url)
     {
-        $uri = !is_null($this->prefixUrl) ? $this->prefixUrl . $url : $url;
+        $uri = !is_null($this->prefixUrl) ? $this->prefixUrl.$url : $url;
 
         return $uri == '/' ? '/' : ltrim(rtrim($uri, '/'), '/');
     }
@@ -616,7 +616,7 @@ 
     public function getOrderedItems()
     {
         if (config('menu.ordering') || $this->ordering) {
-            return $this->toCollection()->sortBy(function ($item) {
+            return $this->toCollection()->sortBy(function($item) {
                 return $item->order;
             })->all();
         }

src/Presenters/Admin/Adminlte.php

Doc Comments +2 added lines, -1 removed lines

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

Spacing +10 added lines, -10 removed lines

@@ -11,7 +11,7 @@ 
      */
     public function getOpenTagWrapper()
     {
-        return PHP_EOL . '<ul class="sidebar-menu tree" data-widget="tree">' . PHP_EOL;
+        return PHP_EOL.'<ul class="sidebar-menu tree" data-widget="tree">'.PHP_EOL;
     }
 
     /**
@@ -19,7 +19,7 @@ 
      */
     public function getCloseTagWrapper()
     {
-        return PHP_EOL . '</ul>' . PHP_EOL;
+        return PHP_EOL.'</ul>'.PHP_EOL;
     }
 
     /**
@@ -27,7 +27,7 @@ 
      */
     public function getMenuWithoutDropdownWrapper($item)
     {
-        return '<li' . $this->getActiveState($item) . '><a href="' . $item->getUrl() . '" ' . $item->getAttributes() . '>' . $item->getIcon() . ' <span>' . $item->title . '</span></a></li>' . PHP_EOL;
+        return '<li'.$this->getActiveState($item).'><a href="'.$item->getUrl().'" '.$item->getAttributes().'>'.$item->getIcon().' <span>'.$item->title.'</span></a></li>'.PHP_EOL;
     }
 
     /**
@@ -64,7 +64,7 @@ 
      */
     public function getHeaderWrapper($item)
     {
-        return '<li class="header">' . $item->title . '</li>';
+        return '<li class="header">'.$item->title.'</li>';
     }
 
     /**
@@ -72,15 +72,15 @@ 
      */
     public function getMenuWithDropDownWrapper($item)
     {
-        return '<li class="treeview' . $this->getActiveStateOnChild($item, ' active') . '">
+        return '<li class="treeview'.$this->getActiveStateOnChild($item, ' active').'">
 		          <a href="#">
-					' . $item->getIcon() . ' <span>' . $item->title . '</span>
+					' . $item->getIcon().' <span>'.$item->title.'</span>
                     <span class="pull-right-container">
                       <i class="fa fa-angle-left pull-right"></i>
                     </span>
 			      </a>
 			      <ul class="treeview-menu">
-			      	' . $this->getChildMenuItems($item) . '
+			      	' . $this->getChildMenuItems($item).'
 			      </ul>
 		      	</li>'
         . PHP_EOL;
@@ -95,15 +95,15 @@ 
      */
     public function getMultiLevelDropdownWrapper($item)
     {
-        return '<li class="treeview' . $this->getActiveStateOnChild($item, ' active') . '">
+        return '<li class="treeview'.$this->getActiveStateOnChild($item, ' active').'">
 		          <a href="#">
-					' . $item->getIcon() . ' <span>' . $item->title . '</span>
+					' . $item->getIcon().' <span>'.$item->title.'</span>
 			      	<span class="pull-right-container">
                       <i class="fa fa-angle-left pull-right"></i>
                     </span>
 			      </a>
 			      <ul class="treeview-menu">
-			      	' . $this->getChildMenuItems($item) . '
+			      	' . $this->getChildMenuItems($item).'
 			      </ul>
 		      	</li>'
         . PHP_EOL;

src/Presenters/Admin/Argon.php

Doc Comments +3 added lines, -2 removed lines

@@ -45,6 +45,7 @@ 
 
     /**
      * {@inheritdoc }.
+     * @param \Spinzar\Menu\MenuItem $item
      */
     public function getActiveState($item, $state = ' active')
     {
@@ -54,7 +55,7 @@ 
     /**
      * Get active state on child items.
      *
-     * @param $item
+     * @param \Spinzar\Menu\MenuItem $item
      * @param string $state
      *
      * @return null|string
@@ -67,7 +68,7 @@ 
     /**
      * Get active state on child items.
      *
-     * @param $item
+     * @param \Spinzar\Menu\MenuItem $item
      * @param string $state
      *
      * @return null|string

Spacing +17 added lines, -17 removed lines

@@ -11,7 +11,7 @@ 
      */
     public function getOpenTagWrapper()
     {
-        return PHP_EOL . '<div class="navbar-inner">
+        return PHP_EOL.'<div class="navbar-inner">
             <!-- Collapse -->
             <div class="collapse navbar-collapse" id="sidenav-collapse-main">
                 <!-- Nav items -->
@@ -23,7 +23,7 @@ 
      */
     public function getCloseTagWrapper()
     {
-        return PHP_EOL . '</ul>
+        return PHP_EOL.'</ul>
             </div>
         </div>' . PHP_EOL;
     }
@@ -34,11 +34,11 @@ 
     public function getMenuWithoutDropdownWrapper($item)
     {
         $html = '<li class="nav-item">';
-        $html .= '  <a class="nav-link' . $this->getActiveState($item) . '" href="' . $item->getUrl() . '" ' . $item->getAttributes() . '>';
-        $html .=      $item->getIcon();
-        $html .= '    <span class="nav-link-text">' . $item->title . '</span>';
+        $html .= '  <a class="nav-link'.$this->getActiveState($item).'" href="'.$item->getUrl().'" '.$item->getAttributes().'>';
+        $html .= $item->getIcon();
+        $html .= '    <span class="nav-link-text">'.$item->title.'</span>';
         $html .= '  </a>';
-        $html .= '</li>' . PHP_EOL;
+        $html .= '</li>'.PHP_EOL;
 
         return $html;
     }
@@ -90,7 +90,7 @@ 
      */
     public function getHeaderWrapper($item)
     {
-        return '<h6 class="navbar-heading p-0 text-muted">' . $item->title . '</h6>';
+        return '<h6 class="navbar-heading p-0 text-muted">'.$item->title.'</h6>';
     }
 
     /**
@@ -99,13 +99,13 @@ 
     public function getMenuWithDropDownWrapper($item)
     {
         return '<li class="nav-item">
-    <a class="nav-link' . $this->getActiveStateOnChild($item) . '" href="#navbar-' . mb_strtolower($item->title) . '" data-toggle="collapse" role="button" aria-expanded="false" aria-controls="navbar-' . mb_strtolower($item->title) . '">
-        ' . $item->getIcon() . '
-        <span class="nav-link-text">' . $item->title . '</span>
+    <a class="nav-link' . $this->getActiveStateOnChild($item).'" href="#navbar-'.mb_strtolower($item->title).'" data-toggle="collapse" role="button" aria-expanded="false" aria-controls="navbar-'.mb_strtolower($item->title).'">
+        ' . $item->getIcon().'
+        <span class="nav-link-text">' . $item->title.'</span>
     </a>
-    <div class="collapse' . $this->getShowStateOnChild($item) . '" id="navbar-' . mb_strtolower($item->title) . '">
+    <div class="collapse' . $this->getShowStateOnChild($item).'" id="navbar-'.mb_strtolower($item->title).'">
         <ul class="nav nav-sm flex-column">
-            ' . $this->getChildMenuItems($item) . '
+            ' . $this->getChildMenuItems($item).'
         </ul>
     </div>
 </li>'
@@ -122,13 +122,13 @@ 
     public function getMultiLevelDropdownWrapper($item)
     {
         return '<li class="nav-item">
-    <a class="nav-link' . $this->getActiveState($item) . '" href="#navbar-' . mb_strtolower($item->title) . '" data-toggle="collapse" role="button" aria-expanded="false" aria-controls="navbar-' . mb_strtolower($item->title) . '">
-        ' . $item->getIcon() . '
-        <span class="nav-link-text">' . $item->title . '</span>
+    <a class="nav-link' . $this->getActiveState($item).'" href="#navbar-'.mb_strtolower($item->title).'" data-toggle="collapse" role="button" aria-expanded="false" aria-controls="navbar-'.mb_strtolower($item->title).'">
+        ' . $item->getIcon().'
+        <span class="nav-link-text">' . $item->title.'</span>
     </a>
-    <div class="collapse" id="navbar-' . mb_strtolower($item->title) . '">
+    <div class="collapse" id="navbar-' . mb_strtolower($item->title).'">
         <ul class="nav nav-sm flex-column">
-            ' . $this->getChildMenuItems($item) . '
+            ' . $this->getChildMenuItems($item).'
         </ul>
     </div>
 </li>'

src/Presenters/Bootstrap3/Navbar.php

Doc Comments +2 added lines, -1 removed lines

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

Spacing +10 added lines, -10 removed lines

@@ -11,7 +11,7 @@ 
      */
     public function getOpenTagWrapper()
     {
-        return PHP_EOL . '<ul class="nav navbar-nav">' . PHP_EOL;
+        return PHP_EOL.'<ul class="nav navbar-nav">'.PHP_EOL;
     }
 
     /**
@@ -19,7 +19,7 @@ 
      */
     public function getCloseTagWrapper()
     {
-        return PHP_EOL . '</ul>' . PHP_EOL;
+        return PHP_EOL.'</ul>'.PHP_EOL;
     }
 
     /**
@@ -27,7 +27,7 @@ 
      */
     public function getMenuWithoutDropdownWrapper($item)
     {
-        return '<li' . $this->getActiveState($item) . '><a href="' . $item->getUrl() . '" ' . $item->getAttributes() . '>' . $item->getIcon() . ' ' . $item->title . '</a></li>' . PHP_EOL;
+        return '<li'.$this->getActiveState($item).'><a href="'.$item->getUrl().'" '.$item->getAttributes().'>'.$item->getIcon().' '.$item->title.'</a></li>'.PHP_EOL;
     }
 
     /**
@@ -64,7 +64,7 @@ 
      */
     public function getHeaderWrapper($item)
     {
-        return '<li class="dropdown-header">' . $item->title . '</li>';
+        return '<li class="dropdown-header">'.$item->title.'</li>';
     }
 
     /**
@@ -72,13 +72,13 @@ 
      */
     public function getMenuWithDropDownWrapper($item)
     {
-        return '<li class="dropdown' . $this->getActiveStateOnChild($item, ' active') . '">
+        return '<li class="dropdown'.$this->getActiveStateOnChild($item, ' active').'">
 		          <a href="#" class="dropdown-toggle" data-toggle="dropdown">
-					' . $item->getIcon() . ' ' . $item->title . '
+					' . $item->getIcon().' '.$item->title.'
 			      	<b class="caret"></b>
 			      </a>
 			      <ul class="dropdown-menu">
-			      	' . $this->getChildMenuItems($item) . '
+			      	' . $this->getChildMenuItems($item).'
 			      </ul>
 		      	</li>'
         . PHP_EOL;
@@ -93,13 +93,13 @@ 
      */
     public function getMultiLevelDropdownWrapper($item)
     {
-        return '<li class="dropdown' . $this->getActiveStateOnChild($item, ' active') . '">
+        return '<li class="dropdown'.$this->getActiveStateOnChild($item, ' active').'">
 		          <a href="#" class="dropdown-toggle" data-toggle="dropdown">
-					' . $item->getIcon() . ' ' . $item->title . '
+					' . $item->getIcon().' '.$item->title.'
 			      	<b class="caret pull-right caret-right"></b>
 			      </a>
 			      <ul class="dropdown-menu">
-			      	' . $this->getChildMenuItems($item) . '
+			      	' . $this->getChildMenuItems($item).'
 			      </ul>
 		      	</li>'
         . PHP_EOL;

src/Presenters/Bootstrap3/Sidebar.php

Doc Comments +2 added lines, -1 removed lines

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

Spacing +9 added lines, -9 removed lines

@@ -35,9 +35,9 @@ 
      */
     public function getMenuWithoutDropdownWrapper($item)
     {
-        return '<li' . $this->getActiveState($item) . '>
-			<a href="' . $item->getUrl() . '" ' . $item->getAttributes() . '>'
-        . $item->getIcon() . ' ' . $item->title . '</a></li>' . PHP_EOL;
+        return '<li'.$this->getActiveState($item).'>
+			<a href="' . $item->getUrl().'" '.$item->getAttributes().'>'
+        . $item->getIcon().' '.$item->title.'</a></li>'.PHP_EOL;
     }
 
     /**
@@ -74,7 +74,7 @@ 
      */
     public function getHeaderWrapper($item)
     {
-        return '<li class="dropdown-header">' . $item->title . '</li>';
+        return '<li class="dropdown-header">'.$item->title.'</li>';
     }
 
     /**
@@ -85,14 +85,14 @@ 
         $id = str_random();
 
         return '
-		<li class="' . $this->getActiveStateOnChild($item) . ' panel panel-default" id="dropdown">
-			<a data-toggle="collapse" href="#' . $id . '">
-				' . $item->getIcon() . ' ' . $item->title . ' <span class="caret"></span>
+		<li class="' . $this->getActiveStateOnChild($item).' panel panel-default" id="dropdown">
+			<a data-toggle="collapse" href="#' . $id.'">
+				' . $item->getIcon().' '.$item->title.' <span class="caret"></span>
 			</a>
-			<div id="' . $id . '" class="panel-collapse collapse ' . $this->getActiveStateOnChild($item, 'in') . '">
+			<div id="' . $id.'" class="panel-collapse collapse '.$this->getActiveStateOnChild($item, 'in').'">
 				<div class="panel-body">
 					<ul class="nav navbar-nav">
-						' . $this->getChildMenuItems($item) . '
+						' . $this->getChildMenuItems($item).'
 					</ul>
 				</div>
 			</div>

src/MenuItem.php

Spacing +3 added lines, -3 removed lines

@@ -349,13 +349,13 @@ 
     public function getIcon($default = null)
     {
         if ($this->icon !== null && $this->icon !== '') {
-            return '<i class="' . $this->icon . '"></i>';
+            return '<i class="'.$this->icon.'"></i>';
         }
         if ($default === null) {
             return $default;
         }
 
-        return '<i class="' . $default . '"></i>';
+        return '<i class="'.$default.'"></i>';
     }
 
     /**
@@ -561,7 +561,7 @@ 
      */
     protected function getActiveStateFromRoute()
     {
-        return Request::is(str_replace(url('/') . '/', '', $this->getUrl()));
+        return Request::is(str_replace(url('/').'/', '', $this->getUrl()));
     }
 
     /**

src/Menu.php

Spacing +1 added lines, -1 removed lines

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

src/Provider.php

Spacing +3 added lines, -3 removed lines

@@ -19,8 +19,8 @@ 
      */
     public function boot()
     {
-        $configPath = __DIR__ . '/Config/menu.php';
-        $viewsPath = __DIR__ . '/Resources/views';
+        $configPath = __DIR__.'/Config/menu.php';
+        $viewsPath = __DIR__.'/Resources/views';
         
         $this->mergeConfigFrom($configPath, 'menu');
         $this->loadViewsFrom($viewsPath, 'menu');
@@ -45,7 +45,7 @@ 
     {
         $this->registerHtmlPackage();
 
-        $this->app->singleton('menu', function ($app) {
+        $this->app->singleton('menu', function($app) {
             return new Menu($app['view'], $app['config']);
         });
     }

src/Presenters/Bootstrap3/NavPills.php

Spacing +1 added lines, -1 removed lines

@@ -9,6 +9,6 @@
      */
     public function getOpenTagWrapper()
     {
-        return PHP_EOL . '<ul class="nav nav-pills">' . PHP_EOL;
+        return PHP_EOL.'<ul class="nav nav-pills">'.PHP_EOL;
     }
 }