bug #48874 [FrameworkBundle] remove double required annotation + attribute (alexislefebvre)

fabpot · fabpot · commit 062275005135 · 2023-01-07T18:01:04.000+01:00
This PR was merged into the 6.3 branch. Discussion ---------- [FrameworkBundle] remove double required annotation + attribute | Q | A | ------------- | --- | Branch? | 6.3 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | Fix #48792 (checked with [this reproducer](#48792 (comment))) | License | MIT | Doc PR | no Annotation and attribute were kept: #45680 (comment) To my understanding, it's not necessary since #48810 Alternative to #48868 Commits ------- 8e8772d remove double required annotation + attribute
diff --git a/src/Symfony/Bundle/FrameworkBundle/Controller/AbstractController.php b/src/Symfony/Bundle/FrameworkBundle/Controller/AbstractController.php
@@ -58,9 +58,6 @@ abstract class AbstractController implements ServiceSubscriberInterface
      */
     protected $container;
 
-    /**
-     * @required
-     */
     #[Required]
     public function setContainer(ContainerInterface $container): ?ContainerInterface
     {

Original file line number	Diff line number	Diff line change
`@@ -58,9 +58,6 @@ abstract class AbstractController implements ServiceSubscriberInterface`
`58`	`58`	`*/`
`59`	`59`	`protected $container;`
`60`	`60`
`61`		`- /**`
`62`		`- * @required`
`63`		`- */`
`64`	`61`	`#[Required]`
`65`	`62`	`public function setContainer(ContainerInterface $container): ?ContainerInterface`
`66`	`63`	`{`