Ticket #24686: 24686.2.diff

src/wp-includes/option.php

@@ -675 +675 @@
          * @param mixed  $value     Value of transient.
          * @param string $transient Transient name.
          */
-        return apply_filters( "transient_{$transient}", $value, $transient );
+        $value = apply_filters( "transient_{$transient}", $value, $transient );
+
+        /**
+         * Filters an existing transient's value.
+         *
+         * @since 4.8.0
+         *
+         * @param mixed  $value     Transient value.
+         * @param string $transient The name of the transient.
+         */
+        return apply_filters( 'getted_transient', $value, $transient );
+
 }
 
 /**
@@ -1627 +1638 @@
          * @param mixed  $value     Value of site transient.
          * @param string $transient Transient name.
          */
-        return apply_filters( "site_transient_{$transient}", $value, $transient );
+        $value = apply_filters( "site_transient_{$transient}", $value, $transient );
+
+        /**
+         * Filters the value of an existing site transient.
+         *
+         * @since 4.8.0
+         *
+         * @param mixed  $value     Transient value.
+         * @param string $transient The name of the transient.
+         */
+        return apply_filters( "getted_site_transient", $value, $transient );
 }
 
 /**

tests/phpunit/tests/option/transient.php

@@ -154 +154 @@
                 );
                 $this->assertEquals( $expected, $a->get_events() );
         }
+
+        /**
+         * @ticket 24686
+         */
+        function test_getted_transient_filter() {
+
+                $key = 'test_getted_transient';
+                set_transient( $key, 'test_value', MINUTE_IN_SECONDS );
+
+                $transient = get_transient( $key );
+
+                $this->assertSame( 'test_value', $transient );
+        }
+
+        /**
+         * @ticket 24686
+         */
+        function test_getted_transient_return_false() {
+
+                $key = 'test_getted_transient';
+                set_transient( $key, 'test_value', MINUTE_IN_SECONDS );
+
+                add_filter( 'getted_transient', '__return_false' );
+
+                $transient = get_transient( $key );
+
+                remove_filter( 'getted_transient', '__return_false' );
+
+                $this->assertSame( false, $transient );
+        }
+
+        /**
+         * @ticket 24686
+         */
+        function test_getted_transient_bypass_single_transient() {
+                $key1 = 'test_getted_transient_1';
+                $key2 = 'test_getted_transient_2';
+                set_transient( $key1, 'test_value_1', MINUTE_IN_SECONDS );
+                set_transient( $key2, 'test_value_2', MINUTE_IN_SECONDS );
+
+                add_filter( 'getted_transient', array( $this, 'bypass_first_transient_only' ), 10, 2 );
+
+                $transient_1 = get_transient( $key1 );
+                $transient_2 = get_transient( $key2 );
+
+                remove_filter( 'getted_transient', array( $this, 'bypass_first_transient_only' ) );
+
+                $this->assertSame( false, $transient_1 );
+                $this->assertSame( 'test_value_2', $transient_2 );
+        }
+
+        /**
+         * Callback for the test above.
+         *
+         * @param string $value     The value of the transient.
+         * @param string $transient The name of the transient.
+         *
+         * @return bool|string|mixed
+         */
+        function bypass_first_transient_only( $value, $transient ) {
+                if ( 'test_getted_transient_1' === $transient ) {
+                        $value = false;
+                }
+                return $value;
+        }
+
 }