WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (9 years ago)
Author:
pento
Message:

Code is Poetry.
WordPress' code just... wasn't.
This is now dealt with.

Props jrf, pento, netweb, GaryJ, jdgrimes, westonruter, Greg Sherwood from PHPCS, and everyone who's ever contributed to WPCS and PHPCS.
Fixes #41057.

File:
1 edited

Legend:

Unmodified
Added
Removed

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

    r41942 r42343  
    8888        'wp_widget_rss_control',
    8989        'wp_widget_recent_comments',
    90         'wp_widget_recent_comments_control'
     90        'wp_widget_recent_comments_control',
    9191);
    9292
    … …  
    168168        $number = (int) $number;
    169169
    170         if ( is_string($args) )
    171                 parse_str($args, $args);
     170        if ( is_string( $args ) ) {
     171                parse_str( $args, $args );
     172        }
    172173
    173174        for ( $i = 1; $i <= $number; $i++ ) {
    174175                $_args = $args;
    175176
    176                 if ( $number > 1 )
    177                         $_args['name'] = isset($args['name']) ? sprintf($args['name'], $i) : sprintf(__('Sidebar %d'), $i);
    178                 else
    179                         $_args['name'] = isset($args['name']) ? $args['name'] : __('Sidebar');
     177                if ( $number > 1 ) {
     178                        $_args['name'] = isset( $args['name'] ) ? sprintf( $args['name'], $i ) : sprintf( __( 'Sidebar %d' ), $i );
     179                } else {
     180                        $_args['name'] = isset( $args['name'] ) ? $args['name'] : __( 'Sidebar' );
     181                }
    180182
    181183                // Custom specified ID's are suffixed if they exist already.
    182184                // Automatically generated sidebar names need to be suffixed regardless starting at -0
    183                 if ( isset($args['id']) ) {
     185                if ( isset( $args['id'] ) ) {
    184186                        $_args['id'] = $args['id'];
    185                         $n = 2; // Start at -2 for conflicting custom ID's
     187                        $n           = 2; // Start at -2 for conflicting custom ID's
    186188                        while ( is_registered_sidebar( $_args['id'] ) ) {
    187189                                $_args['id'] = $args['id'] . '-' . $n++;
    … …  
    193195                        } while ( is_registered_sidebar( $_args['id'] ) );
    194196                }
    195                 register_sidebar($_args);
     197                register_sidebar( $_args );
    196198        }
    197199}
    … …  
    238240 * @return string Sidebar ID added to $wp_registered_sidebars global.
    239241 */
    240 function register_sidebar($args = array()) {
     242function register_sidebar( $args = array() ) {
    241243        global $wp_registered_sidebars;
    242244
    243         $i = count($wp_registered_sidebars) + 1;
     245        $i = count( $wp_registered_sidebars ) + 1;
    244246
    245247        $id_is_empty = empty( $args['id'] );
    246248
    247249        $defaults = array(
    248                 'name' => sprintf(__('Sidebar %d'), $i ),
    249                 'id' => "sidebar-$i",
    250                 'description' => '',
    251                 'class' => '',
     250                'name'          => sprintf( __( 'Sidebar %d' ), $i ),
     251                'id'            => "sidebar-$i",
     252                'description'   => '',
     253                'class'         => '',
    252254                'before_widget' => '<li id="%1$s" class="widget %2$s">',
    253                 'after_widget' => "</li>\n",
    254                 'before_title' => '<h2 class="widgettitle">',
    255                 'after_title' => "</h2>\n",
     255                'after_widget'  => "</li>\n",
     256                'before_title'  => '<h2 class="widgettitle">',
     257                'after_title'   => "</h2>\n",
    256258        );
    257259
    … …  
    263265        }
    264266
    265         $wp_registered_sidebars[$sidebar['id']] = $sidebar;
    266 
    267         add_theme_support('widgets');
     267        $wp_registered_sidebars[ $sidebar['id'] ] = $sidebar;
     268
     269        add_theme_support( 'widgets' );
    268270
    269271        /**
    … …  
    340342        global $wp_registered_widgets, $wp_registered_widget_controls, $wp_registered_widget_updates, $_wp_deprecated_widgets_callbacks;
    341343
    342         $id = strtolower($id);
    343 
    344         if ( empty($output_callback) ) {
    345                 unset($wp_registered_widgets[$id]);
     344        $id = strtolower( $id );
     345
     346        if ( empty( $output_callback ) ) {
     347                unset( $wp_registered_widgets[ $id ] );
    346348                return;
    347349        }
    348350
    349         $id_base = _get_widget_id_base($id);
    350         if ( in_array($output_callback, $_wp_deprecated_widgets_callbacks, true) && !is_callable($output_callback) ) {
     351        $id_base = _get_widget_id_base( $id );
     352        if ( in_array( $output_callback, $_wp_deprecated_widgets_callbacks, true ) && ! is_callable( $output_callback ) ) {
    351353                unset( $wp_registered_widget_controls[ $id ] );
    352354                unset( $wp_registered_widget_updates[ $id_base ] );
    … …  
    354356        }
    355357
    356         $defaults = array('classname' => $output_callback);
    357         $options = wp_parse_args($options, $defaults);
    358         $widget = array(
    359                 'name' => $name,
    360                 'id' => $id,
     358        $defaults = array( 'classname' => $output_callback );
     359        $options  = wp_parse_args( $options, $defaults );
     360        $widget   = array(
     361                'name'     => $name,
     362                'id'       => $id,
    361363                'callback' => $output_callback,
    362                 'params' => array_slice(func_get_args(), 4)
     364                'params'   => array_slice( func_get_args(), 4 ),
    363365        );
    364         $widget = array_merge($widget, $options);
    365 
    366         if ( is_callable($output_callback) && ( !isset($wp_registered_widgets[$id]) || did_action( 'widgets_init' ) ) ) {
     366        $widget   = array_merge( $widget, $options );
     367
     368        if ( is_callable( $output_callback ) && ( ! isset( $wp_registered_widgets[ $id ] ) || did_action( 'widgets_init' ) ) ) {
    367369
    368370                /**
    … …  
    374376                 */
    375377                do_action( 'wp_register_sidebar_widget', $widget );
    376                 $wp_registered_widgets[$id] = $widget;
     378                $wp_registered_widgets[ $id ] = $widget;
    377379        }
    378380}
    … …  
    393395 */
    394396function wp_widget_description( $id ) {
    395         if ( !is_scalar($id) )
     397        if ( ! is_scalar( $id ) ) {
    396398                return;
     399        }
    397400
    398401        global $wp_registered_widgets;
    399402
    400         if ( isset($wp_registered_widgets[$id]['description']) )
    401                 return esc_html( $wp_registered_widgets[$id]['description'] );
     403        if ( isset( $wp_registered_widgets[ $id ]['description'] ) ) {
     404                return esc_html( $wp_registered_widgets[ $id ]['description'] );
     405        }
    402406}
    403407
    … …  
    416420 */
    417421function wp_sidebar_description( $id ) {
    418         if ( !is_scalar($id) )
     422        if ( ! is_scalar( $id ) ) {
    419423                return;
     424        }
    420425
    421426        global $wp_registered_sidebars;
    422427
    423         if ( isset($wp_registered_sidebars[$id]['description']) )
    424                 return esc_html( $wp_registered_sidebars[$id]['description'] );
     428        if ( isset( $wp_registered_sidebars[ $id ]['description'] ) ) {
     429                return esc_html( $wp_registered_sidebars[ $id ]['description'] );
     430        }
    425431}
    426432
    … …  
    432438 * @param int|string $id Widget ID.
    433439 */
    434 function wp_unregister_sidebar_widget($id) {
     440function wp_unregister_sidebar_widget( $id ) {
    435441
    436442        /**
    … …  
    443449        do_action( 'wp_unregister_sidebar_widget', $id );
    444450
    445         wp_register_sidebar_widget($id, '', '');
    446         wp_unregister_widget_control($id);
     451        wp_register_sidebar_widget( $id, '', '' );
     452        wp_unregister_widget_control( $id );
    447453}
    448454
    … …  
    475481        global $wp_registered_widget_controls, $wp_registered_widget_updates, $wp_registered_widgets, $_wp_deprecated_widgets_callbacks;
    476482
    477         $id = strtolower($id);
    478         $id_base = _get_widget_id_base($id);
    479 
    480         if ( empty($control_callback) ) {
    481                 unset($wp_registered_widget_controls[$id]);
    482                 unset($wp_registered_widget_updates[$id_base]);
     483        $id      = strtolower( $id );
     484        $id_base = _get_widget_id_base( $id );
     485
     486        if ( empty( $control_callback ) ) {
     487                unset( $wp_registered_widget_controls[ $id ] );
     488                unset( $wp_registered_widget_updates[ $id_base ] );
    483489                return;
    484490        }
    485491
    486         if ( in_array($control_callback, $_wp_deprecated_widgets_callbacks, true) && !is_callable($control_callback) ) {
     492        if ( in_array( $control_callback, $_wp_deprecated_widgets_callbacks, true ) && ! is_callable( $control_callback ) ) {
    487493                unset( $wp_registered_widgets[ $id ] );
    488494                return;
    489495        }
    490496
    491         if ( isset($wp_registered_widget_controls[$id]) && !did_action( 'widgets_init' ) )
     497        if ( isset( $wp_registered_widget_controls[ $id ] ) && ! did_action( 'widgets_init' ) ) {
    492498                return;
    493 
    494         $defaults = array('width' => 250, 'height' => 200 ); // height is never used
    495         $options = wp_parse_args($options, $defaults);
    496         $options['width'] = (int) $options['width'];
     499        }
     500
     501        $defaults          = array(
     502                'width'  => 250,
     503                'height' => 200,
     504        ); // height is never used
     505        $options           = wp_parse_args( $options, $defaults );
     506        $options['width']  = (int) $options['width'];
    497507        $options['height'] = (int) $options['height'];
    498508
    499509        $widget = array(
    500                 'name' => $name,
    501                 'id' => $id,
     510                'name'     => $name,
     511                'id'       => $id,
    502512                'callback' => $control_callback,
    503                 'params' => array_slice(func_get_args(), 4)
     513                'params'   => array_slice( func_get_args(), 4 ),
    504514        );
    505         $widget = array_merge($widget, $options);
    506 
    507         $wp_registered_widget_controls[$id] = $widget;
    508 
    509         if ( isset($wp_registered_widget_updates[$id_base]) )
     515        $widget = array_merge( $widget, $options );
     516
     517        $wp_registered_widget_controls[ $id ] = $widget;
     518
     519        if ( isset( $wp_registered_widget_updates[ $id_base ] ) ) {
    510520                return;
    511 
    512         if ( isset($widget['params'][0]['number']) )
     521        }
     522
     523        if ( isset( $widget['params'][0]['number'] ) ) {
    513524                $widget['params'][0]['number'] = -1;
    514 
    515         unset($widget['width'], $widget['height'], $widget['name'], $widget['id']);
    516         $wp_registered_widget_updates[$id_base] = $widget;
     525        }
     526
     527        unset( $widget['width'], $widget['height'], $widget['name'], $widget['id'] );
     528        $wp_registered_widget_updates[ $id_base ] = $widget;
    517529}
    518530
    … …  
    532544        global $wp_registered_widget_updates;
    533545
    534         if ( isset($wp_registered_widget_updates[$id_base]) ) {
    535                 if ( empty($update_callback) )
    536                         unset($wp_registered_widget_updates[$id_base]);
     546        if ( isset( $wp_registered_widget_updates[ $id_base ] ) ) {
     547                if ( empty( $update_callback ) ) {
     548                        unset( $wp_registered_widget_updates[ $id_base ] );
     549                }
    537550                return;
    538551        }
    … …  
    540553        $widget = array(
    541554                'callback' => $update_callback,
    542                 'params' => array_slice(func_get_args(), 3)
     555                'params'   => array_slice( func_get_args(), 3 ),
    543556        );
    544557
    545         $widget = array_merge($widget, $options);
    546         $wp_registered_widget_updates[$id_base] = $widget;
     558        $widget                                   = array_merge( $widget, $options );
     559        $wp_registered_widget_updates[ $id_base ] = $widget;
    547560}
    548561
    … …  
    560573 *                                  Default empty array.
    561574 */
    562 function _register_widget_form_callback($id, $name, $form_callback, $options = array()) {
     575function _register_widget_form_callback( $id, $name, $form_callback, $options = array() ) {
    563576        global $wp_registered_widget_controls;
    564577
    565         $id = strtolower($id);
    566 
    567         if ( empty($form_callback) ) {
    568                 unset($wp_registered_widget_controls[$id]);
     578        $id = strtolower( $id );
     579
     580        if ( empty( $form_callback ) ) {
     581                unset( $wp_registered_widget_controls[ $id ] );
    569582                return;
    570583        }
    571584
    572         if ( isset($wp_registered_widget_controls[$id]) && !did_action( 'widgets_init' ) )
     585        if ( isset( $wp_registered_widget_controls[ $id ] ) && ! did_action( 'widgets_init' ) ) {
    573586                return;
    574 
    575         $defaults = array('width' => 250, 'height' => 200 );
    576         $options = wp_parse_args($options, $defaults);
    577         $options['width'] = (int) $options['width'];
     587        }
     588
     589        $defaults          = array(
     590                'width'  => 250,
     591                'height' => 200,
     592        );
     593        $options           = wp_parse_args( $options, $defaults );
     594        $options['width']  = (int) $options['width'];
    578595        $options['height'] = (int) $options['height'];
    579596
    580597        $widget = array(
    581                 'name' => $name,
    582                 'id' => $id,
     598                'name'     => $name,
     599                'id'       => $id,
    583600                'callback' => $form_callback,
    584                 'params' => array_slice(func_get_args(), 4)
     601                'params'   => array_slice( func_get_args(), 4 ),
    585602        );
    586         $widget = array_merge($widget, $options);
    587 
    588         $wp_registered_widget_controls[$id] = $widget;
     603        $widget = array_merge( $widget, $options );
     604
     605        $wp_registered_widget_controls[ $id ] = $widget;
    589606}
    590607
    … …  
    596613 * @param int|string $id Widget ID.
    597614 */
    598 function wp_unregister_widget_control($id) {
     615function wp_unregister_widget_control( $id ) {
    599616        wp_register_widget_control( $id, '', '' );
    600617}
    … …  
    635652                do_action( 'dynamic_sidebar_before', $index, false );
    636653                /** This action is documented in wp-includes/widget.php */
    637                 do_action( 'dynamic_sidebar_after',  $index, false );
     654                do_action( 'dynamic_sidebar_after', $index, false );
    638655                /** This filter is documented in wp-includes/widget.php */
    639656                return apply_filters( 'dynamic_sidebar_has_widgets', false, $index );
    … …  
    653670         */
    654671        do_action( 'dynamic_sidebar_before', $index, true );
    655         $sidebar = $wp_registered_sidebars[$index];
     672        $sidebar = $wp_registered_sidebars[ $index ];
    656673
    657674        $did_one = false;
    658         foreach ( (array) $sidebars_widgets[$index] as $id ) {
    659 
    660                 if ( !isset($wp_registered_widgets[$id]) ) continue;
     675        foreach ( (array) $sidebars_widgets[ $index ] as $id ) {
     676
     677                if ( ! isset( $wp_registered_widgets[ $id ] ) ) {
     678                        continue;
     679                }
    661680
    662681                $params = array_merge(
    663                         array( array_merge( $sidebar, array('widget_id' => $id, 'widget_name' => $wp_registered_widgets[$id]['name']) ) ),
    664                         (array) $wp_registered_widgets[$id]['params']
     682                        array(
     683                                array_merge(
     684                                        $sidebar, array(
     685                                                'widget_id'   => $id,
     686                                                'widget_name' => $wp_registered_widgets[ $id ]['name'],
     687                                        )
     688                                ),
     689                        ),
     690                        (array) $wp_registered_widgets[ $id ]['params']
    665691                );
    666692
    667693                // Substitute HTML id and class attributes into before_widget
    668694                $classname_ = '';
    669                 foreach ( (array) $wp_registered_widgets[$id]['classname'] as $cn ) {
    670                         if ( is_string($cn) )
     695                foreach ( (array) $wp_registered_widgets[ $id ]['classname'] as $cn ) {
     696                        if ( is_string( $cn ) ) {
    671697                                $classname_ .= '_' . $cn;
    672                         elseif ( is_object($cn) )
    673                                 $classname_ .= '_' . get_class($cn);
    674                 }
    675                 $classname_ = ltrim($classname_, '_');
    676                 $params[0]['before_widget'] = sprintf($params[0]['before_widget'], $id, $classname_);
     698                        } elseif ( is_object( $cn ) ) {
     699                                $classname_ .= '_' . get_class( $cn );
     700                        }
     701                }
     702                $classname_                 = ltrim( $classname_, '_' );
     703                $params[0]['before_widget'] = sprintf( $params[0]['before_widget'], $id, $classname_ );
    677704
    678705                /**
    … …  
    710737                $params = apply_filters( 'dynamic_sidebar_params', $params );
    711738
    712                 $callback = $wp_registered_widgets[$id]['callback'];
     739                $callback = $wp_registered_widgets[ $id ]['callback'];
    713740
    714741                /**
    … …  
    739766                do_action( 'dynamic_sidebar', $wp_registered_widgets[ $id ] );
    740767
    741                 if ( is_callable($callback) ) {
    742                         call_user_func_array($callback, $params);
     768                if ( is_callable( $callback ) ) {
     769                        call_user_func_array( $callback, $params );
    743770                        $did_one = true;
    744771                }
    … …  
    802829        $sidebars_widgets = wp_get_sidebars_widgets();
    803830
    804         if ( is_array($sidebars_widgets) ) {
     831        if ( is_array( $sidebars_widgets ) ) {
    805832                foreach ( $sidebars_widgets as $sidebar => $widgets ) {
    806833                        if ( $skip_inactive && ( 'wp_inactive_widgets' === $sidebar || 'orphaned_widgets' === substr( $sidebar, 0, 16 ) ) ) {
    … …  
    808835                        }
    809836
    810                         if ( is_array($widgets) ) {
     837                        if ( is_array( $widgets ) ) {
    811838                                foreach ( $widgets as $widget ) {
    812                                         if ( ( $callback && isset($wp_registered_widgets[$widget]['callback']) && $wp_registered_widgets[$widget]['callback'] == $callback ) || ( $id_base && _get_widget_id_base($widget) == $id_base ) ) {
    813                                                 if ( !$widget_id || $widget_id == $wp_registered_widgets[$widget]['id'] )
     839                                        if ( ( $callback && isset( $wp_registered_widgets[ $widget ]['callback'] ) && $wp_registered_widgets[ $widget ]['callback'] == $callback ) || ( $id_base && _get_widget_id_base( $widget ) == $id_base ) ) {
     840                                                if ( ! $widget_id || $widget_id == $wp_registered_widgets[ $widget ]['id'] ) {
    814841                                                        return $sidebar;
     842                                                }
    815843                                        }
    816844                                }
    … …  
    833861function is_dynamic_sidebar() {
    834862        global $wp_registered_widgets, $wp_registered_sidebars;
    835         $sidebars_widgets = get_option('sidebars_widgets');
     863        $sidebars_widgets = get_option( 'sidebars_widgets' );
    836864        foreach ( (array) $wp_registered_sidebars as $index => $sidebar ) {
    837865                if ( ! empty( $sidebars_widgets[ $index ] ) ) {
    838                         foreach ( (array) $sidebars_widgets[$index] as $widget )
    839                                 if ( array_key_exists($widget, $wp_registered_widgets) )
     866                        foreach ( (array) $sidebars_widgets[ $index ] as $widget ) {
     867                                if ( array_key_exists( $widget, $wp_registered_widgets ) ) {
    840868                                        return true;
     869                                }
     870                        }
    841871                }
    842872        }
    … …  
    853883 */
    854884function is_active_sidebar( $index ) {
    855         $index = ( is_int($index) ) ? "sidebar-$index" : sanitize_title($index);
    856         $sidebars_widgets = wp_get_sidebars_widgets();
    857         $is_active_sidebar = ! empty( $sidebars_widgets[$index] );
     885        $index             = ( is_int( $index ) ) ? "sidebar-$index" : sanitize_title( $index );
     886        $sidebars_widgets  = wp_get_sidebars_widgets();
     887        $is_active_sidebar = ! empty( $sidebars_widgets[ $index ] );
    858888
    859889        /**
    … …  
    889919 */
    890920function wp_get_sidebars_widgets( $deprecated = true ) {
    891         if ( $deprecated !== true )
     921        if ( $deprecated !== true ) {
    892922                _deprecated_argument( __FUNCTION__, '2.8.1' );
     923        }
    893924
    894925        global $_wp_sidebars_widgets, $sidebars_widgets;
    … …  
    896927        // If loading from front page, consult $_wp_sidebars_widgets rather than options
    897928        // to see if wp_convert_widget_settings() has made manipulations in memory.
    898         if ( !is_admin() ) {
    899                 if ( empty($_wp_sidebars_widgets) )
    900                         $_wp_sidebars_widgets = get_option('sidebars_widgets', array());
     929        if ( ! is_admin() ) {
     930                if ( empty( $_wp_sidebars_widgets ) ) {
     931                        $_wp_sidebars_widgets = get_option( 'sidebars_widgets', array() );
     932                }
    901933
    902934                $sidebars_widgets = $_wp_sidebars_widgets;
    903935        } else {
    904                 $sidebars_widgets = get_option('sidebars_widgets', array());
    905         }
    906 
    907         if ( is_array( $sidebars_widgets ) && isset($sidebars_widgets['array_version']) )
    908                 unset($sidebars_widgets['array_version']);
     936                $sidebars_widgets = get_option( 'sidebars_widgets', array() );
     937        }
     938
     939        if ( is_array( $sidebars_widgets ) && isset( $sidebars_widgets['array_version'] ) ) {
     940                unset( $sidebars_widgets['array_version'] );
     941        }
    909942
    910943        /**
    … …  
    955988        $defaults = array();
    956989
    957         foreach ( (array) $wp_registered_sidebars as $index => $sidebar )
    958                 $defaults[$index] = array();
     990        foreach ( (array) $wp_registered_sidebars as $index => $sidebar ) {
     991                $defaults[ $index ] = array();
     992        }
    959993
    960994        return $defaults;
    … …  
    9731007 * @return array
    9741008 */
    975 function wp_convert_widget_settings($base_name, $option_name, $settings) {
     1009function wp_convert_widget_settings( $base_name, $option_name, $settings ) {
    9761010        // This test may need expanding.
    9771011        $single = $changed = false;
    978         if ( empty($settings) ) {
     1012        if ( empty( $settings ) ) {
    9791013                $single = true;
    9801014        } else {
    981                 foreach ( array_keys($settings) as $number ) {
    982                         if ( 'number' == $number )
     1015                foreach ( array_keys( $settings ) as $number ) {
     1016                        if ( 'number' == $number ) {
    9831017                                continue;
    984                         if ( !is_numeric($number) ) {
     1018                        }
     1019                        if ( ! is_numeric( $number ) ) {
    9851020                                $single = true;
    9861021                                break;
    … …  
    9941029                // If loading from the front page, update sidebar in memory but don't save to options
    9951030                if ( is_admin() ) {
    996                         $sidebars_widgets = get_option('sidebars_widgets');
     1031                        $sidebars_widgets = get_option( 'sidebars_widgets' );
    9971032                } else {
    998                         if ( empty($GLOBALS['_wp_sidebars_widgets']) )
    999                                 $GLOBALS['_wp_sidebars_widgets'] = get_option('sidebars_widgets', array());
     1033                        if ( empty( $GLOBALS['_wp_sidebars_widgets'] ) ) {
     1034                                $GLOBALS['_wp_sidebars_widgets'] = get_option( 'sidebars_widgets', array() );
     1035                        }
    10001036                        $sidebars_widgets = &$GLOBALS['_wp_sidebars_widgets'];
    10011037                }
    10021038
    10031039                foreach ( (array) $sidebars_widgets as $index => $sidebar ) {
    1004                         if ( is_array($sidebar) ) {
     1040                        if ( is_array( $sidebar ) ) {
    10051041                                foreach ( $sidebar as $i => $name ) {
    10061042                                        if ( $base_name == $name ) {
    1007                                                 $sidebars_widgets[$index][$i] = "$name-2";
    1008                                                 $changed = true;
     1043                                                $sidebars_widgets[ $index ][ $i ] = "$name-2";
     1044                                                $changed                          = true;
    10091045                                                break 2;
    10101046                                        }
    … …  
    10131049                }
    10141050
    1015                 if ( is_admin() && $changed )
    1016                         update_option('sidebars_widgets', $sidebars_widgets);
     1051                if ( is_admin() && $changed ) {
     1052                        update_option( 'sidebars_widgets', $sidebars_widgets );
     1053                }
    10171054        }
    10181055
    10191056        $settings['_multiwidget'] = 1;
    1020         if ( is_admin() )
     1057        if ( is_admin() ) {
    10211058                update_option( $option_name, $settings );
     1059        }
    10221060
    10231061        return $settings;
    … …  
    10551093        }
    10561094
    1057         $widget_obj = $wp_widget_factory->widgets[$widget];
     1095        $widget_obj = $wp_widget_factory->widgets[ $widget ];
    10581096        if ( ! ( $widget_obj instanceof WP_Widget ) ) {
    10591097                return;
    10601098        }
    10611099
    1062         $default_args = array(
     1100        $default_args          = array(
    10631101                'before_widget' => '<div class="widget %s">',
    1064                 'after_widget'  => "</div>",
     1102                'after_widget'  => '</div>',
    10651103                'before_title'  => '<h2 class="widgettitle">',
    10661104                'after_title'   => '</h2>',
    10671105        );
    1068         $args = wp_parse_args( $args, $default_args );
     1106        $args                  = wp_parse_args( $args, $default_args );
    10691107        $args['before_widget'] = sprintf( $args['before_widget'], $widget_obj->widget_options['classname'] );
    10701108
    1071         $instance = wp_parse_args($instance);
     1109        $instance = wp_parse_args( $instance );
    10721110
    10731111        /**
    … …  
    10821120        do_action( 'the_widget', $widget, $instance, $args );
    10831121
    1084         $widget_obj->_set(-1);
    1085         $widget_obj->widget($args, $instance);
     1122        $widget_obj->_set( -1 );
     1123        $widget_obj->widget( $args, $instance );
    10861124}
    10871125
    … …  
    11091147        global $sidebars_widgets;
    11101148
    1111         if ( ! is_array( $sidebars_widgets ) )
     1149        if ( ! is_array( $sidebars_widgets ) ) {
    11121150                $sidebars_widgets = wp_get_sidebars_widgets();
    1113 
    1114         retrieve_widgets(true);
     1151        }
     1152
     1153        retrieve_widgets( true );
    11151154}
    11161155
    … …  
    11341173        $registered_widgets_ids   = array_keys( $wp_registered_widgets );
    11351174
    1136         if ( ! is_array( get_theme_mod( 'sidebars_widgets' ) ) )  {
     1175        if ( ! is_array( get_theme_mod( 'sidebars_widgets' ) ) ) {
    11371176                if ( empty( $sidebars_widgets ) ) {
    11381177                        return array();
    … …  
    12181257                        $new_sidebars_widgets[ $sidebar ] = $existing_sidebars_widgets[ $sidebar ];
    12191258                        unset( $existing_sidebars_widgets[ $sidebar ] );
    1220                 } else if ( ! array_key_exists( $sidebar, $new_sidebars_widgets ) ) {
     1259                } elseif ( ! array_key_exists( $sidebar, $new_sidebars_widgets ) ) {
    12211260                        $new_sidebars_widgets[ $sidebar ] = array();
    12221261                }
    … …  
    13421381                } // endif ( ! empty( $old_sidebars_widgets ) )
    13431382
    1344 
    13451383                // Restore widget settings from when theme was previously active.
    13461384                $new_sidebars_widgets = array_merge( $new_sidebars_widgets, $old_sidebars_widgets );
    … …  
    13831421function wp_widget_rss_output( $rss, $args = array() ) {
    13841422        if ( is_string( $rss ) ) {
    1385                 $rss = fetch_feed($rss);
    1386         } elseif ( is_array($rss) && isset($rss['url']) ) {
     1423                $rss = fetch_feed( $rss );
     1424        } elseif ( is_array( $rss ) && isset( $rss['url'] ) ) {
    13871425                $args = $rss;
    1388                 $rss = fetch_feed($rss['url']);
    1389         } elseif ( !is_object($rss) ) {
     1426                $rss  = fetch_feed( $rss['url'] );
     1427        } elseif ( ! is_object( $rss ) ) {
    13901428                return;
    13911429        }
    13921430
    1393         if ( is_wp_error($rss) ) {
    1394                 if ( is_admin() || current_user_can('manage_options') )
     1431        if ( is_wp_error( $rss ) ) {
     1432                if ( is_admin() || current_user_can( 'manage_options' ) ) {
    13951433                        echo '<p><strong>' . __( 'RSS Error:' ) . '</strong> ' . $rss->get_error_message() . '</p>';
     1434                }
    13961435                return;
    13971436        }
    13981437
    1399         $default_args = array( 'show_author' => 0, 'show_date' => 0, 'show_summary' => 0, 'items' => 0 );
    1400         $args = wp_parse_args( $args, $default_args );
     1438        $default_args = array(
     1439                'show_author'  => 0,
     1440                'show_date'    => 0,
     1441                'show_summary' => 0,
     1442                'items'        => 0,
     1443        );
     1444        $args         = wp_parse_args( $args, $default_args );
    14011445
    14021446        $items = (int) $args['items'];
    1403         if ( $items < 1 || 20 < $items )
     1447        if ( $items < 1 || 20 < $items ) {
    14041448                $items = 10;
    1405         $show_summary  = (int) $args['show_summary'];
    1406         $show_author   = (int) $args['show_author'];
    1407         $show_date     = (int) $args['show_date'];
    1408 
    1409         if ( !$rss->get_item_quantity() ) {
     1449        }
     1450        $show_summary = (int) $args['show_summary'];
     1451        $show_author  = (int) $args['show_author'];
     1452        $show_date    = (int) $args['show_date'];
     1453
     1454        if ( ! $rss->get_item_quantity() ) {
    14101455                echo '<ul><li>' . __( 'An error has occurred, which probably means the feed is down. Try again later.' ) . '</li></ul>';
    14111456                $rss->__destruct();
    1412                 unset($rss);
     1457                unset( $rss );
    14131458                return;
    14141459        }
    … …  
    14541499                if ( $show_author ) {
    14551500                        $author = $item->get_author();
    1456                         if ( is_object($author) ) {
     1501                        if ( is_object( $author ) ) {
    14571502                                $author = $author->get_name();
    14581503                                $author = ' <cite>' . esc_html( strip_tags( $author ) ) . '</cite>';
    … …  
    14701515        echo '</ul>';
    14711516        $rss->__destruct();
    1472         unset($rss);
     1517        unset( $rss );
    14731518}
    14741519
    … …  
    14861531 */
    14871532function wp_widget_rss_form( $args, $inputs = null ) {
    1488         $default_inputs = array( 'url' => true, 'title' => true, 'items' => true, 'show_summary' => true, 'show_author' => true, 'show_date' => true );
    1489         $inputs = wp_parse_args( $inputs, $default_inputs );
     1533        $default_inputs = array(
     1534                'url'          => true,
     1535                'title'        => true,
     1536                'items'        => true,
     1537                'show_summary' => true,
     1538                'show_author'  => true,
     1539                'show_date'    => true,
     1540        );
     1541        $inputs         = wp_parse_args( $inputs, $default_inputs );
    14901542
    14911543        $args['title'] = isset( $args['title'] ) ? $args['title'] : '';
    1492         $args['url'] = isset( $args['url'] ) ? $args['url'] : '';
     1544        $args['url']   = isset( $args['url'] ) ? $args['url'] : '';
    14931545        $args['items'] = isset( $args['items'] ) ? (int) $args['items'] : 0;
    14941546
    … …  
    14971549        }
    14981550
    1499         $args['show_summary']   = isset( $args['show_summary'] ) ? (int) $args['show_summary'] : (int) $inputs['show_summary'];
    1500         $args['show_author']    = isset( $args['show_author'] ) ? (int) $args['show_author'] : (int) $inputs['show_author'];
    1501         $args['show_date']      = isset( $args['show_date'] ) ? (int) $args['show_date'] : (int) $inputs['show_date'];
     1551        $args['show_summary'] = isset( $args['show_summary'] ) ? (int) $args['show_summary'] : (int) $inputs['show_summary'];
     1552        $args['show_author']  = isset( $args['show_author'] ) ? (int) $args['show_author'] : (int) $inputs['show_author'];
     1553        $args['show_date']    = isset( $args['show_date'] ) ? (int) $args['show_date'] : (int) $inputs['show_date'];
    15021554
    15031555        if ( ! empty( $args['error'] ) ) {
    … …  
    15331585<?php
    15341586        endif;
    1535         foreach ( array_keys($default_inputs) as $input ) :
    1536                 if ( 'hidden' === $inputs[$input] ) :
    1537                         $id = str_replace( '_', '-', $input );
     1587foreach ( array_keys( $default_inputs ) as $input ) :
     1588        if ( 'hidden' === $inputs[ $input ] ) :
     1589                $id = str_replace( '_', '-', $input );
    15381590?>
    1539         <input type="hidden" id="rss-<?php echo esc_attr( $id ); ?>-<?php echo $esc_number; ?>" name="widget-rss[<?php echo $esc_number; ?>][<?php echo esc_attr( $input ); ?>]" value="<?php echo esc_attr( $args[ $input ] ); ?>" />
     1591<input type="hidden" id="rss-<?php echo esc_attr( $id ); ?>-<?php echo $esc_number; ?>" name="widget-rss[<?php echo $esc_number; ?>][<?php echo esc_attr( $input ); ?>]" value="<?php echo esc_attr( $args[ $input ] ); ?>" />
    15401592<?php
    1541                 endif;
     1593        endif;
    15421594        endforeach;
    15431595}
    … …  
    15611613function wp_widget_rss_process( $widget_rss, $check_feed = true ) {
    15621614        $items = (int) $widget_rss['items'];
    1563         if ( $items < 1 || 20 < $items )
     1615        if ( $items < 1 || 20 < $items ) {
    15641616                $items = 10;
    1565         $url           = esc_url_raw( strip_tags( $widget_rss['url'] ) );
    1566         $title         = isset( $widget_rss['title'] ) ? trim( strip_tags( $widget_rss['title'] ) ) : '';
    1567         $show_summary  = isset( $widget_rss['show_summary'] ) ? (int) $widget_rss['show_summary'] : 0;
    1568         $show_author   = isset( $widget_rss['show_author'] ) ? (int) $widget_rss['show_author'] :0;
    1569         $show_date     = isset( $widget_rss['show_date'] ) ? (int) $widget_rss['show_date'] : 0;
     1617        }
     1618        $url          = esc_url_raw( strip_tags( $widget_rss['url'] ) );
     1619        $title        = isset( $widget_rss['title'] ) ? trim( strip_tags( $widget_rss['title'] ) ) : '';
     1620        $show_summary = isset( $widget_rss['show_summary'] ) ? (int) $widget_rss['show_summary'] : 0;
     1621        $show_author  = isset( $widget_rss['show_author'] ) ? (int) $widget_rss['show_author'] : 0;
     1622        $show_date    = isset( $widget_rss['show_date'] ) ? (int) $widget_rss['show_date'] : 0;
    15701623
    15711624        if ( $check_feed ) {
    1572                 $rss = fetch_feed($url);
     1625                $rss   = fetch_feed( $url );
    15731626                $error = false;
    1574                 $link = '';
    1575                 if ( is_wp_error($rss) ) {
     1627                $link  = '';
     1628                if ( is_wp_error( $rss ) ) {
    15761629                        $error = $rss->get_error_message();
    15771630                } else {
    1578                         $link = esc_url(strip_tags($rss->get_permalink()));
    1579                         while ( stristr($link, 'http') != $link )
    1580                                 $link = substr($link, 1);
     1631                        $link = esc_url( strip_tags( $rss->get_permalink() ) );
     1632                        while ( stristr( $link, 'http' ) != $link ) {
     1633                                $link = substr( $link, 1 );
     1634                        }
    15811635
    15821636                        $rss->__destruct();
    1583                         unset($rss);
     1637                        unset( $rss );
    15841638                }
    15851639        }
Note: See TracChangeset for help on using the changeset viewer.