Release version 3.4.1

afterpay · May 5, 2021 · 182d2f9 · 182d2f9
1 parent 4c2f96c
commit 182d2f9
Show file tree

Hide file tree

Showing 18 changed files with 166 additions and 93 deletions.
diff --git a/Block/Adminhtml/System/Config/Form/Field/cbtLabel.php b/Block/Adminhtml/System/Config/Form/Field/cbtLabel.php
@@ -0,0 +1,34 @@
+<?php
+/**
+ * Magento 2 extensions for Afterpay Payment
+ *
+ * @author Afterpay
+ * @copyright 2016-2020 Afterpay https://www.afterpay.com
+ */
+namespace Afterpay\Afterpay\Block\Adminhtml\System\Config\Form\Field;
+
+use Magento\Framework\Data\Form\Element\AbstractElement;
+use \Afterpay\Afterpay\Helper\Data as AfterpayHelper;
+
+class CbtLabel extends \Magento\Config\Block\System\Config\Form\Field
+{
+    protected $helper;
+
+    /**
+     * Call constructor.
+     * @param AfterpayHelper $helper
+     */
+    public function __construct(
+        \Magento\Backend\Block\Template\Context $context,
+        AfterpayHelper $helper
+    ) {
+        $this->helper = $helper;
+        parent::__construct($context);
+    }
+
+
+    protected function _getElementHtml(AbstractElement $element)
+    {
+        return $this->helper->getCbtCountry();
+    }
+}
diff --git a/Block/Config.php b/Block/Config.php
@@ -4,6 +4,7 @@
 
 use Magento\Framework\View\Element\Template;
 use Afterpay\Afterpay\Model\Config\Payovertime;
+use Afterpay\Afterpay\Model\Payovertime as AfterpayPayovertime;
 use Magento\Framework\Json\Helper\Data;
 
 class Config extends Template
@@ -29,11 +30,13 @@ public function __construct(
         Payovertime $payovertime,
         Data $dataHelper,
         Template\Context $context,
+        AfterpayPayovertime $afterpayPayovertime,
         array $data
     ) {
 
         $this->_payOverTime = $payovertime;
         $this->_dataHelper = $dataHelper;
+        $this->afterpayPayovertime = $afterpayPayovertime;
 
         parent::__construct($context, $data);
     }
@@ -59,12 +62,7 @@ public function getAfterpayJsUrl()
      */
 	public function checkCurrency()
     {
-		$supportedCurrency=['AUD','NZD','USD','CAD'];
-		if(in_array($this->_payOverTime->getCurrencyCode(),$supportedCurrency)){
-			return true;
-		}
-		else{
-			return false;
-		}
+        return $this->afterpayPayovertime->canUseForCurrency($this->_payOverTime->getCurrencyCode()) && $this->_payOverTime->isActive();
+
     }
 }
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,25 @@
 # Afterpay Magento 2 Extension Changelog
 
+## Version 3.4.1
+
+_Wed 05 May 2021_
+
+### Supported Editions & Versions
+
+Tested and verified in clean installations of Magento 2:
+
+- Magento Enterprise Edition (EE) version 2.4.2
+
+### Highlights
+
+- Improved support for Cross Border Trade (CBT).
+- Improved handling of exceptions at Checkout.
+- Improved JS configurations.
+- Improved support for native Terms and Conditions.
+- Improved support for local currency formatting on product detail pages.
+
+---
+
 ## Version 3.4.0
 
 _Wed 24 Mar 2021_

diff --git a/Controller/Payment/Response.php b/Controller/Payment/Response.php
@@ -40,6 +40,7 @@ class Response extends \Magento\Framework\App\Action\Action
 	protected $_quoteValidator;
 	protected $_timezone;
 	protected $_afterpayApiPayment;
+	protected $_expressPayment;
 
     /**
      * Response constructor.
@@ -60,6 +61,10 @@ class Response extends \Magento\Framework\App\Action\Action
      * @param \Magento\Sales\Model\Order\Payment\Transaction\Repository $transactionRepository
      * @param \Magento\Framework\Notification\NotifierInterface $notifierPool
      * @param \Afterpay\Afterpay\Model\Adapter\V2\AfterpayOrderPaymentCapture $paymentCapture
+     * @param \Magento\Quote\Model\QuoteValidator $quoteValidator
+     * @param \Magento\Framework\Stdlib\DateTime\TimezoneInterface $timezone
+     * @param \Afterpay\Afterpay\Model\Adapter\AfterpayPayment $afterpayApiPayment
+     * @param \Afterpay\Afterpay\Model\Adapter\AfterpayExpressPayment $expressPayment
      */
     public function __construct(
         \Magento\Framework\App\Action\Context $context,
@@ -82,7 +87,8 @@ public function __construct(
 		\Afterpay\Afterpay\Model\Adapter\V2\AfterpayOrderPaymentCapture $paymentCapture,
 		\Magento\Quote\Model\QuoteValidator $quoteValidator,
 		\Magento\Framework\Stdlib\DateTime\TimezoneInterface $timezone,
-		\Afterpay\Afterpay\Model\Adapter\AfterpayPayment $afterpayApiPayment
+		\Afterpay\Afterpay\Model\Adapter\AfterpayPayment $afterpayApiPayment,
+		\Afterpay\Afterpay\Model\Adapter\AfterpayExpressPayment $expressPayment
     ) {
         $this->_resultForwardFactory = $resultForwardFactory; 
 		$this->response = $response;
@@ -104,6 +110,7 @@ public function __construct(
 		$this->_quoteValidator = $quoteValidator;
 		$this->_timezone = $timezone;
 		$this->_afterpayApiPayment = $afterpayApiPayment;
+		$this->_expressPayment = $expressPayment;
 
         parent::__construct($context);
     }
@@ -164,12 +171,12 @@ private function _processAuthCapture($query)
                     if (!$response_check) {
                         // Check the order token being use
                         throw new \Magento\Framework\Exception\LocalizedException(__('There are issues when processing your payment. Invalid Token'));
-                    } elseif ($merchant_order_id != $response_check['merchantReference']) {
-                        // Check order id
-                        throw new \Magento\Framework\Exception\LocalizedException(__('There are issues when processing your payment. Invalid Merchant Reference'));
                     } elseif (round($quote->getBaseGrandTotal(), 2) != round($response_check['amount']['amount'], 2)) {
                         // Check the order amount
                         throw new \Magento\Framework\Exception\LocalizedException(__('There are issues when processing your payment. Invalid Amount'));
+                    } elseif ($this->_expressPayment->isCartUpdated($quote, $response_check['items'])) {
+                        // Check cart Items
+                        throw new \Magento\Framework\Exception\LocalizedException(__('There are issues when processing your payment. Invalid Cart Items'));
                     }
 
 					if(!$this->_helper->getConfig('payment/afterpaypayovertime/payment_flow') || $this->_helper->getConfig('payment/afterpaypayovertime/payment_flow')=="immediate" || $quote->getIsVirtual()){

diff --git a/Helper/Data.php b/Helper/Data.php
@@ -12,17 +12,20 @@ class Data extends \Magento\Framework\App\Helper\AbstractHelper
     protected $_logger;
     protected $_afterpayConfig;
     protected $_moduleList;
+    protected $_countryFactory;
 
     public function __construct(
         \Magento\Framework\App\Helper\Context $context,
         \Afterpay\Afterpay\Model\Logger\Logger $logger,
         \Afterpay\Afterpay\Model\Config\Payovertime $afterpayConfig,
-        \Magento\Framework\Module\ModuleListInterface $moduleList
+        \Magento\Framework\Module\ModuleListInterface $moduleList,
+		\Magento\Directory\Model\CountryFactory $countryFactory
     ) {
         parent::__construct($context);
         $this->_logger = $logger;
         $this->_afterpayConfig = $afterpayConfig;
         $this->_moduleList = $moduleList;
+		$this->_countryFactory = $countryFactory;
     }
 
     public function debug($message, array $context = [])
@@ -38,6 +41,26 @@ public function getModuleVersion()
         return $moduleInfo['setup_version'];
     }
 
+	 public function getCbtCountry()
+    {
+		$cbtEnabled="Disabled";
+        if($this->_afterpayConfig->isCbtEnabled()){
+			$cbtEnabled = "Enabled";
+			$cbtCountries = $this->_afterpayConfig->getCbtCountry();
+			if(!empty($cbtCountries)){
+				$cbtCountryCode=explode(",",$cbtCountries);
+                $counrtyNames=[];
+                foreach($cbtCountryCode AS $countryCode){
+					if($country = $this->_countryFactory->create()->loadByCode($countryCode)){
+						$counrtyNames[] = $country->getName();
+					}
+				}
+				$cbtEnabled = $cbtEnabled." [ ".implode(" | ",$counrtyNames)." ]";
+			}
+		}
+        return $cbtEnabled;
+    }
+
 	public function getConfig($config_path)
     {
         return $this->scopeConfig->getValue(

diff --git a/Model/Adapter/AfterpayExpressPayment.php b/Model/Adapter/AfterpayExpressPayment.php
@@ -193,7 +193,7 @@ public function formatAmount($amount)
     public function isCartUpdated($quoteData, $responseItems)
     {
         $isCartupdated = false;
-        $quoteItems = $quoteData->getAllItems();
+        $quoteItems = $quoteData->getAllVisibleItems();
         if ($quoteData->getItemsCount()!= count($responseItems)) {
             $isCartupdated = true;
             $this->_helper->debug('Cart Items count does not match. Quote Count : '.$quoteData->getItemsCount().' & Response count : '.count($responseItems));

diff --git a/Model/Config/Payovertime.php b/Model/Config/Payovertime.php
@@ -426,6 +426,24 @@ public function getExpressCheckoutKey()
     {
         return $this->_getConfigData(self::EXPRESS_CHECKOUT_KEY);
     }
+	/**
+     * Get cbt enabled/disabled
+     *
+     * @return int
+     */
+    public function isCbtEnabled()
+    {
+        return $this->_getConfigData(self::ENABLE_CBT);
+    } 
+
+	/* Get cbt countries
+     *
+     * @return string
+     */
+    public function getCbtCountry()
+    {
+        return $this->_getConfigData(self::CBT_COUNTRY);
+    }
 
 
 }
diff --git a/Model/Config/Save/Plugin.php b/Model/Config/Save/Plugin.php
@@ -119,14 +119,9 @@ public function aroundSave(
 								}
 
 								$countryName="";
-								if($enable_cbt=="1"){								   
-								    $countryName = $this->_scopeConfig->getValue('general/country/default', $scope,$scopeId);
-								     if(isset($response['CBT']['countries']) && !empty($response['CBT']['countries'])){
-								         if(is_array($response['CBT']['countries'])){
-								             $countryName .=",".implode(",",$response['CBT']['countries']);
-								         }
-								     }								    
-								}								
+								if($enable_cbt=="1" && !empty($response['CBT']['countries']) && is_array($response['CBT']['countries'])){
+								    $countryName =implode(",",$response['CBT']['countries']);
+								}
 
 
 

diff --git a/Model/ConfigProvider.php b/Model/ConfigProvider.php
@@ -20,14 +20,19 @@ class ConfigProvider implements ConfigProviderInterface
      * @var Config\Payovertime
      */
     protected $afterpayConfig;
+    /**
+     * @var Payovertime
+     */
+    protected $afterpayPayovertime;
 
     /**
      * ConfigProvider constructor.
      * @param Config\Payovertime $config
      */
-    public function __construct(\Afterpay\Afterpay\Model\Config\Payovertime $config)
+    public function __construct(\Afterpay\Afterpay\Model\Config\Payovertime $config,\Afterpay\Afterpay\Model\Payovertime $afterpayPayovertime)
     {
         $this->afterpayConfig = $config;
+        $this->afterpayPayovertime = $afterpayPayovertime;
     }
 
     /**
@@ -51,6 +56,7 @@ public function getConfig()
                     'paymentAction'     => $this->afterpayConfig->getPaymentAction(),
                     'termsConditionUrl' => self::TERMS_CONDITION_LINK,
                     'currencyCode'     => $this->afterpayConfig->getCurrencyCode(),
+                    'baseCurrencyCode'     => $this->afterpayPayovertime->getStoreCurrencyCode(),
                 ],
             ],
         ]);

diff --git a/Model/Payovertime.php b/Model/Payovertime.php
@@ -54,7 +54,7 @@ class Payovertime extends \Magento\Payment\Model\Method\AbstractMethod
     /**
      * For dependency injection
      */
-    protected $supportedContryCurrencyCodes = array('AU'=>'AUD','NZ'=>'NZD','US'=>'USD','CA'=>'CAD');
+    protected $supportedContryCurrencyCodes = array('AU'=>'AUD','NZ'=>'NZD','US'=>'USD','CA'=>'CAD','GB'=>'GBP');
     protected $afterPayPaymentTypeCode = self::AFTERPAY_PAYMENT_TYPE_CODE;
 
     protected $logger;
@@ -321,14 +321,14 @@ public function isAvailable(\Magento\Quote\Api\Data\CartInterface $quote = null)
     public function canUseForCurrency($currencyCode)
     {
         $canUseForCurrency= false;
-
+        $storeCurrencyCode=$this->getStoreCurrencyCode();
         if (in_array($currencyCode, $this->supportedContryCurrencyCodes) ) {
 
-            $canUseForCurrency=parent::canUseForCurrency($currencyCode);
-            //Currency Check for Cross Border trade
-            if(!empty($this->getConfigData('enable_cbt'))){
+            if ($currencyCode==$storeCurrencyCode ) {
+                $canUseForCurrency=parent::canUseForCurrency($currencyCode);
+            }else if(!empty($this->getConfigData('enable_cbt')) && !empty($this->getConfigData('cbt_country'))){
+                //Currency Check for Cross Border trade
                 $specifiedCountires=explode(",",$this->getConfigData('cbt_country'));
-                $canUseForCurrency=false;
                 foreach($specifiedCountires AS $country){
                     if(isset($this->supportedContryCurrencyCodes[$country]) && ($currencyCode==$this->supportedContryCurrencyCodes[$country])){
                         $canUseForCurrency=parent::canUseForCurrency($currencyCode);
@@ -408,4 +408,16 @@ public function fetchTransactionInfo(InfoInterface $payment, $transactionId)
         // return to the parent
         return parent::fetchTransactionInfo($payment, $transactionId);
     }
+    /**
+     * Calculated the merchant's store currency code
+     *
+     * @return $text
+     */
+    public function getStoreCurrencyCode()
+    {
+        $objectManager =  \Magento\Framework\App\ObjectManager::getInstance();
+        $storeManager = $objectManager->get('\Magento\Store\Model\StoreManagerInterface');
+        $store = $storeManager->getStore();
+        return $store->getBaseCurrencyCode();
+    }
 }
diff --git a/assets.ini b/assets.ini
diff --git a/composer.json b/composer.json
@@ -3,7 +3,7 @@
 	"license" : "OSL-3.0",
 	"type" : "magento2-module",
 	"description" : "Magento 2 Afterpay Payment Module",
-	"version" : "3.4.0",
+	"version" : "3.4.1",
 	"authors" : [{
 			"name" : "Afterpay",
 			"homepage" : "https://www.afterpay.com"

diff --git a/etc/adminhtml/system.xml b/etc/adminhtml/system.xml
@@ -94,6 +94,12 @@
                         <comment><![CDATA[Check limit configuration from Afterpay API.]]></comment>
                     </field>
 
+					<!-- CBT countries -->
+                    <field id="cbt_country" translate="label" type="label" sortOrder="49" showInDefault="1" showInWebsite="1" showInStore="0">
+                        <label>Cross-Border Trade</label>
+                        <frontend_model>Afterpay\Afterpay\Block\Adminhtml\System\Config\Form\Field\cbtLabel</frontend_model>
+                    </field>
+
                     <!-- Debug mode -->
                     <field id="debug" translate="label" type="select" sortOrder="50" showInDefault="1" showInWebsite="1" showInStore="0">
                         <label>Debug</label>