[wp-trac] [WordPress Trac] #7092: Should support separate setting for limiting comments feed
WordPress Trac
noreply at wordpress.org
Thu Oct 9 15:00:27 UTC 2014
#7092: Should support separate setting for limiting comments feed
---------------------------+------------------------
Reporter: redsweater | Owner:
Type: enhancement | Status: closed
Priority: normal | Milestone:
Component: Feeds | Version: 2.9
Severity: normal | Resolution: duplicate
Keywords: needs-refresh | Focuses:
---------------------------+------------------------
Changes (by chriscct7):
* keywords: has-patch => needs-refresh
* status: reopened => closed
* resolution: => duplicate
Comment:
Replying to [comment:28 redsweater]:
> Replying to [comment:26 chriscct7]:
> > This was fixed in #9635 and further revised in #9886
>
> I don't think either of the cited tickets are suitable duplicates of
this one. Trunk WordPress still doesn't support the feature requested by
this ticket: separate user control over the number of items in the
comments feed.
The ticket is requesting a way to modify the number of comments in the
comment thread. #9635 adds a filter called `comment_feed_limits` (later
edited in #9886) that allows you to be able to directly manipulate just
the number of comments on the comment thread. Nacin's patch doesn't add a
UI, nor did the ticket request it, nor does a UI for editing the number of
comments on the comment feed make sense because I highly doubt most people
would ever want to do this, and for the extreme vast majority that do, a
filter is there for it thanks to #9635, so we don't need "yet another
setting".
Its important to note, the ticket never asked for a UI setting, it asked
for a variable (he meant to say filter) on the last line.
That being said, #9635 solves all objectives stated in the ticket, and
thus is a duplicate of that ticket.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/7092#comment:29>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list