Paging broken on Manage Posts

[ryanscheuermann]

I posted this in wp-testers but it seems no one reads that list anymore...

Using latest trunk code, BTW.

First off, the paging on the Manage Posts admin page is determined by the option "posts_per_page" but the results list is hardcoded to show the Latest 15 posts:

edit.php, line 59:

   wp('what_to_show=posts&posts_per_page=15&posts_per_archive_page=-1');

So, if I set the "posts_per_page" option to 5 (on Options->Reading) and I have 15 posts, the Manage Posts page allows me to page through 3 pages:

• the first page contains my latest 15 posts
• the second page (previous) is blank
• the third page (previous) is blank

The other bug is related to ​Ticket #2578

If I set "show at most N posts" to 0, the Manage->Posts page now displays 15 posts and this error:

*    Warning*:  Division by zero in *wp-includes\template-functions-links.php* on line *476*

This is also related to ​Changeset 3694.

[matt]

Is this latest trunk? I'm pretty sure I removed paging on the Page page page.

[ryanscheuermann]

OMG, my bad, I meant that to be the Manage Posts page is broken! Yeah, you're right, paging on Manage Pages is removed, but paging on Manage Posts is broken. :-) Typo!

[Nazgul]

I've attached a patch which fixes paging.

The other issue mentioned seems to have been fixed already.

[westi]

-1 to this patch

I think we still want to show 15 posts.

And you can't treat the translation like that you need to use sprintf or it won't be translateable.

Can we not make this paging work correctly?

[westi]

Alternative patch - use the query info

[westi]

Alternative patch fixes the issue as keeps the paging going on.

[markjaquith]

(In [4316]) Manage Posts paging fix from Westi. fixes #2693