Merge pull request #252 from TransbankDevelopers/feat/improve-oneclic…

…k-payment-flow feat: use user id instead of the email
TransbankDevelopers · Aug 19, 2024 · 08e8c68 · 08e8c68
2 parents 92868f6 + ac42a86
commit 08e8c68
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/plugin/src/PaymentGateways/WC_Gateway_Transbank_Oneclick_Mall_REST.php b/plugin/src/PaymentGateways/WC_Gateway_Transbank_Oneclick_Mall_REST.php
@@ -717,10 +717,10 @@ private function getTotalAmountFromOrder(WC_Order $order): int
     private function validatePayerMatchesCardInscription(WC_Payment_Token_Oneclick $paymentToken): bool
     {
         $currentUser = wp_get_current_user();
-        $userEmail = $currentUser->user_email;
-        $inscriptionEmail = $paymentToken->get_email();
+        $userId = $currentUser->id;
+        $inscriptionId = $paymentToken->get_userId();
 
-        return $userEmail == $inscriptionEmail;
+        return $userId == $inscriptionId;
     }
 
     public function getOneclickPaymentTokenClass()

diff --git a/plugin/src/Tokenization/WC_Payment_Token_Oneclick.php b/plugin/src/Tokenization/WC_Payment_Token_Oneclick.php
@@ -197,4 +197,16 @@ public function set_email($email)
     {
         $this->set_prop('email', $email);
     }
+
+    /**
+     * Returns the user id  of the inscriptions.
+     *
+     * @param string $context What the value is for. Valid values are view and edit.
+     *
+     * @return int The user id.
+     */
+    public function get_userId($context = 'view')
+    {
+        return $this->get_prop('user_id', $context);
+    }
 }