NTR: fix order expire for sepa

mollie · Dec 9, 2024 · c9e9577 · c9e9577
1 parent ccde869
commit c9e9577
Show file tree

Hide file tree

Showing 4 changed files with 27 additions and 5 deletions.
diff --git a/src/Resources/config/services/services.xml b/src/Resources/config/services/services.xml
@@ -128,7 +128,9 @@
             <argument type="service" id="Kiener\MolliePayments\Service\UrlParsingService"/>
         </service>
 
-        <service id="Kiener\MolliePayments\Service\Order\OrderTimeService"/>
+        <service id="Kiener\MolliePayments\Service\Order\OrderTimeService">
+            <argument type="service" id="mollie_payments.logger"/>
+        </service>
 
         <service id="Kiener\MolliePayments\Service\Order\OrderStatusUpdater">
             <argument type="service" id="Kiener\MolliePayments\Service\Order\OrderStateService"/>

diff --git a/src/Service/Order/OrderExpireService.php b/src/Service/Order/OrderExpireService.php
@@ -114,7 +114,11 @@ public function cancelExpiredOrders(OrderCollection $orders, Context $context):
             $finalizeTransactionTimeInMinutes = $settings->getPaymentFinalizeTransactionTime();
 
             if ($this->orderUsesSepaPayment($lastTransaction)) {
-                $finalizeTransactionTimeInMinutes = (int)ceil($settings->getPaymentMethodBankTransferDueDateDays() / 24 / 60);
+                $bankTransferDueDays = $settings->getPaymentMethodBankTransferDueDateDays();
+                if ($bankTransferDueDays === null) {
+                    $bankTransferDueDays = BankTransferPayment::DUE_DATE_MIN_DAYS;
+                }
+                $finalizeTransactionTimeInMinutes = 60 * 60 * 24 * $bankTransferDueDays;
             }
 
             if ($this->orderTimeService->isOrderAgeGreaterThan($order, $finalizeTransactionTimeInMinutes) === false) {

diff --git a/src/Service/Order/OrderTimeService.php b/src/Service/Order/OrderTimeService.php
@@ -4,8 +4,10 @@
 namespace Kiener\MolliePayments\Service\Order;
 
 use DateTime;
+use Psr\Log\LoggerInterface;
 use Shopware\Core\Checkout\Order\Aggregate\OrderTransaction\OrderTransactionEntity;
 use Shopware\Core\Checkout\Order\OrderEntity;
+use Shopware\Core\Defaults;
 
 class OrderTimeService
 {
@@ -14,9 +16,15 @@ class OrderTimeService
      */
     private $now;
 
-    public function __construct(?DateTime $now = null)
+    /**
+     * @var LoggerInterface
+     */
+    private $logger;
+
+    public function __construct(LoggerInterface $logger, ?DateTime $now = null)
     {
         $this->now = $now ?? new DateTime();
+        $this->logger = $logger;
     }
 
     /**
@@ -42,7 +50,7 @@ public function isOrderAgeGreaterThan(OrderEntity $order, int $minutes): bool
             return false;
         }
 
-        $transitionDate = $lastTransaction->getCreatedAt();
+        $transitionDate = $lastTransaction->getUpdatedAt() ?? $lastTransaction->getCreatedAt();
 
         if ($transitionDate === null) {
             return false;
@@ -52,6 +60,13 @@ public function isOrderAgeGreaterThan(OrderEntity $order, int $minutes): bool
         $diffInHours = $interval->h + ($interval->days * 24);
         $diffInMinutes = $interval->i + ($diffInHours * 60);
 
+        $this->logger->debug('Check if order is expired', [
+            'lastTransactionTime' => $transitionDate->format(Defaults::STORAGE_DATE_TIME_FORMAT),
+            'now' => $this->now->format(Defaults::STORAGE_DATE_TIME_FORMAT),
+            'diffInMinutes' => $diffInMinutes,
+            'minutes'=>$minutes
+        ]);
+
         return $diffInMinutes > $minutes;
     }
 }
diff --git a/tests/PHPUnit/Service/Order/OrderTimeServiceTest.php b/tests/PHPUnit/Service/Order/OrderTimeServiceTest.php
@@ -6,6 +6,7 @@
 
 use Kiener\MolliePayments\Service\Order\OrderTimeService;
 use PHPUnit\Framework\TestCase;
+use Psr\Log\NullLogger;
 use Shopware\Core\Checkout\Order\Aggregate\OrderTransaction\OrderTransactionCollection;
 use Shopware\Core\Checkout\Order\Aggregate\OrderTransaction\OrderTransactionEntity;
 use Shopware\Core\Checkout\Order\OrderEntity;
@@ -24,7 +25,7 @@ public function testDateComparisonLogic(\DateTime $now, \DateTime $orderDate, bo
     {
         $order = $this->orderMockWithLastTransactionTimestamp($orderDate);
 
-        $result = (new OrderTimeService($now))->isOrderAgeGreaterThan($order, $compareValueInMinutes);
+        $result = (new OrderTimeService(new NullLogger(),$now))->isOrderAgeGreaterThan($order, $compareValueInMinutes);
 
         $this->assertSame($expected, $result);
     }