MOL-1196: fix automatic shipping and add error logs (#685)

* MOL-1196: fix automatic shipping and add error logs * MOL-1196: remove comment --------- Co-authored-by: Vitalij Mik <[email protected]>
mollie · Jan 16, 2024 · 1900474 · 1900474
1 parent 32b0be3
commit 1900474
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/Service/OrderDeliveryService.php b/src/Service/OrderDeliveryService.php
@@ -27,6 +27,7 @@ public function getDelivery(string $orderDeliveryId, Context $context): ?OrderDe
     {
         $criteria = new Criteria([$orderDeliveryId]);
         $criteria->addAssociation('order.transactions.paymentMethod');
+        $criteria->addAssociation('order.deliveries');
         $criteria->addAssociation('shippingMethod');
         $result = $this->orderDeliveryRepository->search($criteria, $context);
 

diff --git a/src/Subscriber/OrderDeliverySubscriber.php b/src/Subscriber/OrderDeliverySubscriber.php
@@ -120,7 +120,7 @@ public function onOrderDeliveryChanged(StateMachineStateChangeEvent $event): voi
 
             $this->mollieShipment->shipOrderRest($order, null, $event->getContext());
         } catch (\Throwable $ex) {
-            # do nothing in error in subscriber
+            $this->logger->error('Failed to transfer delivery state to mollie: '.$ex->getMessage());
             return;
         }
     }