WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (7 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.