inpsyde/php-coding-standards

PHP 7+ coding standards for Inpsyde WordPress projects.

Monday, May 21, 2018
by inpsyde
Repository
7 Watchers
16 Stars
3,064 Installations

PHP
13 Dependents
0 Suggesters
1 Forks
3 Open issues
22 Versions
35 % Grown

The README.md

Syde PHP Coding Standards

PHP 7.4+ coding standards for Syde WordPress projects., _(*1)

, _(*2)

Usage

When the package is installed via Composer, and dependencies are updated, everything is ready and the coding standards can be checked via:, _(*3)

vendor/bin/phpcs --standard="Inpsyde" <path>

Here, <path> is at least one file or directory to check, for example:, _(*4)

vendor/bin/phpcs --standard="Inpsyde" ./src/ ./my-plugin.php

There are many options that can be used to customise the behavior of the command, to get documentation use:, _(*5)

vendor/bin/phpcs --help

Configuration File

A phpcs.xml.dist file can be used to avoid passing many arguments via the command line. For example:, _(*6)


<ruleset name="MyProjectCodingStandard">

    <description>My Project coding standard.</description>

    <file>./src</file>
    <file>./tests/src</file>

    <arg value="sp"/>
    <arg name="colors"/>

    <config name="testVersion" value="7.4-"/>
    <config name="text_domain" value="my-project"/>

    <rule ref="Inpsyde">
        <exclude name="WordPress.PHP.DiscouragedPHPFunctions.serialize_serialize"/>
    </rule>

    <rule ref="Inpsyde.CodeQuality.Psr4">
        <properties>
            <property
                name="psr4"
                type="array"
                value="
                    Inpsyde\MyProject=>src,
                    Inpsyde\MyProject\Tests=>tests/src|tests/unit
                "/>
        </properties>
    </rule>

</ruleset>

Such a configuration allows to run the code style check like so:, _(*7)

vendor/bin/phpcs

Moreover, thanks to the text_domain setting, PHP_CodeSniffer will also check that all WordPress internationalization functions are called with the proper text domain., _(*8)

Included rules

For the detailed lists of included rules, refer to ruleset.xml., _(*9)

PSR-1, PSR-2, PSR-12

For more information about included rules from PHP Standards Recommendations (PSR), refer to the official documentation:, _(*10)

WordPress Coding Standards

To ensure code quality, and compatibility with WordPress VIP, some rules have been included from:, _(*11)

Slevomat

A few rules have been included from the Slevomat Coding Standard., _(*12)

PHPCompatibility

For PHP cross-version compatibility checks, the full PHP Compatibility Coding Standard for PHP CodeSniffer standard has been included., _(*13)

The target PHP version (range) can be changed via a custom phpcs.xml file., _(*14)

Generic Rules

Some rules are also included from PHP_CodeSniffer itself, as well as PHPCSExtra., _(*15)

Custom Rules

The following custom rules are in use:, _(*16)

Sniff Name	Description	Has Config	Auto-Fixable
`ArgumentTypeDeclaration`	Enforce argument type declaration.
`DisableCallUserFunc`	Disable usage of `call_user_func`.
`DisableMagicSerialize`	Disable usage of `__sleep`, `__wakeup`.
`DisableSerializeInterface`	Disable usage of `Serializable` interface.
`DisallowShortOpenTag`	Disallow short open PHP tag (short echo tag allowed).
`ElementNameMinimalLength`	Use minimum 3 chars for names (with a few exclusions)	✓
`EncodingComment`	Detect usage of opening `-- coding: utf-8 --`	✓	✓
`ForbiddenPublicProperty`	No public class properties
`FunctionBodyStart`	Handle blank line at start of function body.		✓
`FunctionLength`	Max 50 lines per function/method, excluding blank lines and comments-only lines.	✓
`HookClosureReturn`	Ensure that actions callbacks do not return anything, while filter callbacks return something.
`HookPriority`	Report usage of `PHP_INT_MAX` and `PHP_INT_MIN` as hook priority.
`LineLength`	Max 100 chars per line	✓
`NestingLevel`	Max indent level of 3 inside functions	✓
`NoAccessors`	Discourage usage of getters and setters.
`NoElse`	Discourage usage of `else`.
`NoRootNamespaceFunctions`	Report usage of global functions in the root namespace.
`NoTopLevelDefine`	Discourage usage of `define` where `const` is preferable.
`PropertyPerClassLimit`	Discourage usage of more than 10 properties per class.	✓
`Psr4`	Check PSR-4 compliance	✓
`ReturnTypeDeclaration`	Enforce return type declaration
`StaticClosure`	Points closures that can be `static`.		✓
`VariablesName`	Check variable (and properties) names	✓

For notes and configuration, refer to the inpsyde-custom-sniffs.md file in this repository., _(*17)

Template Rules

The InpsydeTemplates ruleset extends the standard Inpsyde ruleset with some template-specific sniffs., _(*18)

The recommended way to use the InpsydeTemplates ruleset is as follows:, _(*19)

<ruleset>
    <file>./src</file>
    <file>./templates</file>
    <file>./tests</file>
    <file>./views</file>

    <rule ref="Inpsyde"/>

    <rule ref="InpsydeTemplates">
        <include-pattern>*/templates/*</include-pattern>
        <include-pattern>*/views/*</include-pattern>
    </rule>
</ruleset>

The following template-specific rules are available:, _(*20)

Sniff Name	Description	Auto-Fixable
`AlternativeControlStructure`	Encourage usage of alternative syntax with inline HTML.
`ShortEchoTag`	Replace echo with short echo tag in single-line statements.	✓
`TrailingSemicolon`	Remove trailing semicolon before closing PHP tag.	✓

Removing or Disabling Rules

Rules Tree

Sometimes it is necessary not to follow some rules. To avoid error reporting, it is possible to:, _(*21)

remove rules for an entire project via configuration;
disable rules from code, only is specific places.

In both cases, it is possible to remove or disable:, _(*22)

a complete standard;
a standard subset;
a single sniff;
a single rule.

These things are in a hierarchical relationship: standards are made of one or more subsets, which contain one or more sniffs, which in turn contain one or more rules., _(*23)

Removing Rules via Configuration File

Rules can be removed for the entire project by using a custom phpcs.xml file, like this:, _(*24)


<ruleset name="MyProjectCodingStandard">

    <rule ref="Inpsyde">
        <exclude name="PSR1.Classes.ClassDeclaration"/>
    </rule>

</ruleset>

In the example above, the PSR1.Classes.ClassDeclaration sniff (and all the rules it contains) has been removed., _(*25)

By using PSR1 instead of PSR1.Classes.ClassDeclaration, one would remove the entire PSR1 standard, whereas using PSR1.Classes.ClassDeclaration.MultipleClasses would remove this one rule only, but no other rules in the PSR1.Classes.ClassDeclaration sniff., _(*26)

Removing Rules via Code Comments

Removing a rule/sniff/subset/standard only for a specific file or a part of it can be done by using special phpcs annotations/comments, for example, // phpcs:disable followed by an optional name of a standard/subset/sniff/rule. Like so:, _(*27)

// phpcs:disable PSR1.Classes.ClassDeclaration

For more information about ignoring files, please refer to the official PHP_CodeSniffer Wiki., _(*28)

IDE Integration

PhpStorm

After installing the package as explained above, open PhpStorm settings, and navigate to, _(*29)

Language & Frameworks -> PHP -> Quality Tools -> PHP_CodeSniffer, _(*30)

Choose "Local" in the "Configuration" dropdown., _(*31)

Click the "..." button next to the dropdown. It will show a dialog where you need to specify the path for the PHP_CodeSniffer executable., _(*32)

Open the file selection dialog, navigate to vendor/bin/ in your project, and select phpcs. On Windows, choose phpcs.bat., _(*33)

Click the "Validate" button next to the path input field. If everything is working fine, a success message will be shown at the bottom of the window., _(*34)

Still in the PhpStorm settings, navigate to:, _(*35)

Editor -> Inspections, _(*36)

Type codesniffer in the search field before the list of inspections, then select:, _(*37)

PHP -> Quality Tools -> PHP_CodeSniffer validation, _(*38)

Enable it using the checkbox in the list, press "Apply"., _(*39)

Select "PHP_CodeSniffer validation", click the refresh icon next to the "Coding standard" dropdown on the right, and choose Inpsyde., _(*40)

If you don't see Inpsyde here, you may need to specify the phpcs.xml file by selecting "Custom" as standard and then use the "..." button next to the dropdown., _(*41)

Once the PhpStorm integration is complete, warnings and errors in your code will automatically be shown in your IDE editor., _(*42)

Installation

Via Composer, require as development dependency:, _(*43)

composer require "inpsyde/php-coding-standards:^2@dev" --dev

(Please note that @dev can be removed as soon as a stable 2.0.0 version has been released, or if your root package minimum stability is dev)., _(*44)

The Versions

21/05 2018

dev-master

9999999-dev

PHP 7+ coding standards for Inpsyde WordPress projects.

phpcodesniffer-standard php-coding-standards

PHP 7+ coding standards for Inpsyde WordPress projects.

inpsyde/php-coding-standards

The README.md

Syde PHP Coding Standards

Usage

Configuration File

Included rules

PSR-1, PSR-2, PSR-12

WordPress Coding Standards

Slevomat

PHPCompatibility

Generic Rules

Custom Rules

Template Rules

Removing or Disabling Rules

Rules Tree

Removing Rules via Configuration File

Removing Rules via Code Comments

IDE Integration

PhpStorm

Installation

The Versions

dev-master

The Requires

The Development Requires

by Inpsyde GmbH

0.13.1

The Requires

The Development Requires

by Inpsyde GmbH

0.13.0

The Requires

The Development Requires

by Inpsyde GmbH

0.12.0

The Requires

The Development Requires

by Inpsyde GmbH

0.11.0

The Requires

The Development Requires

by Inpsyde GmbH

dev-issue-8

The Requires

The Development Requires

by Inpsyde GmbH

0.10.0

The Requires

The Development Requires

by Inpsyde GmbH

0.9.0

The Requires

The Development Requires

by Inpsyde GmbH

0.8.0

The Requires

The Development Requires

by Inpsyde GmbH

0.7.2

The Requires

The Development Requires

by Inpsyde GmbH

0.7.1

The Requires

The Development Requires

by Inpsyde GmbH

0.7.0

The Requires

The Development Requires

by Inpsyde GmbH

0.6.0

The Requires

The Development Requires

by Inpsyde GmbH

0.5.1

The Requires

by Inpsyde GmbH

0.5.0

The Requires