sonata-project / SonataAdminBundle

tests/Show/ShowMapperTest.php

Doc Comments +4 added lines

@@ -501,6 +501,10 @@
         $this->admin->setLabelTranslatorStrategy(new NoopLabelTranslatorStrategy());
     }
 
+    /**
+     * @param string $name
+     * @param string $label
+     */
     private function getFieldDescriptionMock(?string $name = null, ?string $label = null): BaseFieldDescription
     {
         $fieldDescription = $this->getMockForAbstractClass(BaseFieldDescription::class);

src/Security/Handler/SecurityHandlerInterface.php

Doc Comments +3 added lines, -1 removed lines

@@ -21,7 +21,7 @@ 
 interface SecurityHandlerInterface
 {
     /**
-     * @param string|array $attributes
+     * @param string $attributes
      * @param mixed|null   $object
      *
      * @return bool
@@ -44,6 +44,7 @@ 
      * Create object security, fe. make the current user owner of the object.
      *
      * @param object $object
+     * @return void
      */
     public function createObjectSecurity(AdminInterface $admin, $object);
 
@@ -51,6 +52,7 @@ 
      * Remove object security.
      *
      * @param object $object
+     * @return void
      */
     public function deleteObjectSecurity(AdminInterface $admin, $object);
 }

tests/Datagrid/PagerTest.php

Doc Comments +3 added lines

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

src/Command/QuestionableCommand.php

Doc Comments +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 {
     /**
      * @param string   $questionText
-     * @param mixed    $default
+     * @param string    $default
      * @param callable $validator
      *
      * @return mixed

tests/Form/Widget/FormSonataFilterChoiceWidgetTest.php

Doc Comments +3 added lines

@@ -56,6 +56,9 @@
         );
     }
 
+    /**
+     * @return string
+     */
     protected function getChoiceClass()
     {
         return ChoiceType::class;

src/Model/ModelManagerInterface.php

Doc Comments +11 added lines, -2 removed lines

@@ -37,6 +37,7 @@ 
      * @param object $object
      *
      * @throws ModelManagerException
+     * @return void
      */
     public function create($object);
 
@@ -44,6 +45,7 @@ 
      * @param object $object
      *
      * @throws ModelManagerException
+     * @return void
      */
     public function update($object);
 
@@ -51,6 +53,7 @@ 
      * @param object $object
      *
      * @throws ModelManagerException
+     * @return void
      */
     public function delete($object);
 
@@ -72,7 +75,7 @@ 
      * @param string $class
      * @param mixed  $id
      *
-     * @return object|null the object with id or null if not found
+     * @return \Sonata\AdminBundle\Tests\App\Model\Foo|null the object with id or null if not found
      */
     public function find($class, $id);
 
@@ -80,6 +83,7 @@ 
      * @param string $class
      *
      * @throws ModelManagerException
+     * @return void
      */
     public function batchDelete($class, ProxyQueryInterface $queryProxy);
 
@@ -156,7 +160,7 @@ 
      *
      * @param string $class
      *
-     * @return object
+     * @return \Sonata\AdminBundle\Tests\App\Model\Foo
      */
     public function getModelInstance($class);
 
@@ -172,6 +176,7 @@ 
      *
      * @param array  $collection
      * @param object $element
+     * @return void
      */
     public function collectionRemoveElement(&$collection, &$element);
 
@@ -180,6 +185,7 @@ 
      *
      * @param array  $collection
      * @param object $element
+     * @return void
      */
     public function collectionAddElement(&$collection, &$element);
 
@@ -197,6 +203,7 @@ 
      * Clear the collection.
      *
      * @param array $collection
+     * @return void
      */
     public function collectionClear(&$collection);
 
@@ -229,6 +236,7 @@ 
 
     /**
      * @param mixed $query
+     * @return void
      */
     public function executeQuery($query);
 
@@ -261,6 +269,7 @@ 
 
     /**
      * @param string $class
+     * @return void
      */
     public function addIdentifiersToQuery($class, ProxyQueryInterface $query, array $idx);
 }

src/Filter/FilterInterface.php

Doc Comments +7 added lines, -2 removed lines

@@ -62,6 +62,7 @@ 
 
     /**
      * @param string $label
+     * @return void
      */
     public function setLabel($label);
 
@@ -72,7 +73,7 @@ 
 
     /**
      * @param string     $name
-     * @param mixed|null $default
+     * @param boolean $default
      *
      * @return mixed
      */
@@ -80,12 +81,14 @@ 
 
     /**
      * @param string $name
-     * @param mixed  $value
+     * @param boolean  $value
+     * @return void
      */
     public function setOption($name, $value);
 
     /**
      * @param string $name
+     * @return void
      */
     public function initialize($name, array $options = []);
 
@@ -129,6 +132,7 @@ 
      *
      * @param string $name
      * @param mixed  $value
+     * @return void
      */
     public function setFieldOption($name, $value);
 
@@ -155,6 +159,7 @@ 
      * Set the condition to use with the left side of the query : OR or AND.
      *
      * @param string $condition
+     * @return void
      */
     public function setCondition($condition);
 

tests/Fixtures/Admin/PostAdmin.php

Doc Comments +3 added lines

@@ -19,6 +19,9 @@
 {
     protected $metadataClass;
 
+    /**
+     * @param string $associationMapping
+     */
     public function setParentAssociationMapping($associationMapping): void
     {
         $this->parentAssociationMapping = $associationMapping;

src/Form/ChoiceList/ModelChoiceLoader.php

Doc Comments +1 added lines

@@ -65,6 +65,7 @@
      * @param string|null $property
      * @param mixed|null  $query
      * @param array       $choices
+     * @param PropertyAccessorInterface $propertyAccessor
      */
     public function __construct(
         ModelManagerInterface $modelManager,