Dispatch OrderPaymentPaid event based on order payment state

config/config.yaml

@@ -20,6 +20,7 @@ sylius_mailer:
 knp_snappy:
     pdf:
         enabled: true
+        binary: '%env(WKHTMLTOPDF_PATH)%'
 
 sylius_grid:
     templates:

src/EventProducer/OrderPaymentPaidProducer.php

@@ -15,6 +15,7 @@
 
 use Sylius\Component\Core\Model\OrderInterface;
 use Sylius\Component\Core\Model\PaymentInterface;
+use Sylius\Component\Core\OrderPaymentStates;
 use Sylius\InvoicingPlugin\Doctrine\ORM\InvoiceRepositoryInterface;
 use Sylius\InvoicingPlugin\Event\OrderPaymentPaid;
 use Symfony\Component\Clock\ClockInterface;
@@ -51,6 +52,8 @@ private function shouldEventBeDispatched(PaymentInterface $payment): bool
         /** @var OrderInterface $order */
         $order = $payment->getOrder();
 
-        return null !== $order && null !== $this->invoiceRepository->findOneByOrder($order);
+        return null !== $order &&
+            $order->getPaymentState() === OrderPaymentStates::STATE_PAID &&
+            null !== $this->invoiceRepository->findOneByOrder($order);
     }
 }

tests/Unit/EventProducer/OrderPaymentPaidProducerTest.php

@@ -17,6 +17,7 @@
 use PHPUnit\Framework\TestCase;
 use Sylius\Component\Core\Model\OrderInterface;
 use Sylius\Component\Core\Model\PaymentInterface;
+use Sylius\Component\Core\OrderPaymentStates;
 use Sylius\InvoicingPlugin\Doctrine\ORM\InvoiceRepositoryInterface;
 use Sylius\InvoicingPlugin\Entity\InvoiceInterface;
 use Sylius\InvoicingPlugin\Event\OrderPaymentPaid;
@@ -42,6 +43,7 @@ public function it_dispatches_order_payment_paid_event_for_payment(): void
 
         $payment->method('getOrder')->willReturn($order);
         $order->method('getNumber')->willReturn('0000001');
+        $order->method('getPaymentState')->willReturn(OrderPaymentStates::STATE_PAID);
         $clock->method('now')->willReturn($dateTime);
         $invoiceRepository->method('findOneByOrder')->with($order)->willReturn($invoice);
 
@@ -83,6 +85,7 @@ public function it_does_not_dispatch_event_when_there_is_no_invoice_related_to_o
 
         $payment->method('getOrder')->willReturn($order);
         $order->method('getNumber')->willReturn('0000001');
+        $order->method('getPaymentState')->willReturn(OrderPaymentStates::STATE_PAID);
         $invoiceRepository->method('findOneByOrder')->with($order)->willReturn(null);
 
         $eventBus->expects($this->never())->method('dispatch');
@@ -91,4 +94,26 @@ public function it_does_not_dispatch_event_when_there_is_no_invoice_related_to_o
         $producer = new OrderPaymentPaidProducer($eventBus, $clock, $invoiceRepository);
         $producer($payment);
     }
+
+    #[Test]
+    public function it_does_not_dispatch_event_when_order_is_only_partially_paid(): void
+    {
+        $eventBus = $this->createMock(MessageBusInterface::class);
+        $clock = $this->createMock(ClockInterface::class);
+        $invoiceRepository = $this->createMock(InvoiceRepositoryInterface::class);
+        $payment = $this->createMock(PaymentInterface::class);
+        $order = $this->createMock(OrderInterface::class);
+        $invoice = $this->createMock(InvoiceInterface::class);
+
+        $payment->method('getOrder')->willReturn($order);
+        $order->method('getNumber')->willReturn('0000001');
+        $order->method('getPaymentState')->willReturn(OrderPaymentStates::STATE_PARTIALLY_PAID);
+        $invoiceRepository->method('findOneByOrder')->with($order)->willReturn($invoice);
+
+        $eventBus->expects($this->never())->method('dispatch');
+        $clock->expects($this->never())->method('now');
+
+        $producer = new OrderPaymentPaidProducer($eventBus, $clock, $invoiceRepository);
+        $producer($payment);
+    }
 }