You are browsing a version that is no longer maintained. |
Introduction
The Doctrine Coding Standard is a set of rules for PHP_CodeSniffer and applies to all Doctrine projects. It is based on PSR-1 and PSR-12, with some noticeable exceptions/differences/extensions.
- Keep the nesting of control structures per method as small as possible
- Abstract exception class names and exception interface names should be suffixed with
Exception
- Abstract classes should not be prefixed or suffixed with
Abstract
- Interfaces should not be prefixed or suffixed with
Interface
- Concrete exception class names should not be prefixed or suffixed with
Exception
- Align equals (
=
) signs in assignments - Add spaces around a concatenation operator
$foo = 'Hello ' . 'World!';
- Add spaces between assignment, control and return statements
- Add spaces after a negation operator
if (! $cond)
- Add spaces around a colon in return type declaration
function () : void {}
- Add spaces after a type cast
$foo = (int) '12345';
- Use apostrophes for enclosing strings
- Always use strict comparisons
- Always add
declare(strict_types=1)
at the beginning of a file - Always add native types where possible
- Omit phpDoc for parameters/returns with native types, unless adding description
- Don't use
@author
,@since
and similar annotations that duplicate Git information - Assignment in condition is not allowed
- Use parentheses when creating new instances that do not require arguments
$foo = new Foo()
- Use Null Coalesce Operator
$foo = $bar ?? $baz
- Prefer early exit over nesting conditions or using else
For full reference of enforcements, go through lib/Doctrine/ruleset.xml
where each sniff is briefly described.
Installation
You have two possibilities to use the Doctrine Coding Standard with PHP_CodeSniffer in a particular project.
Composer Project Dependency
You can install the Doctrine Coding Standard as a composer dependency to your particular project. Just run the following command to add the Doctrine Coding Standard to your project:
$ php composer require --dev doctrine/coding-standard
Then you can use it like:
$ vendor/bin/phpcs --standard=Doctrine /path/to/some/file/to/sniff.php
You might also do automatic fixes using phpcbf
:
$ vendor/bin/phpcbf --standard=Doctrine /path/to/some/file/to/sniff.php
Composer Global Installation
You can also install the Doctrine Coding Standard globally:
$ composer global require doctrine/coding-standard
Then you can use it like:
$ phpcs --standard=Doctrine /path/to/some/file/to/sniff.php
You might also do automatic fixes using phpcbf
:
$ phpcbf --standard=Doctrine /path/to/some/file/to/sniff.php
Project-level ruleset
To enable Doctrine Coding Standard for your project, create a phpcs.xml.dist
file with the following content:
1 <?xml version="1.0"?>
<ruleset>
<arg name="basepath" value="."/>
<arg name="extensions" value="php"/>
<arg name="parallel" value="80"/>
<arg name="cache" value=".phpcs-cache"/>
<arg name="colors"/>
<!-- Ignore warnings, show progress of the run and show sniff names -->
<arg value="nps"/>
<!-- Directories to be checked -->
<file>lib</file>
<file>tests</file>
<!-- Include full Doctrine Coding Standard -->
<rule ref="Doctrine"/>
</ruleset>
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
This will enable verbatim Doctrine Coding Standard with all rules included with their defaults.
From now on you can just run vendor/bin/phpcs
and vendor/bin/phpcbf
without any arguments.
Don't forget to add .phpcs-cache
and phpcs.xml
(without .dist
suffix) to your .gitignore
.
The first ignored file is a cache used by PHP CodeSniffer to speed things up,
the second one allows any developer to adjust configuration locally without touching the versioned file.
For further reading about the CodeSniffer configuration, please refer to the configuration format overview and the list of configuration options
To learn about customizing the rules, please refer to the chapter.
Contributing
Versioning
This library follows semantic versioning, and additions to the code ruleset are only performed in major releases.
The patch branch can receive internal changes, such as CI pipeline improvements, test workflow improvements, and can receive bugfixes. For instance, such as removing rules that were not meant to be added.
The minor branch can receive refactorings, and dependency updates that do not affect the code ruleset.
Testing
If you are contributing to the Doctrine Coding Standard and want to test your contribution, you just
need to execute the test
GNU make target:
$ make test
Sometimes you are enabling a sniff that enforces some features for a specific PHP version, that will probably cause our build pipeline to fail when running on such versions.
We have created the update-compatibility-patch-*
GNU make targets to
help you with that. For instance, for PHP 6.0, you would run
$ make update-compatibility-patch-60
That target will attempt to apply the latest version of our
compatibility patch for that version of PHP and ask you to adjust the
code in tests/fixed
and tests/expected_report.txt
using the
editor of your preference.
If the patch does not apply cleanly, you can edit it to edit or remove hunks that relate to files you changed in your PR.
Once you are done, please type "y" to resume the command, which will update the compatibility patch and create a commit with your changes for you to push to your remote branch.