WordPress.org

Make WordPress Core


Ignore:
Timestamp:
05/15/2014 04:32:18 PM (7 years ago)
Author:
wonderboymusic
Message:

Eliminate use of extract() in default-widgets.php.

Props rzen, wonderboymusic.

See #22400.

File:
1 edited

Legend:

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

    r28205 r28433  
    2020
    2121    function widget( $args, $instance ) {
    22         extract( $args );
    2322
    2423        /**
     
    5554        ) ) );
    5655
    57         if ( !empty( $out ) ) {
    58             echo $before_widget;
    59             if ( $title)
    60                 echo $before_title . $title . $after_title;
     56        if ( ! empty( $out ) ) {
     57            echo $args['before_widget'];
     58            if ( $title ) {
     59                echo $args['before_title'] . $title . $args['after_title'];
     60            }
    6161        ?>
    6262        <ul>
     
    6464        </ul>
    6565        <?php
    66             echo $after_widget;
     66            echo $args['after_widget'];
    6767        }
    6868    }
     
    120120
    121121    function widget( $args, $instance ) {
    122         extract($args, EXTR_SKIP);
    123122
    124123        $show_description = isset($instance['description']) ? $instance['description'] : false;
     
    131130        $limit = isset( $instance['limit'] ) ? $instance['limit'] : -1;
    132131
    133         $before_widget = preg_replace('/id="[^"]*"/','id="%id"', $before_widget);
     132        $before_widget = preg_replace( '/id="[^"]*"/', 'id="%id"', $args['before_widget'] );
    134133
    135134        /**
     
    143142         */
    144143        wp_list_bookmarks( apply_filters( 'widget_links_args', array(
    145             'title_before' => $before_title, 'title_after' => $after_title,
    146             'category_before' => $before_widget, 'category_after' => $after_widget,
     144            'title_before' => $args['before_title'], 'title_after' => $args['after_title'],
     145            'category_before' => $before_widget, 'category_after' => $args['after_widget'],
    147146            'show_images' => $show_images, 'show_description' => $show_description,
    148147            'show_name' => $show_name, 'show_rating' => $show_rating,
     
    230229
    231230    function widget( $args, $instance ) {
    232         extract($args);
    233231
    234232        /** This filter is documented in wp-includes/default-widgets.php */
    235233        $title = apply_filters( 'widget_title', empty( $instance['title'] ) ? '' : $instance['title'], $instance, $this->id_base );
    236234
    237         echo $before_widget;
    238         if ( $title )
    239             echo $before_title . $title . $after_title;
     235        echo $args['before_widget'];
     236        if ( $title ) {
     237            echo $args['before_title'] . $title . $args['after_title'];
     238        }
    240239
    241240        // Use current theme search form if it exists
    242241        get_search_form();
    243242
    244         echo $after_widget;
     243        echo $args['after_widget'];
    245244    }
    246245
     
    275274
    276275    function widget( $args, $instance ) {
    277         extract($args);
    278276        $c = ! empty( $instance['count'] ) ? '1' : '0';
    279277        $d = ! empty( $instance['dropdown'] ) ? '1' : '0';
    280278
    281279        /** This filter is documented in wp-includes/default-widgets.php */
    282         $title = apply_filters( 'widget_title', empty($instance['title'] ) ? __( 'Archives' ) : $instance['title'], $instance, $this->id_base );
    283 
    284         echo $before_widget;
    285         if ( $title )
    286             echo $before_title . $title . $after_title;
     280        $title = apply_filters( 'widget_title', empty( $instance['title'] ) ? __( 'Archives' ) : $instance['title'], $instance, $this->id_base );
     281
     282        echo $args['before_widget'];
     283        if ( $title ) {
     284            echo $args['before_title'] . $title . $args['after_title'];
     285        }
    287286
    288287        if ( $d ) {
     
    331330        }
    332331
    333         echo $after_widget;
     332        echo $args['after_widget'];
    334333    }
    335334
     
    375374
    376375    function widget( $args, $instance ) {
    377         extract($args);
    378376
    379377        /** This filter is documented in wp-includes/default-widgets.php */
    380378        $title = apply_filters( 'widget_title', empty($instance['title']) ? __( 'Meta' ) : $instance['title'], $instance, $this->id_base );
    381379
    382         echo $before_widget;
    383         if ( $title )
    384             echo $before_title . $title . $after_title;
     380        echo $args['before_widget'];
     381        if ( $title ) {
     382            echo $args['before_title'] . $title . $args['after_title'];
     383        }
    385384?>
    386385            <ul>
     
    407406            </ul>
    408407<?php
    409         echo $after_widget;
     408        echo $args['after_widget'];
    410409    }
    411410
     
    439438
    440439    function widget( $args, $instance ) {
    441         extract($args);
    442440
    443441        /** This filter is documented in wp-includes/default-widgets.php */
    444442        $title = apply_filters( 'widget_title', empty( $instance['title'] ) ? '' : $instance['title'], $instance, $this->id_base );
    445443
    446         echo $before_widget;
    447         if ( $title )
    448             echo $before_title . $title . $after_title;
     444        echo $args['before_widget'];
     445        if ( $title ) {
     446            echo $args['before_title'] . $title . $args['after_title'];
     447        }
    449448        echo '<div id="calendar_wrap">';
    450449        get_calendar();
    451450        echo '</div>';
    452         echo $after_widget;
     451        echo $args['after_widget'];
    453452    }
    454453
     
    484483
    485484    function widget( $args, $instance ) {
    486         extract($args);
    487485
    488486        /** This filter is documented in wp-includes/default-widgets.php */
     
    498496         */
    499497        $text = apply_filters( 'widget_text', empty( $instance['text'] ) ? '' : $instance['text'], $instance );
    500         echo $before_widget;
    501         if ( !empty( $title ) ) { echo $before_title . $title . $after_title; } ?>
     498        echo $args['before_widget'];
     499        if ( ! empty( $title ) ) {
     500            echo $args['before_title'] . $title . $args['after_title'];
     501        } ?>
    502502            <div class="textwidget"><?php echo !empty( $instance['filter'] ) ? wpautop( $text ) : $text; ?></div>
    503503        <?php
    504         echo $after_widget;
     504        echo $args['after_widget'];
    505505    }
    506506
     
    544544
    545545    function widget( $args, $instance ) {
    546         extract( $args );
    547546
    548547        /** This filter is documented in wp-includes/default-widgets.php */
     
    553552        $d = ! empty( $instance['dropdown'] ) ? '1' : '0';
    554553
    555         echo $before_widget;
    556         if ( $title )
    557             echo $before_title . $title . $after_title;
     554        echo $args['before_widget'];
     555        if ( $title ) {
     556            echo $args['before_title'] . $title . $args['after_title'];
     557        }
    558558
    559559        $cat_args = array('orderby' => 'name', 'show_count' => $c, 'hierarchical' => $h);
     
    606606        }
    607607
    608         echo $after_widget;
     608        echo $args['after_widget'];
    609609    }
    610610
     
    680680
    681681        ob_start();
    682         extract($args);
    683682
    684683        $title = ( ! empty( $instance['title'] ) ) ? $instance['title'] : __( 'Recent Posts' );
     
    710709        if ($r->have_posts()) :
    711710?>
    712         <?php echo $before_widget; ?>
    713         <?php if ( $title ) echo $before_title . $title . $after_title; ?>
     711        <?php echo $args['before_widget']; ?>
     712        <?php if ( $title ) {
     713            echo $args['before_title'] . $title . $args['after_title'];
     714        } ?>
    714715        <ul>
    715716        <?php while ( $r->have_posts() ) : $r->the_post(); ?>
     
    722723        <?php endwhile; ?>
    723724        </ul>
    724         <?php echo $after_widget; ?>
     725        <?php echo $args['after_widget']; ?>
    725726<?php
    726727        // Reset the global $the_post as this query will have stomped on it
     
    833834        }
    834835
    835         extract($args, EXTR_SKIP);
    836836        $output = '';
    837837
     
    860860        ) ) );
    861861
    862         $output .= $before_widget;
    863         if ( $title )
    864             $output .= $before_title . $title . $after_title;
     862        $output .= $args['before_widget'];
     863        if ( $title ) {
     864            $output .= $args['before_title'] . $title . $args['after_title'];
     865        }
    865866
    866867        $output .= '<ul id="recentcomments">';
     
    875876        }
    876877        $output .= '</ul>';
    877         $output .= $after_widget;
     878        $output .= $args['after_widget'];
    878879
    879880        echo $output;
     
    12211222
    12221223    function widget( $args, $instance ) {
    1223         extract($args);
    12241224        $current_taxonomy = $this->_get_current_taxonomy($instance);
    12251225        if ( !empty($instance['title']) ) {
     
    12371237        $title = apply_filters( 'widget_title', $title, $instance, $this->id_base );
    12381238
    1239         echo $before_widget;
    1240         if ( $title )
    1241             echo $before_title . $title . $after_title;
     1239        echo $args['before_widget'];
     1240        if ( $title ) {
     1241            echo $args['before_title'] . $title . $args['after_title'];
     1242        }
    12421243        echo '<div class="tagcloud">';
    12431244
     
    12571258
    12581259        echo "</div>\n";
    1259         echo $after_widget;
     1260        echo $args['after_widget'];
    12601261    }
    12611262
     
    13241325
    13251326    function update( $new_instance, $old_instance ) {
    1326         $instance['title'] = strip_tags( stripslashes($new_instance['title']) );
    1327         $instance['nav_menu'] = (int) $new_instance['nav_menu'];
     1327        $instance = array();
     1328        if ( ! empty( $new_instance['title'] ) ) {
     1329            $instance['title'] = strip_tags( stripslashes($new_instance['title']) );
     1330        }
     1331        if ( ! empty( $new_instance['nav_menu'] ) ) {
     1332            $instance['nav_menu'] = (int) $new_instance['nav_menu'];
     1333        }
    13281334        return $instance;
    13291335    }
Note: See TracChangeset for help on using the changeset viewer.