WordPress.org
Get WordPress

Make WordPress Core

Ticket #22400: widgets.diff

File widgets.diff, 11.6 KB (added by wonderboymusic, 12 years ago)

  • src/wp-includes/default-widgets.php

     
    1919        }
    2020
    2121        function widget( $args, $instance ) {
    22                 extract( $args );
    2322
    2423                /**
    2524                 * Filter the widget title.
    … …  
    5453                        'exclude'     => $exclude
    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>
    6363                        <?php echo $out; ?>
    6464                </ul>
    6565                <?php
    66                         echo $after_widget;
     66                        echo $args['after_widget'];
    6767                }
    6868        }
    6969
    … …  
    119119        }
    120120
    121121        function widget( $args, $instance ) {
    122                 extract($args, EXTR_SKIP);
    123122
    124123                $show_description = isset($instance['description']) ? $instance['description'] : false;
    125124                $show_name = isset($instance['name']) ? $instance['name'] : false;
    … …  
    130129                $order = $orderby == 'rating' ? 'DESC' : 'ASC';
    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                /**
    136135                 * Filter the arguments for the Links widget.
    … …  
    142141                 * @param array $args An array of arguments to retrieve the links list.
    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,
    149148                        'category' => $category, 'class' => 'linkcat widget',
    … …  
    229228        }
    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
    247246        function form( $instance ) {
    … …  
    274273        }
    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 );
     280                $title = apply_filters( 'widget_title', empty( $instance['title'] ) ? __( 'Archives' ) : $instance['title'], $instance, $this->id_base );
    283281
    284                 echo $before_widget;
    285                 if ( $title )
    286                         echo $before_title . $title . $after_title;
     282                echo $args['before_widget'];
     283                if ( $title ) {
     284                        echo $args['before_title'] . $title . $args['after_title'];
     285                }
    287286
    288287                if ( $d ) {
    289288?>
    … …  
    330329<?php
    331330                }
    332331
    333                 echo $after_widget;
     332                echo $args['after_widget'];
    334333        }
    335334
    336335        function update( $new_instance, $old_instance ) {
    … …  
    374373        }
    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>
    387386                        <?php wp_register(); ?>
    … …  
    406405?>
    407406                        </ul>
    408407<?php
    409                 echo $after_widget;
     408                echo $args['after_widget'];
    410409        }
    411410
    412411        function update( $new_instance, $old_instance ) {
    … …  
    438437        }
    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
    455454        function update( $new_instance, $old_instance ) {
    … …  
    483482        }
    484483
    485484        function widget( $args, $instance ) {
    486                 extract($args);
    487485
    488486                /** This filter is documented in wp-includes/default-widgets.php */
    489487                $title = apply_filters( 'widget_title', empty( $instance['title'] ) ? '' : $instance['title'], $instance, $this->id_base );
    … …  
    497495                 * @param WP_Widget $instance    WP_Widget instance.
    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
    507507        function update( $new_instance, $old_instance ) {
    … …  
    543543        }
    544544
    545545        function widget( $args, $instance ) {
    546                 extract( $args );
    547546
    548547                /** This filter is documented in wp-includes/default-widgets.php */
    549548                $title = apply_filters( 'widget_title', empty( $instance['title'] ) ? __( 'Categories' ) : $instance['title'], $instance, $this->id_base );
    … …  
    552551                $h = ! empty( $instance['hierarchical'] ) ? '1' : '0';
    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);
    560560
    … …  
    605605<?php
    606606                }
    607607
    608                 echo $after_widget;
     608                echo $args['after_widget'];
    609609        }
    610610
    611611        function update( $new_instance, $old_instance ) {
    … …  
    679679                }
    680680
    681681                ob_start();
    682                 extract($args);
    683682
    684683                $title = ( ! empty( $instance['title'] ) ) ? $instance['title'] : __( 'Recent Posts' );
    685684
    … …  
    709708
    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(); ?>
    716717                        <li>
    … …  
    721722                        </li>
    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
    727728                wp_reset_postdata();
    … …  
    832833                        return;
    833834                }
    834835
    835                 extract($args, EXTR_SKIP);
    836836                $output = '';
    837837
    838838                $title = ( ! empty( $instance['title'] ) ) ? $instance['title'] : __( 'Recent Comments' );
    … …  
    859859                        'post_status' => 'publish'
    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">';
    867868                if ( $comments ) {
    … …  
    874875                        }
    875876                }
    876877                $output .= '</ul>';
    877                 $output .= $after_widget;
     878                $output .= $args['after_widget'];
    878879
    879880                echo $output;
    880881
    … …  
    928929                if ( isset($instance['error']) && $instance['error'] )
    929930                        return;
    930931
    931                 extract($args, EXTR_SKIP);
    932 
    933932                $url = ! empty( $instance['url'] ) ? $instance['url'] : '';
    934933                while ( stristr($url, 'http') != $url )
    935934                        $url = substr($url, 1);
    … …  
    966965                if ( $title )
    967966                        $title = "<a class='rsswidget' href='$url' title='" . esc_attr__( 'Syndicate this content' ) ."'><img style='border:0' width='14' height='14' src='$icon' alt='RSS' /></a> <a class='rsswidget' href='$link' title='$desc'>$title</a>";
    968967
    969                 echo $before_widget;
    970                 if ( $title )
    971                         echo $before_title . $title . $after_title;
     968                echo $args['before_widget'];
     969                if ( $title ) {
     970                        echo $args['before_title'] . $title . $args['after_title'];
     971                }
    972972                wp_widget_rss_output( $rss, $instance );
    973                 echo $after_widget;
     973                echo $args['after_widget'];
    974974
    975975                if ( ! is_wp_error($rss) )
    976976                        $rss->__destruct();
    … …  
    10181018
    10191019        $default_args = array( 'show_author' => 0, 'show_date' => 0, 'show_summary' => 0 );
    10201020        $args = wp_parse_args( $args, $default_args );
    1021         extract( $args, EXTR_SKIP );
    10221021
    1023         $items = (int) $items;
     1022        $items = (int) $args['items'];
    10241023        if ( $items < 1 || 20 < $items )
    10251024                $items = 10;
    1026         $show_summary  = (int) $show_summary;
    1027         $show_author   = (int) $show_author;
    1028         $show_date     = (int) $show_date;
     1025        $show_summary  = (int) $args['show_summary'];
     1026        $show_author   = (int) $args['show_author'];
     1027        $show_date     = (int) $args['show_date'];
    10291028
    10301029        if ( !$rss->get_item_quantity() ) {
    10311030                echo '<ul><li>' . __( 'An error has occurred, which probably means the feed is down. Try again later.' ) . '</li></ul>';
    … …  
    11101109
    11111110        $default_inputs = array( 'url' => true, 'title' => true, 'items' => true, 'show_summary' => true, 'show_author' => true, 'show_date' => true );
    11121111        $inputs = wp_parse_args( $inputs, $default_inputs );
    1113         extract( $args );
    1114         extract( $inputs, EXTR_SKIP );
    11151112
    1116         $number = esc_attr( $number );
    1117         $title  = esc_attr( $title );
    1118         $url    = esc_url( $url );
    1119         $items  = (int) $items;
    1120         if ( $items < 1 || 20 < $items )
     1113        $number = esc_attr( $inputs['number'] );
     1114        $title  = esc_attr( $inputs['title'] );
     1115        $url    = esc_url( $inputs['url'] );
     1116        $items  = (int) $inputs['items'];
     1117        if ( $items < 1 || 20 < $items ) {
    11211118                $items  = 10;
    1122         $show_summary   = (int) $show_summary;
    1123         $show_author    = (int) $show_author;
    1124         $show_date      = (int) $show_date;
     1119        }
     1120        $show_summary   = (int) $inputs['show_summary'];
     1121        $show_author    = (int) $inputs['show_author'];
     1122        $show_date      = (int) $inputs['show_date'];
    11251123
    11261124        if ( !empty($error) )
    11271125                echo '<p class="widget-error"><strong>' . sprintf( __('RSS Error: %s'), $error) . '</strong></p>';
    … …  
    12201218        }
    12211219
    12221220        function widget( $args, $instance ) {
    1223                 extract($args);
    12241221                $current_taxonomy = $this->_get_current_taxonomy($instance);
    12251222                if ( !empty($instance['title']) ) {
    12261223                        $title = $instance['title'];
    … …  
    12361233                /** This filter is documented in wp-includes/default-widgets.php */
    12371234                $title = apply_filters( 'widget_title', $title, $instance, $this->id_base );
    12381235
    1239                 echo $before_widget;
    1240                 if ( $title )
    1241                         echo $before_title . $title . $after_title;
     1236                echo $args['before_widget'];
     1237                if ( $title ) {
     1238                        echo $args['before_title'] . $title . $args['after_title'];
     1239                }
    12421240                echo '<div class="tagcloud">';
    12431241
    12441242                /**
    … …  
    12561254                ) ) );
    12571255
    12581256                echo "</div>\n";
    1259                 echo $after_widget;
     1257                echo $args['after_widget'];
    12601258        }
    12611259
    12621260        function update( $new_instance, $old_instance ) {