Process Rector rule: AddOverrideAttributeToOverriddenMethodsRector

codedmonkey · codedmonkey · commit b04a909fa046 · 2026-02-02T13:50:04.000+01:00
Signed-off-by: Tim Goudriaan &lt;tim@codedmonkey.com&gt;
diff --git a/src/ConsoleApplication.php b/src/ConsoleApplication.php
@@ -17,6 +17,7 @@ public function __construct(
         parent::__construct('Dirigent', Kernel::VERSION);
     }
 
+    #[\Override]
     public function doRun(InputInterface $input, OutputInterface $output): int
     {
         $this->registerCommands();
diff --git a/src/Controller/Dashboard/DashboardAccessTokenController.php b/src/Controller/Dashboard/DashboardAccessTokenController.php
@@ -35,6 +35,7 @@ public static function getSubscribedEvents(): array
         ];
     }
 
+    #[\Override]
     public function configureCrud(Crud $crud): Crud
     {
         return $crud
@@ -43,19 +44,22 @@ public function configureCrud(Crud $crud): Crud
             ->overrideTemplate('crud/index', 'dashboard/access_token/index.html.twig');
     }
 
+    #[\Override]
     public function configureActions(Actions $actions): Actions
     {
         return $actions
             ->remove(Crud::PAGE_INDEX, Action::EDIT)
             ->remove(Crud::PAGE_NEW, Action::SAVE_AND_ADD_ANOTHER);
     }
 
+    #[\Override]
     public function configureFields(string $pageName): iterable
     {
         yield TextField::new('name');
         yield DateTimeField::new('expiresAt');
     }
 
+    #[\Override]
     public function createIndexQueryBuilder(SearchDto $searchDto, EntityDto $entityDto, FieldCollection $fields, FilterCollection $filters): QueryBuilder
     {
         /** @var User $user */
diff --git a/src/Controller/Dashboard/DashboardCredentialsController.php b/src/Controller/Dashboard/DashboardCredentialsController.php
@@ -19,6 +19,7 @@ public static function getEntityFqcn(): string
         return Credentials::class;
     }
 
+    #[\Override]
     public function configureCrud(Crud $crud): Crud
     {
         return parent::configureCrud($crud)
@@ -27,6 +28,7 @@ public function configureCrud(Crud $crud): Crud
             ->overrideTemplate('layout', 'dashboard/credentials/layout.html.twig');
     }
 
+    #[\Override]
     public function configureFields(string $pageName): iterable
     {
         yield TextField::new('name');
diff --git a/src/Controller/Dashboard/DashboardRegistryController.php b/src/Controller/Dashboard/DashboardRegistryController.php
@@ -28,6 +28,7 @@ public static function getEntityFqcn(): string
         return Registry::class;
     }
 
+    #[\Override]
     public function configureCrud(Crud $crud): Crud
     {
         return $crud
@@ -36,6 +37,7 @@ public function configureCrud(Crud $crud): Crud
             ->setEntityPermission('ROLE_ADMIN');
     }
 
+    #[\Override]
     public function configureActions(Actions $actions): Actions
     {
         $upAction = Action::new('moveUp', icon: 'fa fa-arrow-up')
@@ -48,6 +50,7 @@ public function configureActions(Actions $actions): Actions
             ->add(Crud::PAGE_INDEX, $upAction);
     }
 
+    #[\Override]
     public function configureFields(string $pageName): iterable
     {
         yield TextField::new('name')
diff --git a/src/Controller/Dashboard/DashboardRootController.php b/src/Controller/Dashboard/DashboardRootController.php
@@ -37,18 +37,21 @@ public function __construct(
     ) {
     }
 
+    #[\Override]
     public function configureDashboard(): Dashboard
     {
         return Dashboard::new()
             ->setTitle($this->title);
     }
 
+    #[\Override]
     public function configureAssets(): Assets
     {
         return Assets::new()
             ->addWebpackEncoreEntry('dashboard');
     }
 
+    #[\Override]
     public function configureMenuItems(): iterable
     {
         $request = $this->container->get('request_stack')->getCurrentRequest();
@@ -93,6 +96,7 @@ public function configureMenuItems(): iterable
     /**
      * @param User $user
      */
+    #[\Override]
     public function configureUserMenu(UserInterface $user): UserMenu
     {
         $menu = parent::configureUserMenu($user)
@@ -109,6 +113,7 @@ public function configureUserMenu(UserInterface $user): UserMenu
     }
 
     #[IsGrantedAccess]
+    #[\Override]
     public function index(): Response
     {
         $packageCount = $this->packageRepository->count();
diff --git a/src/Controller/Dashboard/DashboardUserController.php b/src/Controller/Dashboard/DashboardUserController.php
@@ -24,6 +24,7 @@ public static function getEntityFqcn(): string
         return User::class;
     }
 
+    #[\Override]
     public function configureCrud(Crud $crud): Crud
     {
         return parent::configureCrud($crud)
@@ -32,6 +33,7 @@ public function configureCrud(Crud $crud): Crud
             ->setEntityPermission('ROLE_ADMIN');
     }
 
+    #[\Override]
     public function configureActions(Actions $actions): Actions
     {
         $impersonate = Action::new('impersonate', 'Impersonate')
@@ -46,6 +48,7 @@ public function configureActions(Actions $actions): Actions
             ->setPermission('impersonate', 'ROLE_ALLOWED_TO_SWITCH');
     }
 
+    #[\Override]
     public function configureFields(string $pageName): iterable
     {
         yield TextField::new('username');
diff --git a/src/DependencyInjection/DirigentExtension.php b/src/DependencyInjection/DirigentExtension.php
@@ -35,6 +35,7 @@ protected function loadInternal(array $mergedConfig, ContainerBuilder $container
         $container->setParameter('dirigent.dist_mirroring.dev_packages', $mergedConfig['dist_mirroring']['dev_packages']);
     }
 
+    #[\Override]
     public function getConfiguration(array $config, ContainerBuilder $container): ConfigurationInterface
     {
         return new DirigentConfiguration();
diff --git a/src/Doctrine/Entity/AccessToken.php b/src/Doctrine/Entity/AccessToken.php
@@ -62,6 +62,7 @@ public function getToken(): ?string
         return $this->token;
     }
 
+    #[\Override]
     public function getCreatedAt(): \DateTimeImmutable
     {
         return $this->createdAt;
diff --git a/src/Doctrine/Entity/Package.php b/src/Doctrine/Entity/Package.php
@@ -283,6 +283,7 @@ public function getVersion(string $normalizedVersion): ?Version
         return $this->cachedVersions[strtolower($normalizedVersion)] ?? null;
     }
 
+    #[\Override]
     public function getCreatedAt(): \DateTimeImmutable
     {
         return $this->createdAt;
diff --git a/src/Doctrine/Type/EncryptedTextType.php b/src/Doctrine/Type/EncryptedTextType.php
@@ -31,6 +31,7 @@ public function getSQLDeclaration(array $column, AbstractPlatform $platform): st
         return $platform->getClobTypeDeclarationSQL($column);
     }
 
+    #[\Override]
     public function convertToPHPValue($value, AbstractPlatform $platform): mixed
     {
         if (null === $value) {
@@ -40,6 +41,7 @@ public function convertToPHPValue($value, AbstractPlatform $platform): mixed
         return $this->encryption->reveal($value);
     }
 
+    #[\Override]
     public function convertToDatabaseValue($value, AbstractPlatform $platform): mixed
     {
         if (null === $value) {
diff --git a/src/Form/NewPasswordType.php b/src/Form/NewPasswordType.php
@@ -41,6 +41,7 @@ public function configureOptions(OptionsResolver $resolver): void
         ]);
     }
 
+    #[\Override]
     public function getParent(): string
     {
         return RepeatedType::class;
diff --git a/src/Form/TotpCodeType.php b/src/Form/TotpCodeType.php
@@ -26,6 +26,7 @@ public function configureOptions(OptionsResolver $resolver): void
         ]);
     }
 
+    #[\Override]
     public function getParent(): string
     {
         return TextType::class;
diff --git a/src/Kernel.php b/src/Kernel.php
@@ -37,6 +37,7 @@ protected function build(ContainerBuilder $container): void
         $container->addCompilerPass(new ParametersPass());
     }
 
+    #[\Override]
     public function boot(): void
     {
         parent::boot();
diff --git a/src/Validator/UniquePackage.php b/src/Validator/UniquePackage.php
@@ -7,6 +7,7 @@
 #[\Attribute(\Attribute::TARGET_CLASS)]
 class UniquePackage extends Constraint
 {
+    #[\Override]
     public function getTargets(): string
     {
         return self::CLASS_CONSTRAINT;
diff --git a/tests/Docker/Standalone/DockerStandaloneIsolatedTestCase.php b/tests/Docker/Standalone/DockerStandaloneIsolatedTestCase.php
@@ -4,6 +4,7 @@
 
 abstract class DockerStandaloneIsolatedTestCase extends DockerStandaloneTestCase
 {
+    #[\Override]
     protected function setUp(): void
     {
     }
diff --git a/tests/Docker/Standalone/InitTest.php b/tests/Docker/Standalone/InitTest.php
@@ -11,11 +11,13 @@ class InitTest extends DockerStandaloneIsolatedTestCase
 {
     protected Filesystem $filesystem;
 
+    #[\Override]
     protected function setUp(): void
     {
         $this->filesystem = new Filesystem();
     }
 
+    #[\Override]
     protected function tearDown(): void
     {
         parent::tearDown();
diff --git a/tests/FunctionalTests/Controller/ApiControllerPublicTest.php b/tests/FunctionalTests/Controller/ApiControllerPublicTest.php
@@ -9,6 +9,7 @@
 
 class ApiControllerPublicTest extends KernelTestCase
 {
+    #[\Override]
     protected static function getKernelClass(): string
     {
         return PublicKernel::class;
diff --git a/tests/FunctionalTests/Controller/Dashboard/DashboardRootControllerPublicTest.php b/tests/FunctionalTests/Controller/Dashboard/DashboardRootControllerPublicTest.php
@@ -8,6 +8,7 @@
 
 class DashboardRootControllerPublicTest extends WebTestCase
 {
+    #[\Override]
     protected static function getKernelClass(): string
     {
         return PublicKernel::class;
diff --git a/tests/FunctionalTests/PublicKernel.php b/tests/FunctionalTests/PublicKernel.php
@@ -7,6 +7,7 @@
 
 class PublicKernel extends Kernel
 {
+    #[\Override]
     protected function configureContainer(ContainerConfigurator $container): void
     {
         parent::configureContainer($container);
diff --git a/tests/UnitTests/Command/EncryptionGenerateKeysCommandTest.php b/tests/UnitTests/Command/EncryptionGenerateKeysCommandTest.php
@@ -25,6 +25,7 @@ protected function setUp(): void
         $this->clearKeys();
     }
 
+    #[\Override]
     protected function tearDown(): void
     {
         $this->clearKeys();

Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,7 @@ public function __construct(`
`17`	`17`	`parent::__construct('Dirigent', Kernel::VERSION);`
`18`	`18`	`}`
`19`	`19`
	`20`	`+ #[\Override]`
`20`	`21`	`public function doRun(InputInterface $input, OutputInterface $output): int`
`21`	`22`	`{`
`22`	`23`	`$this->registerCommands();`
Original file line number	Diff line number	Diff line change
`@@ -35,6 +35,7 @@ public static function getSubscribedEvents(): array`
`35`	`35`	`];`
`36`	`36`	`}`
`37`	`37`
	`38`	`+ #[\Override]`
`38`	`39`	`public function configureCrud(Crud $crud): Crud`
`39`	`40`	`{`
`40`	`41`	`return $crud`
`@@ -43,19 +44,22 @@ public function configureCrud(Crud $crud): Crud`
`43`	`44`	`->overrideTemplate('crud/index', 'dashboard/access_token/index.html.twig');`
`44`	`45`	`}`
`45`	`46`
	`47`	`+ #[\Override]`
`46`	`48`	`public function configureActions(Actions $actions): Actions`
`47`	`49`	`{`
`48`	`50`	`return $actions`
`49`	`51`	`->remove(Crud::PAGE_INDEX, Action::EDIT)`
`50`	`52`	`->remove(Crud::PAGE_NEW, Action::SAVE_AND_ADD_ANOTHER);`
`51`	`53`	`}`
`52`	`54`
	`55`	`+ #[\Override]`
`53`	`56`	`public function configureFields(string $pageName): iterable`
`54`	`57`	`{`
`55`	`58`	`yield TextField::new('name');`
`56`	`59`	`yield DateTimeField::new('expiresAt');`
`57`	`60`	`}`
`58`	`61`
	`62`	`+ #[\Override]`
`59`	`63`	`public function createIndexQueryBuilder(SearchDto $searchDto, EntityDto $entityDto, FieldCollection $fields, FilterCollection $filters): QueryBuilder`
`60`	`64`	`{`
`61`	`65`	`/** @var User $user */`
Original file line number	Diff line number	Diff line change
`@@ -19,6 +19,7 @@ public static function getEntityFqcn(): string`
`19`	`19`	`return Credentials::class;`
`20`	`20`	`}`
`21`	`21`
	`22`	`+ #[\Override]`
`22`	`23`	`public function configureCrud(Crud $crud): Crud`
`23`	`24`	`{`
`24`	`25`	`return parent::configureCrud($crud)`
`@@ -27,6 +28,7 @@ public function configureCrud(Crud $crud): Crud`
`27`	`28`	`->overrideTemplate('layout', 'dashboard/credentials/layout.html.twig');`
`28`	`29`	`}`
`29`	`30`
	`31`	`+ #[\Override]`
`30`	`32`	`public function configureFields(string $pageName): iterable`
`31`	`33`	`{`
`32`	`34`	`yield TextField::new('name');`
Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,7 @@ public static function getEntityFqcn(): string`
`28`	`28`	`return Registry::class;`
`29`	`29`	`}`
`30`	`30`
	`31`	`+ #[\Override]`
`31`	`32`	`public function configureCrud(Crud $crud): Crud`
`32`	`33`	`{`
`33`	`34`	`return $crud`
`@@ -36,6 +37,7 @@ public function configureCrud(Crud $crud): Crud`
`36`	`37`	`->setEntityPermission('ROLE_ADMIN');`
`37`	`38`	`}`
`38`	`39`
	`40`	`+ #[\Override]`
`39`	`41`	`public function configureActions(Actions $actions): Actions`
`40`	`42`	`{`
`41`	`43`	`$upAction = Action::new('moveUp', icon: 'fa fa-arrow-up')`
`@@ -48,6 +50,7 @@ public function configureActions(Actions $actions): Actions`
`48`	`50`	`->add(Crud::PAGE_INDEX, $upAction);`
`49`	`51`	`}`
`50`	`52`
	`53`	`+ #[\Override]`
`51`	`54`	`public function configureFields(string $pageName): iterable`
`52`	`55`	`{`
`53`	`56`	`yield TextField::new('name')`
Original file line number	Diff line number	Diff line change
`@@ -37,18 +37,21 @@ public function __construct(`
`37`	`37`	`) {`
`38`	`38`	`}`
`39`	`39`
	`40`	`+ #[\Override]`
`40`	`41`	`public function configureDashboard(): Dashboard`
`41`	`42`	`{`
`42`	`43`	`return Dashboard::new()`
`43`	`44`	`->setTitle($this->title);`
`44`	`45`	`}`
`45`	`46`
	`47`	`+ #[\Override]`
`46`	`48`	`public function configureAssets(): Assets`
`47`	`49`	`{`
`48`	`50`	`return Assets::new()`
`49`	`51`	`->addWebpackEncoreEntry('dashboard');`
`50`	`52`	`}`
`51`	`53`
	`54`	`+ #[\Override]`
`52`	`55`	`public function configureMenuItems(): iterable`
`53`	`56`	`{`
`54`	`57`	`$request = $this->container->get('request_stack')->getCurrentRequest();`
`@@ -93,6 +96,7 @@ public function configureMenuItems(): iterable`
`93`	`96`	`/**`
`94`	`97`	`* @param User $user`
`95`	`98`	`*/`
	`99`	`+ #[\Override]`
`96`	`100`	`public function configureUserMenu(UserInterface $user): UserMenu`
`97`	`101`	`{`
`98`	`102`	`$menu = parent::configureUserMenu($user)`
`@@ -109,6 +113,7 @@ public function configureUserMenu(UserInterface $user): UserMenu`
`109`	`113`	`}`
`110`	`114`
`111`	`115`	`#[IsGrantedAccess]`
	`116`	`+ #[\Override]`
`112`	`117`	`public function index(): Response`
`113`	`118`	`{`
`114`	`119`	`$packageCount = $this->packageRepository->count();`
Original file line number	Diff line number	Diff line change
`@@ -24,6 +24,7 @@ public static function getEntityFqcn(): string`
`24`	`24`	`return User::class;`
`25`	`25`	`}`
`26`	`26`
	`27`	`+ #[\Override]`
`27`	`28`	`public function configureCrud(Crud $crud): Crud`
`28`	`29`	`{`
`29`	`30`	`return parent::configureCrud($crud)`
`@@ -32,6 +33,7 @@ public function configureCrud(Crud $crud): Crud`
`32`	`33`	`->setEntityPermission('ROLE_ADMIN');`
`33`	`34`	`}`
`34`	`35`
	`36`	`+ #[\Override]`
`35`	`37`	`public function configureActions(Actions $actions): Actions`
`36`	`38`	`{`
`37`	`39`	`$impersonate = Action::new('impersonate', 'Impersonate')`
`@@ -46,6 +48,7 @@ public function configureActions(Actions $actions): Actions`
`46`	`48`	`->setPermission('impersonate', 'ROLE_ALLOWED_TO_SWITCH');`
`47`	`49`	`}`
`48`	`50`
	`51`	`+ #[\Override]`
`49`	`52`	`public function configureFields(string $pageName): iterable`
`50`	`53`	`{`
`51`	`54`	`yield TextField::new('username');`
Original file line number	Diff line number	Diff line change
`@@ -35,6 +35,7 @@ protected function loadInternal(array $mergedConfig, ContainerBuilder $container`
`35`	`35`	`$container->setParameter('dirigent.dist_mirroring.dev_packages', $mergedConfig['dist_mirroring']['dev_packages']);`
`36`	`36`	`}`
`37`	`37`
	`38`	`+ #[\Override]`
`38`	`39`	`public function getConfiguration(array $config, ContainerBuilder $container): ConfigurationInterface`
`39`	`40`	`{`
`40`	`41`	`return new DirigentConfiguration();`
Original file line number	Diff line number	Diff line change
`@@ -62,6 +62,7 @@ public function getToken(): ?string`
`62`	`62`	`return $this->token;`
`63`	`63`	`}`
`64`	`64`
	`65`	`+ #[\Override]`
`65`	`66`	`public function getCreatedAt(): \DateTimeImmutable`
`66`	`67`	`{`
`67`	`68`	`return $this->createdAt;`
Original file line number	Diff line number	Diff line change
`@@ -283,6 +283,7 @@ public function getVersion(string $normalizedVersion): ?Version`
`283`	`283`	`return $this->cachedVersions[strtolower($normalizedVersion)] ?? null;`
`284`	`284`	`}`
`285`	`285`
	`286`	`+ #[\Override]`
`286`	`287`	`public function getCreatedAt(): \DateTimeImmutable`
`287`	`288`	`{`
`288`	`289`	`return $this->createdAt;`
Original file line number	Diff line number	Diff line change
`@@ -31,6 +31,7 @@ public function getSQLDeclaration(array $column, AbstractPlatform $platform): st`
`31`	`31`	`return $platform->getClobTypeDeclarationSQL($column);`
`32`	`32`	`}`
`33`	`33`
	`34`	`+ #[\Override]`
`34`	`35`	`public function convertToPHPValue($value, AbstractPlatform $platform): mixed`
`35`	`36`	`{`
`36`	`37`	`if (null === $value) {`
`@@ -40,6 +41,7 @@ public function convertToPHPValue($value, AbstractPlatform $platform): mixed`
`40`	`41`	`return $this->encryption->reveal($value);`
`41`	`42`	`}`
`42`	`43`
	`44`	`+ #[\Override]`
`43`	`45`	`public function convertToDatabaseValue($value, AbstractPlatform $platform): mixed`
`44`	`46`	`{`
`45`	`47`	`if (null === $value) {`