Submodule to support Facebook OAuth module integration (http://drupal.org/project/fboauth).

To use, just unzip the files into your openid_selector module folder and activate from the module administration page.

CommentFileSizeAuthor
openid_selector_fboauth.zip1.47 KBabulte
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
CreditAttribution: xurizaemon commented
Status: Needs review » Needs work

Looks good, thanks abulte.

Requires that the block from fboauth (title: "Facebook login") is displayed on screen to function. (That's a requirement of fboauth module but took me a minute to spot)

The JS code in line 16 uses $('.facebook-action-connect').attr('href') but doesn't test the result, which means if the fboauth block isn't rendered then you'll end up on a 404 for "/undefined"

Committing this to my sandbox @ http://drupal.org/sandbox/grobot/1673156

andriy_gerasika’s picture

andriy_gerasika CreditAttribution: andriy_gerasika commented

fixed for D6 in http://drupalcode.org/project/openid_selector.git/commit/feab917

andriy_gerasika’s picture

andriy_gerasika CreditAttribution: andriy_gerasika commented
Status: Needs work » Fixed

fixed for D7 in http://drupalcode.org/project/openid_selector.git/commit/9cf4022

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

udvranto’s picture

udvranto CreditAttribution: udvranto commented

grobot,
This is exactly what is happening in my case. I didn't want to load the block and getting the undefined error. Any way to fix it?

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
CreditAttribution: xurizaemon commented

You probably need the block rendered to get the JS it contains. Either render the block and hide it, or extract just enough of the JS it adds and theme that in.