Make WordPress Core


Ignore:
Timestamp:
08/17/2018 01:50:26 AM (8 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/rest-api/class-wp-rest-request.php

    r42746 r43571  
    803803                if ( $invalid_params ) {
    804804                        return new WP_Error(
    805                                 'rest_invalid_param', sprintf( __( 'Invalid parameter(s): %s' ), implode( ', ', array_keys( $invalid_params ) ) ), array(
     805                                'rest_invalid_param',
     806                                sprintf( __( 'Invalid parameter(s): %s' ), implode( ', ', array_keys( $invalid_params ) ) ),
     807                                array(
    806808                                        'status' => 400,
    807809                                        'params' => $invalid_params,
     
    846848                if ( ! empty( $required ) ) {
    847849                        return new WP_Error(
    848                                 'rest_missing_callback_param', sprintf( __( 'Missing parameter(s): %s' ), implode( ', ', $required ) ), array(
     850                                'rest_missing_callback_param',
     851                                sprintf( __( 'Missing parameter(s): %s' ), implode( ', ', $required ) ),
     852                                array(
    849853                                        'status' => 400,
    850854                                        'params' => $required,
     
    879883                if ( $invalid_params ) {
    880884                        return new WP_Error(
    881                                 'rest_invalid_param', sprintf( __( 'Invalid parameter(s): %s' ), implode( ', ', array_keys( $invalid_params ) ) ), array(
     885                                'rest_invalid_param',
     886                                sprintf( __( 'Invalid parameter(s): %s' ), implode( ', ', array_keys( $invalid_params ) ) ),
     887                                array(
    882888                                        'status' => 400,
    883889                                        'params' => $invalid_params,
Note: See TracChangeset for help on using the changeset viewer.