Forum module doesn't support feeds itself, but relies on taxonomy/term/tid for feeds.
Currently, the pathauto_forum hook claims that it supports feeds, which is kinda true since forums do have feeds, but is also false since the code then goes ahead and creates the feed alias for forum/tid/feed which won't work. In general, we should probably provide patterns for feed aliases which default to "regular pattern + /feed" but which can be customized.
For a quick fix for a core module, though, I think a hack solution is in order :/
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | forum_feed_aliasing_uses_tax.patch | 1.11 KB | greggles |
Comments
Comment #1
gregglesHere's a patch which fixes this.
As I said, it's a bit of a hack because it puts knowledge of how the forum module works into the pathauto_create_alias function.
Comment #2
suzanne.aldrich commentedHi there greggles. I tried this patch out (on 5.x-2.0-beta5) and it worked perfectly. I also checked out the code for sanity's sake, and it looked great, even though it will probably get re-factored out at a later date, it will make forum feed aliasing work right for now.
Here is how I tested: I deleted my forum aliases by hand (since there's no way to delete aliases in a group yet...but that's another issue to track), then I did a bulk generation of forums and containers, with the feed option checked. The aliases I got were perfect, with a feed pointing to taxonomy/term/TID/0/feed as required, such that when I go through the forum interface, the RSS link presented a nice and friendly url alias. Yay!
Comment #3
gregglesgreat - committed to both 5.x and MAIN.
Comment #4
(not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.