[wp-trac] [WordPress Trac] #57150: Implement wp_cache_get_multiple() in wp_queue_posts_for_term_meta_lazyload()

[WordPress Trac]

#57150: Implement wp_cache_get_multiple() in
wp_queue_posts_for_term_meta_lazyload()
-------------------------------------------------+-------------------------
 Reporter:  ocean90                              |       Owner:
                                                 |  spacedmonkey
     Type:  enhancement                          |      Status:  reopened
 Priority:  normal                               |   Milestone:  6.2
Component:  Taxonomy                             |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch dev-feedback has-unit-     |     Focuses:
  tests                                          |  performance
-------------------------------------------------+-------------------------

Comment (by Chouby):

 @peterwilsoncc Thank you for the ping.

 On our side, we have been aware of this issue for a few days. Although we
 did not detect it in our PHPUnit tests because the Polylang developement
 branch was already fixed by @ocean90
 [https://github.com/polylang/polylang/pull/1154 (caching term ids instead
 of WP_Term objects) for WP 6.0+], it was detected by @chrystl in her tests
 and also reported by a few users :).

 So you can consider that we will fix that on our side.

 I have also tested [attachment:57150.patch] proposed by @skithund. I
 confirm that it also fixes the issue on WordPress side. This could help
 for backward compatibility with potential other plugins still caching
 WP_Term objects.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/57150#comment:24>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform