there is few points - share app id and initialization of js sdk. What do you think about it?

CommentFileSizeAuthor
#16 fb_social-solve fbconnect conflict with drupal_add_js-852582-D6.patch1.16 KBdavidajnered
#13 fb-modules-integration.patch4.25 KBvectoroc
#12 fb-modules-integration.patch4.12 KBvectoroc
#4 fb_social.patch437 bytesvectoroc

Comments

HS’s picture

HS commented

+1

ferdi’s picture

ferdi commented

I'd be happy to help. I havent used fbconnect and I dont quite know how it works. As other users have requested #787648: New facebook login I am planing to implement some single sign-on functionality using fb's login button (http://developers.facebook.com/docs/reference/plugins/login, http://developers.facebook.com/docs/guides/web#login).
What do you have in mind about the integration between fbconnect and fb_social ?
thanks!

vectoroc’s picture

vectoroc commented

I want to remove functions of social plugins from fbconnect. About sharing - perhaps I can do it through hook_form_alter. We need to solve way to initialize FB sdk. I see few ways :

1 - you add next code to your hook_footer jQuery.event.trigger('fb:init'); (in this way I have to overwrite fb_social_appid and fb_social_locale variables that's not good)
2 - you check some conditions and dont print footer if there is already exists modules that initializes FB sdk

vectoroc’s picture

vectoroc commented
Status: Active » Needs review
StatusFileSize
new fb_social.patch437 bytes

I've created submodule that integrates fbconnect & fb_social

One thing that I need from you is attached here

sohlex’s picture

sohlex commented

I beg you to make this official!

ferdi’s picture

ferdi commented

Committed, http://drupal.org/cvs?commit=400204
I have no idea what it does. Please test. And to be honest I'd do it differently. We need a fb_api module that load all the needed libraries and other modules have it as a dependency
thanks!

vectoroc’s picture

vectoroc commented

It allows me to call own initialization procs after initialization of fb js sdk

$(document).bind('fb:init', function() { ... })

ferdi’s picture

ferdi commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

liquidcms’s picture

liquidcms commented
Category: task » bug
Status: Closed (fixed) » Active

I have been having a lot of trouble getting fbconnect to work and then realized this weekend that it was due to an incompatibility with the fb_social module.

i tried using the fbconnect-fb_social comp module that comes with fbconnect but no luck. I also just tried the latest dev version of fb_social and verified that the patch here had been applied - still no luck.

so as far as i know.. i have latest devs of everything: both these modules as well as the fb sdk.

vectoroc’s picture

vectoroc commented
Title: Integrate with fbconnect » Integrate with another fb* modules

I think that much better way to integrate is way like facebook_stream does

vectoroc’s picture

vectoroc commented
Issue tags: +fbConnect, +Drupal for Facebook
StatusFileSize
new fb-modules-integration.patch4.12 KB

a simple patch

vectoroc’s picture

vectoroc commented
Status: Active » Needs review
StatusFileSize
new fb-modules-integration.patch4.25 KB

forgot to catch hook_footer

HS’s picture

HS commented

So, fb_social and fbconnect don't work together yet?

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented

I've applied the patch and fb_social (I only tested likebox) works and I can login using drupal for facebook, but the form in the settings of the fb_social that allows me select the "other facebook module" yields a very ugly js error.

davidajnered’s picture

davidajnered commented
StatusFileSize
new fb_social-solve fbconnect conflict with drupal_add_js-852582-D6.patch1.16 KB

The conflict with fbconnect can be solved by adding the javascript with drupal_add_js. I have only tried with the likebox, so the patch needs to be tested more thoroughly.

D.H.’s picture

D.H. commented

If nothing of the above mentioned works, it's also possible to exclude the content of fb_init and fb_footer from all pages except "/user/login", "/user/register" and "/logout". Normally you wouldn't have a fb_social like button on those paths. The only bug with this solution is that you need to call "/user/login" or "/user/register" to get logged in again on your site when you are already logged in with facebook.

socialnicheguru’s picture

socialnicheguru commented

The previous comments were about Drupal 6.

Is there a Drupal 7 fix?

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Issue summary: View changes

Closed two duplicates: #1293826: Add ability to not include fb-root settings for compatibility with other Facebook modules and #1505274: conflict with modules/fb