Make WordPress Core


Ignore:
Timestamp:
08/17/2018 01:50:26 AM (6 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/src/wp-includes/class-wp-customize-manager.php

    r43535 r43571  
    14791479
    14801480            $this->set_post_value(
    1481                 $nav_menu_setting_id, array(
     1481                $nav_menu_setting_id,
     1482                array(
    14821483                    'name' => isset( $nav_menu['name'] ) ? $nav_menu['name'] : $nav_menu_location,
    14831484                )
     
    22772278    public function validate_setting_values( $setting_values, $options = array() ) {
    22782279        $options = wp_parse_args(
    2279             $options, array(
     2280            $options,
     2281            array(
    22802282                'validate_capability' => false,
    22812283                'validate_existence'  => false,
     
    27142716        );
    27152717        $setting_validities    = $this->validate_setting_values(
    2716             $validated_values, array(
     2718            $validated_values,
     2719            array(
    27172720                'validate_capability' => true,
    27182721                'validate_existence'  => true,
     
    40154018            $l10n = array(
    40164019                /* translators: %s: User who is customizing the changeset in customizer. */
    4017                 'locked' => __( '%s is already customizing this changeset. Please wait until they are done to try customizing. Your latest changes have been autosaved.' ),
     4020                'locked'                => __( '%s is already customizing this changeset. Please wait until they are done to try customizing. Your latest changes have been autosaved.' ),
    40184021                /* translators: %s: User who is customizing the changeset in customizer. */
    40194022                'locked_allow_override' => __( '%s is already customizing this changeset. Do you want to take over?' ),
     
    40224025            $l10n = array(
    40234026                /* translators: %s: User who is customizing the changeset in customizer. */
    4024                 'locked' => __( '%s is already customizing this site. Please wait until they are done to try customizing. Your latest changes have been autosaved.' ),
     4027                'locked'                => __( '%s is already customizing this site. Please wait until they are done to try customizing. Your latest changes have been autosaved.' ),
    40254028                /* translators: %s: User who is customizing the changeset in customizer. */
    40264029                'locked_allow_override' => __( '%s is already customizing this site. Do you want to take over?' ),
     
    40304033        foreach ( $this->registered_control_types as $control_type ) {
    40314034            $control = new $control_type(
    4032                 $this, 'temp', array(
     4035                $this,
     4036                'temp',
     4037                array(
    40334038                    'settings' => array(),
    40344039                )
     
    43074312        $controls       = array();
    43084313        $this->controls = wp_list_sort(
    4309             $this->controls, array(
     4314            $this->controls,
     4315            array(
    43104316                'priority'        => 'ASC',
    43114317                'instance_number' => 'ASC',
    4312             ), 'ASC', true
     4318            ),
     4319            'ASC',
     4320            true
    43134321        );
    43144322
     
    43254333        // Prepare sections.
    43264334        $this->sections = wp_list_sort(
    4327             $this->sections, array(
     4335            $this->sections,
     4336            array(
    43284337                'priority'        => 'ASC',
    43294338                'instance_number' => 'ASC',
    4330             ), 'ASC', true
     4339            ),
     4340            'ASC',
     4341            true
    43314342        );
    43324343        $sections       = array();
     
    43384349
    43394350            $section->controls = wp_list_sort(
    4340                 $section->controls, array(
     4351                $section->controls,
     4352                array(
    43414353                    'priority'        => 'ASC',
    43424354                    'instance_number' => 'ASC',
     
    43584370        // Prepare panels.
    43594371        $this->panels = wp_list_sort(
    4360             $this->panels, array(
     4372            $this->panels,
     4373            array(
    43614374                'priority'        => 'ASC',
    43624375                'instance_number' => 'ASC',
    4363             ), 'ASC', true
     4376            ),
     4377            'ASC',
     4378            true
    43644379        );
    43654380        $panels       = array();
     
    43714386
    43724387            $panel->sections      = wp_list_sort(
    4373                 $panel->sections, array(
     4388                $panel->sections,
     4389                array(
    43744390                    'priority'        => 'ASC',
    43754391                    'instance_number' => 'ASC',
    4376                 ), 'ASC', true
     4392                ),
     4393                'ASC',
     4394                true
    43774395            );
    43784396            $panels[ $panel->id ] = $panel;
     
    43834401        $this->containers = array_merge( $this->panels, $this->sections );
    43844402        $this->containers = wp_list_sort(
    4385             $this->containers, array(
     4403            $this->containers,
     4404            array(
    43864405                'priority'        => 'ASC',
    43874406                'instance_number' => 'ASC',
    4388             ), 'ASC', true
     4407            ),
     4408            'ASC',
     4409            true
    43894410        );
    43904411    }
     
    44034424            wp_enqueue_script( 'updates' );
    44044425            wp_localize_script(
    4405                 'updates', '_wpUpdatesItemCounts', array(
     4426                'updates',
     4427                '_wpUpdatesItemCounts',
     4428                array(
    44064429                    'totals' => wp_get_update_data(),
    44074430                )
     
    46434666                'interim-login'   => 1,
    46444667                'customize-login' => 1,
    4645             ), wp_login_url()
     4668            ),
     4669            wp_login_url()
    46464670        );
    46474671
     
    48414865            }
    48424866            echo "})( _wpCustomizeSettings.controls );\n";
    4843         ?>
     4867            ?>
    48444868        </script>
    48454869        <?php
     
    48924916        $this->add_panel(
    48934917            new WP_Customize_Themes_Panel(
    4894                 $this, 'themes', array(
     4918                $this,
     4919                'themes',
     4920                array(
    48954921                    'title'       => $this->theme()->display( 'Name' ),
    48964922                    'description' => (
     
    49064932        $this->add_section(
    49074933            new WP_Customize_Themes_Section(
    4908                 $this, 'installed_themes', array(
     4934                $this,
     4935                'installed_themes',
     4936                array(
    49094937                    'title'      => __( 'Installed themes' ),
    49104938                    'action'     => 'installed',
     
    49194947            $this->add_section(
    49204948                new WP_Customize_Themes_Section(
    4921                     $this, 'wporg_themes', array(
     4949                    $this,
     4950                    'wporg_themes',
     4951                    array(
    49224952                        'title'       => __( 'WordPress.org themes' ),
    49234953                        'action'      => 'wporg',
     
    49344964        $this->add_setting(
    49354965            new WP_Customize_Filter_Setting(
    4936                 $this, 'active_theme', array(
     4966                $this,
     4967                'active_theme',
     4968                array(
    49374969                    'capability' => 'switch_themes',
    49384970                )
     
    49434975
    49444976        $this->add_section(
    4945             'title_tagline', array(
     4977            'title_tagline',
     4978            array(
    49464979                'title'    => __( 'Site Identity' ),
    49474980                'priority' => 20,
     
    49504983
    49514984        $this->add_setting(
    4952             'blogname', array(
     4985            'blogname',
     4986            array(
    49534987                'default'    => get_option( 'blogname' ),
    49544988                'type'       => 'option',
     
    49584992
    49594993        $this->add_control(
    4960             'blogname', array(
     4994            'blogname',
     4995            array(
    49614996                'label'   => __( 'Site Title' ),
    49624997                'section' => 'title_tagline',
     
    49655000
    49665001        $this->add_setting(
    4967             'blogdescription', array(
     5002            'blogdescription',
     5003            array(
    49685004                'default'    => get_option( 'blogdescription' ),
    49695005                'type'       => 'option',
     
    49735009
    49745010        $this->add_control(
    4975             'blogdescription', array(
     5011            'blogdescription',
     5012            array(
    49765013                'label'   => __( 'Tagline' ),
    49775014                'section' => 'title_tagline',
     
    49825019        if ( ! current_theme_supports( 'custom-header', 'header-text' ) ) {
    49835020            $this->add_setting(
    4984                 'header_text', array(
     5021                'header_text',
     5022                array(
    49855023                    'theme_supports'    => array( 'custom-logo', 'header-text' ),
    49865024                    'default'           => 1,
     
    49905028
    49915029            $this->add_control(
    4992                 'header_text', array(
     5030                'header_text',
     5031                array(
    49935032                    'label'    => __( 'Display Site Title and Tagline' ),
    49945033                    'section'  => 'title_tagline',
     
    50005039
    50015040        $this->add_setting(
    5002             'site_icon', array(
     5041            'site_icon',
     5042            array(
    50035043                'type'       => 'option',
    50045044                'capability' => 'manage_options',
     
    50095049        $this->add_control(
    50105050            new WP_Customize_Site_Icon_Control(
    5011                 $this, 'site_icon', array(
     5051                $this,
     5052                'site_icon',
     5053                array(
    50125054                    'label'       => __( 'Site Icon' ),
    50135055                    'description' => sprintf(
     
    50265068
    50275069        $this->add_setting(
    5028             'custom_logo', array(
     5070            'custom_logo',
     5071            array(
    50295072                'theme_supports' => array( 'custom-logo' ),
    50305073                'transport'      => 'postMessage',
     
    50355078        $this->add_control(
    50365079            new WP_Customize_Cropped_Image_Control(
    5037                 $this, 'custom_logo', array(
     5080                $this,
     5081                'custom_logo',
     5082                array(
    50385083                    'label'         => __( 'Logo' ),
    50395084                    'section'       => 'title_tagline',
     
    50575102
    50585103        $this->selective_refresh->add_partial(
    5059             'custom_logo', array(
     5104            'custom_logo',
     5105            array(
    50605106                'settings'            => array( 'custom_logo' ),
    50615107                'selector'            => '.custom-logo-link',
     
    50685114
    50695115        $this->add_section(
    5070             'colors', array(
     5116            'colors',
     5117            array(
    50715118                'title'    => __( 'Colors' ),
    50725119                'priority' => 40,
     
    50755122
    50765123        $this->add_setting(
    5077             'header_textcolor', array(
     5124            'header_textcolor',
     5125            array(
    50785126                'theme_supports'       => array( 'custom-header', 'header-text' ),
    50795127                'default'              => get_theme_support( 'custom-header', 'default-text-color' ),
     
    50875135        // With custom value
    50885136        $this->add_control(
    5089             'display_header_text', array(
     5137            'display_header_text',
     5138            array(
    50905139                'settings' => 'header_textcolor',
    50915140                'label'    => __( 'Display Site Title and Tagline' ),
     
    50985147        $this->add_control(
    50995148            new WP_Customize_Color_Control(
    5100                 $this, 'header_textcolor', array(
     5149                $this,
     5150                'header_textcolor',
     5151                array(
    51015152                    'label'   => __( 'Header Text Color' ),
    51025153                    'section' => 'colors',
     
    51085159        // With sanitize_callback
    51095160        $this->add_setting(
    5110             'background_color', array(
     5161            'background_color',
     5162            array(
    51115163                'default'              => get_theme_support( 'custom-background', 'default-color' ),
    51125164                'theme_supports'       => 'custom-background',
     
    51195171        $this->add_control(
    51205172            new WP_Customize_Color_Control(
    5121                 $this, 'background_color', array(
     5173                $this,
     5174                'background_color',
     5175                array(
    51225176                    'label'   => __( 'Background Color' ),
    51235177                    'section' => 'colors',
     
    51635217
    51645218        $this->add_section(
    5165             'header_image', array(
     5219            'header_image',
     5220            array(
    51665221                'title'          => $title,
    51675222                'description'    => $description,
     
    51725227
    51735228        $this->add_setting(
    5174             'header_video', array(
     5229            'header_video',
     5230            array(
    51755231                'theme_supports'    => array( 'custom-header', 'video' ),
    51765232                'transport'         => 'postMessage',
     
    51815237
    51825238        $this->add_setting(
    5183             'external_header_video', array(
     5239            'external_header_video',
     5240            array(
    51845241                'theme_supports'    => array( 'custom-header', 'video' ),
    51855242                'transport'         => 'postMessage',
     
    51915248        $this->add_setting(
    51925249            new WP_Customize_Filter_Setting(
    5193                 $this, 'header_image', array(
     5250                $this,
     5251                'header_image',
     5252                array(
    51945253                    'default'        => sprintf( get_theme_support( 'custom-header', 'default-image' ), get_template_directory_uri(), get_stylesheet_directory_uri() ),
    51955254                    'theme_supports' => 'custom-header',
     
    52005259        $this->add_setting(
    52015260            new WP_Customize_Header_Image_Setting(
    5202                 $this, 'header_image_data', array(
     5261                $this,
     5262                'header_image_data',
     5263                array(
    52035264                    'theme_supports' => 'custom-header',
    52045265                )
     
    52185279        $this->add_control(
    52195280            new WP_Customize_Media_Control(
    5220                 $this, 'header_video', array(
     5281                $this,
     5282                'header_video',
     5283                array(
    52215284                    'theme_supports'  => array( 'custom-header', 'video' ),
    52225285                    'label'           => __( 'Header Video' ),
     
    52305293
    52315294        $this->add_control(
    5232             'external_header_video', array(
     5295            'external_header_video',
     5296            array(
    52335297                'theme_supports'  => array( 'custom-header', 'video' ),
    52345298                'type'            => 'url',
     
    52425306
    52435307        $this->selective_refresh->add_partial(
    5244             'custom_header', array(
     5308            'custom_header',
     5309            array(
    52455310                'selector'            => '#wp-custom-header',
    52465311                'render_callback'     => 'the_custom_header_markup',
     
    52535318
    52545319        $this->add_section(
    5255             'background_image', array(
     5320            'background_image',
     5321            array(
    52565322                'title'          => __( 'Background Image' ),
    52575323                'theme_supports' => 'custom-background',
     
    52615327
    52625328        $this->add_setting(
    5263             'background_image', array(
     5329            'background_image',
     5330            array(
    52645331                'default'           => get_theme_support( 'custom-background', 'default-image' ),
    52655332                'theme_supports'    => 'custom-background',
     
    52705337        $this->add_setting(
    52715338            new WP_Customize_Background_Image_Setting(
    5272                 $this, 'background_image_thumb', array(
     5339                $this,
     5340                'background_image_thumb',
     5341                array(
    52735342                    'theme_supports'    => 'custom-background',
    52745343                    'sanitize_callback' => array( $this, '_sanitize_background_setting' ),
     
    52805349
    52815350        $this->add_setting(
    5282             'background_preset', array(
     5351            'background_preset',
     5352            array(
    52835353                'default'           => get_theme_support( 'custom-background', 'default-preset' ),
    52845354                'theme_supports'    => 'custom-background',
     
    52885358
    52895359        $this->add_control(
    5290             'background_preset', array(
     5360            'background_preset',
     5361            array(
    52915362                'label'   => _x( 'Preset', 'Background Preset' ),
    52925363                'section' => 'background_image',
     
    53035374
    53045375        $this->add_setting(
    5305             'background_position_x', array(
     5376            'background_position_x',
     5377            array(
    53065378                'default'           => get_theme_support( 'custom-background', 'default-position-x' ),
    53075379                'theme_supports'    => 'custom-background',
     
    53115383
    53125384        $this->add_setting(
    5313             'background_position_y', array(
     5385            'background_position_y',
     5386            array(
    53145387                'default'           => get_theme_support( 'custom-background', 'default-position-y' ),
    53155388                'theme_supports'    => 'custom-background',
     
    53205393        $this->add_control(
    53215394            new WP_Customize_Background_Position_Control(
    5322                 $this, 'background_position', array(
     5395                $this,
     5396                'background_position',
     5397                array(
    53235398                    'label'    => __( 'Image Position' ),
    53245399                    'section'  => 'background_image',
     
    53325407
    53335408        $this->add_setting(
    5334             'background_size', array(
     5409            'background_size',
     5410            array(
    53355411                'default'           => get_theme_support( 'custom-background', 'default-size' ),
    53365412                'theme_supports'    => 'custom-background',
     
    53405416
    53415417        $this->add_control(
    5342             'background_size', array(
     5418            'background_size',
     5419            array(
    53435420                'label'   => __( 'Image Size' ),
    53445421                'section' => 'background_image',
     
    53535430
    53545431        $this->add_setting(
    5355             'background_repeat', array(
     5432            'background_repeat',
     5433            array(
    53565434                'default'           => get_theme_support( 'custom-background', 'default-repeat' ),
    53575435                'sanitize_callback' => array( $this, '_sanitize_background_setting' ),
     
    53615439
    53625440        $this->add_control(
    5363             'background_repeat', array(
     5441            'background_repeat',
     5442            array(
    53645443                'label'   => __( 'Repeat Background Image' ),
    53655444                'section' => 'background_image',
     
    53695448
    53705449        $this->add_setting(
    5371             'background_attachment', array(
     5450            'background_attachment',
     5451            array(
    53725452                'default'           => get_theme_support( 'custom-background', 'default-attachment' ),
    53735453                'sanitize_callback' => array( $this, '_sanitize_background_setting' ),
     
    53775457
    53785458        $this->add_control(
    5379             'background_attachment', array(
     5459            'background_attachment',
     5460            array(
    53805461                'label'   => __( 'Scroll with Page' ),
    53815462                'section' => 'background_image',
     
    53995480
    54005481        $this->add_section(
    5401             'static_front_page', array(
     5482            'static_front_page',
     5483            array(
    54025484                'title'           => __( 'Homepage Settings' ),
    54035485                'priority'        => 120,
     
    54085490
    54095491        $this->add_setting(
    5410             'show_on_front', array(
     5492            'show_on_front',
     5493            array(
    54115494                'default'    => get_option( 'show_on_front' ),
    54125495                'capability' => 'manage_options',
     
    54165499
    54175500        $this->add_control(
    5418             'show_on_front', array(
     5501            'show_on_front',
     5502            array(
    54195503                'label'   => __( 'Your homepage displays' ),
    54205504                'section' => 'static_front_page',
     
    54285512
    54295513        $this->add_setting(
    5430             'page_on_front', array(
     5514            'page_on_front',
     5515            array(
    54315516                'type'       => 'option',
    54325517                'capability' => 'manage_options',
     
    54355520
    54365521        $this->add_control(
    5437             'page_on_front', array(
     5522            'page_on_front',
     5523            array(
    54385524                'label'          => __( 'Homepage' ),
    54395525                'section'        => 'static_front_page',
     
    54445530
    54455531        $this->add_setting(
    5446             'page_for_posts', array(
     5532            'page_for_posts',
     5533            array(
    54475534                'type'       => 'option',
    54485535                'capability' => 'manage_options',
     
    54515538
    54525539        $this->add_control(
    5453             'page_for_posts', array(
     5540            'page_for_posts',
     5541            array(
    54545542                'label'          => __( 'Posts page' ),
    54555543                'section'        => 'static_front_page',
     
    54995587
    55005588        $this->add_section(
    5501             'custom_css', array(
     5589            'custom_css',
     5590            array(
    55025591                'title'              => __( 'Additional CSS' ),
    55035592                'priority'           => 200,
     
    55085597
    55095598        $custom_css_setting = new WP_Customize_Custom_CSS_Setting(
    5510             $this, sprintf( 'custom_css[%s]', get_stylesheet() ), array(
     5599            $this,
     5600            sprintf( 'custom_css[%s]', get_stylesheet() ),
     5601            array(
    55115602                'capability' => 'edit_css',
    55125603                'default'    => '',
     
    55175608        $this->add_control(
    55185609            new WP_Customize_Code_Editor_Control(
    5519                 $this, 'custom_css', array(
     5610                $this,
     5611                'custom_css',
     5612                array(
    55205613                    'label'       => __( 'CSS code' ),
    55215614                    'section'     => 'custom_css',
     
    56605753                        'theme'    => $theme->slug,
    56615754                        '_wpnonce' => wp_create_nonce( 'install-theme_' . $theme->slug ),
    5662                     ), $update_php
     5755                    ),
     5756                    $update_php
    56635757                );
    56645758
     
    58335927            if ( '.mp4' !== substr( $video, -4 ) && '.mov' !== substr( $video, -4 ) ) { // Check for .mp4 or .mov format, which (assuming h.264 encoding) are the only cross-browser-supported formats.
    58345928                $validity->add(
    5835                     'invalid_file_type', sprintf(
     5929                    'invalid_file_type',
     5930                    sprintf(
    58365931                        /* translators: 1: .mp4, 2: .mov */
    58375932                        __( 'Only %1$s or %2$s files may be used for header video. Please convert your video file and try again, or, upload your video to YouTube and link it with the option below.' ),
Note: See TracChangeset for help on using the changeset viewer.