Changeset 55671 for trunk/tests/phpunit/tests/term/query.php

Timestamp:

04/21/2023 09:22:04 AM (3 years ago)

Author:

spacedmonkey

Message:

Taxonomy: Always lazily load term meta.

In [34529] introduced lazy loading of term meta. However, this was only in the context of WP_Query. Other parts of the codebase, like WP_Term_Query did not lazily load term meta. In this change, calls to update_termmeta_cache are now replaced with wp_lazyload_term_meta, that instead of priming term meta caches, just adds them to the queue to be primed it ever called. This results in far less database queries, as there a number of places where term meta is being primed unnecessarily and never used. Adding everything to the term meta queue, also means that if term meta is used, that is all loaded in a single database / cache call.

Props spacedmonkey, mukesh27, peterwilsoncc.
Fixes #57645.

File:

• trunk/tests/phpunit/tests/term/query.php (modified) (1 diff)

trunk/tests/phpunit/tests/term/query.php

@@ -45 +45 @@
 
         $this->assertSameSets( array( $term_2 ), $q->terms );
+    }
+
+    /**
+     * @ticket 57645
+     */
+    public function test_lazy_load_term_meta() {
+        $filter = new MockAction();
+        add_filter( 'update_term_metadata_cache', array( $filter, 'filter' ), 10, 2 );
+        register_taxonomy( 'wptests_tax_1', 'post' );
+        register_taxonomy( 'wptests_tax_2', 'post' );
+
+        $term_1 = self::factory()->term->create( array( 'taxonomy' => 'wptests_tax_1' ) );
+        $term_2 = self::factory()->term->create( array( 'taxonomy' => 'wptests_tax_2' ) );
+
+        $q = new WP_Term_Query(
+            array(
+                'taxonomy'   => 'wptests_tax_1',
+                'fields'     => 'ids',
+                'hide_empty' => false,
+            )
+        );
+
+        $this->assertSameSets( array( $term_1 ), $q->terms );
+
+        $q = new WP_Term_Query(
+            array(
+                'taxonomy'   => 'wptests_tax_2',
+                'fields'     => 'ids',
+                'hide_empty' => false,
+            )
+        );
+
+        $this->assertSameSets( array( $term_2 ), $q->terms );
+
+        get_term_meta( $term_1 );
+
+        $args     = $filter->get_args();
+        $first    = reset( $args );
+        $term_ids = end( $first );
+        $this->assertSameSets( $term_ids, array( $term_1, $term_2 ) );
     }