WordPress.org

Make WordPress Core

Changeset 36130


Ignore:
Timestamp:
12/31/2015 03:12:36 AM (4 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:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/widgets.php

    r35992 r36130  
    617617        $index = "sidebar-$index";
    618618    } else {
    619         $sanitized_index = sanitize_title( $index );
     619        $index = sanitize_title( $index );
    620620        foreach ( (array) $wp_registered_sidebars as $key => $value ) {
    621             if ( sanitize_title( $value['name'] ) == $sanitized_index ) {
     621            if ( sanitize_title( $value['name'] ) == $index ) {
    622622                $index = $key;
    623623                break;
  • trunk/tests/phpunit/tests/widgets.php

    r35272 r36130  
    534534        $option_value = get_option( $wp_widget_search->option_name );
    535535        $this->assertArrayNotHasKey( 2, $option_value );
    536     }
    537 
    538     /**
    539      * @ticket 23423
    540      */
    541     function test_dynamic_sidebar_id_special_characters() {
    542         wp_widgets_init();
    543         register_sidebar( array(
    544             'name' => 'Sidebar 2',
    545             'id' => 'sidebar-2',
    546         ) );
    547 
    548         ob_start();
    549         $result = dynamic_sidebar( 'Sidebar 1' );
    550         ob_end_clean();
    551 
    552         $this->assertFalse( $result );
    553536    }
    554537
Note: See TracChangeset for help on using the changeset viewer.