[buddypress-trac] [BuddyPress] #5213: Set better HTTP headers for RSS feeds

[buddypress-trac]

#5213: Set better HTTP headers for RSS feeds
--------------------------+---------------------
 Reporter:  r-a-y         |       Owner:  r-a-y
     Type:  defect (bug)  |      Status:  closed
 Priority:  normal        |   Milestone:  1.9
Component:  Activity      |     Version:  1.8
 Severity:  normal        |  Resolution:  fixed
 Keywords:                |
--------------------------+---------------------
Changes (by r-a-y):

 * owner:   => r-a-y
 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"7477"]:
 {{{
 #!CommitTicketReference repository="" revision="7477"
 Set better HTTP headers for activity RSS feeds.

 Previously, BP's RSS feeds would use WP's native HTTP headers set in the
 WP::send_headers() method.  This would cause BuddyPress to inherit WP
 feed's "ETag" and HTTP status headers.

 When trying to refresh BP's RSS feeds, this would not work correctly due
 to BP relying on WP's post last modified date instead of BP's activity
 last
 updated date.  This would lead to a HTTP 304 Not Modified status header
 being sent by WP, meaning the client (browser, RSS reader) would not fetch
 the latest items from the activity stream.

 This commit sets the appropriate HTTP headers for BP's activity RSS feeds
 and uses a new, internal class method - http_headers() - to set them.

 Fixes #5213.
 }}}

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5213#comment:1>
BuddyPress <http://buddypress.org/>
BuddyPress