Changeset 56595 for trunk/tests/phpunit/tests/option/option.php

Timestamp:

09/15/2023 04:13:52 PM (2 weeks ago)

Author:

spacedmonkey

Message:

Options, Meta APIs: Optimize get_option by relocating notoptions cache lookup.

In the get_option function, a cache lookup for the notoptions key is performed, which stores an array of keys for options known not to exist. This optimization prevents repeated database queries when certain options are requested. However, the cache lookup for notoptions was conducted before checking if the requested option exists in the cache. Given that it's more likely that the option does exist, this commit reorders the checks to first verify the option's existence in the cache before confirming its absence. This adjustment reduces redundant queries and also eliminates an unnecessary cache lookup, improving overall performance.

Props spacedmonkey, costdev, flixos90, azaozz.
Fixes #58277.

File:

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

trunk/tests/phpunit/tests/option/option.php

@@ -102 +102 @@
 
     /**
+     * @ticket 58277
+     *
+     * @covers ::get_option
+     */
+    public function test_get_option_notoptions_cache() {
+        $notoptions = array(
+            'invalid' => true,
+        );
+        wp_cache_set( 'notoptions', $notoptions, 'options' );
+
+        $before = get_num_queries();
+        $value  = get_option( 'invalid' );
+        $after  = get_num_queries();
+
+        $this->assertSame( 0, $after - $before );
+    }
+
+    /**
+     * @ticket 58277
+     *
+     * @covers ::get_option
+     */
+    public function test_get_option_notoptions_set_cache() {
+        get_option( 'invalid' );
+
+        $before = get_num_queries();
+        $value  = get_option( 'invalid' );
+        $after  = get_num_queries();
+
+        $notoptions = wp_cache_get( 'notoptions', 'options' );
+
+        $this->assertSame( 0, $after - $before, 'The notoptions cache was not hit on the second call to `get_option()`.' );
+        $this->assertIsArray( $notoptions, 'The notoptions cache should be set.' );
+        $this->assertArrayHasKey( 'invalid', $notoptions, 'The "invalid" option should be in the notoptions cache.' );
+    }
+
+    /**
+     * @ticket 58277
+     *
+     * @covers ::get_option
+     */
+    public function test_get_option_notoptions_do_not_load_cache() {
+        add_option( 'foo', 'bar', '', 'no' );
+        wp_cache_delete( 'notoptions', 'options' );
+
+        $before = get_num_queries();
+        $value  = get_option( 'foo' );
+        $after  = get_num_queries();
+
+        $notoptions = wp_cache_get( 'notoptions', 'options' );
+
+        $this->assertSame( 0, $after - $before, 'The options cache was not hit on the second call to `get_option()`.' );
+        $this->assertFalse( $notoptions, 'The notoptions cache should not be set.' );
+    }
+
+    /**
      * @covers ::get_option
      * @covers ::add_option