Filters not applied to bloginfo_rss()

[Kafkaesqui]

Currently bloginfo_rss() lacks a filter hook:

function bloginfo_rss($show = '') {
	echo get_bloginfo_rss($show);
}

It should probably be more like:

function bloginfo_rss($show = '') {
	$info = get_bloginfo_rss($show);
	$info = apply_filters('bloginfo_rss', $info, $show);
	echo $info;
}

[Viper007Bond]

I'd think the filters should be applied to get_bloginfo_rss(), not bloginfo_rss() as bloginfo_rss() is just an echo wrapper for get_bloginfo_rss().

[Kafkaesqui]

I'm agreeable to that, but then this should be looked at for bloginfo()/get_bloginfo(): the limited filter handling there presently goes through bloginfo(), not get_bloginfo().

[Kafkaesqui]

Actually, I take it back.

There are reasons where I might need to bypass filters (I just ran into one). Having them outside the get_* functions provides a way to do that. A case could be made that having filters in the get_* functions can benefit plugin developers as well (ah, decisions!) but for now I have to recommend sticking with current WP design and filter only on bloginfo/bloginfo_rss.

[Kafkaesqui]

Patch adds filter hook to bloginfo_rss()

[Kafkaesqui]

Have added a patch for this.

[rob1n]

Patch refreshed, and a filter name changed.

Also, having get_bloginfo_rss apply a get_bloginfo_rss filter and then bloginfo_rss applying bloginfo_rss on top of that mirrors get_bloginfo and bloginfo's functions.

[rob1n]

Assigned to ryan for commit.

[rob1n]

(In [5134]) Apply the bloginfo_rss filter and change bloginfo_rss filter to get_bloginfo. Props Kafkaesqui. fixes #2014