Merge pull request #461 from denbatte/feature/insightsNodeBundle

[NodeBundle] fixes Sensiolabs insight issues on Nodebundle

src/Kunstmaan/NodeBundle/Event/SlugEvent.php

@@ -65,4 +65,4 @@ public function setRenderContext(RenderContext $renderContext)
     {
         $this->renderContext = $renderContext;
     }
-}
+}

src/Kunstmaan/NodeBundle/Event/SlugSecurityEvent.php

@@ -12,4 +12,4 @@
 final class SlugSecurityEvent extends Event
 {
 
-}
+}

src/Kunstmaan/NodeBundle/EventListener/NodeTranslationListener.php

@@ -80,7 +80,7 @@ private function setSlugWhenEmpty(NodeTranslation $nodeTranslation, EntityManage
         /**
          * If no slug is set and no structure node, apply title as slug
          */
-        if ($nodeTranslation->getSlug() == null && $nodeTranslation->getNode()->getParent() != null) {
+        if ($nodeTranslation->getSlug() === null && $nodeTranslation->getNode()->getParent() !== null) {
             $nodeTranslation->setSlug($this->slugifier->slugify($nodeTranslation->getTitle()));
         }
     }

src/Kunstmaan/NodeBundle/EventListener/SlugListener.php

@@ -74,4 +74,4 @@ public function onKernelController(FilterControllerEvent $event)
             $event->setController($this->resolver->getController($request));
         }
     }
-}
+}

src/Kunstmaan/NodeBundle/EventListener/SlugSecurityListener.php

@@ -68,4 +68,4 @@ public function onSlugSecurityEvent()
 
         $this->request->attributes->set('_nodeMenu', $nodeMenu);
     }
-}
+}

src/Kunstmaan/NodeBundle/Helper/NodeMenu.php

@@ -17,8 +17,6 @@
 class NodeMenu
 {
 
-    /***** constructor arguments *****/
-
     /**
      * @var EntityManager
      */
@@ -59,9 +57,6 @@ class NodeMenu
      */
     private $includeHiddenFromNav = false;
 
-
-    /***** temporary storage variables *****/
-
     /**
      * @var NodeMenuItem[]
      */