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 +10 added lines, -1 removed lines

@@ -1024,6 +1024,7 @@ 
 
     /**
      * NEXT_MAJOR: remove this method.
+     * @param string $subClass
      */
     public function addSubClass($subClass)
     {
@@ -1146,6 +1147,10 @@ 
         return $this->routeGenerator->hasAdminRoute($this, $name);
     }
 
+    /**
+     * @param string $name
+     * @param string $adminCode
+     */
     public function isCurrentRoute($name, $adminCode = null)
     {
         if (!$this->hasRequest()) {
@@ -2291,7 +2296,7 @@ 
      *
      * @param string $context
      *
-     * @return array
+     * @return string[]
      */
     public function getPermissionsShow($context)
     {
@@ -2596,6 +2601,9 @@ 
         return true;
     }
 
+    /**
+     * @param string $action
+     */
     public function configureActionButtons($action, $object = null)
     {
         $list = [];
@@ -2847,6 +2855,7 @@ 
     /**
      * NEXT_MAJOR: remove this method.
      *
+     * @param string $action
      * @return mixed
      *
      * @deprecated Use configureTabMenu instead

src/Admin/FieldDescriptionCollection.php

Doc Comments +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
     }
 
     /**
-     * @return array
+     * @return FieldDescriptionInterface[]
      */
     public function getElements()
     {

src/Generator/ControllerGenerator.php

Doc Comments +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
     private $file;
 
     /**
-     * @param array|string $skeletonDirectory
+     * @param string $skeletonDirectory
      */
     public function __construct($skeletonDirectory)
     {

tests/Admin/Extension/LockExtensionTest.php

Doc Comments +4 added lines

@@ -195,6 +195,10 @@
         return new FormMapper($contractor->reveal(), $formBuilder, $this->admin->reveal());
     }
 
+    /**
+     * @param string $uniqid
+     * @param Request $request
+     */
     private function configureAdmin($uniqid = null, $request = null, $modelManager = null)
     {
         $this->admin->getUniqid()->willReturn($uniqid);

tests/Controller/CRUDControllerTest.php

Doc Comments +7 added lines

@@ -3759,6 +3759,9 @@ 
         ];
     }
 
+    /**
+     * @param string $method
+     */
     private function assertLoggerLogsModelManagerException($subject, $method)
     {
         $exception = new ModelManagerException(
@@ -3781,6 +3784,10 @@ 
             ]);
     }
 
+    /**
+     * @param string $id
+     * @param string $domain
+     */
     private function expectTranslate($id, array $parameters = [], $domain = null, $locale = null)
     {
         $this->translator->expects($this->once())

tests/Fixtures/Admin/PostAdmin.php

Doc Comments +3 added lines

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

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

tests/Menu/Matcher/Voter/AdminVoterTest.php

Doc Comments +3 added lines

@@ -70,6 +70,7 @@ 
 
     /**
      * {@inheritdoc}
+     * @param string $code
      */
     private function getAdmin($code, $list = false, $granted = false)
     {
@@ -102,6 +103,8 @@ 
 
     /**
      * {@inheritdoc}
+     * @param string $parentCode
+     * @param string $childCode
      */
     private function getChildAdmin($parentCode, $childCode, $list = false, $granted = false)
     {

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