feat: calculate directly the distance on tasks even if no vehicle

src/Doctrine/EventSubscriber/TaskListSubscriber.php

@@ -34,7 +34,6 @@ public function __construct(
         private readonly TranslatorInterface $translator,
         private readonly RoutingInterface $routing,
         private readonly TaskListRepository $taskListRepository,
-        private readonly EntityManagerInterface $em,
         private readonly LoggerInterface $logger
     )
     {
@@ -49,11 +48,12 @@ public function getSubscribedEvents()
         );
     }
 
-    private function calculate(TaskList $taskList)
+    private function calculate(TaskList $taskList, EntityManagerInterface $em)
     {
         $coordinates = [];
         $tasks = [];
-        $vehicle = $taskList->getVehicle(); 
+        $vehicle = $taskList->getVehicle();
+        $uow = $em->getUnitOfWork();
 
         if (!is_null($vehicle)) {
             $coordinates[] = $taskList->getVehicle()->getWarehouse()->getAddress()->getGeo();
@@ -86,7 +86,16 @@ private function calculate(TaskList $taskList)
                     $emissions = intval($vehicle->getCo2emissions() * $leg['distance'] / 1000);
                     $task->setDistanceFromPrevious(intval($leg['distance'])); // in meter
                     $task->setCo2Emissions($emissions);
-                    $this->em->getUnitOfWork()->recomputeSingleEntityChangeSet($this->em->getClassMetadata(Task::class), $task);
+                    $uow->recomputeSingleEntityChangeSet($em->getClassMetadata(Task::class), $task);
+                }
+            } else {
+                $route = $this->routing->route(...$coordinates)['routes'][0];
+                $legs = $route["legs"];
+                foreach ($legs as $index => $leg) {
+                    $task = $taskList->getTasks()[$index + 1]; // we assume we start at the first task, as there is no warehouse
+                    $task->setDistanceFromPrevious(intval($leg['distance'])); // in meter
+                    $task->setCo2Emissions(0); // reset
+                    $uow->recomputeSingleEntityChangeSet($em->getClassMetadata(Task::class), $task);
                 }
             }
         }
@@ -95,9 +104,10 @@ private function calculate(TaskList $taskList)
     public function prePersist(LifecycleEventArgs $args)
     {
         $entity = $args->getObject();
+        $em = $args->getObjectManager();
 
         if ($entity instanceof TaskList) {
-            $this->calculate($entity);
+            $this->calculate($entity, $em);
         }
     }
 
@@ -166,7 +176,7 @@ public function onFlush(OnFlushEventArgs $args)
         foreach ($this->taskLists as $taskList) { // @phpstan-ignore-line
 
             $this->logger->debug('TaskList was modified, recalculating…');
-            $this->calculate($taskList);
+            $this->calculate($taskList, $em);
 
             if ($uow->isInIdentityMap($taskList)) {
                 $uow->recomputeSingleEntityChangeSet($em->getClassMetadata(TaskList::class), $taskList);

src/Entity/Listener/DeliveryListener.php

@@ -72,7 +72,6 @@ public function postPersist(Delivery $delivery)
             foreach ($delivery->getTasks() as $task) {
                 $taskList = $this->taskListProvider->getTaskList($task, $courier);
                 $taskList->appendTask($task);
-                $this->entityManager->persist($taskList);
             }
 
             $this->entityManager->flush();

src/Entity/Task.php

@@ -490,12 +490,14 @@ class Task implements TaggableInterface, OrganizationAwareInterface, PackagesAwa
     private $incidents;
 
     /**
-    * CO2 emissions from the previous task/warehouse to accomplish this task, in g
+    * CO2 emissions from the previous task/warehouse to accomplish this task
+    * @Groups({"task"})
     */
     private $co2Emissions = 0;
 
     /**
     * Distance from previous task, in meter
+    * @Groups({"task"})
     */
     private $distanceFromPrevious = 0;
 

src/Entity/TaskList.php

@@ -29,7 +29,7 @@
  *       "method"="GET",
  *       "access_control"="is_granted('ROLE_DISPATCHER') or is_granted('ROLE_OAUTH2_TASKS')",
  *       "openapi_context"={
- *         "summary"="Legacy endpoint, please use '/api/task_lists/v2' instead. Retrieves Tasklists as lists of tasks, not tasks and tours, with expanded tasks."
+ *         "summary"="Legacy endpoint, please use '/api/task_lists/v2' instead. Retrieves Tasklists as lists of tasks, not tasks and tours, with expanded tasks. Used by store integrations that wants to track tasks statuses."
  *       },
  *       "normalization_context"={"groups"={"task_list", "task_collection", "task", "delivery", "address"}}
  *     },

src/Security/TokenStoreExtractor.php

@@ -28,6 +28,7 @@ public function __construct(
 
     public function extractStore()
     {
+
         if (null === ($token = $this->tokenStorage->getToken())) {
             return;
         }