[symfony] remove NoAbstractControllerConstructorRule, too strict

README.md

@@ -1993,46 +1993,6 @@ return static function (ContainerConfigurator $containerConfigurator): void {
 
 <br>
 
-### NoAbstractControllerConstructorRule
-
-Abstract controller should not have constructor, as it can lead to tight coupling. Use @required annotation instead
-
-```yaml
-rules:
-    - Symplify\PHPStanRules\Rules\Symfony\NoAbstractControllerConstructorRule
-```
-
-```php
-abstract class AbstractController extends Controller
-{
-    public function __construct(
-        private SomeService $someService
-    ) {
-    }
-}
-```
-
-:x:
-
-<br>
-
-```php
-abstract class AbstractController extends Controller
-{
-    private $someService;
-
-    #[Required]
-    public function autowireAbstractController(SomeService $someService)
-    {
-        $this->someService = $someService;
-    }
-}
-```
-
-:+1:
-
-<br>
-
 ### AlreadyRegisteredAutodiscoveryServiceRule
 
 Remove service, as already registered via autodiscovery ->load(), no need to set it twice.

config/symfony-rules.neon

@@ -1,5 +1,4 @@
 rules:
-    - Symplify\PHPStanRules\Rules\Symfony\NoAbstractControllerConstructorRule
     - Symplify\PHPStanRules\Rules\Symfony\NoRequiredOutsideClassRule
     - Symplify\PHPStanRules\Rules\Symfony\SingleArgEventDispatchRule
     - Symplify\PHPStanRules\Rules\Symfony\NoListenerWithoutContractRule

src/Enum/RuleIdentifier/SymfonyRuleIdentifier.php

@@ -20,8 +20,6 @@ final class SymfonyRuleIdentifier
 
     public const string NO_STRING_IN_GET_SUBSCRIBED_EVENTS = 'symfony.noStringInGetSubscribedEvents';
 
-    public const string SYMFONY_NO_ABSTRACT_CONTROLLER_CONSTRUCTOR = 'symfony.noAbstractControllerConstructor';
-
     public const string SINGLE_REQUIRED_METHOD = 'symfony.singleRequiredMethod';
 
     public const string SYMFONY_REQUIRED_ONLY_IN_ABSTRACT = 'symfony.requiredOnlyInAbstract';