Avoid final classes for jobs but add final to methods (#31)

Author: yann-eugone

src/batch/src/Job/AbstractJob.php

@@ -13,7 +13,7 @@ abstract class AbstractJob implements JobInterface
     /**
      * @inheritDoc
      */
-    public function execute(JobExecution $jobExecution): void
+    final public function execute(JobExecution $jobExecution): void
     {
         if (!$jobExecution->getStatus()->isExecutable()) {
             $jobExecution->getLogger()->error('Job is not executable', ['job' => $jobExecution->getJobName()]);

src/batch/src/Job/Item/ItemJob.php

@@ -68,7 +68,7 @@ public function __construct(
     /**
      * @inheritDoc
      */
-    protected function doExecute(JobExecution $jobExecution): void
+    final protected function doExecute(JobExecution $jobExecution): void
     {
         $rootExecution = $jobExecution->getRootExecution();
         $summary = $jobExecution->getSummary();

src/batch/src/Job/JobWithChildJobs.php

@@ -9,7 +9,7 @@
 use Yokai\Batch\Registry\JobRegistry;
 use Yokai\Batch\Storage\JobExecutionStorageInterface;
 
-final class JobWithChildJobs extends AbstractJob
+class JobWithChildJobs extends AbstractJob
 {
     /**
      * @var JobExecutionStorageInterface
@@ -44,7 +44,7 @@ public function __construct(
     /**
      * @inheritDoc
      */
-    protected function doExecute(JobExecution $jobExecution): void
+    final protected function doExecute(JobExecution $jobExecution): void
     {
         $logger = $jobExecution->getLogger();
         foreach ($this->childJobs as $jobName) {