feat: use user id instead of the email

TransbankDevelopers · Aug 1, 2024 · cb64b41 · cb64b41
1 parent bd89b4a
commit cb64b41
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 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,8 +717,8 @@ 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();
+        $userEmail = $currentUser->id;
+        $inscriptionEmail = $paymentToken->get_userId();
 
         return $userEmail == $inscriptionEmail;
     }

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);
+    }
 }