Skip user edit page on first login using fboauth [#2508838]

I am using the 15 days old dev version on this module on my site. I am using Popup for facebook login.

When the user logs in for the first time using facebook currently the user is being redirected to the account edit page. Is there a way I can skip this test, so that user will be back to the page where he came from. Later on he can go to account edit page and set the password if he wants to. Else he continue using the facebook login only.

Once the account is already connected, I am not facing this issue as the user is rightly directed to the page he came from. The issue is only with the first login using facebook.

Please let me know if there is some setting that I can enable.

Comment	File	Size	Author
#3	skip_profile_edit_option_on_first_login_spaces_not_fixed.patch	1.52 KB	gokulnk
#1	skip_profile_edit_option_on_first_login.patch	7.9 KB	gokulnk

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

gokulnk CreditAttribution: gokulnk commented 19 June 2015 at 20:37

File	Size
skip_profile_edit_option_on_first_login.patch	7.9 KB

Added a checkbox for configuration to skip the profile edit screen on first login using facebook.

Comment #2

AlexKirienko CreditAttribution: AlexKirienko as a volunteer and at WikiJob commented 20 June 2015 at 20:26

Hi.

"Popup for facebook login" was added recently by DarrellDuane. Unfortunately I don't have time to take a look at this feature.
But I'm agree with you, such settings will be useful.

Your patch contains changes unrelated with this issue. Please update your patch. I will check it as soon as I will have time.

Thank you.

Comment #3

gokulnk CreditAttribution: gokulnk at Azri Solutions commented 22 June 2015 at 06:20

File	Size
skip_profile_edit_option_on_first_login_spaces_not_fixed.patch	1.52 KB

Hi the issue was because of additional spaces at the EOLs in the current code and the diff shows them as my editor fixed those space issues.

Anyways I have created a new patch ignoring the additional spaces and attachign it. Please commit the file that you think would be the right one.

Comment #4

gokulnk CreditAttribution: gokulnk at Azri Solutions commented 22 June 2015 at 06:28

Status:

Needs work

» Needs review

Why do I always forget to update the status? :P

Comment #5

sketman CreditAttribution: sketman commented 12 January 2016 at 20:14

Patch #3 is working for me.
I am using Logintoboggan and have "Set the password" setting on (checked).
When new account is created through FBOAuth with this patch applied, after successful authentication the actual page is reloaded without user being redirected to the user edit form.
Thanks @gokulnk