[wp-trac] Re: [WordPress Trac] #8708: Remove paging information from Atom comment feed ids

[WordPress Trac]

#8708: Remove paging information from Atom comment feed ids
-----------------------------------+----------------------------------------
 Reporter:  alexforrow             |        Owner:  josephscott
     Type:  defect (bug)           |       Status:  reopened   
 Priority:  normal                 |    Milestone:  2.7.1      
Component:  XML-RPC                |      Version:  2.7        
 Severity:  normal                 |   Resolution:             
 Keywords:  comments, feeds, atom  |  
-----------------------------------+----------------------------------------
Changes (by GregMulhauser):

  * keywords:  => comments, feeds, atom
  * status:  closed => reopened
  * resolution:  fixed =>

Comment:

 Hi folks,

 I'm not convinced that this 'bug' is necessarily a bug, or that this 'fix'
 is necessarily a fix...

 I understand that in principle, IDs should never change -- but changing
 *URIs* are a logical consequence of allowing users to choose 1) comment
 pagination, and 2) ascending date order to display comments. (URIs do not
 change if comments are displayed in descending date order and with a
 constant value for comments per page.) Unfortunately, IDs constructed
 without page numbers make for broken URIs for all comments except those
 which happen to live at the post's permalink. In alexforrow's example, the
 URI only works because we are on comment page 1; if we were on other
 pages, the URI would be broken.

 When submitting the bug report, alexforrow mentioned that including page
 numbers "could break things if the the page number were to change". But if
 "break things" means "break the way in which IDs can pass for URIs", then
 the fix only serves to trade future potential breaks for present
 guaranteed breaks for users with paging enabled.

 I'm guessing that many users would be more unhappy to find that their Atom
 feeds include broken links than to find that they may break comment URIs
 if they choose pagination combined with ascending date order.

 I'm happy to be told I'm all wrong about this, but it just seemed to me
 there was more afoot than whether including paging information was a good
 idea or a bad idea...

 All the best,
 Greg

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