[Payum] consider CoreShop decimal factor

coreshop · Mar 3, 2020 · 111003d · 111003d
1 parent d0476c9
commit 111003d
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 2 deletions.
diff --git a/src/CoreShop/Bundle/PayumBundle/Action/AuthorizePaymentAction.php b/src/CoreShop/Bundle/PayumBundle/Action/AuthorizePaymentAction.php
@@ -27,6 +27,16 @@ final class AuthorizePaymentAction implements ActionInterface, GatewayAwareInter
 {
     use GatewayAwareTrait;
 
+     /**
+     * @var int
+     */
+    protected $decimalFactor;
+
+    public function __construct(int $decimalFactor)
+    {
+        $this->decimalFactor = $decimalFactor;
+    }
+
     /**
      * {@inheritdoc}
      *
@@ -46,7 +56,8 @@ public function execute($request): void
             } catch (RequestNotSupportedException $e) {
                 $payumPayment = new PayumPayment();
                 $payumPayment->setNumber($payment->getNumber());
-                $payumPayment->setTotalAmount($payment->getTotalAmount());
+                //Payum Payment works with ints with a precision of 2
+                $payumPayment->setTotalAmount(100 * round($payment->getTotalAmount() / $this->decimalFactor, 2));
                 $payumPayment->setCurrencyCode($payment->getCurrency()->getIsoCode());
                 $payumPayment->setDescription($payment->getDescription());
                 $payumPayment->setDetails($payment->getDetails());

diff --git a/src/CoreShop/Bundle/PayumBundle/Action/CapturePaymentAction.php b/src/CoreShop/Bundle/PayumBundle/Action/CapturePaymentAction.php
@@ -27,6 +27,16 @@ final class CapturePaymentAction implements ActionInterface, GatewayAwareInterfa
 {
     use GatewayAwareTrait;
 
+     /**
+     * @var int
+     */
+    protected $decimalFactor;
+
+    public function __construct(int $decimalFactor)
+    {
+        $this->decimalFactor = $decimalFactor;
+    }
+
     /**
      * {@inheritdoc}
      *
@@ -48,7 +58,8 @@ public function execute($request): void
             } catch (RequestNotSupportedException $e) {
                 $payumPayment = new PayumPayment();
                 $payumPayment->setNumber($payment->getNumber());
-                $payumPayment->setTotalAmount($payment->getTotalAmount());
+                //Payum Payment works with ints with a precision of 2
+                $payumPayment->setTotalAmount(100 * round($payment->getTotalAmount() / $this->decimalFactor, 2));
                 $payumPayment->setCurrencyCode($payment->getCurrency()->getIsoCode());
                 $payumPayment->setDescription($payment->getDescription());
                 $payumPayment->setDetails($payment->getDetails());

diff --git a/src/CoreShop/Bundle/PayumBundle/Resources/config/services.yml b/src/CoreShop/Bundle/PayumBundle/Resources/config/services.yml
@@ -28,11 +28,15 @@ services:
 
     coreshop.payum_action.authorize_payment: '@CoreShop\Bundle\PayumBundle\Action\AuthorizePaymentAction'
     CoreShop\Bundle\PayumBundle\Action\AuthorizePaymentAction:
+        arguments:
+            - '%coreshop.currency.decimal_factor%'
         tags:
             - { name: payum.action, all: true, alias: coreshop.authorize_payment }
 
     coreshop.payum_action.capture_payment: '@CoreShop\Bundle\PayumBundle\Action\CapturePaymentAction'
     CoreShop\Bundle\PayumBundle\Action\CapturePaymentAction:
+        arguments:
+            - '%coreshop.currency.decimal_factor%'
         tags:
             - { name: payum.action, all: true, alias: coreshop.capture_payment }