alpixel / AlpixelMenuBundle

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();

DependencyInjection/AlpixelMenuExtension.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
     public function prepend(ContainerBuilder $container)
     {
         $parser = new Parser();
-        $config = $parser->parse(file_get_contents(__DIR__ . '/../Resources/config/config.yml'));
+        $config = $parser->parse(file_get_contents(__DIR__.'/../Resources/config/config.yml'));
 
         foreach ($config as $key => $configuration) {
             $container->prependExtensionConfig($key, $configuration);

Controller/Admin/CRUD/ItemAdmin.php

Spacing +4 added lines, -4 removed lines

@@ -86,7 +86,7 @@ 
 
         $idItem = null;
         $idMenu = null;
-        if($request->query->has('uniqid')) {
+        if ($request->query->has('uniqid')) {
             $formId = $request->query->get('uniqid');
             if ($request->request->has($formId)) {
                 $formData = $request->request->get($formId);
@@ -110,7 +110,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 === null) {
                         return $query;
@@ -132,7 +132,7 @@ 
                 'required'      => true,
                 'property'      => 'name',
                 'data'          => $parentItem,
-                'query_builder' => function (EntityRepository $repository) use ($parentItem) {
+                'query_builder' => function(EntityRepository $repository) use ($parentItem) {
                     $qb = $repository->createQueryBuilder('i');
 
                     if ($parentItem->getParent() === null) {
@@ -200,7 +200,7 @@ 
 
     public function checkUri($object = null)
     {
-        if($object === null) {
+        if ($object === null) {
             return;
         }
 

Indentation +4 added lines, -4 removed lines

@@ -136,9 +136,9 @@ 
                     $qb = $repository->createQueryBuilder('i');
 
                     if ($parentItem->getParent() === null) {
-                         return $qb->where('i.parent IS NULL')
-                             ->andWhere('i.menu = :id')
-                             ->setParameter('id', $parentItem->getMenu());
+                            return $qb->where('i.parent IS NULL')
+                                ->andWhere('i.menu = :id')
+                                ->setParameter('id', $parentItem->getMenu());
                     }
 
                     return $qb->where('i.id IN (:ids)')
@@ -208,7 +208,7 @@ 
 
         if (!Item::uriTypeExists($uriType)) {
             $form->get('uri')->addError(new FormError(
-               'Le type d\'url est invalide !'
+                'Le type d\'url est invalide !'
             ));
 
             return;

Menu/Menu.php

Doc Comments +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
     }
 
     /**
-     * @return array An array of available delete strategies
+     * @return string[] An array of available delete strategies
      */
     public static function getDeleteStrategiesAvailable()
     {

Indentation +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
     public static function getDeleteStrategiesAvailable()
     {
         return [
-          self::DELETE_STRATEGY_MOVE_CHILDREN,
+            self::DELETE_STRATEGY_MOVE_CHILDREN,
         ];
     }
 }

Builder/MenuBuilder.php

Doc Comments +2 added lines, -2 removed lines

@@ -58,7 +58,7 @@ 
      * The parameter $locale must be defined in your
      * symfony configuration file under parameters.
      *
-     * @param $locale String
+     * @param string $locale String
      *
      * @return $this
      */
@@ -89,7 +89,7 @@ 
     /**
      * Check if the machineName is valid.
      *
-     * @param $machineName
+     * @param string $machineName
      *
      * @return bool
      */

Entity/Item.php

Doc Comments +2 added lines, -3 removed lines

@@ -128,7 +128,7 @@ 
     /**
      * Get parent Item.
      *
-     * @return null\Item
+     * @return null|ItemInterface
      */
     public function getParent()
     {
@@ -138,7 +138,6 @@ 
     /**
      * Set parent Item.
      *
-     * @param ItemInterface $menu
      *
      * @return self
      */
@@ -223,6 +222,7 @@ 
      * Set name displayed in Item.
      *
      * @param string
+     * @param string $name
      *
      * @return self
      */
@@ -246,7 +246,6 @@ 
     /**
      * Set URL.
      *
-     * @param string $url
      *
      * @return self
      */

Entity/Menu.php

Doc Comments +8 added lines, -5 removed lines

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