Completed
Pull Request — master (#5661)
by Grégoire
03:31
created
src/Admin/AbstractAdmin.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -13,7 +13,6 @@
 block discarded – undo
13 13
 
14 14
 use Doctrine\Common\Util\ClassUtils;
15 15
 use Knp\Menu\FactoryInterface as MenuFactoryInterface;
16
-use Knp\Menu\ItemInterface;
17 16
 use Knp\Menu\ItemInterface as MenuItemInterface;
18 17
 use Sonata\AdminBundle\Builder\DatagridBuilderInterface;
19 18
 use Sonata\AdminBundle\Builder\FormContractorInterface;
Please login to merge, or discard this patch.
Doc Comments   +12 added lines, -1 removed lines patch added patch discarded remove patch
@@ -949,6 +949,7 @@  discard block
 block discarded – undo
949 949
 
950 950
     /**
951 951
      * NEXT_MAJOR: remove this method.
952
+     * @param string $subClass
952 953
      */
953 954
     public function addSubClass($subClass): void
954 955
     {
@@ -1104,6 +1105,10 @@  discard block
 block discarded – undo
1104 1105
         return $this->routeGenerator->hasAdminRoute($this, $name);
1105 1106
     }
1106 1107
 
1108
+    /**
1109
+     * @param string $name
1110
+     * @param string $adminCode
1111
+     */
1107 1112
     public function isCurrentRoute($name, $adminCode = null)
1108 1113
     {
1109 1114
         if (!$this->hasRequest()) {
@@ -2130,7 +2135,7 @@  discard block
 block discarded – undo
2130 2135
      *
2131 2136
      * @param string $context
2132 2137
      *
2133
-     * @return array
2138
+     * @return string[]
2134 2139
      */
2135 2140
     public function getPermissionsShow($context)
2136 2141
     {
@@ -2408,6 +2413,7 @@  discard block
 block discarded – undo
2408 2413
 
2409 2414
     /**
2410 2415
      * {@inheritdoc}
2416
+     * @param string $action
2411 2417
      */
2412 2418
     public function hasAccess($action, $object = null)
2413 2419
     {
@@ -2532,6 +2538,7 @@  discard block
 block discarded – undo
2532 2538
 
2533 2539
     /**
2534 2540
      * {@inheritdoc}
2541
+     * @param boolean $isShown
2535 2542
      */
2536 2543
     final public function showMosaicButton($isShown): void
2537 2544
     {
@@ -2652,6 +2659,9 @@  discard block
 block discarded – undo
2652 2659
     {
2653 2660
     }
2654 2661
 
2662
+    /**
2663
+     * @param string $action
2664
+     */
2655 2665
     protected function configureActionButtons($buttonList, $action, $object = null)
2656 2666
     {
2657 2667
         return $buttonList;
@@ -2673,6 +2683,7 @@  discard block
 block discarded – undo
2673 2683
      * NEXT_MAJOR: remove this method.
2674 2684
      *
2675 2685
      * @deprecated Use configureTabMenu instead
2686
+     * @param string $action
2676 2687
      */
2677 2688
     protected function configureSideMenu(MenuItemInterface $menu, $action, AdminInterface $childAdmin = null)
2678 2689
     {
Please login to merge, or discard this patch.
src/Admin/FieldDescriptionCollection.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@
 block discarded – undo
30 30
     }
31 31
 
32 32
     /**
33
-     * @return array
33
+     * @return FieldDescriptionInterface[]
34 34
      */
35 35
     public function getElements()
36 36
     {
Please login to merge, or discard this patch.
src/Generator/ControllerGenerator.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@
 block discarded – undo
31 31
     private $file;
32 32
 
33 33
     /**
34
-     * @param array|string $skeletonDirectory
34
+     * @param string $skeletonDirectory
35 35
      */
36 36
     public function __construct($skeletonDirectory)
37 37
     {
Please login to merge, or discard this patch.
src/Controller/CRUDController.php 1 patch
Doc Comments   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -208,7 +208,7 @@  discard block
 block discarded – undo
208 208
     /**
209 209
      * Delete action.
210 210
      *
211
-     * @param int|string|null $id
211
+     * @param integer|null $id
212 212
      *
213 213
      * @throws NotFoundHttpException If the object does not exist
214 214
      * @throws AccessDeniedException If access is not granted
@@ -726,7 +726,7 @@  discard block
 block discarded – undo
726 726
     /**
727 727
      * View history revision of object.
728 728
      *
729
-     * @param int|string|null $id
729
+     * @param null|integer $id
730 730
      * @param string|null     $revision
731 731
      *
732 732
      * @throws AccessDeniedException If access is not granted
@@ -786,9 +786,9 @@  discard block
 block discarded – undo
786 786
     /**
787 787
      * Compare history revisions of object.
788 788
      *
789
-     * @param int|string|null $id
790
-     * @param int|string|null $base_revision
791
-     * @param int|string|null $compare_revision
789
+     * @param null|integer $id
790
+     * @param null|integer $base_revision
791
+     * @param null|integer $compare_revision
792 792
      *
793 793
      * @throws AccessDeniedException If access is not granted
794 794
      * @throws NotFoundHttpException If the object or revision does not exist or the audit reader is not available
Please login to merge, or discard this patch.
tests/Datagrid/PagerTest.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -639,6 +639,10 @@
 block discarded – undo
639 639
         $this->assertNull($this->pager->getQuery());
640 640
     }
641 641
 
642
+    /**
643
+     * @param Pager $obj
644
+     * @param string $name
645
+     */
642 646
     protected function callMethod($obj, $name, array $args = [])
643 647
     {
644 648
         $class = new \ReflectionClass($obj);
Please login to merge, or discard this patch.
src/Form/ChoiceList/ModelChoiceList.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -101,7 +101,7 @@
 block discarded – undo
101 101
      * @param string            $class
102 102
      * @param string|null       $property
103 103
      * @param QueryBuilder|null $query
104
-     * @param array             $choices
104
+     * @param integer[]             $choices
105 105
      */
106 106
     public function __construct(
107 107
         ModelManagerInterface $modelManager,
Please login to merge, or discard this patch.
src/Generator/AdminGenerator.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@
 block discarded – undo
37 37
     private $file;
38 38
 
39 39
     /**
40
-     * @param array|string $skeletonDirectories
40
+     * @param string $skeletonDirectories
41 41
      */
42 42
     public function __construct(ModelManagerInterface $modelManager, $skeletonDirectories)
43 43
     {
Please login to merge, or discard this patch.
src/Controller/PolyfillControllerTrait.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -27,6 +27,9 @@
 block discarded – undo
27 27
  */
28 28
 trait PolyfillControllerTrait
29 29
 {
30
+    /**
31
+     * @param string $methodName
32
+     */
30 33
     public function __call($methodName, $arguments): void
31 34
     {
32 35
         $this->proxyToController($methodName, $arguments);
Please login to merge, or discard this patch.
src/Datagrid/Pager.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -434,7 +434,7 @@  discard block
 block discarded – undo
434 434
      * Returns a parameter.
435 435
      *
436 436
      * @param string $name
437
-     * @param mixed  $default
437
+     * @param null|string  $default
438 438
      *
439 439
      * @return mixed
440 440
      */
@@ -459,7 +459,7 @@  discard block
 block discarded – undo
459 459
      * Sets a parameter.
460 460
      *
461 461
      * @param string $name
462
-     * @param mixed  $value
462
+     * @param string  $value
463 463
      */
464 464
     public function setParameter($name, $value): void
465 465
     {
Please login to merge, or discard this patch.