Comment RSS is handling the drupal_add_feed logic in the menu system, which means that every URL beginning with node/$nid will get a comment feed link, whether it is appropriate or not. Example: editing a node results in a comment feed link appearing at the bottom.

The nodeapi hook seems a more logical place to execute this call; that way it can be limited to particular types of comment and only execute during a page view of the node in question.

There might also be an appropriate comment hook which would be useful for attaching the comment feed.

A simple (and not completely tested) fix for users who want the feed to appear only on a node view is to add arg(2)==NULL to the condition branch in the menu hook.

Comments

gábor hojtsy’s picture

Version: 5.x-1.x-dev » 5.x-2.x-dev
Assigned: Unassigned » gábor hojtsy
Status: Active » Fixed

I have implemented the simple arg(2) == NULL check, which is fine as it is IMHO. It is in 5.x-2.x-dev, please try that and reopen if it is not a right fix.

http://drupal.org/cvs?commit=113501

Anonymous’s picture

Status: Fixed » Closed (fixed)

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