Skip to:
Content

bbPress.org

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#3064 closed defect (bug) (fixed)

Make RSS feed titles filterable

Reported by: sergeybiryukov's profile SergeyBiryukov Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 2.6 Priority: normal
Severity: normal Version:
Component: API - Feeds Keywords: has-patch
Cc:

Description

In #meta2078, it was brought up that the RSS feed title for the WP.org support forum of a plugin or theme is too generic: "All Topics", which isn't useful when dealing with support, as it does not include the project title.

Currently, there's no filter to change the default "All Topics" title for view feeds, unless you use a precisely targeted gettext filter.

In the patch:

  • Make feed titles for topics and replies filterable using wp_title_rss filter.
  • Remove a redundant bbp_user_can_view_forum() condition: it's already true at this point, there's a check for that a few lines above, and it looks like a direct copy/paste of that line.

Attachments (1)

3064.patch (2.6 KB) - added by SergeyBiryukov 8 years ago.

Download all attachments as: .zip

Change History (3)

@SergeyBiryukov
8 years ago

#1 @johnjamesjacoby
8 years ago

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

In 6304:

Feeds: Allow feed titles to be filtered (using the WordPress filter name.)

Also removes redundant conditional in replies feed.

Props SergeyBiryukov. Fixes #3064.

#2 @johnjamesjacoby
8 years ago

In 6306:

Feeds: Call get_wp_title_rss() in replies feeds.

See #3064. Missed in r6304.

Note: See TracTickets for help on using tickets.