Opened 14 years ago
Closed 12 years ago
#22362 closed defect (bug) (fixed)
Titles of RSS feeds lacks spacing - words are run together
| Reported by: | DavidAnderson | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.0 |
| Component: | Feeds | Version: | 3.4.2 |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: | Focuses: |
Description
Line 24 of wp-includes/feed-rss2.php has:
<title><?php bloginfo_rss('name'); wp_title_rss(); ?></title>
There's no space between those two parts. In my (development) site, for the feed for a category "News" (which is the main one I'm offering up), that results in:
<title>SitenameNews</title>
Attachments (1)
Change History (6)
#3
@
12 years ago
- Keywords has-patch added
- Milestone Awaiting Review → 4.0
Could not reproduce with default themes.
get_wp_title_rss() has » as a default separator: tags/3.9.1/src/wp-includes/feed.php#L91.
This is the same issue as in #21366. It only happens if the theme has a custom function attached to wp_title, and that function doesn't include an is_feed() check (as in [21276]). The fix for the theme would be like [21327].
If #21366 is not an option due to backwards compatibility issues, we could probably enforce the separator in get_wp_title_rss() if the filtered title doesn't start with a space. 22362.patch does that.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Related: #21366