Ticket #15058: 15058.1.diff

src/wp-includes/option.php

@@ -240 +240 @@
         if ( is_object( $value ) )
                 $value = clone $value;
 
+        if( strlen( $option ) > 64 ) {
+                $option = md5( $option );
+                $option = "md5_" . $option;
+        }
+
         $value = sanitize_option( $option, $value );
         $old_value = get_option( $option );
 
@@ -377 +382 @@
         $serialized_value = maybe_serialize( $value );
         $autoload = ( 'no' === $autoload ) ? 'no' : 'yes';
 
+        if( strlen( $option ) > 64 ) {
+                $option = md5( $option );
+                $option = "md5_" . $option;
+        }
+
         /**
          * Fires before an option is added.
          *
@@ -582 +592 @@
         if ( wp_using_ext_object_cache() ) {
                 $value = wp_cache_get( $transient, 'transient' );
         } else {
+
+                if( strlen( $transient ) > 40 ) {
+                        $transient = md5( $transient );
+                        $transient = "md5_" . $transient;
+                }
+
                 $transient_option = '_transient_' . $transient;
                 if ( ! defined( 'WP_INSTALLING' ) ) {
                         // If option is not in alloptions, it is not autoloaded and thus has a timeout
@@ -645 +661 @@
         if ( wp_using_ext_object_cache() ) {
                 $result = wp_cache_set( $transient, $value, 'transient', $expiration );
         } else {
+
+                if( strlen( $transient ) > 40 ) {
+                        $transient = md5( $transient );
+                        $transient = "md5_" . $transient;
+                }
+
                 $transient_timeout = '_transient_timeout_' . $transient;
                 $transient = '_transient_' . $transient;
+
                 if ( false === get_option( $transient ) ) {
                         $autoload = 'yes';
                         if ( $expiration ) {

tests/phpunit/tests/option/transient.php

@@ -83 +83 @@
                 update_option( '_transient_timeout_' . $key, $now - 1 );
                 $this->assertFalse( get_transient( $key ) );
         }
+
+        /**
+         * ticket 15058
+         */
+        function test_transient_long() {
+                $long_name = "thisisaverylongnamethatwillnotfitincolumnbecauseitistoolongforthecolumn";
+                set_transient( $long_name, true, 60 );
+                $this->assertTrue( get_transient( $long_name ) );
+        }
 }