From fdaea6b7f250d066cf54ea361fe9097c17d968f9 Mon Sep 17 00:00:00 2001 From: Scott Taylor Date: Fri, 16 May 2014 15:38:14 +0000 Subject: [PATCH] Eliminate use of `extract()` in `wp_widget_rss_form()`. See #22400. Built from https://develop.svn.wordpress.org/trunk@28449 git-svn-id: http://core.svn.wordpress.org/trunk@28276 1a063a9b-81f0-0310-95a4-ce76da25c4cd --- wp-includes/default-widgets.php | 25 ++++++++++++------------- 1 file changed, 12 insertions(+), 13 deletions(-) diff --git a/wp-includes/default-widgets.php b/wp-includes/default-widgets.php index dc8481305d..29bae94373 100644 --- a/wp-includes/default-widgets.php +++ b/wp-includes/default-widgets.php @@ -1106,24 +1106,23 @@ function wp_widget_rss_output( $rss, $args = array() ) { * @param array $inputs Override default display options. */ function wp_widget_rss_form( $args, $inputs = null ) { - $default_inputs = array( 'url' => true, 'title' => true, 'items' => true, 'show_summary' => true, 'show_author' => true, 'show_date' => true ); $inputs = wp_parse_args( $inputs, $default_inputs ); - extract( $args ); - extract( $inputs, EXTR_SKIP ); - $number = esc_attr( $number ); - $title = esc_attr( $title ); - $url = esc_url( $url ); - $items = (int) $items; - if ( $items < 1 || 20 < $items ) + $number = esc_attr( $args['number'] ); + $title = isset( $args['title'] ) ? esc_attr( $args['title'] ) : ''; + $url = isset( $args['url'] ) ? esc_url( $args['url'] ) : ''; + $items = isset( $args['items'] ) ? (int) $args['items'] : 0; + if ( $items < 1 || 20 < $items ) { $items = 10; - $show_summary = (int) $show_summary; - $show_author = (int) $show_author; - $show_date = (int) $show_date; + } + $show_summary = isset( $args['show_summary'] ) ? (int) $args['show_summary'] : (int) $inputs['show_summary']; + $show_author = isset( $args['show_author'] ) ? (int) $args['show_author'] : (int) $inputs['show_author']; + $show_date = isset( $args['show_date'] ) ? (int) $args['show_date'] : (int) $inputs['show_date']; - if ( !empty($error) ) - echo '

' . sprintf( __('RSS Error: %s'), $error) . '

'; + if ( ! empty( $args['error'] ) ) { + echo '

' . sprintf( __( 'RSS Error: %s' ), $args['error'] ) . '

'; + } if ( $inputs['url'] ) : ?>