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/comment.php

    r43535 r43571  
    382382        {$where}
    383383        GROUP BY comment_approved
    384     ", ARRAY_A
     384    ",
     385        ARRAY_A
    385386    );
    386387
     
    25212522
    25222523    $response = wp_safe_remote_head(
    2523         $url, array(
     2524        $url,
     2525        array(
    25242526            'timeout'     => 2,
    25252527            'httpversion' => '1.0',
     
    25422544    // Now do a GET since we're going to look in the html headers (and we're sure it's not a binary file)
    25432545    $response = wp_safe_remote_get(
    2544         $url, array(
     2546        $url,
     2547        array(
    25452548            'timeout'     => 2,
    25462549            'httpversion' => '1.0',
     
    26572660                    $wpdb->prepare(
    26582661                        "UPDATE $wpdb->posts SET to_ping = TRIM(REPLACE(to_ping, %s,
    2659                     '')) WHERE ID = %d", $tb_ping, $post->ID
     2662                    '')) WHERE ID = %d",
     2663                        $tb_ping,
     2664                        $post->ID
    26602665                    )
    26612666                );
Note: See TracChangeset for help on using the changeset viewer.