#5173 closed enhancement (fixed)
Replace get_category_rss_link with get_category_feed_link
| Reported by: | ionfish | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 2.5 |
| Component: | Template | Version: | 2.3 |
| Severity: | normal | Keywords: | needs-patch |
| Cc: | Focuses: |
Description
Replacing get_category_rss_link with a get_category_feed_link function would allow theme authors to use WordPress's Atom feeds instead, if they prefer. get_category_rss_link would be deprecated, but not removed, to keep backwards compatibility with themes which use it.
Attachments (1)
Change History (10)
#3
@
19 years ago
Good point. I'll tweak the patch. I was also thinking of doing the same for get_author_rss_link, should I add that to this ticket or make a new one?
#4
@
19 years ago
Look at get_tag_feed_link() and pattern get_category_feed_link() after it. We don't really need to pass the nicename and echo.
#6
@
19 years ago
Getting better. On line 150, you need to trailingslashit($link) (like on the old line 138). Otherwise, you could get:
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
+1 on the idea. But the patch only addresses default permalinks. People with pretty permalinks wouldn't get the feed type they asked for.