alpixel / AlpixelMenuBundle

Builder/MenuBuilder.php

Doc Comments +3 added lines, -3 removed lines

@@ -48,7 +48,7 @@ 
      * The parameter $locale must be defined in your
      * symfony configuration file under parameters.
      *
-     * @param $locale String
+     * @param string $locale String
      * @return $this
      */
     public function setDefaultLocale($locale)
@@ -78,7 +78,7 @@ 
     /**
      * Check if the machineName is valid
      *
-     * @param $machineName
+     * @param string $machineName
      * @return bool
      */
     public static function isValidMachineName($machineName)
@@ -93,7 +93,7 @@ 
     /**
      * Retrun null or a KnpMenuItem instance
      *
-     * @return null|KnpMenuItem
+     * @return KnpMenuItem
      */
     public function getKnpMenu()
     {

Spacing +5 added lines, -5 removed lines

@@ -140,7 +140,7 @@ 
             throw new UnexpectedValueException('The parameter $machineName must be a non empty string');
         }
 
-        if($locale === null && $this->request !== null) {
+        if ($locale === null && $this->request !== null) {
             $locale = $this->request->getLocale();
         }
 
@@ -178,7 +178,7 @@ 
      */
     protected function getTree(KnpMenuItem $knpMenu, ItemInterface $item, KnpMenuItem $parent = null)
     {
-        if($parent === null) {
+        if ($parent === null) {
             $menuItem = $knpMenu->addChild($item);
         } else {
             $menuItem = $parent->addChild($item);
@@ -186,10 +186,10 @@ 
 
         if (($uri = $item->getUri()) !== null) {
             if ($uri[0] == '/') {
-                $baseUri = $this->request->getBasePath() .
-                    $this->request->getBaseURL() .
+                $baseUri = $this->request->getBasePath().
+                    $this->request->getBaseURL().
                     $uri;
-                $uri = $this->request->getSchemeAndHttpHost() . $baseUri;
+                $uri = $this->request->getSchemeAndHttpHost().$baseUri;
 
                 if ($baseUri === $this->currentUri) {
                     $menuItem->setCurrent(true);

Entity/Item.php

Doc Comments +2 added lines, -3 removed lines

@@ -115,7 +115,7 @@ 
     /**
      * Get parent Item
      *
-     * @return null\Item
+     * @return null|ItemInterface
      */
     public function getParent()
     {
@@ -125,7 +125,6 @@ 
     /**
      * Set parent Item
      *
-     * @param ItemInterface $menu
      *
      * @return self
      */
@@ -209,6 +208,7 @@ 
      * Set name displayed in Item
      *
      * @param string
+     * @param string $name
      *
      * @return self
      */
@@ -232,7 +232,6 @@ 
     /**
      * Set URL
      *
-     * @param string $url
      *
      * @return self
      */

Entity/Menu.php

Doc Comments +8 added lines, -5 removed lines

@@ -87,6 +87,7 @@ 
      * Set the machineName the key for querying a menu.
      *
      * @param string
+     * @param string $machineName
      *
      * @return self
      */
@@ -100,7 +101,7 @@ 
     /**
      * Get the name the value displayed to the administrator.
      *
-     * @return self
+     * @return string
      */
     public function getName()
     {
@@ -110,6 +111,7 @@ 
     /**
      * Set the name the value displayed to the administrator.
      *
+     * @param string $name
      * @return self
      */
     public function setName($name)
@@ -167,9 +169,9 @@ 
     /**
      * Set items for the menu.
      * @deprecated
-     * @param null\ItemInterface $item
+     * @param ItemInterface $item
      *
-     * @return Item
+     * @return Menu
      */
     public function setItem(ItemInterface $item)
     {
@@ -179,9 +181,9 @@ 
     /**
      * Set items for the menu.
      *
-     * @param null\ItemInterface $item
+     * @param ItemInterface $item
      *
-     * @return Item
+     * @return Menu
      */
     public function addItem(ItemInterface $item)
     {
@@ -204,6 +206,7 @@ 
     /**
      * Set the locale language.
      *
+     * @param string $locale
      * @return self
      */
     public function setLocale($locale)

Tests/Builder/MenuBuilderTest.php

Doc Comments +6 added lines

@@ -74,6 +74,9 @@ 
         $this->assertInstanceOf(MenuItem::class, $knpMenu);
     }
 
+    /**
+     * @param string $locale
+     */
     public function getMenuContext($locale)
     {
         $menuData = $this->getMenuData()[$locale];
@@ -103,6 +106,9 @@ 
         return $menu;
     }
 
+    /**
+     * @param integer $i
+     */
     public function newItem(Menu $menu, $i, Item $parentItem = null)
     {
         $item = new Item();

Controller/Admin/CRUD/ItemAdmin.php

Spacing +2 added lines, -2 removed lines

@@ -96,7 +96,7 @@ 
                 'label'         => 'Menu',
                 'required'      => true,
                 'property'      => 'name',
-                'query_builder' => function (EntityRepository $entityRepository) use ($idMenu) {
+                'query_builder' => function(EntityRepository $entityRepository) use ($idMenu) {
                     $query = $entityRepository->createQuerybuilder('m');
                     if ($idMenu === 0) {
                         return $query;
@@ -113,7 +113,7 @@ 
                 'label'    => 'Item parent',
                 'required' => true,
                 'property' => 'name',
-                'query_builder' => function (EntityRepository $entityRepository) use ($idItem) {
+                'query_builder' => function(EntityRepository $entityRepository) use ($idItem) {
                     $query = $entityRepository->createQuerybuilder('i');
                     if ($idItem === 0) {
                         return $query;

Controller/CRUDController.php

Spacing +2 added lines, -2 removed lines

@@ -69,7 +69,7 @@ 
             }
             else {
                 $router = $this->get('router');
-                $url= $router->generate('admin_alpixel_menu_menu_list');
+                $url = $router->generate('admin_alpixel_menu_menu_list');
             }
         }
 
@@ -84,7 +84,7 @@ 
 
         if (!$url) {
             $router = $this->get('router');
-            $url= $router->generate('admin_alpixel_menu_menu_list');
+            $url = $router->generate('admin_alpixel_menu_menu_list');
         }
 
         return new RedirectResponse($url);

Braces +1 added lines, -2 removed lines

@@ -66,8 +66,7 @@
         if ($this->getRestMethod() === 'DELETE') {
             if ($object instanceof Item) {
                 $url = $this->admin->generateUrl('list', $params['list']);
-            }
-            else {
+            } else {
                 $router = $this->get('router');
                 $url= $router->generate('admin_alpixel_menu_menu_list');
             }