Closed (fixed)
Project:
Facebook Connect
Version:
6.x-1.0-beta8
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
17 Nov 2009 at 22:31 UTC
Updated:
5 Aug 2010 at 16:54 UTC
Jump to comment: Most recent file
Comments
Comment #1
vectoroc commentedhere is some bugfixes
Comment #2
vectoroc commented+ Ability to link/unlink Facebook acount from user profile
Comment #3
vectoroc commentedsome bugfixes
Comment #4
vectoroc commentedforgot to remove debugging
Comment #5
gregarios commentedThis 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 Facebooktext 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.
Comment #6
WildBill commentedI would like to echo Gregarios' request in comment #5...
Comment #7
vectoroc commentedCheck this. I believe that is more flexible way. With it you can fully configure fbconnect button through hook_form_alter or style sheets
Comment #8
gregarios commentedActually, we're having problems modifying the text next to the button, using CSS. Modifying the button itself is not a problem.
Comment #9
vectoroc commentedI'm not sure that I understand you. Now
Sign in using Facebookis wrapped with<div class="description">and you can access it through#fbconnect_button-wrapper .descriptionE.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
Comment #10
gregarios commentedOh 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? :-)
Comment #11
vectoroc commentedActually you can download it from http://github.com/vectoroc/fbconnect
Comment #12
gregarios commentedUh 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.
Comment #13
vectoroc commentedIt's not informative. I can't reproduce the bug.
Comment #14
gregarios commentedSorry... 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.
Comment #15
vectoroc commentedcan you give me acces to page where error occurs?
Comment #16
gregarios commentedNot 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.jsfile 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.Comment #17
vectoroc commentedThx, I found problem on page /imce and fixed it. Download last version which is here
Comment #18
gregarios commentedThat's it. Your update works great. All the functionality and no problems.
How can we get this turned into a Version 1.0 release? :-)
Comment #19
nirad commentedVectoroc will need maintainer privileges to create a 1.0 release. Are the current maintainers still maintaining the module at all?
Comment #20
gregarios commentedIt would seem not, as the last commit was over 21 weeks ago...
Can someone step in and assume control?
Comment #21
guedressel commented+1 from me.
Just launched a new Website including fbconnect.
Would love to see the latest contributions in upstream repositor and new supported releases.
Comment #22
ambo commentedHi 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?
Comment #23
vectoroc commentedYes, I'm. Good idea. Are you talking about fork on github or drupal-module fork ?
Comment #24
ambo commentedI 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?
Comment #25
gregarios commentedIf you start working on Github, please make announcements of new versions in this thread if you would. :)
Comment #26
yeeloon commented+1 subscribing
Comment #27
WildBill commented+1 for rolling this into a new Drupal 1.0 version...
Comment #28
Carlos Miranda Levy commentedQuick 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?
Comment #29
vectoroc commentedActually it looks so
1. Download http://github.com/vectoroc/fbconnect/zipball/master
2. Get Facebook Platform from http://svn.facebook.com/svnroot/platform/clients/packages/facebook-platf...
Comment #30
vectoroc commentedI 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
Comment #31
gregarios commentedA 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. :-)
Comment #32
ambo commentedI 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?
Comment #33
WildBill commentedI'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.
Comment #34
ambo commentedI 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
Comment #35
akongz commented*deleted*
Comment #36
akongz commentedsetting auto login & logout not working?
Comment #37
vectoroc commentedCheck latest commit. It should work
Comment #38
akongz commented@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.
Comment #39
tsi commentedThis 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.
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.
Comment #40
vectoroc commentedHappy new year!
I'll release next beta in next year
Comment #41
WildBill commentedVectoroc, 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......
Comment #42
ambo commentedHi 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:
If anything is wrong or missing just post it here.
regards - andreas
Comment #43
alexfisher commentedsubscribing
Great patch/release vectoroc.
Comment #44
joetsuihk commentedi 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?
Comment #45
.Raj commentedHi 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..
Comment #46
e0ipso+1 subscribing
Comment #47
Aquilasfx commentedI 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?
Comment #48
vectoroc commentedAquilasfx: 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.
Comment #49
marcus178 commentedI 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?