View Issue Detail

ID	Project	Category	Submitted / Last Updated
01454	UserSpice	Plugins	2022-01-14 10:49:05 / 2022-02-19 14:31:43
Reported	Dan Hoover	Assigned To	Unassigned
Priority	none	Reported	5.4.0
Status	addon	Resolution Version and Commit
Summary	Add case for failed fb login email permission
Description	And maybe to help someone else, to do that in fb-callback.php just after //Check to see if the user has an account $fbEmail = $fbuser['email']; you add: if (!$fbEmail) { $declined_scopes = ['email']; $callback=$settings->fbcallback; header("Location: " . $helper->getReRequestUrl( $callback, $declined_scopes)); exit; } MerNion

January 2022

Facebook says that between the attempts (using the Rerequestul) you have to "You should have a screen of education on why you think they should grant the permission to you and then re-ask." So you have to implement some logic in between the attempts. What i did is instead of this code above i did this: $fbEmail = $fbuser['email']; if (!$fbEmail) { Redirect::to($us_url_root.'users/login.php?error=facebook_permissions'); } and the in login.php I check if the $_GET["error"] exists and use this: $declined_scopes = ['email']; $callback=$settings->fbcallback; $loginUrl = $helper->getReRequestUrl( $callback, $declined_scopes)); but you have to somehow manage that the hooks of the facebook plugin won't change the $loginUrl again.. I leave that to you ???? This is just for thought, I don't know it is correct way of doing it