Merge pull request #58 from two-inc/brtkwr-two/cet-533-magento-remove-merchant_short_name-input-from-admin-settings

CET-533/fix: Magento: Remove merchant_short_name input from admin settings

Author: brtkwr

Api/Config/RepositoryInterface.php

@@ -22,7 +22,6 @@ interface RepositoryInterface
     public const XML_PATH_ENABLED = 'payment/two_payment/active';
     public const XML_PATH_TITLE = 'payment/two_payment/title';
     public const XML_PATH_MODE = 'payment/two_payment/mode';
-    public const XML_PATH_MERCHANT_SHORT_NAME = 'payment/two_payment/merchant_short_name';
     public const XML_PATH_API_KEY = 'payment/two_payment/api_key';
     public const XML_PATH_DAYS_ON_INVOICE = 'payment/two_payment/days_on_invoice';
     public const XML_PATH_FULFILL_TRIGGER = 'payment/two_payment/fulfill_trigger';
@@ -59,15 +58,6 @@ public function isActive(?int $storeId = null): bool;
      */
     public function getMode(?int $storeId = null): string;
 
-    /**
-     * Get merchant short name
-     *
-     * @param int|null $storeId
-     *
-     * @return string
-     */
-    public function getMerchantShortName(?int $storeId = null): string;
-
     /**
      * Get API key
      *

Block/Adminhtml/System/Config/Field/ApiKeyCheck.php

@@ -19,7 +19,6 @@
  */
 class ApiKeyCheck extends Field
 {
-
     /**
      * @var string
      */
@@ -28,24 +27,25 @@ class ApiKeyCheck extends Field
     /**
      * @var ConfigRepository
      */
-    private $_configRepository;
+    private $configRepository;
 
     /**
      * @var Two
      */
-    private $_two;
+    private $two;
 
     /**
      * @var Adapter
      */
-    private $_adapter;
+    private $adapter;
 
     /**
      * Version constructor.
      *
-     * @param Context $context
-    * @param Adapter $adapter
      * @param ConfigRepository $configRepository
+     * @param Adapter $adapter
+     * @param Two $two
+     * @param Context $context
      * @param array $data
      */
     public function __construct(
@@ -55,9 +55,9 @@ public function __construct(
         Context $context,
         array $data = []
     ) {
-        $this->_configRepository = $configRepository;
-        $this->_adapter = $adapter;
-        $this->_two = $two;
+        $this->configRepository = $configRepository;
+        $this->adapter = $adapter;
+        $this->two = $two;
         parent::__construct($context, $data);
     }
 
@@ -68,34 +68,27 @@ public function __construct(
      */
     public function getApiKeyStatus(): array
     {
-        $short_name = $this->_configRepository->getMerchantShortName();
-        if ($short_name && $this->_configRepository->getApiKey()) {
-            $result = $this->_adapter->execute('/v1/merchant/' . $short_name, [], 'GET');
-            $error = $this->_two->getErrorFromResponse($result);
-            if ($error) {
-                return [
-                    'message' => __('Credentials are invalid'),
-                    'status' => 'error',
-                    'error' => $error
-                ];
-            } else {
-                if ($result['short_name'] != $short_name) {
-                    return [
-                        'message' => __('Username should be set to %1', $result['short_name']),
-                        'status' => 'warning',
-                    ];
-                }
-                return [
-                    'message' => __('Credentials are valid'),
-                    'status' => 'success'
-                ];
-            }
-        } else {
+        if (!$this->configRepository->getApiKey()) {
             return [
-                'message' => __('Credentials are missing'),
+                'message' => __('API key is missing'),
                 'status' => 'warning'
             ];
         }
+
+        $result = $this->adapter->execute('/v1/merchant/verify_api_key', [], 'GET');
+        $error = $this->two->getErrorFromResponse($result);
+        if ($error) {
+            return [
+                'message' => __('API key is not valid'),
+                'status' => 'error',
+                'error' => $error
+            ];
+        } else {
+            return [
+                'message' => __('API key is valid'),
+                'status' => 'success'
+            ];
+        }
     }
 
     /**

Model/Config/Repository.php

@@ -79,14 +79,6 @@ private function isSetFlag(string $path, ?int $storeId = null, ?string $scope =
         return $this->scopeConfig->isSetFlag($path, $scope, $storeId);
     }
 
-    /**
-     * @inheritDoc
-     */
-    public function getMerchantShortName(?int $storeId = null): string
-    {
-        return (string)$this->getConfig(self::XML_PATH_MERCHANT_SHORT_NAME, $storeId);
-    }
-
     /**
      * Retrieve config value
      *

Model/Ui/ConfigProvider.php

@@ -11,6 +11,8 @@
 use Magento\Framework\View\Asset\Repository as AssetRepository;
 use Two\Gateway\Api\Config\RepositoryInterface as ConfigRepository;
 use Two\Gateway\Service\UrlCookie;
+use Two\Gateway\Service\Api\Adapter;
+use Two\Gateway\Model\Two;
 
 /**
  * Ui Config Provider
@@ -22,6 +24,16 @@ class ConfigProvider implements ConfigProviderInterface
      */
     private $configRepository;
 
+    /**
+     * @var Two
+     */
+    private $two;
+
+    /**
+     * @var Adapter
+     */
+    private $adapter;
+
     /**
      * @var AssetRepository
      */
@@ -31,13 +43,19 @@ class ConfigProvider implements ConfigProviderInterface
      * ConfigProvider constructor.
      *
      * @param ConfigRepository $configRepository
+     * @param Adapter $adapter
+     * @param Two $two
      * @param AssetRepository $assetRepository
      */
     public function __construct(
         ConfigRepository $configRepository,
+        Adapter $adapter,
+        Two $two,
         AssetRepository $assetRepository
     ) {
         $this->configRepository = $configRepository;
+        $this->adapter = $adapter;
+        $this->two = $two;
         $this->assetRepository = $assetRepository;
     }
 
@@ -48,12 +66,16 @@ public function __construct(
      */
     public function getConfig(): array
     {
+        $merchant = null;
+        if ($this->configRepository->getApiKey()) {
+            $merchant = $this->adapter->execute('/v1/merchant/verify_api_key', [], 'GET');
+        }
         $orderIntentConfig = [
             'extensionPlatformName' => $this->configRepository->getExtensionPlatformName(),
             'extensionDBVersion' => $this->configRepository->getExtensionDBVersion(),
             'invoiceType' => 'FUNDED_INVOICE',
-            'merchantShortName' => $this->configRepository->getMerchantShortName(),
             'weightUnit' => $this->configRepository->getWeightUnit(),
+            'merchant' => $merchant,
         ];
 
         $provider = $this->configRepository::PROVIDER;

bumpver.toml

@@ -1,5 +1,5 @@
 [tool.bumpver]
-current_version = "1.7.2"
+current_version = "1.7.3"
 version_pattern = "MAJOR.MINOR.PATCH[-TAGNUM]"
 commit_message = "chore: Bump version {old_version} -> {new_version}"
 commit = true

composer.json

@@ -2,7 +2,7 @@
     "name": "two-inc/magento2",
     "description": "Two B2B BNPL payments extension",
     "type": "magento2-module",
-    "version": "1.7.2",
+    "version": "1.7.3",
     "license": [
         "OSL-3.0",
         "AFL-3.0"

etc/adminhtml/system.xml

@@ -40,15 +40,6 @@
                     </depends>
                     <config_path>payment/two_payment/mode</config_path>
                 </field>
-                <field id="merchant_short_name" translate="label" type="text" sortOrder="40" showInDefault="1"
-                       showInWebsite="1" showInStore="1">
-                    <label>Username</label>
-                    <comment>Username for your merchant account</comment>
-                    <depends>
-                        <field id="active">1</field>
-                    </depends>
-                    <config_path>payment/two_payment/merchant_short_name</config_path>
-                </field>
                 <field id="api_key" translate="label" type="obscure" sortOrder="50" showInDefault="1" showInWebsite="1"
                        showInStore="1">
                     <label>API key</label>
@@ -62,7 +53,6 @@
                 </field>
                 <field id="api_key_check" translate="label" type="button" sortOrder="55" showInDefault="1"
                        showInWebsite="1" showInStore="1">
-                    <label>Credentials</label>
                     <frontend_model>Two\Gateway\Block\Adminhtml\System\Config\Field\ApiKeyCheck</frontend_model>
                 </field>
                 <field id="debug" translate="label" type="select" sortOrder="60" showInDefault="1" showInWebsite="1"

etc/config.xml

@@ -11,7 +11,7 @@
         <payment>
             <two_payment>
                 <active>1</active>
-                <version>1.7.2</version>
+                <version>1.7.3</version>
                 <title>Business invoice - 30 days</title>
                 <mode>sandbox</mode>
                 <invoice_type>FUNDED_INVOICE</invoice_type>

i18n/en_US.csv

@@ -8,6 +8,9 @@
 "%1 requires whole order to be shipped before it can be fulfilled.","%1 requires whole order to be shipped before it can be fulfilled."
 "* Required Fields","* Required Fields"
 "API key","API key"
+"API key is missing","API key is missing"
+"API key is not valid","API key is not valid"
+"API key is valid","API key is valid"
 "Add PO number field","Add PO number field"
 "Add department field","Add department field"
 "Add order note field","Add order note field"
@@ -29,15 +32,10 @@ Branding,Branding
 "Could not initiate refund with %1","Could not initiate refund with %1"
 "Could not update %1 order status to cancelled. Please contact support with order ID %2. Error: %3","Could not update %1 order status to cancelled. Please contact support with order ID %2. Error: %3"
 "Country is not valid.","Country is not valid."
-Credentials,Credentials
-"Credentials are invalid","Credentials are invalid"
-"Credentials are missing","Credentials are missing"
-"Credentials are valid","Credentials are valid"
 "Credit Note","Credit Note"
 "Debug mode","Debug mode"
 Department,Department
 "Developed by Magmodules.","Developed by Magmodules."
-"Display info link","Display info link"
 Download,Download
 "Download as .txt file","Download as .txt file"
 "Email Address is not valid.","Email Address is not valid."
@@ -96,8 +94,6 @@ Title,Title
 "Two is a payment solution for B2B purchases online, allowing you to buy from your favourite merchants and suppliers on trade credit. Using Two, you can access flexible trade credit instantly to make purchasing simple.","Two is a payment solution for B2B purchases online, allowing you to buy from your favourite merchants and suppliers on trade credit. Using Two, you can access flexible trade credit instantly to make purchasing simple."
 "Unable to find the requested %1 order","Unable to find the requested %1 order"
 "Unable to retrieve payment information for your invoice purchase with %1. The cart will be restored.","Unable to retrieve payment information for your invoice purchase with %1. The cart will be restored."
-Username,Username
-"Username should be set to %1","Username should be set to %1"
 Version,Version
 "You must first accept the payment terms.","You must first accept the payment terms."
 "You will be redirected to %1 when you place order.","You will be redirected to %1 when you place order."

i18n/nb_NO.csv

@@ -8,6 +8,9 @@
 "%1 requires whole order to be shipped before it can be fulfilled.","%1 krever at hele bestillingen sendes før den kan oppfylles."
 "* Required Fields","* Obligatoriske felter"
 "API key",API-nøkkel
+"API key is missing","API-nøkkel mangler"
+"API key is not valid","API-nøkkel er ikke gyldig"
+"API key is valid","API-nøkkel er gyldig"
 "Add PO number field","Legg til PO-nummerfelt"
 "Add department field","Legg til felt for avdeling"
 "Add order note field","Legg til bestillingsnotatfelt"
@@ -29,15 +32,10 @@ Branding,Merkevarebygging
 "Could not initiate refund with %1","Kunne ikke starte refusjon med %1"
 "Could not update %1 order status to cancelled. Please contact support with order ID %2. Error: %3","Kunne ikke oppdatere %1 ordrestatus til kansellert. Ta kontakt med brukerstøtten med ordre-ID %2. Feil: %3"
 "Country is not valid.","Land er ikke gyldig."
-Credentials,Legitimasjon
-"Credentials are invalid","Legitimasjonen er ugyldig"
-"Credentials are missing","Legitimasjon mangler"
-"Credentials are valid","Legitimasjonen er gyldig"
 "Credit Note",Kreditnota
 "Debug mode",Feilsøkingsmodus
 Department,Avdeling
 "Developed by Magmodules.","Utviklet av Magmodules."
-"Display info link","Vis infolink"
 Download,"Last ned"
 "Download as .txt file","Last ned som .txt-fil"
 "Email Address is not valid.","E-postadresse er ikke gyldig."
@@ -96,8 +94,6 @@ Title,Tittel
 "Two is a payment solution for B2B purchases online, allowing you to buy from your favourite merchants and suppliers on trade credit. Using Two, you can access flexible trade credit instantly to make purchasing simple.","Two er en betalingsløsning for B2B-kjøp på nettet, som lar deg kjøpe fra dine favorittforhandlere og leverandører på handelskreditt. Med Two kan du få tilgang til fleksibel handelskreditt umiddelbart for å gjøre innkjøp enkelt."
 "Unable to find the requested %1 order","Kan ikke finne den forespurte bestillingen %1"
 "Unable to retrieve payment information for your invoice purchase with %1. The cart will be restored.","Kan ikke hente betalingsinformasjon for fakturakjøpet med %1. Vognen vil bli restaurert."
-Username,Brukernavn
-"Username should be set to %1","Brukernavn skal settes til %1"
 Version,Versjon
 "You must first accept the payment terms.","Du må først godta betalingsvilkårene."
 "You will be redirected to %1 when you place order.","Du vil bli omdirigert til %1 når du legger inn bestilling."