howdy!

when crss should be included in a page, it calls drupal_add_feed() in a preprocess_page function.

drupal_add_feed tries to add a feed link to the head of the page, so that feed readers etc. can discover the feed:

<link rel="alternate" type="application/rss+xml" title="localhost - All comments" href="http://localhost/drupal-6/crss" />

however, because $variables['head'] has already been computed at this point, the link is not getting added to head.

the solution to this is to reset $variables['head'] in commentrss_preprocess_page(), and that's what this patch does.

Comments

firebus’s picture

Status: Active » Needs review
StatusFileSize
new592 bytes

if you'd prefer a patch against 7.x-2.x-dev, let me know - i thought i'd hold off since there's no working dev release yet, and maybe the bug doesn't exist in 7?

dave reid’s picture

Status: Needs review » Fixed

Fixed in 6.x-2.x: http://drupalcode.org/project/commentrss.git/commit/4887cb4. Confirmed that its not necessary with Drupal 7.

Status: Fixed » Closed (fixed)

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