Make WordPress Core

Opened 13 years ago

Closed 13 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's profile greuben Owned by:
Milestone: 3.1 Priority: normal
Severity: major Version: 3.1
Component: Comments Keywords: has-patch
Focuses: Cc:

Description

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 13 years ago.
15998.only.10.patch (476 bytes) - added by ocean90 13 years ago.
15998-2.diff (1.2 KB) - added by greuben 13 years ago.

Download all attachments as: .zip

Change History (9)

@greuben
13 years ago

#1 @greuben
13 years ago

  • Keywords has-patch added

#2 @ocean90
13 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
13 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
13 years ago

  • Keywords needs-patch added; has-patch removed

@greuben
13 years ago

#5 @greuben
13 years ago

  • Keywords has-patch added; needs-patch removed

#6 @ryan
13 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.