I found that there is a conflict between feedburner and organic groups when page caching is enabled. My solution was to remove the conditional include of common.inc in _feedburner_redirect() and then conditionally use header() to do the redirection to feedburner.com rather than drupal_goto if drupal_goto isn't available.

This seems to work but it is possible it causes problems I didn't see.

Please let me know if you see any problems with this patch or if you know of a better solution.

CommentFileSizeAuthor
feedburner_og_pagecache.patch2.45 KBjosh.estep

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

What was the problem caused by including common.inc? BTW, the fix for this in the 6.x-1.x version of the module (that will be backported) is just to call drupal_bootstrap(DRUPAL_BOOTSTRAP_PATH) and duplicate the code from drupal_goto inside the redirect function.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Fixed

This should now be fixed in 5.x-1.x.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.