Completed
Pull Request — 3.x (#6299)
by Javier
12:29
created
src/Twig/Extension/SonataAdminExtension.php 1 patch
Doc Comments   +6 added lines, -5 removed lines patch added patch discarded remove patch
@@ -182,7 +182,7 @@  discard block
 block discarded – undo
182 182
     /**
183 183
      * render a view element.
184 184
      *
185
-     * @param object $object
185
+     * @param \stdClass $object
186 186
      *
187 187
      * @return string
188 188
      */
@@ -221,8 +221,8 @@  discard block
 block discarded – undo
221 221
     /**
222 222
      * render a compared view element.
223 223
      *
224
-     * @param mixed $baseObject
225
-     * @param mixed $compareObject
224
+     * @param \stdClass $baseObject
225
+     * @param \stdClass $compareObject
226 226
      *
227 227
      * @return string
228 228
      */
@@ -343,7 +343,8 @@  discard block
 block discarded – undo
343 343
     /**
344 344
      * Get the identifiers as a string that is safe to use in a url.
345 345
      *
346
-     * @param object $model
346
+     * @param \stdClass $model
347
+     * @param AdminInterface $admin
347 348
      *
348 349
      * @return string string representation of the id that is safe to use in a url
349 350
      */
@@ -365,7 +366,7 @@  discard block
 block discarded – undo
365 366
     }
366 367
 
367 368
     /**
368
-     * @return string|bool
369
+     * @return string|false
369 370
      */
370 371
     public function getXEditableType($type)
371 372
     {
Please login to merge, or discard this patch.
tests/Datagrid/ListMapperTest.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -414,6 +414,10 @@
 block discarded – undo
414 414
         );
415 415
     }
416 416
 
417
+    /**
418
+     * @param string $name
419
+     * @param string $label
420
+     */
417 421
     private function getFieldDescriptionMock(?string $name = null, ?string $label = null): BaseFieldDescription
418 422
     {
419 423
         $fieldDescription = $this->getMockForAbstractClass(BaseFieldDescription::class);
Please login to merge, or discard this patch.
src/Block/AdminListBlockService.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@
 block discarded – undo
44 44
      * NEXT_MAJOR: Change signature for (Environment $twig, Pool $pool, ?TemplateRegistryInterface $templateRegistry = null).
45 45
      *
46 46
      * @param Environment|string                  $twigOrName
47
-     * @param EngineInterface|Pool|null           $poolOrTemplating
47
+     * @param Pool|null           $poolOrTemplating
48 48
      * @param Pool|TemplateRegistryInterface|null $templateRegistryOrPool
49 49
      */
50 50
     public function __construct(
Please login to merge, or discard this patch.
src/Block/AdminSearchBlockService.php 1 patch
Doc Comments   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -45,8 +45,9 @@
 block discarded – undo
45 45
      * NEXT_MAJOR: Change signature for (Environment $twig, Pool $pool, SearchHandler $searchHandler).
46 46
      *
47 47
      * @param Environment|string        $twigOrName
48
-     * @param Pool|EngineInterface|null $poolOrTemplating
48
+     * @param Pool|null $poolOrTemplating
49 49
      * @param SearchHandler|Pool        $searchHandlerOrPool
50
+     * @param SearchHandler $searchHandler
50 51
      */
51 52
     public function __construct($twigOrName, ?object $poolOrTemplating, object $searchHandlerOrPool, ?SearchHandler $searchHandler = null)
52 53
     {
Please login to merge, or discard this patch.
src/Block/AdminStatsBlockService.php 1 patch
Doc Comments   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -37,7 +37,8 @@
 block discarded – undo
37 37
      * NEXT_MAJOR: Change signature for (Environment $twig, Pool $pool).
38 38
      *
39 39
      * @param Environment|string        $twigOrName
40
-     * @param Pool|EngineInterface|null $poolOrTemplating
40
+     * @param Pool|null $poolOrTemplating
41
+     * @param Pool $pool
41 42
      */
42 43
     public function __construct($twigOrName, ?object $poolOrTemplating, ?Pool $pool = null)
43 44
     {
Please login to merge, or discard this patch.
src/Admin/BaseFieldDescription.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -540,6 +540,9 @@
 block discarded – undo
540 540
         return $object->{$fieldName};
541 541
     }
542 542
 
543
+    /**
544
+     * @param string $fieldName
545
+     */
543 546
     private function cacheFieldGetter(object $object, ?string $fieldName, string $method, ?string $getter = null): void
544 547
     {
545 548
         $getterKey = $this->getFieldGetterKey($object, $fieldName);
Please login to merge, or discard this patch.
src/Model/ModelManagerInterface.php 1 patch
Doc Comments   +11 added lines, -1 removed lines patch added patch discarded remove patch
@@ -36,6 +36,7 @@  discard block
 block discarded – undo
36 36
      * @param object $object
37 37
      *
38 38
      * @throws ModelManagerException
39
+     * @return void
39 40
      */
40 41
     public function create($object);
41 42
 
@@ -43,6 +44,7 @@  discard block
 block discarded – undo
43 44
      * @param object $object
44 45
      *
45 46
      * @throws ModelManagerException
47
+     * @return void
46 48
      */
47 49
     public function update($object);
48 50
 
@@ -50,6 +52,7 @@  discard block
 block discarded – undo
50 52
      * @param object $object
51 53
      *
52 54
      * @throws ModelManagerException
55
+     * @return void
53 56
      */
54 57
     public function delete($object);
55 58
 
@@ -71,7 +74,7 @@  discard block
 block discarded – undo
71 74
      * @param string $class
72 75
      * @param mixed  $id
73 76
      *
74
-     * @return object|null the object with id or null if not found
77
+     * @return \Sonata\AdminBundle\Tests\App\Model\Foo|null the object with id or null if not found
75 78
      */
76 79
     public function find($class, $id);
77 80
 
@@ -79,6 +82,7 @@  discard block
 block discarded – undo
79 82
      * @param string $class
80 83
      *
81 84
      * @throws ModelManagerException
85
+     * @return void
82 86
      */
83 87
     public function batchDelete($class, ProxyQueryInterface $queryProxy);
84 88
 
@@ -90,6 +94,7 @@  discard block
 block discarded – undo
90 94
      *
91 95
      * @param array  $parentAssociationMapping
92 96
      * @param string $class
97
+     * @return FieldDescriptionInterface
93 98
      */
94 99
     public function getParentFieldDescription($parentAssociationMapping, $class);
95 100
 
@@ -180,6 +185,7 @@  discard block
 block discarded – undo
180 185
      *
181 186
      * @param array  $collection
182 187
      * @param object $element
188
+     * @return void
183 189
      */
184 190
     public function collectionRemoveElement(&$collection, &$element);
185 191
 
@@ -188,6 +194,7 @@  discard block
 block discarded – undo
188 194
      *
189 195
      * @param array  $collection
190 196
      * @param object $element
197
+     * @return void
191 198
      */
192 199
     public function collectionAddElement(&$collection, &$element);
193 200
 
@@ -205,6 +212,7 @@  discard block
 block discarded – undo
205 212
      * Clear the collection.
206 213
      *
207 214
      * @param array $collection
215
+     * @return void
208 216
      */
209 217
     public function collectionClear(&$collection);
210 218
 
@@ -235,6 +243,7 @@  discard block
 block discarded – undo
235 243
 
236 244
     /**
237 245
      * @param mixed $query
246
+     * @return void
238 247
      */
239 248
     public function executeQuery($query);
240 249
 
@@ -272,6 +281,7 @@  discard block
 block discarded – undo
272 281
 
273 282
     /**
274 283
      * @param string $class
284
+     * @return void
275 285
      */
276 286
     public function addIdentifiersToQuery($class, ProxyQueryInterface $query, array $idx);
277 287
 }
Please login to merge, or discard this patch.
src/Admin/AbstractAdmin.php 1 patch
Doc Comments   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1111,6 +1111,7 @@  discard block
 block discarded – undo
1111 1111
 
1112 1112
     /**
1113 1113
      * NEXT_MAJOR: remove this method.
1114
+     * @param string $subClass
1114 1115
      */
1115 1116
     public function addSubClass($subClass)
1116 1117
     {
@@ -2677,7 +2678,7 @@  discard block
 block discarded – undo
2677 2678
      *
2678 2679
      * @param string $context
2679 2680
      *
2680
-     * @return array
2681
+     * @return string[]
2681 2682
      */
2682 2683
     public function getPermissionsShow($context)
2683 2684
     {
Please login to merge, or discard this patch.
src/Admin/AdminInterface.php 1 patch
Doc Comments   +73 added lines, -2 removed lines patch added patch discarded remove patch
@@ -67,6 +67,9 @@  discard block
 block discarded – undo
67 67
  */
68 68
 interface AdminInterface extends AccessRegistryInterface, FieldDescriptionRegistryInterface, LifecycleHookProviderInterface, MenuBuilderInterface, ParentAdminInterface, UrlGeneratorInterface
69 69
 {
70
+    /**
71
+     * @return void
72
+     */
70 73
     public function setMenuFactory(MenuFactoryInterface $menuFactory);
71 74
 
72 75
     /**
@@ -74,8 +77,14 @@  discard block
 block discarded – undo
74 77
      */
75 78
     public function getMenuFactory();
76 79
 
80
+    /**
81
+     * @return void
82
+     */
77 83
     public function setFormContractor(FormContractorInterface $formContractor);
78 84
 
85
+    /**
86
+     * @return void
87
+     */
79 88
     public function setListBuilder(ListBuilderInterface $listBuilder);
80 89
 
81 90
     /**
@@ -83,6 +92,9 @@  discard block
 block discarded – undo
83 92
      */
84 93
     public function getListBuilder();
85 94
 
95
+    /**
96
+     * @return void
97
+     */
86 98
     public function setDatagridBuilder(DatagridBuilderInterface $datagridBuilder);
87 99
 
88 100
     /**
@@ -90,6 +102,9 @@  discard block
 block discarded – undo
90 102
      */
91 103
     public function getDatagridBuilder();
92 104
 
105
+    /**
106
+     * @return void
107
+     */
93 108
     public function setTranslator(TranslatorInterface $translator);
94 109
 
95 110
     /**
@@ -97,8 +112,14 @@  discard block
 block discarded – undo
97 112
      */
98 113
     public function getTranslator();
99 114
 
115
+    /**
116
+     * @return void
117
+     */
100 118
     public function setRequest(Request $request);
101 119
 
120
+    /**
121
+     * @return void
122
+     */
102 123
     public function setConfigurationPool(Pool $pool);
103 124
 
104 125
     /**
@@ -111,6 +132,9 @@  discard block
 block discarded – undo
111 132
      */
112 133
     public function getClass();
113 134
 
135
+    /**
136
+     * @return void
137
+     */
114 138
     public function attachAdminClass(FieldDescriptionInterface $fieldDescription);
115 139
 
116 140
     // NEXT_MAJOR: uncomment this method in 4.0
@@ -125,6 +149,7 @@  discard block
 block discarded – undo
125 149
      * Set base controller name.
126 150
      *
127 151
      * @param string $baseControllerName
152
+     * @return void
128 153
      */
129 154
     public function setBaseControllerName($baseControllerName);
130 155
 
@@ -199,6 +224,9 @@  discard block
 block discarded – undo
199 224
      */
200 225
     public function getSecurityInformation();
201 226
 
227
+    /**
228
+     * @return void
229
+     */
202 230
     public function setParentFieldDescription(FieldDescriptionInterface $parentFieldDescription);
203 231
 
204 232
     /**
@@ -276,12 +304,13 @@  discard block
 block discarded – undo
276 304
      *
277 305
      * @param object $model
278 306
      *
279
-     * @return mixed
307
+     * @return string
280 308
      */
281 309
     public function id($model);
282 310
 
283 311
     /**
284 312
      * @param ValidatorInterface $validator
313
+     * @return void
285 314
      */
286 315
     public function setValidator($validator);
287 316
 
@@ -298,6 +327,9 @@  discard block
 block discarded – undo
298 327
 //    NEXT_MAJOR: uncomment this method in 4.0
299 328
 //    public function getList(): ?FieldDescriptionCollection;
300 329
 
330
+    /**
331
+     * @return void
332
+     */
301 333
     public function setFormTheme(array $formTheme);
302 334
 
303 335
     /**
@@ -305,6 +337,9 @@  discard block
 block discarded – undo
305 337
      */
306 338
     public function getFormTheme();
307 339
 
340
+    /**
341
+     * @return void
342
+     */
308 343
     public function setFilterTheme(array $filterTheme);
309 344
 
310 345
     /**
@@ -312,6 +347,9 @@  discard block
 block discarded – undo
312 347
      */
313 348
     public function getFilterTheme();
314 349
 
350
+    /**
351
+     * @return void
352
+     */
315 353
     public function addExtension(AdminExtensionInterface $extension);
316 354
 
317 355
     /**
@@ -321,6 +359,9 @@  discard block
 block discarded – undo
321 359
      */
322 360
     public function getExtensions();
323 361
 
362
+    /**
363
+     * @return void
364
+     */
324 365
     public function setRouteBuilder(RouteBuilderInterface $routeBuilder);
325 366
 
326 367
     /**
@@ -335,6 +376,9 @@  discard block
 block discarded – undo
335 376
      */
336 377
     public function toString($object);
337 378
 
379
+    /**
380
+     * @return void
381
+     */
338 382
     public function setLabelTranslatorStrategy(LabelTranslatorStrategyInterface $labelTranslatorStrategy);
339 383
 
340 384
     /**
@@ -358,6 +402,7 @@  discard block
 block discarded – undo
358 402
 
359 403
     /**
360 404
      * @param string $uniqId
405
+     * @return void
361 406
      */
362 407
     public function setUniqid($uniqId);
363 408
 
@@ -377,6 +422,7 @@  discard block
 block discarded – undo
377 422
 
378 423
     /**
379 424
      * @param object|null $subject
425
+     * @return void
380 426
      */
381 427
     public function setSubject($subject);
382 428
 
@@ -414,7 +460,7 @@  discard block
 block discarded – undo
414 460
      *
415 461
      * Returns the collection of list FieldDescriptions.
416 462
      *
417
-     * @return array
463
+     * @return FieldDescriptionInterface[]
418 464
      */
419 465
     public function getListFieldDescriptions();
420 466
 
@@ -432,6 +478,9 @@  discard block
 block discarded – undo
432 478
      */
433 479
     public function getDataSourceIterator();
434 480
 
481
+    /**
482
+     * @return void
483
+     */
435 484
     public function configure();
436 485
 
437 486
     /**
@@ -439,6 +488,7 @@  discard block
 block discarded – undo
439 488
      *
440 489
      * @param string $actionName
441 490
      * @param bool   $allElements
491
+     * @return void
442 492
      */
443 493
     public function preBatchAction($actionName, ProxyQueryInterface $query, array &$idx, $allElements);
444 494
 
@@ -463,6 +513,7 @@  discard block
 block discarded – undo
463 513
      *
464 514
      * @deprecated this feature cannot be stable, use a custom validator,
465 515
      *             the feature will be removed with Symfony 2.2
516
+     * @return void
466 517
      */
467 518
     public function validate(ErrorElement $errorElement, $object);
468 519
 
@@ -477,6 +528,7 @@  discard block
 block discarded – undo
477 528
      * Add object security, fe. make the current user owner of the object.
478 529
      *
479 530
      * @param object $object
531
+     * @return void
480 532
      */
481 533
     public function createObjectSecurity($object);
482 534
 
@@ -485,6 +537,9 @@  discard block
 block discarded – undo
485 537
      */
486 538
     public function getParent();
487 539
 
540
+    /**
541
+     * @return void
542
+     */
488 543
     public function setParent(self $admin);
489 544
 
490 545
     /**
@@ -509,6 +564,7 @@  discard block
 block discarded – undo
509 564
      * Set the translation domain.
510 565
      *
511 566
      * @param string $translationDomain the translation domain
567
+     * @return void
512 568
      */
513 569
     public function setTranslationDomain($translationDomain);
514 570
 
@@ -530,6 +586,7 @@  discard block
 block discarded – undo
530 586
 
531 587
     /**
532 588
      * Set the form groups.
589
+     * @return void
533 590
      */
534 591
     public function setFormGroups(array $formGroups);
535 592
 
@@ -538,6 +595,9 @@  discard block
 block discarded – undo
538 595
      */
539 596
     public function getFormTabs();
540 597
 
598
+    /**
599
+     * @return void
600
+     */
541 601
     public function setFormTabs(array $formTabs);
542 602
 
543 603
     /**
@@ -545,12 +605,16 @@  discard block
 block discarded – undo
545 605
      */
546 606
     public function getShowTabs();
547 607
 
608
+    /**
609
+     * @return void
610
+     */
548 611
     public function setShowTabs(array $showTabs);
549 612
 
550 613
     /**
551 614
      * Remove a form group field.
552 615
      *
553 616
      * @param string $key
617
+     * @return void
554 618
      */
555 619
     public function removeFieldFromFormGroup($key);
556 620
 
@@ -565,6 +629,7 @@  discard block
 block discarded – undo
565 629
 
566 630
     /**
567 631
      * Set the show groups.
632
+     * @return void
568 633
      */
569 634
     public function setShowGroups(array $showGroups);
570 635
 
@@ -572,6 +637,7 @@  discard block
 block discarded – undo
572 637
      * Reorder items in showGroup.
573 638
      *
574 639
      * @param string $group
640
+     * @return void
575 641
      */
576 642
     public function reorderShowGroup($group, array $keys);
577 643
 
@@ -581,6 +647,7 @@  discard block
 block discarded – undo
581 647
      * add a FieldDescription.
582 648
      *
583 649
      * @param string $name
650
+     * @return void
584 651
      */
585 652
     public function addFormFieldDescription($name, FieldDescriptionInterface $fieldDescription);
586 653
 
@@ -590,6 +657,7 @@  discard block
 block discarded – undo
590 657
      * Remove a FieldDescription.
591 658
      *
592 659
      * @param string $name
660
+     * @return void
593 661
      */
594 662
     public function removeFormFieldDescription($name);
595 663
 
@@ -602,6 +670,7 @@  discard block
 block discarded – undo
602 670
 
603 671
     /**
604 672
      * Sets the list of supported sub classes.
673
+     * @return void
605 674
      */
606 675
     public function setSubClasses(array $subClasses);
607 676
 
@@ -670,6 +739,7 @@  discard block
 block discarded – undo
670 739
      * Set the current child status.
671 740
      *
672 741
      * @param bool $currentChild
742
+     * @return void
673 743
      */
674 744
     public function setCurrentChild($currentChild);
675 745
 
@@ -719,6 +789,7 @@  discard block
 block discarded – undo
719 789
 
720 790
     /**
721 791
      * @param string $mode
792
+     * @return void
722 793
      */
723 794
     public function setListMode($mode);
724 795
 
Please login to merge, or discard this patch.