New hook for paginate_links() result

[lightningspirit]

I'd like to propose two new hooks to paginate_links() in ​http://core.trac.wordpress.org/browser/trunk/wp-includes/general-template.php#L1950

This two hooks paginate_links_array and paginate_links_html will allow plugins to add, remove or alter html and links in admin pagination output, including the media uploader.

Background: sites with more than a thousand media objects have 140+ pages in media uploader interface. I had to be able to create a input box to enter page number manually.

[lightningspirit]

This patch add two new filters at end of paginate_links()

[lightningspirit]

The paginate_links() hooks allow this

[scribu]

If we're going to add a filter there, it should also pass the parameters that paginate_links() received.

And the 'paginate_links_array' filter should be applied only once, before the switch statement.

[scribu]

Also, your screenshot is puzzling. Since WP 3.1, the pagination in the admin has been revamped, adding an input for the current page, like you mentioned.

[scribu]

Oh, you mean the pagination links on the Media Library tab?

We should update them to the new style, which can be seen when going to /wp-admin/edit.php for example.

[lightningspirit]

Oh okay, I think updating to the new style will be better than revamping this one. Should I create a new ticket?

[scribu]

Yeah, you should make a new ticket; the filters might be a good idea on their own.

[lightningspirit]

New hook for paginate_links() "paginate_links_{$context}". Added $context variable for developers to be able to apply filters for paginate_links() in diferent contextes. Added $context = 'media-upload' for paginate_links() argument for developers to be able to hack the pagination result in the wp-admin/media-upload.php aka Media Library.

[lightningspirit]

This patch adds a new filter hook in paginate_links() function based on context passed through the argument of the function call.

Provides developers the ability to filter the ouptut/result of paginate_links() on diferent contextes!

I've added the context 'media-upload' in the call of media-upload.php file. This way developers can hack the pagination to, for example, provide a more suitable pagination UI if the Media Library have displayed a lot of pages.

[helenyhou]

Version number indicates when the bug was initially introduced/reported.

[nacin]

Seems like this is no longer applicable as of 3.5. lightningspirit, if there is still a desire (after all these years) for a filter in paginate_links() generally, feel free to reopen. I will mention though that paginate_links() is a low-level utility function that can be used by any context (paginating a post, or comments, or whatever), and it's possible that it's probably just not a good idea.

[SergeyBiryukov]

Follow-up: #44018