Currently if user do not confirm or uncheck email from Facebook login dialogue then module generates email like XXXXX@facebook.com which is not always required as user must provide real email address and if dont provide then user should not be created

CommentFileSizeAuthor
#3 auth-type-confirm-email_2744593.patch446 bytesjalpesh

Comments

khoomy created an issue. See original summary.

khoomy’s picture

khoomy commented

Adding following line $return['query']['auth_type'] = 'rerequest'; solved the issue

/**
 * Return a set of properties suitable for use to a url() call.
 */
function fboauth_action_link_properties($action_name, $redirect = NULL, $app_id = NULL) {
  // Use the default App ID if not specified.
  $app_id = isset($app_id) ? $app_id : variable_get('fboauth_id', '');
  $action = fboauth_action_load($action_name);
...............................
.............................

if (variable_get('fboauth_popup', 1)) {
    $return['attributes']['class'] = array('fboauth-popup');
    $return['query']['display'] = 'popup';
  }

  $return['query']['auth_type'] = 'rerequest';

.........
jalpesh’s picture

jalpesh commented
StatusFileSize
new auth-type-confirm-email_2744593.patch446 bytes

I think it must be in below condition.

if (variable_get('fboauth_popup', 1)) {
$return['attributes']['class'] = array('fboauth-popup');
$return['query']['display'] = 'popup';
}

Can you please reconfirm with attach patch. Thanks.

jalpesh’s picture

jalpesh commented
Status: Active » Needs review
keva’s picture

keva commented

Even with patch #3, it still provides the @facebook.com email address