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/formatting/EscUrl.php

    r42343 r43571  
    5353                'query'    => 'query=2&r[]=3',
    5454                'fragment' => 'fragment',
    55             ), parse_url( $url )
     55            ),
     56            parse_url( $url )
    5657        );
    5758        $this->assertEquals( 'https://user:pass@host.example.com:1234/path;p=1?query=2&r%5B%5D=3#fragment', esc_url_raw( $url ) );
     
    8485        $this->assertEquals( '', esc_url( 'nasty://example.com/' ) );
    8586        $this->assertEquals(
    86             '', esc_url(
    87                 'example.com', array(
     87            '',
     88            esc_url(
     89                'example.com',
     90                array(
    8891                    'https',
    8992                )
     
    9194        );
    9295        $this->assertEquals(
    93             '', esc_url(
    94                 'http://example.com', array(
     96            '',
     97            esc_url(
     98                'http://example.com',
     99                array(
    95100                    'https',
    96101                )
     
    98103        );
    99104        $this->assertEquals(
    100             'https://example.com', esc_url(
    101                 'https://example.com', array(
     105            'https://example.com',
     106            esc_url(
     107                'https://example.com',
     108                array(
    102109                    'http',
    103110                    'https',
     
    109116            $this->assertEquals( "{$scheme}://example.com", esc_url( "{$scheme}://example.com" ), $scheme );
    110117            $this->assertEquals(
    111                 "{$scheme}://example.com", esc_url(
    112                     "{$scheme}://example.com", array(
     118                "{$scheme}://example.com",
     119                esc_url(
     120                    "{$scheme}://example.com",
     121                    array(
    113122                        $scheme,
    114123                    )
    115                 ), $scheme
     124                ),
     125                $scheme
    116126            );
    117127        }
     
    122132        $this->assertTrue( ! in_array( 'foo', wp_allowed_protocols(), true ) );
    123133        $this->assertEquals(
    124             'foo://example.com', esc_url(
    125                 'foo://example.com', array(
     134            'foo://example.com',
     135            esc_url(
     136                'foo://example.com',
     137                array(
    126138                    'foo',
    127139                )
Note: See TracChangeset for help on using the changeset viewer.