sonata-project / SonataAdminBundle

src/Controller/PolyfillControllerTrait.php

Doc Comments +3 added lines

@@ -25,6 +25,9 @@
  */
 trait PolyfillControllerTrait
 {
+    /**
+     * @param string $methodName
+     */
     public function __call($methodName, $arguments)
     {
         $this->proxyToController($methodName, $arguments);

src/Twig/GlobalVariables.php

Doc Comments +5 added lines, -5 removed lines

@@ -67,7 +67,7 @@ 
     /**
      * @param string $code
      * @param string $action
-     * @param array  $parameters
+     * @param string[]  $parameters
      * @param int    $absolute
      *
      * @return string
@@ -82,8 +82,8 @@ 
     /**
      * @param string $code
      * @param string $action
-     * @param mixed  $object
-     * @param array  $parameters
+     * @param string  $object
+     * @param string[]  $parameters
      * @param int    $absolute
      *
      * @return string
@@ -96,8 +96,8 @@ 
     }
 
     /**
-     * @param $code
-     * @param $action
+     * @param string $code
+     * @param string $action
      *
      * @return array
      */

tests/Datagrid/DatagridMapperTest.php

Doc Comments +6 added lines

@@ -266,6 +266,12 @@
         ], array_keys($this->datagrid->getFilters()));
     }
 
+    /**
+     * @param string $name
+     * @param string $label
+     *
+     * @return string
+     */
     private function getFieldDescriptionMock($name = null, $label = null)
     {
         $fieldDescription = $this->getMockForAbstractClass(BaseFieldDescription::class);

tests/Datagrid/ListMapperTest.php

Doc Comments +6 added lines

@@ -255,6 +255,12 @@
         ], true), print_r($this->fieldDescriptionCollection->getElements(), true));
     }
 
+    /**
+     * @param string $name
+     * @param string $label
+     *
+     * @return string
+     */
     private function getFieldDescriptionMock($name = null, $label = null)
     {
         $fieldDescription = $this->getMockForAbstractClass(BaseFieldDescription::class);

tests/Show/ShowMapperTest.php

Doc Comments +4 added lines

@@ -455,6 +455,10 @@
         $this->showMapper = new ShowMapper($this->showBuilder, $this->fieldDescriptionCollection, $this->admin);
     }
 
+    /**
+     * @param string $name
+     * @param string $label
+     */
     private function getFieldDescriptionMock($name = null, $label = null)
     {
         $fieldDescription = $this->getMockForAbstractClass(BaseFieldDescription::class);

src/Admin/AbstractAdmin.php

Doc Comments +7 added lines, -4 removed lines

@@ -1019,6 +1019,7 @@ 
 
     /**
      * NEXT_MAJOR: remove this method
+     * @param string $subClass
      */
     public function addSubClass($subClass)
     {
@@ -1172,6 +1173,8 @@ 
 
     /**
      * {@inheritdoc}
+     * @param string $name
+     * @param string $adminCode
      */
     public function isCurrentRoute($name, $adminCode = null)
     {
@@ -2543,7 +2546,7 @@ 
      *
      * @param string $context
      *
-     * @return array
+     * @return string[]
      */
     public function getPermissionsShow($context)
     {
@@ -2786,7 +2789,7 @@ 
     /**
      * Set custom per page options.
      *
-     * @param array $options
+     * @param integer[] $options
      */
     public function setPerPageOptions(array $options)
     {
@@ -2948,6 +2951,7 @@ 
 
     /**
      * {@inheritdoc}
+     * @param string $action
      */
     public function configureActionButtons($action, $object = null)
     {
@@ -3079,7 +3083,6 @@ 
     }
 
     /**
-     * @param FormMapper $form
      */
     final public function getSearchResultLink($object)
     {
@@ -3196,7 +3199,7 @@ 
      * NEXT_MAJOR: remove this method.
      *
      * @param MenuItemInterface $menu
-     * @param                   $action
+     * @param                   string $action
      * @param AdminInterface    $childAdmin
      *
      * @return mixed

src/Controller/CRUDController.php

Doc Comments +5 added lines, -5 removed lines

@@ -208,7 +208,7 @@ 
     /**
      * Delete action.
      *
-     * @param int|string|null $id
+     * @param integer|null $id
      *
      * @throws NotFoundHttpException If the object does not exist
      * @throws AccessDeniedException If access is not granted
@@ -726,7 +726,7 @@ 
     /**
      * View history revision of object.
      *
-     * @param int|string|null $id
+     * @param null|integer $id
      * @param string|null     $revision
      *
      * @throws AccessDeniedException If access is not granted
@@ -786,9 +786,9 @@ 
     /**
      * Compare history revisions of object.
      *
-     * @param int|string|null $id
-     * @param int|string|null $base_revision
-     * @param int|string|null $compare_revision
+     * @param null|integer $id
+     * @param null|integer $base_revision
+     * @param null|integer $compare_revision
      *
      * @throws AccessDeniedException If access is not granted
      * @throws NotFoundHttpException If the object or revision does not exist or the audit reader is not available

tests/Datagrid/PagerTest.php

Doc Comments +4 added lines

@@ -639,6 +639,10 @@
         $this->assertNull($this->pager->getQuery());
     }
 
+    /**
+     * @param Pager $obj
+     * @param string $name
+     */
     protected function callMethod($obj, $name, array $args = [])
     {
         $class = new \ReflectionClass($obj);