sonata-project / SonataAdminBundle

src/Admin/BaseFieldDescription.php

Doc Comments +3 added lines

@@ -441,6 +441,9 @@
         return $object->{$fieldName};
     }
 
+    /**
+     * @param string $fieldName
+     */
     private function cacheFieldGetter($object, ?string $fieldName, string $method, ?string $getter = null): void
     {
         $getterKey = $this->getFieldGetterKey($object, $fieldName);

tests/Datagrid/DatagridMapperTest.php

Doc Comments +4 added lines

@@ -292,6 +292,10 @@
         $this->assertTrue($this->datagridMapper->has('baz'));
     }
 
+    /**
+     * @param string $name
+     * @param string $label
+     */
     private function getFieldDescriptionMock(?string $name = null, ?string $label = null): BaseFieldDescription
     {
         $fieldDescription = $this->getMockForAbstractClass(BaseFieldDescription::class);

tests/Datagrid/ListMapperTest.php

Doc Comments +4 added lines

@@ -286,6 +286,10 @@
         $this->assertTrue($this->listMapper->has('baz'));
     }
 
+    /**
+     * @param string $name
+     * @param string $label
+     */
     private function getFieldDescriptionMock(?string $name = null, ?string $label = null): BaseFieldDescription
     {
         $fieldDescription = $this->getMockForAbstractClass(BaseFieldDescription::class);

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

src/Admin/AbstractAdmin.php

Doc Comments +7 added lines, -1 removed lines

@@ -949,6 +949,7 @@ 
 
     /**
      * NEXT_MAJOR: remove this method.
+     * @param string $subClass
      */
     public function addSubClass($subClass): void
     {
@@ -1104,6 +1105,9 @@ 
         return $this->routeGenerator->hasAdminRoute($this, $name);
     }
 
+    /**
+     * @param string $adminCode
+     */
     public function isCurrentRoute(string $name, ?string $adminCode = null): bool
     {
         if (!$this->hasRequest()) {
@@ -2127,7 +2131,7 @@ 
      *
      * @param string $context
      *
-     * @return array
+     * @return string[]
      */
     public function getPermissionsShow($context)
     {
@@ -2532,6 +2536,7 @@ 
 
     /**
      * {@inheritdoc}
+     * @param boolean $isShown
      */
     final public function showMosaicButton($isShown): void
     {
@@ -2663,6 +2668,7 @@ 
      * NEXT_MAJOR: remove this method.
      *
      * @deprecated Use configureTabMenu instead
+     * @param string $action
      */
     protected function configureSideMenu(MenuItemInterface $menu, $action, AdminInterface $childAdmin = null)
     {

src/Twig/GlobalVariables.php

Doc Comments +4 added lines, -3 removed lines

@@ -44,6 +44,7 @@ 
 
     /**
      * @param ContainerInterface|Pool $adminPool
+     * @param string $mosaicBackground
      */
     public function __construct($adminPool, ?string $mosaicBackground = null)
     {
@@ -80,7 +81,7 @@ 
     /**
      * @param string $code
      * @param string $action
-     * @param array  $parameters
+     * @param string[]  $parameters
      * @param int    $absolute
      *
      * @return string
@@ -95,8 +96,8 @@ 
     /**
      * @param string $code
      * @param string $action
-     * @param object $object
-     * @param array  $parameters
+     * @param string $object
+     * @param string[]  $parameters
      * @param int    $absolute
      *
      * @return string

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

tests/Menu/Integration/BaseMenuTest.php

Doc Comments +3 added lines

@@ -44,6 +44,9 @@
         $this->environment = new Environment($loader, ['strict_variables' => true]);
     }
 
+    /**
+     * @return string
+     */
     abstract protected function getTemplate();
 
     protected function getTranslator(): TranslatorInterface