Merge pull request #13 from mdevaud/fix/payplug-product

fix: do not add order product as payplug product when his price is 0

Author: zawaze

Config/module.xml

@@ -17,7 +17,7 @@
         <language>en_US</language>
         <language>fr_FR</language>
     </languages>
-    <version>2.0.4</version>
+    <version>2.0.5</version>
     <authors>
         <author>
             <name>Vincent Lopes-Vicente</name>

Event/PayPlugPaymentEvent.php

@@ -612,7 +612,9 @@ public function buildFromOrder(Order $order)
                 ->findOne();
 
             foreach ($order->getOrderProducts() as $orderProduct) {
-                $this->addProduct((new PayPlugProduct())->buildFromOrderProduct($orderProduct, $payPlugDeliveryType));
+                if($orderProduct->getPrice() > 0 ){
+                    $this->addProduct((new PayPlugProduct())->buildFromOrderProduct($orderProduct, $payPlugDeliveryType));
+                }
             }
         }
 

PayPlugModule.php

@@ -82,6 +82,7 @@ public function isValidPayment()
     public function pay(Order $order)
     {
         try {
+
             /** @var PaymentService $paymentService */
             $paymentService = $this->container->get('payplugmodule_payment_service');
 
@@ -90,7 +91,6 @@ public function pay(Order $order)
             $isMultiPayment = $this->getRequest()->getSession()->get(OrderFormListener::PAY_PLUG_MULTI_PAYMENT_FIELD_NAME, 0);
             if ($isMultiPayment) {
                 $orderTotalAmount = $order->getTotalAmount();
-
                 $minAmount = PayPlugModule::getConfigValue(PayPlugConfigValue::MULTI_PAYMENT_MINIMUM);
                 $maxAmount = PayPlugModule::getConfigValue(PayPlugConfigValue::MULTI_PAYMENT_MAXIMUM);
 
@@ -124,6 +124,7 @@ public function pay(Order $order)
             if ($this->getRequest()->isXmlHttpRequest()) {
                 return new JsonResponse(['error' => $exception->getMessage()], 400);
             }
+            Tlog::getInstance()->addError('Error PayPlugModule::pay() : '. $exception->getMessage());
             return new RedirectResponse(URL::getInstance()->absoluteUrl('error'));
         }