WordPress.org

Make WordPress Core

Changeset 46138


Ignore:
Timestamp:
09/15/2019 11:40:08 AM (2 months ago)
Author:
SergeyBiryukov
Message:

Code Modernisation: Replace call_user_func_array() in wp-admin/includes/widgets.php and associated unit tests with a direct function call.

Props jrf.
See #47678.

Location:
trunk
Files:
2 edited

Legend:

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

    r45932 r46138  
    5252        }
    5353
    54         $args = wp_list_widget_controls_dynamic_sidebar(
    55             array(
    56                 0 => $args,
    57                 1 => $widget['params'][0],
    58             )
     54        $control_args = array(
     55            0 => $args,
     56            1 => $widget['params'][0],
    5957        );
    60         call_user_func_array( 'wp_widget_control', $args );
     58        $sidebar_args = wp_list_widget_controls_dynamic_sidebar( $control_args );
     59
     60        wp_widget_control( ...$sidebar_args );
    6161    }
    6262}
     
    145145    $params[0]['before_title']  = '%BEG_OF_TITLE%'; // deprecated
    146146    $params[0]['after_title']   = '%END_OF_TITLE%'; // deprecated
     147
    147148    if ( is_callable( $wp_registered_widgets[ $widget_id ]['callback'] ) ) {
    148149        $wp_registered_widgets[ $widget_id ]['_callback'] = $wp_registered_widgets[ $widget_id ]['callback'];
  • trunk/tests/phpunit/tests/widgets.php

    r45798 r46138  
    621621        wp_widgets_init();
    622622        require_once ABSPATH . '/wp-admin/includes/widgets.php';
    623         $widget_id = 'search-2';
    624         $widget    = $wp_registered_widgets[ $widget_id ];
    625         $params    = array(
     623        $widget_id    = 'search-2';
     624        $widget       = $wp_registered_widgets[ $widget_id ];
     625        $params       = array(
    626626            'widget_id'   => $widget['id'],
    627627            'widget_name' => $widget['name'],
    628628        );
    629         $args      = wp_list_widget_controls_dynamic_sidebar(
    630             array(
    631                 0 => $params,
    632                 1 => $widget['params'][0],
    633             )
    634         );
     629        $control_args = array(
     630            0 => $params,
     631            1 => $widget['params'][0],
     632        );
     633        $sidebar_args = wp_list_widget_controls_dynamic_sidebar( $control_args );
    635634
    636635        ob_start();
    637         call_user_func_array( 'wp_widget_control', $args );
     636        wp_widget_control( ...$sidebar_args );
    638637        $control = ob_get_clean();
    639638        $this->assertNotEmpty( $control );
     
    660659        );
    661660        $params          = array_merge( $params, $param_overrides );
    662         $args            = wp_list_widget_controls_dynamic_sidebar(
    663             array(
    664                 0 => $params,
    665                 1 => $widget['params'][0],
    666             )
    667         );
     661        $control_args    = array(
     662            0 => $params,
     663            1 => $widget['params'][0],
     664        );
     665        $sidebar_args    = wp_list_widget_controls_dynamic_sidebar( $control_args );
    668666
    669667        ob_start();
    670         call_user_func_array( 'wp_widget_control', $args );
     668        wp_widget_control( ...$sidebar_args );
    671669        $control = ob_get_clean();
    672670        $this->assertNotEmpty( $control );
Note: See TracChangeset for help on using the changeset viewer.