Info from changes.txt

- Supports SSL, http://wiki.developers.facebook.com/index.php/Facebook_Connect_Via_SSL.
- User profile pictures now are loaded through FBML (from DFF module - drupal.org/project/fb)
- Some addition settings (check at admin/settings/fbconnect)
- Most of js code is moved to fbconnect.js
- New hooks: hook_fbconnect_config_alter, hook_fbconnect_login_button_alter

Patches from community:
- http://drupal.org/node/483282#comment-2118418
- http://drupal.org/node/367244#comment-2117946
- http://drupal.org/node/574210
- http://drupal.org/node/605174

Comments

vectoroc’s picture

StatusFileSize
new30.33 KB

here is some bugfixes

vectoroc’s picture

StatusFileSize
new46.03 KB

+ Ability to link/unlink Facebook acount from user profile

vectoroc’s picture

StatusFileSize
new49.06 KB

some bugfixes

vectoroc’s picture

StatusFileSize
new26.35 KB

forgot to remove debugging

gregarios’s picture

This new patch from Comment #4 works very well.

Hey, while you're at it, could you make a <span class="fblogintext"></span> around the "Sign in using Facebook" text next to the login button in the fbconnect.module file? Or even remove it completely. I can't get rid of it using CSS because whatever I do to try to remove it also removes the login button.

Basically just change the Sign in using Facebook text be <span class="fblogintext">Sign in using Facebook</span>.

I've been going into the file and just deleting the text manually, but that is kinda hacky.

WildBill’s picture

I would like to echo Gregarios' request in comment #5...

vectoroc’s picture

StatusFileSize
new766 bytes

Check this. I believe that is more flexible way. With it you can fully configure fbconnect button through hook_form_alter or style sheets

gregarios’s picture

Check this. I believe that is more flexible way. With it you can fully configure fbconnect button through hook_form_alter or style sheets

Actually, we're having problems modifying the text next to the button, using CSS. Modifying the button itself is not a problem.

vectoroc’s picture

I'm not sure that I understand you. Now Sign in using Facebook is wrapped with <div class="description"> and you can access it through #fbconnect_button-wrapper .description

E.g. hide it - #fbconnect_button-wrapper .description {display: none}
(Same through hook_form_alter - unset($form['fbconnect_button']['#description']); )

Give a sample if it's not enough

gregarios’s picture

Oh yes... I applied the previous patch again by mistake. Your new patch should work. Can you combine them again into a new "all-in-one" patch? :-)

vectoroc’s picture

Actually you can download it from http://github.com/vectoroc/fbconnect

gregarios’s picture

StatusFileSize
new2.51 KB
new14.33 KB

Uh oh... Problem with patch from #4: It breaks IMCE Upload functionality somehow. The window containing the upload files still seems to be intact, but the upload form is somehow hidden in it now. Attached is the source code of the IMCE Upload window for reference.

Turning off Facebook Connect restores functionality, then re-enabling it breaks IMCE again.

The attachment named "IMCE_Form_Broken.txt" shows the source code of the page that is broken.
The attachment named "IMCE_Form_Working.txt" shows the only the source code of the working page that is different from the broken page.

vectoroc’s picture

It's not informative. I can't reproduce the bug.

gregarios’s picture

It's not informative. I can't reproduce the bug.

Sorry... its the best I can do right now. There are no PHP errors or any errors registered in the logs. It seems to be something getting overridden in the CSS maybe.

I downgraded to the standard Beta8 and the bug is gone.

vectoroc’s picture

can you give me acces to page where error occurs?

gregarios’s picture

StatusFileSize
new727.89 KB

can you give me acces to page where error occurs?

Not really... However... I've narrowed it down a lot. The attached image illustrates my plight. The window on the upper left is what the file browser should> look like. The window on the right is what I get. The small window showing your fbconnect.js file is the key. IF the red text portion of it is removed, the browser works correctly as in the upper-left window. If it is left in, the right window is what happens. It seems all image previews are removed as well now that I notice.

vectoroc’s picture

Thx, I found problem on page /imce and fixed it. Download last version which is here

gregarios’s picture

Status: Needs review » Reviewed & tested by the community

That's it. Your update works great. All the functionality and no problems.
How can we get this turned into a Version 1.0 release? :-)

nirad’s picture

Vectoroc will need maintainer privileges to create a 1.0 release. Are the current maintainers still maintaining the module at all?

gregarios’s picture

Are the current maintainers still maintaining the module at all?

It would seem not, as the last commit was over 21 weeks ago...

Can someone step in and assume control?

guedressel’s picture

+1 from me.

Just launched a new Website including fbconnect.
Would love to see the latest contributions in upstream repositor and new supported releases.

ambo’s picture

Hi vectoroc,

great work! I think about using this fork for my fork (#617778: Facebook depricates template bundles and their APIs...) - instead of the orignial module. We might coordinate our future work on this to avoid to much different forks...

Are you currently working on this?

vectoroc’s picture

Yes, I'm. Good idea. Are you talking about fork on github or drupal-module fork ?

ambo’s picture

I thought about a drupal module fork. but we will have to put the module here in one of our issues as we don't have cvs permissions for this module and - in my case - not even cvs permissions at all...

So I will use your latest fork for my developement and we bring our both forks together when we're finish??

Could this be the plan?

gregarios’s picture

If you start working on Github, please make announcements of new versions in this thread if you would. :)

yeeloon’s picture

+1 subscribing

WildBill’s picture

+1 for rolling this into a new Drupal 1.0 version...

Carlos Miranda Levy’s picture

Quick question, just to make sure...
Patch above is a few days older than latest version at github.
It seems better to get it from there.
Steps would be:
1. Download fbconnect module from Drupal Project Repository at http://www.drupal.org/project/fbconnect
2. Get Facebook Platform from http://svn.facebook.com/svnroot/platform/clients/packages/facebook-platf...
3. Get updated code from http://github.com/vectoroc/fbconnect and copy it over the sites/all/modules/fbconnect files overwriting old files and keeping unchanged ones.
4. Run update.php ?

Would that do it?

vectoroc’s picture

vectoroc’s picture

I don't think that is good idea to create one new module. Better thing is to give ownership to this project.
http://drupal.org/node/652968
So I don't want to be maintainer of this project because for me it's temporary work

gregarios’s picture

So I don't want to be maintainer of this project because for me it's temporary work

A temp maintainer is better than no maintainer in my book. I would LOVE to see you get it and update it even if it is just to get a new bug-fixed release version out. Looks like it is almost there in patches. If you decided after a week to put a note up abandoning the project again, I for one wouldn't hold it against ya. :-)

ambo’s picture

I agree with you vectoroc. I don't want to be maintainer here if the changes with the api are done...

Thats a +1 for github, is'nt it?

WildBill’s picture

I'd never even heard of github before this thread. If the updated module is "kept" over there, the potential user base will be severely restricted. It just seems like a shame to not keep it here at Drupal.org.

ambo’s picture

I don't think its that bad because you will find the download link right here and there is no need to visit github for downloading the newest version.

The only thing that won't work is a message in drupals module update status.

Greetings - Andreas

akongz’s picture

*deleted*

akongz’s picture

setting auto login & logout not working?

vectoroc’s picture

Check latest commit. It should work

akongz’s picture

@vectoroc

ow soryy I missed this

7 .Edit the "page.tpl.php" file of your current theme. edit the tag and add the facebook xmlns for rendering XFBML.

	  example : <html xmlns="http://www.w3.org/1999/xhtml" xmlns:fb="http://www.facebook.com/2008/fbml">.
tsi’s picture

This is insane !
This version is so much better then the last beta !
I had to go with gigya and back to fbconnect just to accidentally find out about this version.

I don't think its that bad because you will find the download link right here and there is no need to visit github for downloading the newest version.

The only thing that won't work is a message in drupals module update status.

This is not true. people are not looking for modules on github. they are looking for them on drupal.org/project/modules and on drupalmodules.com, they will probably never find it elsewhere unless it is at least mentioned on the project page.
god only knows what would happen if we had to go through all the issues lists to find the best versions of every module.

vectoroc, thank you very much for your hard work ! I hope it will be an official release soon.

vectoroc’s picture

Happy new year!
I'll release next beta in next year

WildBill’s picture

Vectoroc, could you apply to be a temporary maintainer like Gregarios suggested in comment #31? Then you could release the next beta and then stop being maintainer if you wish......

ambo’s picture

Hi to all,

the new version at github is now with lots of changes by vectoroc - and some of mine.. The new facebook api methods as described in #617778: Facebook depricates template bundles and their APIs... are not fully implemented - by facebook, so at the moment there is no way to post pictures or actionlinks to another facebook wall. facebook also moved the final depricate state to "early 2010".

In detail the changes are:

  • themeable user picture if you use the facebook user picture
  • allow/disallow the use of facebook user picture
  • new api-methods for wall-postings (translateable)
  • set your prefered language for facebook at the admin to get translated facebook prompts
  • facebook connect button is now customizable with own text
  • removed or rearranged some items in admin interface

If anything is wrong or missing just post it here.

regards - andreas

alexfisher’s picture

subscribing

Great patch/release vectoroc.

joetsuihk’s picture

i agree #31. temp maintainer is better than none. Release in D.O have the cons that release got noticed by update_status.module.

anyone contacting this module's maintainer?

.Raj’s picture

Hi Vectoroc, I could not log in to my drupal site using the facebook account. The popup window is coming and when i provide my facebook account details it is redirecting to my drupal site which is not logged in still. But when I open facebook it will be in logged in status. I have applied the patches you have provided and did a couple of re installation but nothing helped... pls advice

Thanks for the module..

e0ipso’s picture

+1 subscribing

Aquilasfx’s picture

I have an issue with your last patch, avatar not works. Users when register a new account with facebook, then no imaage avatar is updated.... with original release that functionality works... why?

vectoroc’s picture

Assigned: Unassigned » vectoroc
Status: Reviewed & tested by the community » Closed (fixed)

Aquilasfx: avatar displaying had been rewritten, there might be some reasons why it does not work. Firstly check readme.
dzire: check readme too

If problem still appears them create new issue please.
Thanks

Closing this issue. Actual version from repo is released to beta9.

marcus178’s picture

I was using this version as in resolved the ssl problem on secure pages but I installed the secure pages prevent hijack module and it seems to have stopped working even after uninstalling secure pages prevent hijack module. Any suggestion on why this happening?