Update OverrideMappingListener to use AsDoctrineListener - fix deprecation for registering it as Doctrine subscriber

Author: Matys333

config/services.xml

@@ -4,16 +4,19 @@
            xsi:schemaLocation="http://symfony.com/schema/dic/services
                                https://symfony.com/schema/dic/services/services-1.0.xsd">
     <services>
-        <defaults autowire="true" autoconfigure="true"/>
+        <defaults autowire="true" autoconfigure="true">
+            <bind key="$jobTableName">%job_queue.database.job_table_name%</bind>
+            <bind key="$jobRecurringTableName">%job_queue.database.job_recurring_table_name%</bind>
+        </defaults>
 
         <prototype namespace="TomAtom\JobQueueBundle\" resource="../src/*"
                    exclude="../src/{DependencyInjection,Entity,Tests,Kernel.php}"/>
 
         <service id="tomatom.job_queue.override_mapping_listener"
-                 class="TomAtom\JobQueueBundle\EventListener\OverrideMappingListener">
+                 class="TomAtom\JobQueueBundle\EventListener\OverrideMappingListener"
+                 autowire="false" autoconfigure="false">
             <argument>%job_queue.database.job_table_name%</argument>
             <argument>%job_queue.database.job_recurring_table_name%</argument>
-            <tag name="doctrine.event_subscriber"/>
         </service>
     </services>
 </container>

src/EventListener/OverrideMappingListener.php

@@ -2,25 +2,19 @@
 
 namespace TomAtom\JobQueueBundle\EventListener;
 
-use Doctrine\Common\EventSubscriber;
+use Doctrine\Bundle\DoctrineBundle\Attribute\AsDoctrineListener;
 use Doctrine\ORM\Event\LoadClassMetadataEventArgs;
 use TomAtom\JobQueueBundle\Entity\Job;
 use TomAtom\JobQueueBundle\Entity\JobRecurring;
 
-class OverrideMappingListener implements EventSubscriber
+#[AsDoctrineListener(event: 'loadClassMetadata')]
+class OverrideMappingListener
 {
-    private string $jobTableName;
-    private string $jobRecurringTableName;
-
-    public function __construct(string $jobTableName, string $jobRecurringTableName)
-    {
-        $this->jobTableName = $jobTableName;
-        $this->jobRecurringTableName = $jobRecurringTableName;
-    }
-
-    public function getSubscribedEvents(): array
+    public function __construct(
+        private readonly string $jobTableName,
+        private readonly string $jobRecurringTableName
+    )
     {
-        return ['loadClassMetadata'];
     }
 
     public function loadClassMetadata(LoadClassMetadataEventArgs $args): void
@@ -36,4 +30,4 @@ public function loadClassMetadata(LoadClassMetadataEventArgs $args): void
             $metadata->setPrimaryTable(['name' => $this->jobRecurringTableName]);
         }
     }
-}
+}