Make WordPress Core

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#21366 closed defect (bug) (wontfix)

Remove hardcoded site title from feed titles

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

Description (last modified by SergeyBiryukov)

Background: ticket:21233:9

Since [21276], site title is displayed twice in the feeds when Twenty Twelve is activated:

<title>WordPress TrunkWordPress Trunk</title>

In the feed files, wp_title_rss() is already prepended with site title:
http://core.trac.wordpress.org/browser/tags/3.4.1/wp-includes/feed-rss2.php#L24

We should probably remove hardcoded bloginfo_rss('name') and make the whole title filterable.

Attachments (2)

21366.patch (4.2 KB) - added by SergeyBiryukov 12 years ago.
21366.2.patch (4.2 KB) - added by SergeyBiryukov 12 years ago.
Refreshed after [21332]

Download all attachments as: .zip

Change History (7)

@SergeyBiryukov
12 years ago

Refreshed after [21332]

#3 @nacin
12 years ago

I am -1 on this. Aside from this being a backwards compatibility headache, those who use RSS feeds are going to want the hierarchy of a site title always first, so they are listed well in their reader. I would think this is fairly standard. There's no need for it to be the same as <title>. This isn't an aesthetic thing like a site title — someone else is deliberately consuming a feed on their own terms, and we should have some semblance of standardization if possible.

#4 @SergeyBiryukov
12 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Agreed.

#5 @SergeyBiryukov
12 years ago

  • Description modified (diff)
Note: See TracTickets for help on using tickets.