Call update_post_author_caches in WP_Posts_List_Table class

[spacedmonkey]

In 6.1, update_post_author_caches was introduced. Call this function in WP_Posts_List_Table class, to prime user caches in one request and save database queries.

[thakkarhardik]

Hi @spacedmonkey , I have added update_post_author_caches() function to the display_rows method in WP_Posts_List_Table class and created a patch. Can you please review it and share your feedback?

[spacedmonkey]

        public function display_rows( $posts = array(), $level = 0 ) {
                global $wp_query, $per_page;

                if ( empty( $posts ) ) {
                        $posts = $wp_query->posts;
                        update_post_author_caches( $posts );
                }

@thakkarhardik I was thinking about something like this.

[thakkarhardik]

Updated Patch

[thakkarhardik]

Thanks @spacedmonkey . Your suggestion seems better. I've updated the patch.

Trac ticket: https://core.trac.wordpress.org/ticket/56100

[spacedmonkey]

@thakkarhardik I did some testing and notice some notice errors. It seems in page list views, that only id and parent is loaded.

I have push my own patch up, see what you think.

[desrosj]

What about adding this to update_post_caches()? update_post_caches.diff​ is a proof of concept.

Updates post, term, and metadata caches for a list of post objects.

Authors are really the only thing not being primed for posts (except comments, but that's a separate thing that is not always required). It's reasonable to consider author information is a required part of a post.

I'm not aware if there is a historical reason why this was left out previously, though.

[spacedmonkey]

For context [53482] and #55716.

Priming users can expensive and should be avoided if possible. In [53482] users are only primed in the loop. So for the most part, most uses of WP_Query will already be covered. There are only a couple of uses in core like this do not use the loop and get posts.

Also WP_Posts_List_Table does something odd around pages, see ​this. This loads all pages. This change ensues that only displayed posts users are primed.

[spacedmonkey]

In 54099:

Posts, Post Types: Call update_post_author_caches function in WP_Posts_List_Table class.

Call the update_post_author_caches function in the WP_Posts_List_Table class to prime post author caches in a single database request.

Props spacedmonkey, thakkarhardik, desrosj.
Fixes #56100.

[desrosj]

There are a lot of caching related changes in 6.1 and a dev note should be written collecting them all. Marking this for inclusion.