add tip about /feed aliases.
| Project: | Pathauto |
| Version: | 6.x-2.x-dev |
| Component: | Documentation |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
Hello,
I was confused for a while about the purpose of /feed aliases, such as for /node/1/feed. In pathauto settings, the user is shown this field title:
Internal feed alias text (leave blank to disable):
and this help text:
The text to use for aliases for RSS feeds. Examples are "feed" and "0/feed".
My confusion stemmed from my assumption that drupal, with default, clean configs, offers a feed for /node/1/feed. However it does not. One must install a module in order to get a feed at that address.
I would like to suggest that you add into the help text, something along these lines:
By default, /node/*/feed is an inactive address. To get a feed at such an address, you must install and configure a module such as commentrss or views.
I'm not sure what the exact best wording would be.
For reference, here is my post to the forum about this: http://drupal.org/node/332709
#1
Hi,
I understood it the other way around :), I though this setting was for the _alias_ created by pathauto not for the source feed path.
If this is really the case, then the code has a bug which I tried to fix in the attached patch.
But I am not sure of my interpretation either, drupal has a standard path for nodes
node/$nidbut not for feeds AFAIK, even if the/feedsuffix is the most used one.Maybe pathauto should provide a way to set both the source suffix and a replacement pattern for feeds; for now, for my use case, I would assume that feeds use all the
/feedsuffix usuring that in the feed creator (in my case it is the Views module) and I let pathauto rename them to/rss.xml.Any comments?
Regards,
Antonio
UPDATE: please ignore the attached patch.
#2
Sorry, I've just found the right issue #286843: Provide control over "feed" in url aliases which explains the situation, I am now convinced that I was wrong and also that the wording of this setting description should be improved :)
What about mentioning that the setting refers to the source path of the feed, and that it hardcodes the aliased path?
Sorry for the noise.
Regards,
Antonio
#3
Restoring Component and Category.
#4
What about the attached changes?
Regards,
Antonio
UPDATE: please ignore this patch.
#5
Please use this one, v1 is against the wrong file.
Sorry,
Antonio
#6
Turning this into a documentation bug.
#7
Was also wondering what is the difference between "internal feed" and a plain feed output by views module for the taxonomy/term page?
#8
Here's another point - we should just default the node feeds to off.
Also, we can't change strings in 6.x-1.x so I'm not committing the text change just yet...
#9
Committed that last patch to HEAD and 6.x-1.x
http://drupal.org/cvs?commit=276088
http://drupal.org/cvs?commit=276090