Merge pull request #66 from Affirm/hotfix-update-logging

Hotfix update logging
Affirm · Aug 13, 2021 · 3d9fab4 · 3d9fab4
2 parents 50f5757 + 2dc2619
commit 3d9fab4
Show file tree

Hide file tree

Showing 5 changed files with 3 additions and 37 deletions.
diff --git a/Gateway/Validator/Client/PaymentActionsValidator.php b/Gateway/Validator/Client/PaymentActionsValidator.php
@@ -20,24 +20,12 @@
 
 use Magento\Payment\Gateway\Helper\SubjectReader;
 use Astound\Affirm\Gateway\Helper\Util;
-use Astound\Affirm\Logger\Logger;
 
 /**
  * Class PaymentActionsValidator
  */
 class PaymentActionsValidator extends AbstractResponseValidator
 {
-    /**
-     * Constructor
-     *
-     * @param Logger $logger
-     */
-    public function __construct(
-        Logger $logger
-    ) {
-        $this->logger = $logger;
-    }
-
     /**
      * @inheritdoc
      */
@@ -56,7 +44,6 @@ public function validate(array $validationSubject)
                 [__('Affirm error code:') . $response[self::RESPONSE_CODE] . __(' error: ') .
                     __($response[self::ERROR_MESSAGE])]:
                 [__('Transaction has been declined, please, try again later.')];
-            $this->logger->debug('Astound\Affirm\Gateway\Validator\Client\PaymentActionsValidator::validate', $errorMessages);
         }
 
         return $this->createResult($validationResult, $errorMessages);

diff --git a/Gateway/Validator/Client/PaymentActionsValidatorVoid.php b/Gateway/Validator/Client/PaymentActionsValidatorVoid.php
@@ -19,7 +19,6 @@
 namespace Astound\Affirm\Gateway\Validator\Client;
 
 use Magento\Payment\Gateway\Helper\SubjectReader;
-use Astound\Affirm\Logger\Logger;
 
 /**
  * Class PaymentActionsValidatorVoid
@@ -33,17 +32,6 @@ class PaymentActionsValidatorVoid extends PaymentActionsValidator
     const RESPONSE_TYPE_VOID = 'void';
     /**#@-*/
 
-    /**
-     * Constructor
-     *
-     * @param Logger $logger
-     */
-    public function __construct(
-        Logger $logger
-    ) {
-        $this->logger = $logger;
-    }
-
     /**
      * @inheritdoc
      */
@@ -57,7 +45,6 @@ public function validate(array $validationSubject)
 
         if (!$validationResult) {
             $errorMessages = [__('Transaction has been declined, please, try again later.')];
-            $this->logger->debug('Astound\Affirm\Gateway\Validator\Client\PaymentActionsValidatorVoid::validate', $errorMessages);
         }
 
         return $this->createResult($validationResult, $errorMessages);

diff --git a/Gateway/Validator/CurrencyValidator.php b/Gateway/Validator/CurrencyValidator.php
@@ -21,7 +21,6 @@
 use Magento\Payment\Gateway\Validator\ResultInterface;
 use Magento\Payment\Gateway\Validator\ResultInterfaceFactory;
 use Magento\Payment\Gateway\Validator\AbstractValidator;
-use Astound\Affirm\Logger\Logger;
 
 /**
  * Class CurrencyValidator
@@ -43,15 +42,12 @@ class CurrencyValidator extends AbstractValidator
      *
      * @param ResultInterfaceFactory $resultFactory
      * @param \Magento\Payment\Gateway\ConfigInterface $config
-     * @param Logger $logger
      */
     public function __construct(
         ResultInterfaceFactory $resultFactory,
-        ConfigInterface $config,
-        Logger $logger
+        ConfigInterface $config
     ) {
         $this->config = $config;
-        $this->logger = $logger;
         parent::__construct($resultFactory);
     }
 
@@ -63,7 +59,6 @@ public function __construct(
      */
     public function validate(array $validationSubject)
     {
-        $log = [];
         $isValid = true;
         $storeId = $validationSubject['storeId'];
         $objectManager = \Magento\Framework\App\ObjectManager::getInstance();
@@ -77,9 +72,6 @@ public function validate(array $validationSubject)
 
             if (!in_array($currentCurrencyCode, $availableCurrencies)) {
                 $isValid = false;
-                $log['code'] = $currentCurrencyCode;
-                $log['availableCurrencies'] = $availableCurrencies;
-                $this->logger->debug('Astound\Affirm\Gateway\Validator\CurrencyValidator::validate', $log);
             }
         }
         return $this->createResult($isValid);

diff --git a/composer.json b/composer.json
@@ -2,7 +2,7 @@
     "name": "affirm/magento2",
     "description": "Affirm's extension for the Magento 2 https://www.affirm.com/",
     "type": "magento2-module",
-    "version": "3.0.12",
+    "version": "3.0.13",
     "license": [
         "BSD-3-Clause"
     ],

diff --git a/etc/module.xml b/etc/module.xml
@@ -10,7 +10,7 @@
  */
  -->
 <config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:Module/etc/module.xsd">
-    <module name="Astound_Affirm" setup_version="3.0.12">
+    <module name="Astound_Affirm" setup_version="3.0.13">
         <sequence>
             <module name="Magento_Checkout"/>
             <module name="Magento_Sales"/>