Closed (won't fix)
Project:
Drupal for Facebook
Version:
7.x-3.x-dev
Component:
Miscellaneous
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Reporter:
Created:
19 Apr 2009 at 17:41 UTC
Updated:
22 Sep 2011 at 18:17 UTC
Hello,
I just found fb connect http://drupal.org/project/fbconnect module.
Any plans of merging these?
Comments
Comment #1
Dave Cohen commentedNo plans. I'm open to it, but I don't consider it a bad thing to have more than one "competing" project.
Is there some feature of that module in particular that you'd like included in Drupal for Facebook?
Comment #2
giorgio79 commentedThanks Dave, will let you know if find anything intersesting in there :) To be honest, so far I was just reading its documentation together with those of Drupal for FB, to make sure I make the best decision for my fb app, but it seems to me Drupal For Facebook is already or will support this feature.
I am marking this as fixed for now :)
Comment #3
najibx commentedfb got canvas page ..which is cool !
Comment #4
Dave Cohen commentedDrupal for Facebook also publishes to wall and feed, which I think is one of the most useful parts of Facebook Connect. My goal is to support all important features of facebook APIs, but it will take some time to cover them all.
Comment #6
phpepe commentedHello,
I'm working on a project that has already implemented and configured the fbconnect module for some features.
But now, my task is to integrate my drupal site with fb canvas pages.
Currently I found on fb project a great module for this, at works very well all the releases I’ve tested.
But there is a problem running together "fb" with "fbconnect" module: "Fatal error: Cannot redeclare class facebook in /var/customers/webs/tango/sub01/sites/all/modules/fb/facebook-platform/php/facebook.php on line 37".
The reason is that both projects includes the same library (containing the same classnames, etc) but in different files. So if facebook class was already loaded by one of this modules, the second module that try to make the include will throw the error.
I know that this project contains most of features implemented in “fb”, but really I want to keep working functionalities intact, so I need to combine these 2 modules.
Working around, this is what I changed at “fb” module to avoid this error:
fb.module Line 157:
change:
to:
and changes to "fbconnect" module are:
fbconnect.module line 327 (aprox):
Hope it help someone else, and it would be a great idea if you find a way to share Facebook libraries, between independent facebook modules.
Regards.
PHPepe
Comment #7
Dave Cohen commentedThe README.txt is as clear as it can be about this. You can add a line to settings.php telling fb where to find the libraries. Depending on where you've put the libraries, something like
So, just keep one copy of the libraries.
Comment #8
phpepe commentedThanks Dave.. Was my error not reading carefull the readme file.
This looks better than having the library on each facebook module.
Comment #9
phpepe commentedAnyway I'will continue with my patch because the other module search for the facebook library under its own subdirectory..
Thanks for the quick response.
Comment #10
giorgio79 commentedHey Dave,
Wanted to revisit this issue in light of FBML being deprecated (http://stackoverflow.com/questions/4727413/fbml-fbjs-xfbml) . It seems all emphasis is placed on JS SDK, Facebook Connect and iframe apps hence the boundaries between fbconnect and this module should be blurring. What do you think?
Comment #11
Dave Cohen commentedIts true that with canvas pages now iframes, it's less of a crazy thing to support them. More drupal-like. I don't know the other module well and I don't know what it would take to "merge" them. Probably a lot of work.
I'd be interested in patches that allow the two to work together. My guess is that if you enabled both, you'd get facebook's JS SDK added to the page twice. Might be a simple as adding an option to not initialize javascript.
Comment #12
giorgio79 commentedThanks Dave for the input. Meanwhile I collected some more info at http://facebook.stackoverflow.com/questions/7516591/is-facebook-connect-..., so the php sdk is still useful and it should be used together with FB connect.
Will set this as wont fix then.