Ticket #30380: patch-transient-wtests.diff

src/wp-includes/option.php

@@ -622 +622 @@
                         $alloptions = wp_load_alloptions();
                         if ( !isset( $alloptions[$transient_option] ) ) {
                                 $transient_timeout = '_transient_timeout_' . $transient;
-                                if ( get_option( $transient_timeout ) < time() ) {
+                                $timeout = get_option( $transient_timeout );
+                                if ( false !== $timeout && $timeout < time() ) {
                                         delete_option( $transient_option  );
                                         delete_option( $transient_timeout );
                                         $value = false;

tests/phpunit/tests/option/transient.php

@@ -83 +83 @@
                 update_option( '_transient_timeout_' . $key, $now - 1 );
                 $this->assertFalse( get_transient( $key ) );
         }
+
+        /**
+         * If get_option( $transient_timeout ) returns false, don't bother trying to delete the transient.
+         *
+         * @ticket 30380
+         */
+        function test_nonexistent_key_dont_delete_if_false() {
+                // Create a bogus a transient
+                $key = 'test_transient';
+                set_transient( $key, 'test', 60 * 10 );
+                $this->assertEquals( 'test', get_transient( $key ) );
+
+                // Useful variables for tracking
+                $transient_option = '_transient_' . $key;
+                $transient_timeout = '_transient_timeout_' . $key;
+
+                // Make sure the timeout option returns false
+                add_filter( 'option_' . $transient_timeout, '__return_false' );
+
+                // Add some actions to make sure options are _not_ deleted
+                $options = array();
+                add_action( 'delete_option', function( $option ) use ( &$options ) {
+                        $options[] = $option;
+                } );
+
+                // Act
+                get_transient( $key );
+
+                // Make sure delete option was not called for both the transient and the timeout
+                $this->assertNotContains( $transient_timeout, $options );
+                $this->assertNotContains( $transient_option, $options );
+        }
+
+        /**
+         * @ticket 30380
+         */
+        function test_nonexistent_key_old_timeout() {
+                // Create a transient
+                $key = 'test_transient';
+                set_transient( $key, 'test', 60 * 10 );
+                $this->assertEquals( 'test', get_transient( $key ) );
+
+                // Make sure the timeout option returns false
+                $timeout = '_transient_timeout_' . $key;
+                $transient_option = '_transient_' . $key;
+                add_filter( 'option_' . $timeout, function() { return time() - 60; } );
+
+                // Add some actions to make sure options are deleted
+                $options = array();
+                add_action( 'delete_option', function( $option ) use ( &$options ) {
+                        $options[] = $option;
+                } );
+
+                // Act
+                get_transient( $key );
+
+                // Make sure delete option was called for both the transient and the timeout
+                $this->assertContains( $timeout, $options );
+                $this->assertContains( $transient_option, $options );
+        }
 }