Making WordPress.org

Changeset 8212


Ignore:
Timestamp:
02/06/2019 07:57:21 PM (5 years ago)
Author:
iandunn
Message:

WordCamp PHPCS: Allow multiple function calls on the same line.

For example, $wpdb->query( $wpdb->prepare(.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • sites/trunk/wordcamp.org/phpcs.xml.dist

    r8085 r8212  
    6060
    6161        Setup WordPress.WP.I18n text_domain property and test that it works, see https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/wiki/Customizable-sniff-properties#internationalization-setting-your-text-domain
     62
     63        Don't add @package phpDoc? Clutters code without adding any real value. You know what "package" the file is part of because of the folder it's in; e.g., wp-content/plugins/wordcamp-payments
     64
     65        Don't mush HTML elements together on the same line, unless it's really short. `<h1>The title</h1>` is ok, but `<h1><?php echo esc_html( get_the_title() ); ?></h1>` should have the h1 tags on separate lines.
     66
     67        wp_enqueue_*() cachebusters should use filemtime() rather than a hardcoded int.
     68
     69        Combine similar `use` statements on the same line. Like, all the core PHP objects, then all the WP objects on the 2nd line, then our custom objects on the 3rd line, etc.
    6270    -->
    6371
     
    102110        <!-- There are cases where having multiple items on a single line is appropriate. e.g., a list of 100 currency codes. -->
    103111        <exclude name="WordPress.Arrays.ArrayDeclarationSpacing.ArrayItemNoNewLine" />
     112        <exclude name="PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket" />
     113        <exclude name="PEAR.Functions.FunctionCallSignature.CloseBracketLine" />
    104114
    105115        <!-- todo get_speaker_from_user_id() meta_query should not have to line up with post_status
Note: See TracChangeset for help on using the changeset viewer.