sonata-project / SonataAdminBundle

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

src/Datagrid/Pager.php

Doc Comments +2 added lines, -2 removed lines

@@ -432,7 +432,7 @@ 
      * Returns a parameter.
      *
      * @param string $name
-     * @param mixed  $default
+     * @param null|string  $default
      *
      * @return mixed
      */
@@ -457,7 +457,7 @@ 
      * Sets a parameter.
      *
      * @param string $name
-     * @param mixed  $value
+     * @param string  $value
      */
     public function setParameter($name, $value)
     {

src/Form/ChoiceList/ModelChoiceList.php

Doc Comments +1 added lines, -1 removed lines

@@ -101,7 +101,7 @@
      * @param string            $class
      * @param string|null       $property
      * @param QueryBuilder|null $query
-     * @param array             $choices
+     * @param integer[]             $choices
      */
     public function __construct(
         ModelManagerInterface $modelManager,

src/Generator/AdminGenerator.php

Doc Comments +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
     private $file;
 
     /**
-     * @param array|string $skeletonDirectories
+     * @param string $skeletonDirectories
      */
     public function __construct(ModelManagerInterface $modelManager, $skeletonDirectories)
     {

src/Admin/AbstractAdmin.php

Doc Comments +10 added lines, -2 removed lines

@@ -961,6 +961,7 @@ 
 
     /**
      * NEXT_MAJOR: remove this method.
+     * @param string $subClass
      */
     public function addSubClass($subClass)
     {
@@ -1083,6 +1084,10 @@ 
         return $this->routeGenerator->hasAdminRoute($this, $name);
     }
 
+    /**
+     * @param string $name
+     * @param string $adminCode
+     */
     public function isCurrentRoute($name, $adminCode = null)
     {
         if (!$this->hasRequest()) {
@@ -2177,7 +2182,7 @@ 
      *
      * @param string $context
      *
-     * @return array
+     * @return string[]
      */
     public function getPermissionsShow($context)
     {
@@ -2482,6 +2487,9 @@ 
         return true;
     }
 
+    /**
+     * @param string $action
+     */
     public function configureActionButtons($action, $object = null)
     {
         $list = [];
@@ -2612,7 +2620,6 @@ 
     }
 
     /**
-     * @param FormMapper $form
      */
     final public function getSearchResultLink($object)
     {
@@ -2711,6 +2718,7 @@ 
     /**
      * NEXT_MAJOR: remove this method.
      *
+     * @param string $action
      * @return mixed
      *
      * @deprecated Use configureTabMenu instead

src/Menu/Matcher/Voter/AdminVoter.php

Doc Comments +1 added lines

@@ -42,6 +42,7 @@
     /**
      * @deprecated since version 3.31. Pass a RequestStack to the constructor instead.
      *
+     * @param Request $request
      * @return $this
      */
     public function setRequest($request)

tests/Controller/HelperControllerTest.php

Doc Comments +3 added lines

@@ -495,6 +495,9 @@
         $this->assertSame('{"status":"OK","more":false,"items":[{"id":123,"label":"FOO"}]}', $response->getContent());
     }
 
+    /**
+     * @param string $field
+     */
     private function configureFormConfig($field, $disabled = false)
     {
         $form = $this->prophesize(Form::class);