Context Navigation

← Previous Change
Next Change →

networkOption.php

Timestamp:

07/23/2024 12:25:19 AM (2 years ago)

Author:

peterwilsoncc

Message:

Options, Meta APIs: Prime notoptions cache when deleting options.

Prime the notoptions cache within delete_option and delete_network_option to avoid the need for a database query if get_option or get_network_option is subsequently called.

Adds some associated tests to ensure that an option is cleared from the notoptions cache when an option is added either via add_option, update_option or their network option equivalent.

Props pbearne, mukesh27.
Fixes #61484.

File:

: 1 edited

trunk/tests/phpunit/tests/option/networkOption.php (modified) (2 diffs)

Legend:

: Unmodified
: Added
: Removed

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

-              r56946
+              r58782
         /**
+         * Tests that calling delete_network_option() updates nooptions when option deleted.
+         *
+         * @ticket 61484
+         *
+         * @covers ::delete_network_option
+         */
+        public function test_check_delete_network_option_updates_notoptions() {
+                add_network_option( 1, 'foo', 'value1' );
+                delete_network_option( 1, 'foo' );
+                $cache_key   = is_multisite() ? '1:notoptions' : 'notoptions';
+                $cache_group = is_multisite() ? 'site-options' : 'options';
+                $notoptions  = wp_cache_get( $cache_key, $cache_group );
+                $this->assertIsArray( $notoptions, 'The notoptions cache is expected to be an array.' );
+                $this->assertTrue( $notoptions['foo'], 'The deleted options is expected to be in notoptions.' );
+                $before = get_num_queries();
+                get_network_option( 1, 'foo' );
+                $queries = get_num_queries() - $before;
+                $this->assertSame( 0, $queries, 'get_network_option should not make any database queries.' );
+        }
+        /**
          * @ticket 22846
          * @group ms-excluded
 …
                 $this->assertSame( $num_queries_pre_update, get_num_queries() );
+        }
+        /**
+         * Tests that calling update_network_option() clears the notoptions cache.
+         *
+         * @ticket 61484
+         *
+         * @covers ::update_network_option
+         */
+        public function test_update_network_option_clears_the_notoptions_cache() {
+                $option_name = 'ticket_61484_option_to_be_created';
+                $cache_key   = is_multisite() ? '1:notoptions' : 'notoptions';
+                $cache_group = is_multisite() ? 'site-options' : 'options';
+                $notoptions  = wp_cache_get( $cache_key, $cache_group );
+                if ( ! is_array( $notoptions ) ) {
+                        $notoptions = array();
+                }
+                $notoptions[ $option_name ] = true;
+                wp_cache_set( $cache_key, $notoptions, $cache_group );
+                $this->assertArrayHasKey( $option_name, wp_cache_get( $cache_key, $cache_group ), 'The "foobar" option should be in the notoptions cache.' );
+                update_network_option( 1, $option_name, 'baz' );
+                $updated_notoptions = wp_cache_get( $cache_key, $cache_group );
+                $this->assertArrayNotHasKey( $option_name, $updated_notoptions, 'The "foobar" option should not be in the notoptions cache after updating it.' );
+        }
+        /**
+         * Tests that calling add_network_option() clears the notoptions cache.
+         *
+         * @ticket 61484
+         *
+         * @covers ::add_network_option
+         */
+        public function test_add_network_option_clears_the_notoptions_cache() {
+                $option_name = 'ticket_61484_option_to_be_created';
+                $cache_key   = is_multisite() ? '1:notoptions' : 'notoptions';
+                $cache_group = is_multisite() ? 'site-options' : 'options';
+                $notoptions  = wp_cache_get( $cache_key, $cache_group );
+                if ( ! is_array( $notoptions ) ) {
+                        $notoptions = array();
+                }
+                $notoptions[ $option_name ] = true;
+                wp_cache_set( $cache_key, $notoptions, $cache_group );
+                $this->assertArrayHasKey( $option_name, wp_cache_get( $cache_key, $cache_group ), 'The "foobar" option should be in the notoptions cache.' );
+                add_network_option( 1, $option_name, 'baz' );
+                $updated_notoptions = wp_cache_get( $cache_key, $cache_group );
+                $this->assertArrayNotHasKey( $option_name, $updated_notoptions, 'The "foobar" option should not be in the notoptions cache after updating it.' );
+        }
+}

Note: See TracChangeset for help on using the changeset viewer.

Make WordPress Core

Context Navigation

Changeset 58782 for trunk/tests/phpunit/tests/option/networkOption.php

Legend:

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

Download in other formats: