Active
Project:
Facebook social plugins integration
Version:
6.x-2.0-beta4
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
25 Jan 2012 at 18:59 UTC
Updated:
4 Jun 2012 at 12:24 UTC
Jump to comment: Most recent file
Comments
Comment #1
madhattertech commentedI also get the same issue with 6.x-2.0-beta3
Comment #2
Aress commentedI got the same issue with drupal 6.24 and 6.x-2.0-beta4.
Comment #3
webnotwar commentedCommenting out line 189 in fb_social.module ( $content = drupal_get_title(); , 6.x.2.0-beta4) fixes the problem for me.
Unfortunately, the "og:title" meta-tags doesnt' display the page title, just the site name.
Comment #4
ferdi commentedI looked at this and was not able to find the source of this bug. Will investigate further.
Thanks!
Comment #5
Ionut commentedAny updates on this?
Comment #6
jsagotsky commentedThe problem is in fact drupal_get_title. Running this in hook_init causes common.inc to WSOD. The short explanation is that drupal_get_title affects breadcrumb generation and implies that the current path should be part of the breadcrumb trail. When the breadcrumb trail is generated, that page has no title and causes the error.
Counterintuitively it looks like this can be fixed by swapping drupal_get_title out for drupal_set_title, which always returns the title but only sets a title if a new one is provided. drupal_get_title also sets the active title, which is what causes the breadcrumb error.
Patch to follow. This change does avoid the error, but I'm not 100% sure yet that ignoring the make active call in drupal_get_title is okay.
Comment #7
jsagotsky commentedHere's the patch I mentioned in comment 6.
Comment #8
jsagotsky commentedCommitted that last patch too soon. Turns out it wasn't getting the title correctly after all.
Attached patch will fix it.
Comment #9
wiredescape commentedv6x-2.0-beta4
Confirm that fix works but patch contains an error in line to be replaced.
The code used in module is :
$content = drupal_get_title();
not
$content = drupal_set_title();
as found in patch.
Comment #10
gp.mazzola commentedI have found this thread trying to figure out the issue happening onmy site.
For me the latest patch does not work. (I corrected it according to post #9)
It seems to work a first time, but then it does not work anymore on subsequent pages requests
I tried both witch cache enabled and disabled
Comment #11
myha commentedPatch in #8 works for 6x-2.0-beta4. Thanks a lot!
Comment #11.0
myha commentedadded drupal version