From 15c80ac1e95aa19ad6e3c6a96d528c32c1b47d19 Mon Sep 17 00:00:00 2001
From: Francis Pineda <francis.p@affinidi.com>
Date: Tue, 23 Apr 2024 17:19:02 +0800
Subject: [PATCH] fix: fixed key name and if cond

---
 includes/wp-affinidi-login-callback.php | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/includes/wp-affinidi-login-callback.php b/includes/wp-affinidi-login-callback.php
index 4a24dad..3912e57 100644
--- a/includes/wp-affinidi-login-callback.php
+++ b/includes/wp-affinidi-login-callback.php
@@ -140,7 +140,7 @@
     $user_id = null;
 
     if (email_exists($userInfo['email']) == false) {
-        if (affinidi_login_users_can_signup() == 0) {
+        if (affinidi_login_users_can_signup()) {
             wp_safe_redirect(add_query_arg(array('message' => 'affinidi_login_only'), esc_url($user_redirect)));
             exit;
         }
@@ -149,7 +149,7 @@
         $random_password = wp_generate_password($length = 16, $extra_special_chars = true);
         $user_data = [
             'user_email'   => $userInfo['email'],
-            'user_login'   => (!empty($userInfo['first_name']) ? $userInfo['first_name'] : $userInfo['email']), // default to mail if not present
+            'user_login'   => $userInfo['email'], // default to mail
             'user_pass'    => $random_password,
             'last_name'    => $userInfo['last_name'],
             'first_name'   => $userInfo['first_name'],
@@ -184,11 +184,10 @@
          * we should check the user by email. This may be the case when the users are preregistered outside of OAuth
          */
         if (!$user) {
-             // Get the user by name
-            $user = get_user_by('login', $userInfo['given_name']);
+             // Get the user by email using login
+            $user = get_user_by('login', $userInfo['email']);
         }
 
-
         if (!$user) {
             // redirect user with error code
             wp_safe_redirect(add_query_arg(array('message' => 'affinidi_login_failed'), esc_url($user_redirect)));