[wp-trac] [WordPress Trac] #39157: Feed returns 404 when there are no posts

[WordPress Trac]

#39157: Feed returns 404 when there are no posts
-------------------------------------------------+-------------------------
 Reporter:  pavelevap                            |       Owner:
     Type:  defect (bug)                         |  peterwilsoncc
 Priority:  high                                 |      Status:  assigned
Component:  Feeds                                |   Milestone:  4.7.2
 Severity:  normal                               |     Version:  4.7
 Keywords:  has-patch needs-testing has-unit-    |  Resolution:
  tests                                          |     Focuses:
-------------------------------------------------+-------------------------

Comment (by dd32):

 If there's another committer who understands the implications of the above
 change, and can have it in before the deadline for 4.7.1, they can commit
 the fix.

 I agree with @peterwilsoncc here and would have also punted this if I
 wasn't confident of it.

 > Additionally, new conditionals were added to solve a long standing
 problem

 A targeted regression fix which doesn't introduce new things or fix
 existing bugs would have made this far easier to commit. In this case,
 that would've been to skip all the getter methods here and just check the
 properties directly.

 This is the kind of thing that in the past I'd just reverted the original
 problem causing changeset and see a second run being taken at it.
 Reverting [38929] from 4.7.x is still an option.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/39157#comment:24>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform