| Comment | File | Size | Author |
|---|---|---|---|
| #14 | 2397837-fboauth-v23-14.patch | 3.33 KB | jcisio |
| #1 | compatibility_with-2397837-1.patch | 2.84 KB | nightwalkr |
| Comment | File | Size | Author |
|---|---|---|---|
| #14 | 2397837-fboauth-v23-14.patch | 3.33 KB | jcisio |
| #1 | compatibility_with-2397837-1.patch | 2.84 KB | nightwalkr |
Comments
Comment #1
nightwalkr commentedI did a patch to enable login against Facebook API 2.2
Comment #2
sbilde commentedThanks man.. - I was about to do a similar patch!
Comment #3
mrded commentedDoes it really work? Did anyone tried it?
Comment #4
darrellduane commentedComment #5
darrellduane commentedI'm including this in the dev release of February 14, 2015
Comment #7
quicksketchAwesome guys! This would be great. I've been worried about this pending deadline on one of the sites I have as well.
Comment #8
darrellduane commentedI considered incorporating this into a new, to be created, 2.x release of this module, but as it turns out, there is no actual functionality change for fboauth to use Facebook API 2.2. The only change is adding '/2.2' on to the end of 'graph.facebook.com' in the url. So, I'm planning to keep it in the 1.x version of the fboauth module.
Comment #9
sonicthoughts commentedGreat! Will you be releasing a 1.7 soon?
Comment #10
darrellduane commentedYes, lets make sure the dev version doesn't have any other problems in it for another week or so and then I'll roll a 1.7
Comment #11
darrellduane commentedComment #12
mrded commentedI suggest to change version of the module to 7.x-2.x because new version with Facebook API 2.2 already broke following task #2149661: Users with empty mail field..
Otherwise it will break websites of users who update from 7.x-1.x to 7.x-1.7. It looks like minor update, that is not really so.
Comment #13
jcisio commentedI don't think it break anything. Email is optional permission, so user can opt out it, even in v1.0. My first Facebook user without email registered on Jan 2nd 2013, long before the Facebook API 2.0 was introduced.
BTW we need to fix 6.x branch, too.
Comment #14
jcisio commentedComment #16
AntiNSA commentedIs this going to be fixed for 6.x?????
Comment #17
AlexKirienko commentedI'm not sure. I have no time for this.
Maybe someone will fix it. I can commit patch with fixes.
Comment #18
AntiNSA commentedIf some one fixes it and you commit it, Ill use it :)
Comment #19
santiwww commentedThe patch in #14 works partially, I had to modify the file fboauth.fboauth.inc a little bit more to make it work because response from facebook come in json format, so the function fboauth_access_token has to be modified like this:
Sorry but I can't provide a patch.
I hope it helps someone.