Changeset 36130

Timestamp:

12/31/2015 03:12:36 AM (11 years ago)

Author:

SergeyBiryukov

Message:

Widgets: Revert [34465], as it introduced a regression, making the $index argument of dynamic_sidebar() case-sensitive.

Fixes #34995 for trunk. See #23423.

Location:

trunk

Files:

• src/wp-includes/widgets.php (modified) (1 diff)
• tests/phpunit/tests/widgets.php (modified) (1 diff)

trunk/src/wp-includes/widgets.php

@@ -617 +617 @@
                 $index = "sidebar-$index";
         } else {
-                $sanitized_index = sanitize_title( $index );
+                $index = sanitize_title( $index );
                 foreach ( (array) $wp_registered_sidebars as $key => $value ) {
-                        if ( sanitize_title( $value['name'] ) == $sanitized_index ) {
+                        if ( sanitize_title( $value['name'] ) == $index ) {
                                 $index = $key;
                                 break;

trunk/tests/phpunit/tests/widgets.php

@@ -534 +534 @@
                 $option_value = get_option( $wp_widget_search->option_name );
                 $this->assertArrayNotHasKey( 2, $option_value );
-        }
-
-        /**
-         * @ticket 23423
-         */
-        function test_dynamic_sidebar_id_special_characters() {
-                wp_widgets_init();
-                register_sidebar( array(
-                        'name' => 'Sidebar 2',
-                        'id' => 'sidebar-2',
-                ) );
-
-                ob_start();
-                $result = dynamic_sidebar( 'Sidebar 1' );
-                ob_end_clean();
-
-                $this->assertFalse( $result );
         }