KyleMassacre / laravel-menus

src/Presenters/Bootstrap/NavbarRightPresenter.php

Spacing +3 added lines, -3 removed lines

@@ -11,7 +11,7 @@ 
      */
     public function getOpenTagWrapper(): ?string
     {
-        return PHP_EOL . '<ul class="nav navbar-nav navbar-right">' . PHP_EOL;
+        return PHP_EOL.'<ul class="nav navbar-nav navbar-right">'.PHP_EOL;
     }
 
     /**
@@ -21,11 +21,11 @@ 
     {
         return '<li class="dropdown pull-right">
 			      <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;

src/Presenters/Bootstrap/SidebarMenuPresenter.php

Spacing +9 added lines, -9 removed lines

@@ -37,9 +37,9 @@ 
      */
     public function getMenuWithoutDropdownWrapper(MenuItemContract $item): ?string
     {
-        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;
     }
 
     /**
@@ -76,7 +76,7 @@ 
      */
     public function getHeaderWrapper(MenuItemContract $item): ?string
     {
-        return '<li class="dropdown-header">' . $item->title . '</li>';
+        return '<li class="dropdown-header">'.$item->title.'</li>';
     }
 
     /**
@@ -87,14 +87,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/Presenters/Bootstrap/NavbarPresenter.php

Spacing +10 added lines, -10 removed lines

@@ -12,7 +12,7 @@ 
      */
     public function getOpenTagWrapper(): ?string
     {
-        return PHP_EOL . '<ul class="nav navbar-nav">' . PHP_EOL;
+        return PHP_EOL.'<ul class="nav navbar-nav">'.PHP_EOL;
     }
 
     /**
@@ -20,7 +20,7 @@ 
      */
     public function getCloseTagWrapper(): ?string
     {
-        return PHP_EOL . '</ul>' . PHP_EOL;
+        return PHP_EOL.'</ul>'.PHP_EOL;
     }
 
     /**
@@ -28,7 +28,7 @@ 
      */
     public function getMenuWithoutDropdownWrapper(MenuItemContract $item): ?string
     {
-        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;
     }
 
     /**
@@ -65,7 +65,7 @@ 
      */
     public function getHeaderWrapper(MenuItemContract $item): ?string
     {
-        return '<li class="dropdown-header">' . $item->title . '</li>';
+        return '<li class="dropdown-header">'.$item->title.'</li>';
     }
 
     /**
@@ -73,13 +73,13 @@ 
      */
     public function getMenuWithDropDownWrapper(MenuItemContract $item): ?string
     {
-        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;
@@ -94,13 +94,13 @@ 
      */
     public function getMultiLevelDropdownWrapper(MenuItemContract $item): string
     {
-        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/Admin/AdminltePresenter.php

Spacing +10 added lines, -10 removed lines

@@ -12,7 +12,7 @@ 
      */
     public function getOpenTagWrapper(): ?string
     {
-        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;
     }
 
     /**
@@ -20,7 +20,7 @@ 
      */
     public function getCloseTagWrapper(): ?string
     {
-        return PHP_EOL . '</ul>' . PHP_EOL;
+        return PHP_EOL.'</ul>'.PHP_EOL;
     }
 
     /**
@@ -28,7 +28,7 @@ 
      */
     public function getMenuWithoutDropdownWrapper(MenuItemContract $item): ?string
     {
-        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;
     }
 
     /**
@@ -65,7 +65,7 @@ 
      */
     public function getHeaderWrapper(MenuItemContract $item): ?string
     {
-        return '<li class="header">' . $item->title . '</li>';
+        return '<li class="header">'.$item->title.'</li>';
     }
 
     /**
@@ -73,15 +73,15 @@ 
      */
     public function getMenuWithDropDownWrapper(MenuItemContract $item): ?string
     {
-        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;
@@ -96,15 +96,15 @@ 
      */
     public function getMultiLevelDropdownWrapper(MenuItemContract $item): string
     {
-        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/MenuItem.php

Spacing +5 added lines, -5 removed lines

@@ -274,10 +274,10 @@ 
      */
     public function getBadge(): string
     {
-        if($this->hasBadge()) {
+        if ($this->hasBadge()) {
             extract($this->badge);
 
-            return '<span class="' . $type . '">' . $text . '</span>';
+            return '<span class="'.$type.'">'.$text.'</span>';
         }
     }
 
@@ -291,13 +291,13 @@ 
     public function getIcon(string $default = null): ?string
     {
         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>';
     }
 
     /**
@@ -508,7 +508,7 @@ 
      */
     protected function getActiveStateFromRoute(): bool
     {
-        return Request::is(str_replace(url('/') . '/', '', $this->getUrl()));
+        return Request::is(str_replace(url('/').'/', '', $this->getUrl()));
     }
 
     /**

Tests/MenuItemTest.php

Spacing +9 added lines, -9 removed lines

@@ -106,7 +106,7 @@ 
     public function it_can_create_a_dropdown_menu_item()
     {
         $menuItem = MenuItem::make(['title' => 'Parent Item']);
-        $menuItem->dropdown('Dropdown item', function (MenuItem $sub) {
+        $menuItem->dropdown('Dropdown item', function(MenuItem $sub) {
             $sub->url('settings/account', 'Account');
             $sub->url('settings/password', 'Password');
         });
@@ -118,7 +118,7 @@ 
     public function it_can_make_a_simple_route_menu_item()
     {
         $menuItem = MenuItem::make(['title' => 'Parent Item']);
-        $menuItem->dropdown('Dropdown item', function (MenuItem $sub) {
+        $menuItem->dropdown('Dropdown item', function(MenuItem $sub) {
             $sub->route('settings.account', 'Account', ['user_id' => 1]);
         });
         $children = $menuItem->getChildren()[0]->getChilds();
@@ -133,7 +133,7 @@ 
     public function it_can_make_a_route_menu_item()
     {
         $menuItem = MenuItem::make(['title' => 'Parent Item']);
-        $menuItem->dropdown('Dropdown item', function (MenuItem $sub) {
+        $menuItem->dropdown('Dropdown item', function(MenuItem $sub) {
             $sub->route('settings.account', 'Account', ['user_id' => 1], 1, ['my-attr' => 'value']);
         });
         $children = $menuItem->getChildren()[0]->getChilds();
@@ -150,7 +150,7 @@ 
     public function it_can_make_a_simple_url_menu_item()
     {
         $menuItem = MenuItem::make(['title' => 'Parent Item']);
-        $menuItem->dropdown('Dropdown item', function (MenuItem $sub) {
+        $menuItem->dropdown('Dropdown item', function(MenuItem $sub) {
             $sub->url('settings/account', 'Account');
         });
         $children = $menuItem->getChildren()[0]->getChilds();
@@ -165,7 +165,7 @@ 
     public function it_can_make_a_url_menu_item()
     {
         $menuItem = MenuItem::make(['title' => 'Parent Item']);
-        $menuItem->dropdown('Dropdown item', function (MenuItem $sub) {
+        $menuItem->dropdown('Dropdown item', function(MenuItem $sub) {
             $sub->url('settings/account', 'Account', 1, ['my-attr' => 'value']);
         });
         $children = $menuItem->getChildren()[0]->getChilds();
@@ -182,7 +182,7 @@ 
     public function it_can_add_a_menu_item_divider()
     {
         $menuItem = MenuItem::make(['title' => 'Parent Item']);
-        $menuItem->dropdown('Dropdown item', function (MenuItem $sub) {
+        $menuItem->dropdown('Dropdown item', function(MenuItem $sub) {
             $sub->url('settings/account', 'Account');
             $sub->divider();
         });
@@ -199,7 +199,7 @@ 
     public function it_can_add_a_header_menu_item()
     {
         $menuItem = MenuItem::make(['title' => 'Parent Item']);
-        $menuItem->dropdown('Dropdown item', function (MenuItem $sub) {
+        $menuItem->dropdown('Dropdown item', function(MenuItem $sub) {
             $sub->header('User Stuff');
             $sub->url('settings/account', 'Account');
         });
@@ -226,7 +226,7 @@ 
     {
         $this->app['router']->get('settings/account', ['as' => 'settings.account']);
         $menuItem = MenuItem::make(['title' => 'Parent Item']);
-        $menuItem->dropdown('Dropdown item', function (MenuItem $sub) {
+        $menuItem->dropdown('Dropdown item', function(MenuItem $sub) {
             $sub->route('settings.account', 'Account');
         });
         $children = $menuItem->getChildren()[0]->getChilds();
@@ -287,7 +287,7 @@ 
     public function it_can_check_for_a_submenu()
     {
         $menuItem = MenuItem::make(['title' => 'Parent Item']);
-        $menuItem->dropdown('Dropdown item', function (MenuItem $sub) {
+        $menuItem->dropdown('Dropdown item', function(MenuItem $sub) {
             $sub->header('User Stuff');
             $sub->url('settings/account', 'Account');
         });