View Issue Detail

ID	Project	Category	Submitted / Last Updated
00005	UserSpice	User Management Code	2018-03-10 20:59:14 / 2019-06-20 00:08:48
Reported	Brandin Arsenault	Assigned To	Unassigned
Priority	low	Reported	4.3.14
Status	closed	Resolution Version and Commit
Summary	Overhaul Google OAuth Code
Description	<p> if ($foundUS == 1){</p> <p>                    //Found an existing UserSpice user with the same email</p> <p>                    // die("user already has userspice");</p> <p>                    $this->_db->query("UPDATE users WHERE id = 3 SET lname = ?",array($email)) or die("Google oAuth Error");</p> <p> </p> <p>                }else{</p> <p> </p> <p>I'm very concerned about the update user. Although I know it will never happen unless an admin/user changes their last name to be their email address and their ID happens to be 3, this code seems like it shouldn't exist, right?</p> <p> </p> <p>If this portion isn't used, we should use $foundUS==0 for the else portion and remove this. Let me know what this is meant to do and assign this back to me.</p>