Opened 9 months ago

Closed 9 months ago

#21589 closed defect (bug) (fixed)

get_term_feed_link without filter for tag archive

Reported by: andrea.r Owned by: nacin
Priority: normal Milestone: 3.5
Component: Feeds Version: 3.4.1
Severity: minor Keywords: needs-codex has-patch
Cc:

Description

In function get_term_feed_link, located in link-template.php, line 627:

$link = apply_filters( 'category_feed_link', $link, $feed );

should be changed into:

$link = apply_filters( 'tag_feed_link', $link, $feed );

otherwise, it is not possible to edit or inhibit the feed URL in tag archives.

Attachments (1)

21589.patch​ (571 bytes) - added by SergeyBiryukov 9 months ago.

Download all attachments as: .zip

Change History (5)

SergeyBiryukov — 9 months ago

comment:1   SergeyBiryukov — 9 months ago

  • Keywords has-patch added
  • Milestone changed from Awaiting Review to 3.5

Looks like a copy/paste error in [14711].

comment:2 follow-up: ↓ 3   SergeyBiryukov — 9 months ago

A hacky workaround would be to hook into category_feed_link and search for tag/ or tag= in the link (depending on permalink structure).

comment:3 in reply to: ↑ 2   andrea.r — 9 months ago

Replying to SergeyBiryukov:

A hacky workaround would be to hook into category_feed_link and search for tag/ or tag= in the link (depending on permalink structure).

OK, thank you!!

comment:4   nacin — 9 months ago

  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from new to closed

In [21662]:

Fix copy-paste error in get_term_feed_link() that caused the category_feed_link filter to run for tags, rather than tag_feed_link. props andrea.r, SergeyBiryukov. fixes #21589.

Note: See TracTickets for help on using tickets.