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/src/wp-includes/wp-db.php

    r43017 r43571  
    32183218                . '|UPDATE(?:\s+LOW_PRIORITY)?(?:\s+IGNORE)?'
    32193219                . '|DELETE(?:\s+LOW_PRIORITY|\s+QUICK|\s+IGNORE)*(?:.+?FROM)?'
    3220             . ')\s+((?:[0-9a-zA-Z$_.`-]|[\xC2-\xDF][\x80-\xBF])+)/is', $query, $maybe
     3220            . ')\s+((?:[0-9a-zA-Z$_.`-]|[\xC2-\xDF][\x80-\xBF])+)/is',
     3221            $query,
     3222            $maybe
    32213223        ) ) {
    32223224            return str_replace( '`', '', $maybe[1] );
     
    32533255                . '|(?:GRANT|REVOKE).*ON\s+TABLE'
    32543256                . '|SHOW\s+(?:.*FROM|.*TABLE)'
    3255             . ')\s+\(*\s*((?:[0-9a-zA-Z$_.`-]|[\xC2-\xDF][\x80-\xBF])+)\s*\)*/is', $query, $maybe
     3257            . ')\s+\(*\s*((?:[0-9a-zA-Z$_.`-]|[\xC2-\xDF][\x80-\xBF])+)\s*\)*/is',
     3258            $query,
     3259            $maybe
    32563260        ) ) {
    32573261            return str_replace( '`', '', $maybe[1] );
Note: See TracChangeset for help on using the changeset viewer.