Merge pull request #8 from WeGetFinancing/GET-2400

GET-2400 | fix(validator): merchant token empty validator refactoring
WeGetFinancing · Nov 3, 2023 · 924582e · 924582e
2 parents f381db6 + 8ff8aef
commit 924582e
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 1 deletion.
diff --git a/etc/adminhtml/system.xml b/etc/adminhtml/system.xml
@@ -102,7 +102,7 @@
                         showInDefault="1" showInWebsite="1" showInStore="1"
                 >
                     <label>PPE Merchant Token</label>
-                    <validate>validate-wgf-merchant-token-empty validate-wgf-merchant-token-error</validate>
+                    <validate>validate-wgf-merchant-token-not-set validate-wgf-merchant-token-empty validate-wgf-merchant-token-error</validate>
                 </field>
                 <field
                         id="ppe_is_apply_now"

diff --git a/view/adminhtml/layout/adminhtml_system_config_edit.xml b/view/adminhtml/layout/adminhtml_system_config_edit.xml
@@ -4,5 +4,6 @@
     <head>
         <link src="WeGetFinancing_Checkout::js/validator-merchant-token-empty.js"/>
         <link src="WeGetFinancing_Checkout::js/validator-merchant-token-error.js"/>
+        <link src="WeGetFinancing_Checkout::js/validator-merchant-token-not-set.js"/>
     </head>
 </page>
diff --git a/view/adminhtml/web/js/validator-merchant-token-not-set.js b/view/adminhtml/web/js/validator-merchant-token-not-set.js
@@ -0,0 +1,16 @@
+require(
+    [
+        'jquery',
+        'mage/translate',
+        'jquery/validate'
+    ],
+    function($){
+        $.validator.addMethod(
+            'validate-wgf-merchant-token-not-set',
+            function (value) {
+                return !("" === String(value).trim());
+            },
+            $.mage.__("The PPE Merchant Token cannot be empty.")
+        );
+    }
+);