Port Facebook Connect to Drupal 7

tracking

I've created 7.x branch in github, but I did'nt test it. Fbconnect module should be rewritten cause a lot of changes in FB API

Is the correct repo - http://github.com/voxpelli/drupal-fbconnect ?

subscribing

@vectoroc I've forked your repo and have started a little work. As mentioned in my first pull request on github i'm pretty new to github collaboration so please go easy on me. Thanks.

I was planning on giving a little help getting a module to 7 but I see in http://drupal.org/node/788118 that the 6.x of this needs some love. I'll be working on that for the time being.

Would be great to have a D7 version, but I agree, getting the 6x some love is a good idea

BenK: I think that new fb api does not belong to facebook connect (and this module). Only things that should be done before D7 porting is bug fixing...

BenK: I think that new fb api does not belong to facebook connect (and this module). Only things that should be done before D7 porting is bug fixing...
#788118: This module needs to be updated

Any update? Would be great to have this ready for the first D7 beta..

Tag and sub

subscribing

subscribing

subscribe

any news about D7 version?

duplicate post (feel free to delete)

subscribing

Subscribing

Subscribing

subscribing

subscribing

+1

subscribing

subscribing, too

waiting for this module in D7..

any news?
is there any other module or alternative way to do the same job?

subscribing

@vectoroc: I've just forked your repo on github. How can I help in order to port this module to D7?

Thanks

subscribing

subscribing

@davividal: see 7.x branch. but porting is'nt started yet
btw this branch is outdated a bit

subscribing

subscribing

+1 subscribe

subscribing

sub

I'm working on this actively. I will post a link to my github fork soon.

Github was way out of date.

One patch ignores all whitespace changes, for easier readability.

Mostly changes to function signatures, and to DBTNG.

Was missing a parenthesis. That's what I get for uploading the patch before testing.

I am really in need of a D7 version of fbconnect. Let me know if I can be of any help.

I am new to the Facebook API and I've just started using D7 but I'm not new to PHP or Drupal.

@tim.plunkett: github version was most up to date (if I got correctly your thoughts)
I will review your patch as soon as I can

subscribe

@tim.plunkett: in what order do I apply your patches? or just last one?

Just the last one, to a CVS chectkout of DRUPAL-6--2.

subscribed

subscribing

subscribing

subscribing

subscribing

subscribing

subscribing

subscribing

Instead of just subscribing, how about someone review the patch in #41?

Also, keeping the module name in the issue makes it easier to discern in the tracker.

I tried the patch and it's not really working for me. For an anonymous user that's not logged in to the site, I get the following notices:

    * Notice: Undefined index: theme path in theme() (line 789 of /Users/jen/Sites/drupal7/includes/theme.inc).
    * Notice: Trying to get property of non-object in fbconnect_theme_user_picture_override() (line 814 of /Users/jen/Sites/drupal7/sites/all/modules/fbconnect/fbconnect.module).
    * Warning: Attempt to assign property of non-object in fbconnect_theme_user_picture_override() (line 815 of /Users/jen/Sites/drupal7/sites/all/modules/fbconnect/fbconnect.module).
    * Warning: Attempt to assign property of non-object in fbconnect_theme_user_picture_override() (line 816 of /Users/jen/Sites/drupal7/sites/all/modules/fbconnect/fbconnect.module).
    * Notice: Trying to get property of non-object in fbconnect_theme_user_picture_override() (line 819 of /Users/jen/Sites/drupal7/sites/all/modules/fbconnect/fbconnect.module).
    * Notice: Trying to get property of non-object in fbconnect_theme_user_picture_override() (line 820 of /Users/jen/Sites/drupal7/sites/all/modules/fbconnect/fbconnect.module).
    * Notice: Undefined variable: closure in include() (line 20 of /Users/jen/Sites/drupal7/sites/all/themes/framework/html.tpl.php).

The last notice, "Undefined variable: closure", is probably caused by adding print $closure to the end of my template as per the instructions in the README.

The module inserted the following HTML into my login block, but no actual active link:

<div class="form-item form-type-item" id="fbconnect_button">
    <div class="description">Sign in using Facebook</div>
</div>

I used patch #41 on the dev version of the Facebook Connect module, and the module is on a clean install of drupal on my local machine. The Application ID and Application Secret were set and the XFBML tags were in place in the html tag at the top of my template. The only thing I'm having doubts about is the "Connect URL" because the settings page never displayed one... I just guessed and used the URL for the root of my site.

subscribing

subscribing

subscribing

subscribing

subscribing

subscribing

Subscribing, will start to help test or write tests on a part time basis.

subscribing

Re-rolled the patch in #41 against the current 6.x-2.x branch head for further testing. Not done any testing myself though...

commited to the 7.x-2.x branch

Hello,

I've installed the 7.x-2.x dev version on my site and followed all the instructions (which also must be updated for a formal release of the module: the html tag, referred on the step 6 of the readme.txt, is not defined on page.tpl.php anymore: now it is defined on html.tpl.php, which can be found in the system folder).

Still, I could not make the module work: I get the same errors as #58. I've tried changing the theme, but still couldn't solve it. Any ideas of what could be happening?

Thank you in advance,

subscribing

I ran the module through Coder and saw that it still implements hook_footer(), which is deprecated for D7. I tried to convert it to hook_page_alter() per the D7 requirements. Attached is my first patch contribution to a Drupal module; please be gentle!

Good catch! Sorry I missed that. I'd recommend using git to create patches if you can - it's the future! The full system file paths at the top of your patch file are preferred to be relative paths to the project root - using the git workflow means you don't have to think about it.

Also, not totally sure, but I'd have thought it's worth looking into using drupal_add_js to add the js instead of hook_page_alter...there's a scope parameter that allows you to say it should be placed with the other footer javascript. sorry I've not time to work on the patch!

The tag should be in html.tpl.php, I believe.

Thank you everybody for your great work on getting this ported to 7 - much appreciated! Like #70 I'm also, unfortunately, not able to get it to work. I also used ausrick's excellent suggestion from #78 and verified that the correct rdf namespace does indeed render correctly.

Here's a working version. I'm sure there are some bugs still, but basic functionality works fine.

Using the last module from #80 I got :

Error
The website encountered an unexpected error. Please try again later.
Error message

    Notice: Undefined index: email in fbconnect_register_page() (line 95 of /home/dev/drupal-7.0/sites/all/modules/fbconnect/fbconnect.pages.inc).
    You must enter an e-mail address.
    Recoverable fatal error: Argument 1 passed to drupal_http_build_query() must be an array, string given, called in /home/dev/drupal-7.0/includes/common.inc on line 2157 and defined in drupal_http_build_query() (line 473 of /home/dev/drupal-7.0/includes/common.inc).

@ausrick give me a hint how to make a patch for whole module, e.g. multiple files. I'm too lazy to figure it out now. ;)

@ausrick, thanks! I've created as clean a patch as possible. JS file is not changed that much, it's a big part of patch because of whitespace indentation.

Known issues:
- If you log out of Facebook (in other window), then refresh Drupal page a couple of times, fbconnect settings will lose connection information and ask again to login with Facebook to link your account (even though it is still linked in database). Somehow D6 version of fbconnect manages to keep that facebook session even if you log out of Facebook.
- Of creating account, fbconnect settings (fb_visible, fb_avatar) are not saved.
- If you choose to use fb avatar as your user image, user profile form still allows to upload image (pointless).
- Why store fbuid in both database mapping table and user object (second is redundant)?

subscribing

Hi all.

I don't see "Connect URL" in my settings screen, i found just a label, but value isn't there.
The connect button on User edit page do nothing.

Could you help me?

Another version. Two first issues from #85 have been fixed.

subscribing.

subscribing

Hey mikgreen - thanks for your work on this! I tried your extensive patch from #88, and could finally get it working! Thanks!!

Some misc feedback:

Settings pages:
On the settings page (admin/config/fbconnect) - the field "Connect url" is empty, and looking at the code (around line 62 in fbconnect.admin.inc) the form element "#type" is "item" and uses "#value" instead of "#markup" for displaying of the info (which resulted in nothing begin displayed).

On the "Apperance page" the setting "Disable linking accounts during registration" doesn't seem to do anything

The page at "admin/config/fbconnect/fb-app" is completely empty

User account FB settings page (user/UID/edit/fbconnect):
There's a link there to the "User settings" page - the whole thing should probably be removed if profile pics are disabled on the site

Again, thanks a lot for your work - its really appreciated!

... and for the noobs ....

Would it be possible to provide a new 7.x-2.x-dev version with all the patches included, so it is easier for me to update it?

Thanks in advance..

I second KirstenLangholz's post.

I am not having success patching the fbconnect 7.x-2.x-dev posted by vectoroc on March 16, 2011 at 4:56pm, and this module is really valuable to D7 sites, being that it's might more lightweight than Drupal for Facebook which is much more of a beast.

I've gotten the Drupal7_port_fix-795128-85.patch to work using:
patch -p1 -b < Drupal7_port_fix-795128-85.patch (no error messages)

But, I get errors with Drupal7_further_fixes_795128-88.patch file.

And, I'm probably not qualified to contribute to the development process, but here goes:

On line 124 of "fbconnect.pages.inc", there's an invalid function, "db_result" (D6 function replaced with fetchField() in D7), is used... shouldn't line 124 be:

$result = db_query($query, $name_suggestion)->fetchField();

Subscribing.

subscribe

+1

For those who have successfully patched this file, could you post the working module somewhere on the internet for those of us that keep getting failed hunk messages?

Every single patch file in this thread gives me fail notices with the most recent version of the module.

Notice: Trying to get property of non-object in fbconnect_user_identities() (line 174 of /home/content/46/7809246/html/sites/all/modules/fbconnect/fbconnect.pages.inc).

I'm getting this as I try to connect facebook to a preexisting account. Don't know if this bug needs to be addressed or if it's already been addressed currently.

Subscribing

subscribing

subscribing

subscribing

+1
subscribing

+1 subscribe
Keeping an eye out

*deleted*

subscribe

subscribe

Here's a re-roll of #88.

Changes since #88:

Fixed the #markup form fields on the settings pages and moved the settings to config/people/fbconnect.
I'm not sure about all the JS changes, since they are all from mikgreens patch in #88.

Problems I'm seeing:

Can't create new account - email address isn't fetched.

Better patch that also fixes the issue with getting the email address.

Patch is full of tabs and traling whitespaces

Removed tabs and trailing white spaces.

Hi, I have patched fbconnect with the patch of #111 for peoples who don't use "GIT".
It work for me.

Version patchée avec le patch du #111 pour les personnes n'utilisant pas "GIT".
Fonctionnel.
Good look.

patch applies cleanly. @Adibou unless you plan to maintain this, you shouldn't assign yourself to this bug as it requires further testing before a RTBC

I have been doing some work on this module to get it working.

Do these patches ever get rolled into the dev build? There are a lot in this thread now and it may make sense to push them and work from latest.

I have been doing some more work on this module to get it working. navigate to fbconnect/register/prompt
to give it a test
This patch is to be applied to the 7.x-2.x head and includes the previous patch

I have been doing some more work on this module to get it working. navigate to fbconnect/register/prompt
to give it a test
This patch is to be applied to the 7.x-2.x head and includes the previous patch

Oops wrong patch on previous post (it contains some bugs) please use this one

Zip file with patch applied for people without git.

Been doing some more to get this working on drupal 7. The facebook api that is being used is v2.1.1 this will need to be updated some time since v3 has been released. Put facebook.php v2.1.1 in the fbconnect module directory directly so it is available at fbconnect/facebook.php
Module now allows users to log into website using facebook, also allows users to link account on facebook with account on website.
find latest patch attached. This patch includes all patches and will need to be applied to the 7.x-2.x trunk

Hi Jagermonster,

Is there a difference in the last patch and the zip file you posted a few hours earlier.

I'm still experiencing errors on connect and also the error:

Notice: Use of undefined constant BLOCK_NO_CACHE - assumed 'BLOCK_NO_CACHE' in fbconnect_invite_block_info() (line 66 of /var/www/vhosts/mysite.com/httpdocs/sites/all/modules/fbconnect/fbconnect_invite/fbconnect_invite.module).

Be great if we can get this running.

Appreciate your work
B.

bnicoll at #121 yes this latest patch has some fixes mostly javascript fixes. i have attached a zip file, the facebook.php library is in the zip file.

Thanks jagermonster,

I've ditched it for the time being and going to try using the janrain/rpx module as suggested in post #62.

Thanks
Ben.

great, just needed something functional for the time being.

hi jagermonster,
I have used your given zip module in #123.with that many of the issues are solved.But on the login page
the fbconnect button is not still present.Please help...
Thanx in Advance.

Also the takeover of the facebook profile picture doesn't work :-(

yes i know, only login and account creation.
will see if i get some more time i might have a look at the picture support

#126 Ashu if you would like to add a link to the registration form use the hook_form_alter function to add it to the registration/login form
here is an example
/**
* Implements hook_form_alter().
*/
function YOUR_MODULE_NAME_form_alter(&$form, &$form_state, $form_id){
//add facebook login button to login form
if($form_id == 'user_register_form' || $form_id == 'user_login' || $form_id == 'user_login_block'){
//get facebook info
$conf = fbconnect_get_config();
//check if facebook api key and secret is set
if(!empty($conf)){
$reg_msg = t('Log in with Facebook');
$reg_attr = array();
if ($conf['fast_reg_mode']) {
$reg_attr = array('attributes' => array(
'onclick' => 'Drupal.fbconnect.DoFastRegistration(this); return false;'
));
}
$form['fb_login'] = array(
'#prefix' => '

',
);
}else{
drupal_set_message('Facebook api key and secret not set');
}
}
}

Closed #1200044: Does it actually work on 7.2? as a duplicate.

This might be a slightly more sane version of _facebook_client_load_include() and would waste much less effort doing file checking:

function _facebook_client_load_include() {
  // TODO: add full Libraries API support.
  if (!class_exists('Facebook')) {
    $paths = array(
      DRUPAL_ROOT . '/sites/all/libraries/facebook',
      dirname("__FILE__"),
    );
    foreach ($paths as $path) {
      $path .= '/facebook.php';
      if (file_exists($path)) {
        include $path;
        break;
      }
      else {
        watchdog('fbconnect', t("Unable to load the required Facebook library, please check the README.txt for instructions on how to resolve this."));
      }
    }
  }

  return class_exists('Facebook') && defined('Facebook::VERSION');
}

Slight fix for the code in #131 that doesn't leave a watchdog message just because the file doesn't exist in sites/all/libraries. Also, the correct check is dirname(__FILE__), not dirname("__FILE__").

function _facebook_client_load_include() {
  // TODO: add full Libraries API support.
  if (!class_exists('Facebook')) {
    $paths = array(
      DRUPAL_ROOT . '/sites/all/libraries/facebook',
      dirname(__FILE__),
    );
    foreach ($paths as $path) {
      $path .= '/facebook.php';
      if (file_exists($path)) {
        include $path;
        break;
      }
    }
    if (!class_exists('Facebook')) {
      watchdog('fbconnect', t("Unable to load the required Facebook library, please check the README.txt for instructions on how to resolve this."));
    }
  }

  return class_exists('Facebook') && defined('Facebook::VERSION');
}

It might be worth adding a link on the API settings page to the API registration page, e.g.:

function fbconnect_api_keys_settings($form, &$form_state) {
  $form['fbconnect_appid'] = array(
    '#type' => 'textfield',
    '#required' => TRUE,
    '#title' => t('Application ID'),
    '#default_value' => variable_get('fbconnect_appid', NULL),
    '#description' => t('Also called the <em>OAuth client_id</em> value on Facebook App settings pages. <a href="https://www.facebook.com/developers/createapp.php">Facebook Apps must first be created</a> before they can be added here.'),
  );

  $form['fbconnect_skey'] = array(
    '#type' => 'textfield',
    '#required' => TRUE,
    '#title' => t('Application Secret'),
    '#default_value' => variable_get('fbconnect_skey', NULL),
    '#description' => t('Also called the <em>OAuth client_secret</em> value on Facebook App settings pages.'),
  );

I also included the "OAuth" field names, to help site builders identify what strings to add where.

Missing feature:

• Connecting Facebook accounts to existing user accounts (user/$uid/edit/fbconnect).

Some existing code is there, presumably from the 6.x branch, but it's commented out.

thanx jagermonster for replying...
But i have done that on the same day Using This Patch

@@ -214,7 +222,7 @@ function fbconnect_form_alter(&$form, $form_state, $form_id) {

// Render fbconnect on login block.
case 'user_login_block':
- if (!empty($_GET['destination']) && ($_GET['destination'] != 'fbconnect/link') && facebook_client()) {
+ if ( ((!isset($_GET['destination']))||($_GET['destination'] != 'fbconnect/link')) && facebook_client()) {

#848354 : ask user for email permission when account linking is disabled
$attr = array();
@@ -225,8 +233,8 @@ function fbconnect_form_alter(&$form, $form_state, $form_id) {
$form['fbconnect_button'] = array(
'#type' => 'item',
'#description' => t('Sign in using Facebook'),
- '#value' => fbconnect_render_button($attr),
- '#weight' => 1,
+ '#markup' => fbconnect_render_button($attr),
+ '#weight' => 200,
'#id' => 'fbconnect_button',
);
}

Subscribing.

As there is a D7 release for this module already published, shouldn't we treat the individual issues/improvements separately?

Hi, I'm having the same problem as mentioned in #98
Notice: Trying to get property of non-object in fbconnect_user_identities() (line 174 of /home/content/46/7809246/html/sites/all/modules/fbconnect/fbconnect.pages.inc).

Did anyone find a solution ?

Is anyone getting this after #129 and #135?

PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table '.users' doesn't exist: SELECT COUNT(name) FROM users WHERE name = ':user_name'; Array ( [user_name] => ) in _fbconnect_get_username() (line 123 of ......sites/all/modules/fbconnect/fbconnect.pages.inc).

It seems like the answers are here in some form, a release would be so money. If a new release is provided, I'd be happy to test it out. (I'm not able to do any solid development yet.)

Subscribe

responce to #98 and #138, i created an account on the drupal site. then navigated to user/#uid/edit/fbconnect and used the fbconnect to connect my account with facebook. It worked fine. Try using the module attached to #123 and let me know if you still get the error and the exact steps you used.

Here's a patch with the changes suggested by Damien and also some fixes for stream module.

I have fixed the facebook profile picture support.
This patch includes 795128-Port_fbconnect_to_d7-142.patch and all previous ones by me on this page.

Updated the README
This patch is applied to the 7.x-2.x-dev branch and includes all patches on this page

Readme.txt update patch

subscribe - thanks for everyone's contributions!

What is the status of this?

I tried applying the patch in #144, a few of the attempts fail, but seemingly just for already present corrections.

However, site will not load after, b/c of error:

Call to a member function getSession() on a non-object in .../sites/all/modules/fbconnect/fbconnect.module on line 407

This isn't the typical getSession() error (using the correct 2.x version of the facebook php libs). It only occurs after applying the patch in #144.

Does anyone have this module in working condition under D7?

Okay, applying the later patches to the official 7.x-2.x-dev doesn't seem to work.

This is because one of the above patches moves the expected location of facebook.php, so either be prepared to move it into fbconnect/.

Grabbing the version in #123 gets fbconnect back up and running, but still doesn't seem to work, although it's the most fully working copy yet. Everything seems ready to go, except I never get prompted to authorize the app, and I'm not seeing a login button provided anywhere. Something still isn't complete, but I'm not sure whay.

Dead end again, for now.

The appropriate facebook SDK to use is here, https://raw.github.com/facebook/php-sdk/v2.1.2/src/facebook.php. Put this in sites/all/libraries/facebook/facebook.php and it should rid the getSession error.

Exception thrown while using admin_setAppProperties : The method you are calling must be called with an app secret signed session
Notice: Undefined variable: res in fbconnect_fbapp_settings_submit() (line 289 of /modules/fbconnect/fbconnect.admin.inc).
Checking your facebook session, you are not allowed to change this application settings

This is my error, any clue?

Patch to fix an issue with users that have an account on the website and then afterwards try to create an account with fbconnect. Will have to manually apply this patch i did not create it with git

Ignore previous patch!! That solution dont work.
Find attached my latest version of the fbconnect module in zip format.
very busy atm will create patch when i have some free time.

Patch with all my changes and bug fixes from the current 7.x-2.x dev branch.

Last patch contained a bit of custom code
//remove fblogin button
unset($form['actions']['fb_login']);

that should not be in the module
this is the correct patch.

Hi jagermonster,

When i enable the new fbconnect.zip you uploaded i get this.

Fatal error: Call to a member function getSession() on a non-object in /home/juegosp1/public_html/xxxxx.com.ar/sites/all/modules/fbconnect/fbconnect.module on line 434

Someone know what is/i'm doing wrong? Any idea? The entire site stop working. Work again only after deleting the module.

Disabled some other facebook modules and working now

@RaulOne #155 - what version of facebook.php are you using? You have to use the version 2.1.1 or 2.1.2 facebook.php not the Facebook PHP SDK (v.3.1.1)

I have gotten past most of the errors (or at least I think I have), the last big hurdle was making sure the facebook.php was in the fbconnect directory and not in the src directory. At that point, (after clearing cashe) I got an “invalid URL” message when trying to connect to fb; the App ID and Secret code were correct. To get past this I had to go to My Apps on Facebook and set “Remove Deprecated Auth Methods” to disabled. This can be found under Edit Settings, Advanced migrations. After again clearing cashe and giving Facebook a few minutes to propagate the change, it linked up and I was able to log in using fb and set my picture from fb.

I am still getting a Notice: Undefined variable: preset in fbconnect_theme_user_picture_override() (line 889 … but I was able to log in using FB and link my FB picture. Progress but so painfully slow.

Installed the version of fbconnect_4 from #152 on a clean D7 install.
Installed facebook.php from SDK 2.1.2 in the fbconnect directory.
Completed the html mod to the modules/system/html.tpl.php file ... it was moved from the theme templates/html.tpl.php file in D7
Set up a facebook app and added the ID and secret key to my fbconnect installation
After the "invalid URL" error (see #158 above), I set “Remove Deprecated Auth Methods” to "disabled" in fb my app
At this point I was able to link to fb and upload my fb image to my drupal account but continue to I get the following error:
Notice: Undefined variable: preset in fbconnect_theme_user_picture_override() (line 889 of /home/----------- /sites/all/modules/fbconnect/fbconnect.module).
I get this error when I added a blog to my test account and it shows 3 times when i view "My Account".
The image does show.

subscribing

The error in #158 and #159 seems pretty literal. I don't think the $preset variable is being set by either of those if statements. In D7, should an empty variable be "" or NULL? Line 869 is checking if it's set, but may not be doing it the right way. I checked a couple other modules' code and they're all checking for NULLs when doing this kind of thing.

I haven't learned Git yet so I can't roll a patch, but a test to see if that's the cause is to change line 869 (of the version from #152) to this:

elseif (variable_get('fbconnect_pic_size', NULL)) {

'scribing

Anyone have rights to commit patches? It would be good to get #154 committed so we can keep moving forward, and make it a bit less confusing to follow the thread. The last dev release was in March according to the project page...

Suscribing.

And hoping to get it working on my windows/easyPhp box.

Is fbconnect.zip attached to post 152 the definitive plugin to use, or do I have to apply the patch in post 154 on top of the zip.
I know it sounds fetchy but I can't get the whole thing working on my box, and I want to be sure.

Cheers

It is either/or. So either take the zip of #152 (http://drupal.org/files/issues/fbconnect_4.zip), or download the 7.x-2.x-dev version from March (http://ftp.drupal.org/files/projects/fbconnect-7.x-2.x-dev.tar.gz) and apply the patch of #154 (http://drupal.org/files/issues/more-bug-fixes-795128-153_0.patch) on it.

But for both cases you need an extra patch on fbconnect.module, to overcome the issues of #158 and #159, :

change the line 867 of fbconnect.module from that one:

elseif (variable_get('fbconnect_pic_size', '')) {

to that:

else {

or apply my patch attached.

And you have to use an old Facebook PHP SDK (2.1.2: https://github.com/facebook/php-sdk/tarball/v2.1.2), not the recent ones: download, untar it to /site/all/libraries, rename it to facebook-php-sdk, and copy/symlink the scr/facebook.php into your fbconnect module directory.

subscribing

Why aren't the patches commited??

I've setup a mirror at github for the 7.x-2.x branch, see: https://github.com/doka/fbconnect

The "drupal.org" tag reflects the last committed version of the 7.x-2.x branch, from here, at d.o.
The "7.x-2.x-alpha1" tag is the patched version of jagermonster at http://drupal.org/node/795128#comment-4817546
The "7.x-2.x-alpha2" tag contains a few patches of mine, on top of 7.x-2.x-alpha1, see changelog for more details. This is the last status working fine for me.

I'm happy to accept patches and pull request over there, as long as the 7.x-2.x" branch is not maintained here at d.o.

I've opened #1263774: Offering to comaintain Facebook Connect in the hopes I can start committing this patch and further fixes, to avoid splintering over to github.

I committed according to the instructions in #165, and then followed up with a coding standards fix. I'm closing this issue, if there are any further bugs, open new issues.

http://drupalcode.org/project/fbconnect.git/commit/7910e9d
http://drupalcode.org/project/fbconnect.git/commit/379a2ba

I still have problem
Notice: Undefined index: email in fbconnect_register_page() (line 97 of /home/iraniansociety/public_html/sites/all/modules/fbconnect/fbconnect.pages.inc

file your issue as a bug report in a new thread under 7.x-2.x-dev

subscribing

I get the same error on /admin/config/people/fbconnect/fb-app

* The configuration options have been saved.

* Exception thrown while using admin_setAppProperties : This method must be called with an app access_token.

* Notice: Undefined variable: res in fbconnect_fbapp_settings_submit() (line 298 of /var/www/MYSITE/html/sites/all/modules/fbconnect/fbconnect.admin.inc).

* Checking your facebook session, you are not allowed to change this application settings

same issue as #176 comment above