Skip to content

Commit

Permalink
enable other payments in Convertim OrderFacade
Browse files Browse the repository at this point in the history
  • Loading branch information
TomasLudvik committed Dec 10, 2024
1 parent 1caf5d6 commit bc2fa89
Showing 1 changed file with 9 additions and 6 deletions.
15 changes: 9 additions & 6 deletions src/Model/Order/OrderFacade.php
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@

use Convertim\Order\ConvertimOrderData;
use Convertim\Order\ConvertimOrderDataPaymentStatus;
use Shopsys\ConvertimBundle\Model\Payment\PaymentTypeEnum;
use Shopsys\FrameworkBundle\Component\Domain\Domain;
use Shopsys\FrameworkBundle\Component\Money\Money;
use Shopsys\FrameworkBundle\Model\Order\Order;
Expand All @@ -23,6 +24,7 @@ class OrderFacade
* @param \Shopsys\ConvertimBundle\Model\Order\OrderRepository $orderRepository
* @param \Shopsys\FrameworkBundle\Model\Payment\Transaction\PaymentTransactionFacade $paymentTransactionFacade
* @param \Shopsys\FrameworkBundle\Model\Payment\Transaction\PaymentTransactionDataFactory $paymentTransactionDataFactory
* @param \Shopsys\ConvertimBundle\Model\Payment\PaymentTypeEnum $paymentTypeEnum
*/
public function __construct(
protected readonly ConvertimOrderDataToOrderDataMapper $convertimOrderDataToOrderMapper,
Expand All @@ -31,6 +33,7 @@ public function __construct(
protected readonly OrderRepository $orderRepository,
protected readonly PaymentTransactionFacade $paymentTransactionFacade,
protected readonly PaymentTransactionDataFactory $paymentTransactionDataFactory,
protected readonly PaymentTypeEnum $paymentTypeEnum,
) {
}

Expand All @@ -49,8 +52,8 @@ public function saveOrder(ConvertimOrderData $convertimOrderData): Order
$order = $this->placeOrderFacade->placeOrder($orderData, $deliveryAddressUuid);
}

if ($order->getPayment()->isGoPay()) {
$this->resolveGoPayStatus($order, $convertimOrderData);
if (in_array($order->getPayment()->getType(), $this->paymentTypeEnum->getAllCases(), true)) {
$this->resolveExternalPaymentStatus($order, $convertimOrderData);
}

return $order;
Expand All @@ -60,11 +63,11 @@ public function saveOrder(ConvertimOrderData $convertimOrderData): Order
* @param \Shopsys\FrameworkBundle\Model\Order\Order $order
* @param \Convertim\Order\ConvertimOrderData $convertimOrderData
*/
protected function resolveGoPayStatus(Order $order, ConvertimOrderData $convertimOrderData): void
protected function resolveExternalPaymentStatus(Order $order, ConvertimOrderData $convertimOrderData): void
{
$convertimPaymentStatus = $convertimOrderData->getPaymentStatus();

if ($convertimPaymentStatus === null || $convertimOrderData->getGoPayData() === null) {
if ($convertimPaymentStatus === null) {
return;
}

Expand All @@ -86,9 +89,9 @@ protected function resolveGoPayStatus(Order $order, ConvertimOrderData $converti
}

if ($currentPaymentTransaction === null) {
$this->createPaymentTransaction($order, $convertimPaymentStatus, Money::create($convertimOrderData->getGoPayData()->getAmount()));
$this->createPaymentTransaction($order, $convertimPaymentStatus, Money::create($convertimOrderData->getPaymentData()->getPriceWithVat()));
} else {
$this->updatePaymentTransaction($currentPaymentTransaction, $convertimPaymentStatus, Money::create($convertimOrderData->getGoPayData()->getAmount()));
$this->updatePaymentTransaction($currentPaymentTransaction, $convertimPaymentStatus, Money::create($convertimOrderData->getPaymentData()->getPriceWithVat()));
}
}

Expand Down

0 comments on commit bc2fa89

Please sign in to comment.