Provide control over "feed" in url aliases [#286843]

I have just set the taxonomy term path pattern to be [catpath-raw] and the internal feed alias to be rss.

When saved this generates the following alias:
eg. taxonomy = term1

Alias = term1/feed System = taxonomy/term/1/rss

This is the wrong way round. When I manually edit it to say

Alias = term1/rss System = taxonomy/term/1/feed

it works

Comment	File	Size	Author
#3	provide_control_over_feed_alias_v1.patch	3.33 KB	ao2
#3
#3	feed_source_path_mockup.png	21 KB	ao2

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

greggles

he/him

English

Denver, Colorado, USA

CreditAttribution: greggles commented 24 July 2008 at 21:44

Title:	Custom feed alias created in reverse	» Provide control over "feed" in url aliases
Version:	6.x-1.1	» 7.x-1.x-dev
Category:	bug	» feature

Well, the question is whether the field is the wrong way around or your use of it is the wrong way around. It turns out that it is the second ;) That field is for controlling what gets placed onto the end of /taxonomy/term/1/[here] and not the place that you are hoping it controls. It was added in #138460: allow admin to configure feed alias pattern.

I think that the other situation - controlling the part that gets added to the end of the alias - should also be available to users so I think we should turn your bug into a feature to expose control over that.

Comment #2

mbiddlecombe CreditAttribution: mbiddlecombe commented 25 July 2008 at 08:12

Please do! :-)

Comment #3

ao2 CreditAttribution: ao2 commented 19 April 2009 at 14:49

Status:

Active

» Needs review

File	Size
feed_source_path_mockup.png	21 KB
provide_control_over_feed_alias_v1.patch	3.33 KB

Hi,

Is the attached change OK?
The patch applies cleanly on top of the one here #333002: add tip about /feed aliases., since the feed setting description needs to be updated anyway.

Thanks,
Antonio

P.S.: Ideally the source path for feeds should be even more configurable, for instance the Comment RSS module uses a prefix instead of a suffix to specify a feed, see the mockup in the attached image for a possible way to handle that, but I am not sure whether it is worth the effort to add that.