[wp-trac] [WordPress Trac] #42261: Setting posts_per_rss = -1 fails.

[WordPress Trac]

#42261: Setting posts_per_rss = -1 fails.
------------------------------------------+-----------------------------
 Reporter:  ohryan                        |       Owner:
     Type:  enhancement                   |      Status:  new
 Priority:  normal                        |   Milestone:  Future Release
Component:  Feeds                         |     Version:  4.8.2
 Severity:  normal                        |  Resolution:
 Keywords:  needs-patch needs-unit-tests  |     Focuses:
------------------------------------------+-----------------------------
Changes (by stevenkword):

 * keywords:   => needs-patch needs-unit-tests
 * type:  defect (bug) => enhancement
 * milestone:  Awaiting Review => Future Release


Comment:

 Thank you for the report @ohryan.

 As far as I'm aware, there isn't any document that states this query
 variable supports this value, but that doesn't mean it shouldn't.  I
 simply wouldn't classify this as a bug and more like an enhancement.

 I am in favor of supporting the functionality since the variable is passed
 on to `posts_per_page` in `WP_Query`.  The problem seems to be more at the
 "LIMIT" filters which could be updated.  This enhancement would require
 consideration for comment feeds as well.

 Tests should be written against RSS and Atom and should work for all feeds
 endpoints (home, custom, comments, custom post types, etc.).

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