sonata-project / SonataAdminBundle

src/Admin/AbstractAdmin.php

Unused Use Statements -1 removed lines

@@ -13,7 +13,6 @@
 
 use Doctrine\Common\Util\ClassUtils;
 use Knp\Menu\FactoryInterface as MenuFactoryInterface;
-use Knp\Menu\ItemInterface;
 use Knp\Menu\ItemInterface as MenuItemInterface;
 use Sonata\AdminBundle\Builder\DatagridBuilderInterface;
 use Sonata\AdminBundle\Builder\FormContractorInterface;

Doc Comments +13 added lines

@@ -936,6 +936,7 @@ 
 
     /**
      * NEXT_MAJOR: remove this method.
+     * @param string $subClass
      */
     public function addSubClass($subClass): void
     {
@@ -1064,6 +1065,9 @@ 
         return $this->routeGenerator->hasAdminRoute($this, $name);
     }
 
+    /**
+     * @param string $adminCode
+     */
     public function isCurrentRoute(string $name, ?string $adminCode = null): bool
     {
         if (!$this->hasRequest()) {
@@ -1309,6 +1313,9 @@ 
         return $this->baseControllerName;
     }
 
+    /**
+     * @param string $label
+     */
     public function setLabel(?string $label): void
     {
         $this->label = $label;
@@ -1809,6 +1816,7 @@ 
      *
      * NEXT_MAJOR: remove this method
      *
+     * @param string $domain
      * @return string the translated string
      *
      * @deprecated since sonata-project/admin-bundle 3.9, to be removed with 4.0
@@ -1989,6 +1997,9 @@ 
         return $this->managerType;
     }
 
+    /**
+     * @param string $type
+     */
     public function setManagerType(?string $type): void
     {
         $this->managerType = $type;
@@ -2399,6 +2410,7 @@ 
 
     /**
      * {@inheritdoc}
+     * @param boolean $isShown
      */
     final public function showMosaicButton($isShown): void
     {
@@ -2528,6 +2540,7 @@ 
      * NEXT_MAJOR: remove this method.
      *
      * @deprecated Use configureTabMenu instead
+     * @param string $action
      */
     protected function configureSideMenu(MenuItemInterface $menu, $action, AdminInterface $childAdmin = null): void
     {

src/Maker/AdminMaker.php

Doc Comments +3 added lines

@@ -68,6 +68,9 @@
      */
     private $modelManager;
 
+    /**
+     * @param string $projectDirectory
+     */
     public function __construct($projectDirectory, array $modelManagers = [])
     {
         $this->projectDirectory = $projectDirectory;

src/Object/MetadataInterface.php

Doc Comments +9 added lines

@@ -20,10 +20,19 @@
 {
     public function getTitle(): string;
 
+    /**
+     * @return string|null
+     */
     public function getDescription(): ?string;
 
+    /**
+     * @return string|null
+     */
     public function getImage(): ?string;
 
+    /**
+     * @return string|null
+     */
     public function getDomain(): ?string;
 
     /**

tests/Fixtures/Admin/PostAdmin.php

Doc Comments +3 added lines

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

tests/Controller/CRUDControllerTest.php

Doc Comments +3 added lines

@@ -3986,6 +3986,9 @@
             ]);
     }
 
+    /**
+     * @param string $domain
+     */
     private function expectTranslate(
         string $id,
         array $parameters = [],

src/Object/Metadata.php

Doc Comments +2 added lines

@@ -47,6 +47,8 @@
 
     /**
      * @param array<string, mixed> $options
+     * @param string $description
+     * @param string $domain
      */
     public function __construct(
         string $title,

tests/Twig/Extension/SonataAdminExtensionTest.php

Doc Comments +3 added lines

@@ -2486,6 +2486,9 @@
         return htmlspecialchars(http_build_query($url, '', '&', PHP_QUERY_RFC3986));
     }
 
+    /**
+     * @param string $privateMethod
+     */
     private function getMethodAsPublic($privateMethod): \ReflectionMethod
     {
         $reflection = new \ReflectionMethod('Sonata\AdminBundle\Twig\Extension\SonataAdminExtension', $privateMethod);

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