Malezha / menu

src/Builder.php

Doc Comments +1 added lines, -1 removed lines

@@ -350,7 +350,7 @@
     }
 
     /**
-     * @param $element
+     * @param string $element
      * @return ElementFactory
      */
     protected function getFactory($element)

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 use Malezha\Menu\Contracts\MenuRender;
 use Malezha\Menu\Traits\HasActiveAttributes as TraitHasActiveAttributes;
 use Malezha\Menu\Traits\HasAttributes;
-use Opis\Closure\SerializableClosure;
 
 /**
  * Class Builder

Spacing +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
         $result = call_user_func($callback, $factory);
         $result = is_null($result) ? $factory : $result;
 
-        if (! $result instanceof ElementFactory && ! $result instanceof Element) {
+        if (!$result instanceof ElementFactory && !$result instanceof Element) {
             throw new \RuntimeException("Result of callback must be [" . 
                 Element::class . "] or [" . ElementFactory::class . "]");
         }

tests/ElementTests/TextTest.php

Spacing +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
         $factory = $this->factory();
         $factory->text = 'Serialize';
         $factory->attributes->put('class', 'serialized');
-        $factory->displayRule = function () use ($tmp) {
+        $factory->displayRule = function() use ($tmp) {
             return $tmp === 0;
         };
         $element = $factory->build();

tests/stub/another.php

Spacing +5 added lines, -5 removed lines

@@ -1,9 +1,9 @@
 <div class="div-menu">
     <?php if ($menu) :?>
-        <<?php echo $menu->getType() . $menu->buildAttributes();?>>
+        <<?php echo $menu->getType() . $menu->buildAttributes(); ?>>
             <?php foreach ($menu->all() as $element) :?>
-                <?php echo $element->render($renderView);?>
-            <?php endforeach;?>
-        </<?php echo $menu->getType();?>>
-    <?php endif;?>
+                <?php echo $element->render($renderView); ?>
+            <?php endforeach; ?>
+        </<?php echo $menu->getType(); ?>>
+    <?php endif; ?>
 </div>
\ No newline at end of file

Braces +4 added lines, -1 removed lines

@@ -2,7 +2,10 @@
     <?php if ($menu) :?>
         <<?php echo $menu->getType() . $menu->buildAttributes();?>>
             <?php foreach ($menu->all() as $element) :?>
-                <?php if ($element instanceof \Malezha\Menu\Contracts\ElementFactory) $element = $element->build(); ?>
+                <?php if ($element instanceof \Malezha\Menu\Contracts\ElementFactory) {
+    $element = $element->build();
+}
+?>
                 <?php echo $element->render($renderView);?>
             <?php endforeach;?>
         </<?php echo $menu->getType();?>>

src/Contracts/Builder.php

Indentation +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
      * @internal param string $name
      */
     function __construct(Container $container, Attributes $attributes, Attributes $activeAttributes,
-                         $type = self::UL, $view = null);
+                            $type = self::UL, $view = null);
 
     /**
      * @param string $name

src/Element/AbstractElement.php

Doc Comments +1 added lines

@@ -35,6 +35,7 @@
 
     /**
      * @inheritdoc
+     * @param string $view
      */
     public function setView($view)
     {

tests/stub/directory/view.php

Spacing +1 added lines, -1 removed lines

@@ -1,1 +1,1 @@
-Hello, <?php echo $menu;?> builder!
\ No newline at end of file
+Hello, <?php echo $menu; ?> builder!
\ No newline at end of file

tests/stub/text_element.php

Spacing +1 added lines, -1 removed lines

@@ -1,1 +1,1 @@
-<div<?php echo $attributes;?>><?php echo $text;?></div>
\ No newline at end of file
+<div<?php echo $attributes; ?>><?php echo $text; ?></div>
\ No newline at end of file

src/MenuServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -79,7 +79,7 @@ 
 
     protected function registerRenderSystem()
     {
-        $this->app->bind('menu.render', function (Container $app) {
+        $this->app->bind('menu.render', function(Container $app) {
             $config = $app->make(Repository::class)->get('menu');
             $key = $config['default'];
             $available = $config['renders'];
@@ -95,7 +95,7 @@ 
 
     protected function registerComparativeUrl()
     {
-        $this->app->singleton('menu.compare-url', function (Container $app) {
+        $this->app->singleton('menu.compare-url', function(Container $app) {
             return $app->make(ComparativeUrl::class, [
                 'skippedPaths' => $app->make(Repository::class)->get('menu.skippedPaths'),
             ]);

tests/BuilderTest.php

Spacing +12 added lines, -12 removed lines

@@ -148,9 +148,9 @@ 
         $builder = $this->builderFactory();
 
         $builder->create('index', Link::class,
-            function (LinkFactory $factory) {});
+            function(LinkFactory $factory) {});
         $builder->create('index', SubMenu::class,
-            function (SubMenuFactory $factory) {}); // Duplicate
+            function(SubMenuFactory $factory) {}); // Duplicate
     }
     
     public function testGet()
@@ -158,7 +158,7 @@ 
         $builder = $this->builderFactory();
         
         $item = $builder->create('test', Link::class,
-            function (LinkFactory $factory) {});
+            function(LinkFactory $factory) {});
         
         $this->assertEquals($item, $builder->get('test'));
         $this->assertEquals(null, $builder->get('notFound'));
@@ -171,7 +171,7 @@ 
         $builder = $this->builderFactory();
 
         $item = $builder->create('test', Link::class,
-            function (LinkFactory $factory) {});
+            function(LinkFactory $factory) {});
 
         $this->assertEquals($item, $builder->getByIndex(0));
         $this->assertEquals(null, $builder->getByIndex(1));
@@ -202,7 +202,7 @@ 
         
         $this->assertEquals([], $builder->all());
         $item = $builder->create('test', Link::class,
-            function (LinkFactory $factory) {});
+            function(LinkFactory $factory) {});
         $this->assertEquals(['test' => $item], $builder->all());
     }
     
@@ -211,9 +211,9 @@ 
         $builder = $this->builderFactory();
 
         $builder->create('test', Link::class,
-            function (LinkFactory $factory) {});
+            function(LinkFactory $factory) {});
         $builder->create('another', Text::class,
-            function (TextFactory $factory) {});
+            function(TextFactory $factory) {});
         
         $this->assertTrue($builder->has('test'));
         $builder->forget('test');
@@ -348,14 +348,14 @@ 
             $factory->url = url('logout');
         });
         
-        $builder->insertAfter('index', function (Builder $builder) {
+        $builder->insertAfter('index', function(Builder $builder) {
             $builder->create('users', Link::class, function(LinkFactory $factory) {
                 $factory->title = 'Users';
                 $factory->url = url('users');
             });
         });
         
-        $builder->insertBefore('users', function (Builder $builder) {
+        $builder->insertBefore('users', function(Builder $builder) {
             $builder->create('profile', Link::class, function(LinkFactory $factory) {
                 $factory->title = 'Profile';
                 $factory->url = url('profile');
@@ -370,7 +370,7 @@ 
         $this->expectException(\RuntimeException::class);
 
         $builder = $this->builderFactory();
-        $builder->insertBefore('not_exist', function (Builder $builder) {});
+        $builder->insertBefore('not_exist', function(Builder $builder) {});
     }
 
     public function testInsertExceptionDuplicate()
@@ -386,7 +386,7 @@ 
             $factory->title = 'Some';
             $factory->url = '/';
         });
-        $builder->insertAfter('home', function (Builder $builder) {
+        $builder->insertAfter('home', function(Builder $builder) {
             $builder->create('some', Text::class, function(TextFactory $factory) {
                 $factory->text = 'Duplicate some';
             });
@@ -425,7 +425,7 @@ 
             });
         });
         $builder->create('settings', Link::class, function(LinkFactory $factory) {
-            $factory->displayRule = function () {return false;};
+            $factory->displayRule = function() {return false; };
             $factory->title = 'Settings';
             $factory->url = url('/settings');
         });