Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#15998 closed defect (bug) (fixed)

"Show more comments" does not work on edit posts screen if you have comments per page less than 10

Reported by: greuben Owned by:
Milestone: 3.1 Priority: normal
Severity: major Version: 3.1
Component: Comments Keywords: has-patch
Focuses: Cc:


Set comments per page to 1 and edit a post with more than one comment. "Show more comments" is not displayed at all. If you have 10 or more comments to that post "Show more comments" is displayed but if you click on it will display "No more comments found."

Attachments (3)

15998.diff (1.9 KB) - added by greuben 7 years ago.
15998.only.10.patch (476 bytes) - added by ocean90 7 years ago.
15998-2.diff (1.2 KB) - added by greuben 7 years ago.

Download all attachments as: .zip

Change History (9)

7 years ago

#1 @greuben
7 years ago

  • Keywords has-patch added

#2 @ocean90
7 years ago

  • Component changed from Administration to Comments
  • Milestone changed from Awaiting Review to 3.1
  • Version set to 3.1

It's a regression.

But I think, that we shouldn't use edit_comments_per_page here, because 100 comments on the post page would be to much, but rather use the old fix value 10. I attached a patch with my idea.

#3 @ryan
7 years ago

Let's hard code to 10 somewhere in WP_Post_Comments_List_Table. Maybe have WP_Comments_List_Table use a get_per_page() method that WP_Post_Comments_List_Table can override.

#4 @nacin
7 years ago

  • Keywords needs-patch added; has-patch removed

7 years ago

#5 @greuben
7 years ago

  • Keywords has-patch added; needs-patch removed

#6 @ryan
7 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [17168]) Hard limit to 10 comments on edit post screen. Props greuben. fixes #15998

Note: See TracTickets for help on using tickets.