[wp-trac] Re: [WordPress Trac] #3884: the_content_rss() is used incorrectly in non RSS feeds.

[WordPress Trac]

#3884: the_content_rss() is used incorrectly in non RSS feeds.
-------------------------------------+--------------------------------------
 Reporter:  jeroyang                 |        Owner:  joostdevalk
     Type:  defect                   |       Status:  new        
 Priority:  normal                   |    Milestone:  2.9        
Component:  Template                 |      Version:  2.1.1      
 Severity:  normal                   |   Resolution:             
 Keywords:  needs-testing has-patch  |  
-------------------------------------+--------------------------------------
Changes (by dougal):

 * cc: dougal, westi (added)

Comment:

 So, is there any reasoning behind the inconsistent usage of
 the_content_rss()? It's used in some feed types (rdf and rss) but not
 others (rss2 and atom). What's up with that?

 I ran across this today when I needed a plugin to modify feed content
 differently than regular page views. Ended up using is_feed() in my
 the_content filter but that just seems clunky to me.

 Either we can rely on the_content_rss, or we can't. Which is it? The
 function doesn't need to do anything fancy. Just get_the_content(), and
 apply any 'the_content_rss' filters to it. All those parameters seem like
 overkill to me. Refactor those into their own filter functions. It would
 break backwards compatibility, but I figure this function is broken
 already anyways.

 Oh, and it should probably be deprecated in favor of renaming it to
 'the_content_feed', anyways, right? That could fix the backwards
 compatibility issue.

 get_the_content_feed()
 the_content_feed()
 the_content_rss() -- deprecate

-- 
Ticket URL: <http://trac.wordpress.org/ticket/3884#comment:16>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software