WordPress.org

Make WordPress Core


Ignore:
Timestamp:
08/17/2018 01:50:26 AM (3 years ago)
Author:
pento
Message:

Coding Standards: Upgrade WPCS to 1.0.0

WPCS 1.0.0 includes a bunch of new auto-fixers, which drops the number of coding standards issues across WordPress significantly. Prior to running the auto-fixers, there were 15,312 issues detected. With this commit, we now drop to 4,769 issues.

This change includes three notable additions:

  • Multiline function calls must now put each parameter on a new line.
  • Auto-formatting files is now part of the grunt precommit script.
  • Auto-fixable coding standards issues will now cause Travis failures.

Fixes #44600.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/tests/phpunit/tests/customize/manager.php

    r42863 r43571  
    551551
    552552        $existing_canola_attachment_id     = self::factory()->attachment->create_object(
    553             $this->test_file, 0, array(
     553            $this->test_file,
     554            0,
     555            array(
    554556                'post_mime_type' => 'image/jpeg',
    555557                'post_type'      => 'attachment',
     
    986988        $previous_saved_data = $saved_data;
    987989        $manager->add_setting(
    988             'foo_unauthorized', array(
     990            'foo_unauthorized',
     991            array(
    989992                'capability' => 'do_not_allow',
    990993            )
    991994        );
    992995        $manager->add_setting(
    993             'baz_illegal', array(
     996            'baz_illegal',
     997            array(
    994998                'validate_callback' => array( $this, 'return_illegal_error' ),
    995999            )
     
    10991103        do_action( 'customize_register', $wp_customize );
    11001104        $manager->add_setting(
    1101             'scratchpad', array(
     1105            'scratchpad',
     1106            array(
    11021107                'type'       => 'option',
    11031108                'capability' => 'exist',
     
    13831388        do_action( 'customize_register', $manager );
    13841389        $manager->add_setting(
    1385             'blogfounded', array(
     1390            'blogfounded',
     1391            array(
    13861392                'type' => 'option',
    13871393            )
    13881394        );
    13891395        $manager->add_setting(
    1390             'blogterminated', array(
     1396            'blogterminated',
     1397            array(
    13911398                'type'       => 'option',
    13921399                'capability' => 'do_not_allow',
     
    13941401        );
    13951402        $manager->add_setting(
    1396             'scratchpad', array(
     1403            'scratchpad',
     1404            array(
    13971405                'type'       => 'option',
    13981406                'capability' => 'exist',
     
    18991907    function register_scratchpad_setting( WP_Customize_Manager $wp_customize ) {
    19001908        $wp_customize->add_setting(
    1901             'scratchpad', array(
     1909            'scratchpad',
     1910            array(
    19021911                'type'              => 'option',
    19031912                'capability'        => 'exist',
     
    21952204        $default_value = 'foo_default';
    21962205        $setting       = $this->manager->add_setting(
    2197             'foo', array(
     2206            'foo',
     2207            array(
    21982208                'validate_callback' => array( $this, 'filter_customize_validate_foo' ),
    21992209                'sanitize_callback' => array( $this, 'filter_customize_sanitize_foo' ),
     
    22632273        $default_value = '0';
    22642274        $setting       = $this->manager->add_setting(
    2265             'numeric', array(
     2275            'numeric',
     2276            array(
    22662277                'validate_callback' => array( $this, 'filter_customize_validate_numeric' ),
    22672278                'sanitize_callback' => array( $this, 'filter_customize_sanitize_numeric' ),
     
    23092320        wp_set_current_user( self::$admin_user_id );
    23102321        $setting = $this->manager->add_setting(
    2311             'foo', array(
     2322            'foo',
     2323            array(
    23122324                'validate_callback' => array( $this, 'filter_customize_validate_foo' ),
    23132325                'sanitize_callback' => array( $this, 'filter_customize_sanitize_foo' ),
     
    24342446        wp_set_current_user( self::$admin_user_id );
    24352447        $setting    = $this->manager->add_setting(
    2436             'numeric', array(
     2448            'numeric',
     2449            array(
    24372450                'validate_callback' => array( $this, 'filter_customize_validate_numeric' ),
    24382451                'sanitize_callback' => array( $this, 'filter_customize_sanitize_numeric' ),
     
    24782491        wp_set_current_user( self::$admin_user_id );
    24792492        $this->manager->add_setting(
    2480             'foo', array(
     2493            'foo',
     2494            array(
    24812495                'sanitize_callback' => array( $this, 'sanitize_foo_for_test_set_post_value' ),
    24822496            )
     
    29712985        wp_set_current_user( self::$admin_user_id );
    29722986        $manager->add_section(
    2973             $section_id, array(
     2987            $section_id,
     2988            array(
    29742989                'title'    => 'Section',
    29752990                'priority' => 1,
     
    29842999            $manager->add_setting( $id );
    29853000            $control = new WP_Customize_Control(
    2986                 $manager, $id, array(
     3001                $manager,
     3002                $id,
     3003                array(
    29873004                    'section'  => $section_id,
    29883005                    'priority' => 1,
     
    30083025        $section_id     = 'foo-section';
    30093026        $result_section = $manager->add_section(
    3010             $section_id, array(
     3027            $section_id,
     3028            array(
    30113029                'title'    => 'Section',
    30123030                'priority' => 1,
     
    30183036
    30193037        $section        = new WP_Customize_Section(
    3020             $manager, $section_id, array(
     3038            $manager,
     3039            $section_id,
     3040            array(
    30213041                'title'    => 'Section 2',
    30223042                'priority' => 2,
     
    31103130        $panel_id     = 'foo-panel';
    31113131        $result_panel = $manager->add_panel(
    3112             $panel_id, array(
     3132            $panel_id,
     3133            array(
    31133134                'title'    => 'Test Panel',
    31143135                'priority' => 2,
     
    31203141
    31213142        $panel        = new WP_Customize_Panel(
    3122             $manager, $panel_id, array(
     3143            $manager,
     3144            $panel_id,
     3145            array(
    31233146                'title' => 'Test Panel 2',
    31243147            )
     
    31393162        wp_set_current_user( self::$admin_user_id );
    31403163        $manager->add_section(
    3141             $section_id, array(
     3164            $section_id,
     3165            array(
    31423166                'title'    => 'Section',
    31433167                'priority' => 1,
     
    31493173
    31503174        $result_control = $manager->add_control(
    3151             $control_id, array(
     3175            $control_id,
     3176            array(
    31523177                'section'  => $section_id,
    31533178                'priority' => 1,
     
    31593184
    31603185        $control        = new WP_Customize_Control(
    3161             $manager, $control_id, array(
     3186            $manager,
     3187            $control_id,
     3188            array(
    31623189                'section'  => $section_id,
    31633190                'priority' => 1,
     
    32543281            $this->manager->add_setting( $control_id );
    32553282            $this->manager->add_control(
    3256                 $control_id, array(
     3283                $control_id,
     3284                array(
    32573285                    'priority' => $priority,
    32583286                    'section'  => 'foosection',
     
    32833311        foreach ( $sections as $section_id => $priority ) {
    32843312            $this->manager->add_section(
    3285                 $section_id, array(
     3313                $section_id,
     3314                array(
    32863315                    'priority' => $priority,
    32873316                )
     
    33113340        foreach ( $panels as $panel_id => $priority ) {
    33123341            $this->manager->add_panel(
    3313                 $panel_id, array(
     3342                $panel_id,
     3343                array(
    33143344                    'priority' => $priority,
    33153345                )
Note: See TracChangeset for help on using the changeset viewer.