Code Updates - Request for Review / inclusion

[eddiewebb]

The version of JConnector I downloaded last night has a few coding issues.

None of the files are marked, but maybe v1.5.8 ?

Issues;

• Table Check - tries to call table that may not exist, rather than checking for table - this throws a 500 error for the bad query "select * from pre_comprofiler limit 0,1" (addressed)
• Calls the CB methods anyway (addressed)
• Uses FB ID as user ID (rather than DB's auto-increment) which could cause obvious problems if user already exists with that ID (and makes linking table irrelevant)(addressed)

I have (under terms of GPL) made several corrections/updates to the code, and it now runs smooth on my site save one issue.

It is still not requesting the proxy email permission properly, so no email is returned or saved. I even did a screen dump of the entire user array returned by FB, and no email was included. I don't know enough about the PHP5 FB Client, or the FB API to resolve that last night, but I am working on it.

---

Request to Web Scribbles
How do I submit my code changes for review / inclusion ?




---

Anyone with FB API experience, I would love the hlp fixing this remaining email issue. I can handle the Joomla/PHP aspect if we can just get FB to return the proxied email.

Ideally we should be able to include the permission request in the initial "connect" request - but I am just not sure yet how to do that.

Regards,

Eddie

[Alexander@WS]

Hi Eddie,

Thanks for the contribution! Please send them to support at webscribble.com.

[kindoflost]

I think another user already sent some implementation for that. Look at this: Weird email showing for registered user Why?
He might be able to help with what you are asking for.

[dirthog]

Communicating with Users via Email - Facebook Developer Wiki