Closed (duplicate)
Project:
Drupal for Facebook
Version:
7.x-3.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
1 Jan 2011 at 02:36 UTC
Updated:
1 May 2012 at 16:15 UTC
Jump to comment: Most recent file
Comments
Comment #1
Dave Cohen commentedthanks for the report. Usually I include sites/all/modules/fb/fb_settings.inc in my site's settings.php, so I haven't seen this before. I recommend that as a workaround.
Comment #2
Dave Cohen commentedDo you know which update is trying to run when this happens?
Comment #3
shunshifu commentedI run into this issue also. It happens during any update. I turn of the FB module. Run the update. and turn it back on.
Your workaround of adding
include_once './sites/all/modules/fb/fb_settings.inc';
to settings.php worked great
Thanks
Phil
Comment #4
Dave Cohen commentedHere's the fix, I think.
Comment #5
jeseich commentedI used the patch from Dave Cohen and that seemed to fix my problem running update.php. Thanks!
Comment #6
Dave Cohen commentedThis was checked in but issue status never updated.
Comment #8
jonhattanI've added this module to a install profile. At the very end of installation I obtain
The fix above seems not useful at install time.
Comment #9
jonhattanFixed with attached patch. Perhaps worth it to use hook_boot() instead of all of this.
Comment #10
jonhattanwith better docs.
Comment #11
Dave Cohen commentedWell, that's kind of crappy of drupal to not always call that hook.
I'd rather do something like this in fb_init()
That will lead to less duplicated code between fb_init() and fb_custom_theme(). Can you confirm that works as well?
Comment #12
jonhattanYes it works at install time. Not sure if it is also fine for update.php. Patch attached.
Comment #13
mrfelton commentedPatch at #12 works for me, thanks.
Comment #14
mrfelton commentedAlthough, fb_devel module has the same problem.
Comment #15
fluffy commentedpatch at #12 worked for me too
Comment #16
Dave Cohen commentedPlanning to push this shortly. Thanks all.
Comment #17
Dave Cohen commentedSorry, I got lost in duplicate issues. Fixed under #1182190: hook_custom_theme sometimes isn't called resulting in fatal error