Slevomat Coding Standard for PHP_CodeSniffer provides sniffs that fall into three categories:
- Functional - improving the safety and behaviour of code
- Cleaning - detecting dead code
- Formatting - rules for consistent code looks
- Functional - improving the safety and behaviour of code
- Cleaning - detecting dead code
- Formatting - rules for consistent code looks
π§ = Automatic errors fixing
π§ = Sniff check can be suppressed locally
- Checks for missing parameter typehints in case they can be declared natively. If the phpDoc contains something that can be written as a native PHP 7.0+ typehint, this sniff reports that.
- Checks for useless
@param
annotations. If the native method declaration contains everything and the phpDoc does not add anything useful, it's reported as useless and can optionally be automatically removed withphpcbf
. - Forces to specify what's in traversable types like
array
,iterable
and\Traversable
.
Sniff provides the following settings:
enableObjectTypeHint
: enforces to transform@param object
into nativeobject
typehint. It's on by default if you're on PHP 7.2+enableMixedTypeHint
: enforces to transform@param mixed
into nativemixed
typehint. It's on by default if you're on PHP 8.0+traversableTypeHints
: enforces which typehints must have specified contained type. E. g. if you set this to\Doctrine\Common\Collections\Collection
, then\Doctrine\Common\Collections\Collection
must always be supplied with the contained type:\Doctrine\Common\Collections\Collection|Foo[]
.
This sniff can cause an error if you're overriding or implementing a parent method which does not have typehints. In such cases add @phpcsSuppress SlevomatCodingStandard.TypeHints.ParameterTypeHint.MissingNativeTypeHint
annotation to the method to have this sniff skip it.
- Checks for missing property typehints in case they can be declared natively. If the phpDoc contains something that can be written as a native PHP 7.4+ typehint, this sniff reports that.
- Checks for useless
@var
annotations. If the native method declaration contains everything and the phpDoc does not add anything useful, it's reported as useless and can optionally be automatically removed withphpcbf
. - Forces to specify what's in traversable types like
array
,iterable
and\Traversable
.
Sniff provides the following settings:
enableNativeTypeHint
: enforces to transform@var int
into nativeint
typehint. It's on by default if you're on PHP 7.4+enableMixedTypeHint
: enforces to transform@var mixed
into nativemixed
typehint. It's on by default if you're on PHP 8.0+. It can be enabled only whenenableNativeTypeHint
is enabled too.enableUnionTypeHint
: enforces to transform@var string|int
into nativestring|int
typehint. It's on by default if you're on PHP 8.0+. It can be enabled only whenenableNativeTypeHint
is enabled too.traversableTypeHints
: enforces which typehints must have specified contained type. E. g. if you set this to\Doctrine\Common\Collections\Collection
, then\Doctrine\Common\Collections\Collection
must always be supplied with the contained type:\Doctrine\Common\Collections\Collection|Foo[]
.
This sniff can cause an error if you're overriding parent property which does not have typehints. In such cases add @phpcsSuppress SlevomatCodingStandard.TypeHints.PropertyTypeHint.MissingNativeTypeHint
annotation to the property to have this sniff skip it.
- Checks for missing return typehints in case they can be declared natively. If the phpDoc contains something that can be written as a native PHP 7.0+ typehint, this sniff reports that.
- Checks for useless
@return
annotations. If the native method declaration contains everything and the phpDoc does not add anything useful, it's reported as useless and can optionally be automatically removed withphpcbf
. - Forces to specify what's in traversable types like
array
,iterable
and\Traversable
.
Sniff provides the following settings:
enableObjectTypeHint
: enforces to transform@return object
into nativeobject
typehint. It's on by default if you're on PHP 7.2+enableStaticTypeHint
: enforces to transform@return static
into nativestatic
typehint. It's on by default if you're on PHP 8.0+enableMixedTypeHint
: enforces to transform@return mixed
into nativemixed
typehint. It's on by default if you're on PHP 8.0+enableUnionTypeHint
: enforces to transform@var string|int
into nativestring|int
typehint. It's on by default if you're on PHP 8.0+.traversableTypeHints
: enforces which typehints must have specified contained type. E. g. if you set this to\Doctrine\Common\Collections\Collection
, then\Doctrine\Common\Collections\Collection
must always be supplied with the contained type:\Doctrine\Common\Collections\Collection|Foo[]
.
This sniff can cause an error if you're overriding or implementing a parent method which does not have typehints. In such cases add @phpcsSuppress SlevomatCodingStandard.TypeHints.ReturnTypeHint.MissingNativeTypeHint
annotation to the method to have this sniff skip it.
Reports useless @var
annotation (or whole documentation comment) for constants because the type of constant is always clear.
Checks format of union type hints.
Sniff provides the following settings:
enable
: either to enable or not this sniff. By default, it is enabled for PHP versions 8.0 or higher.withSpaces
:yes
requires spaces arount|
,no
requires no space around|
. None is set by default so both is enabled.shortNullable
:yes
requires usage of?
for nullable type hint,no
disallows it. None is set by default so both are enabled.nullPosition
:first
requiresnull
on first position in the type hint,last
requires last position. None is set by default sonull
can be everywhere.
This sniff forbids use of non-capturing catch introduced in PHP 8.0 PHP RFC: non-capturing catches.
In PHP 7.0, a Throwable
interface was added that allows catching and handling errors in more cases than Exception
previously allowed. So, if the catch statement contained Exception
on PHP 5.x, it means it should probably be rewritten to reference Throwable
on PHP 7.x. This sniff enforces that.
Sniff provides the following settings:
enable
: either to enable or not this sniff. By default, it is enabled for PHP versions 8.0 or higher.
It requires non-capturing catch when the variable with exception is not used.
Enforces having declare(strict_types = 1)
at the top of each PHP file. Allows configuring how many newlines should be between the <?php
opening tag and the declare
statement.
Sniff provides the following settings:
declareOnFirstLine
: requiresdeclare
on the first line right after<?php
linesCountBeforeDeclare
: allows to set 0 to N lines to be betweendeclare
and previous statement. This option is ignored whendeclareOnFirstLine
is enabled.linesCountAfterDeclare
: allows to set 0 to N lines to be betweendeclare
and next statementspacesCountAroundEqualsSign
: allows to set number of required spaces around the=
operator
Disallows implicit array creation.
Checks that class/trait/interface members are in the correct order.
Sniff provides the following settings:
groups
: order of groups. Use multiple groups in one<element value="">
to not differentiate among them. You can use specific groups or shortcuts.enableFinalMethods
: enables groups forfinal
methods
List of supported groups: uses, public constants, protected constants, private constants, public properties, public static properties, protected properties, protected static properties, private properties, private static properties, constructor, static constructors, destructor, magic methods, public methods, protected methods, private methods, public final methods, public static final methods, protected final methods, protected static final methods, public abstract methods, public static abstract methods, protected abstract methods, protected static abstract methods, public static methods, protected static methods, private static methods, private methods
List of supported shortcuts: constants, properties, static properties, methods, all public methods, all protected methods, all private methods, static methods, final methods, abstract methods
<rule ref="SlevomatCodingStandard.Classes.ClassStructure">
<properties>
<property name="groups" type="array">
<element value="uses"/>
<!-- Public constants are first but you don't care about the order of protected or private constants -->
<element value="public constants"/>
<element value="constants"/>
<!-- You don't care about the order among the properties. The same can be done with "properties" shortcut -->
<element value="public properties, protected properties, private properties"/>
<!-- Constructor is first, then all public methods, then protected/private methods and magic methods are last -->
<element value="constructor"/>
<element value="all public methods"/>
<element value="methods"/>
<element value="magic methods"/>
</property>
</properties>
</rule>
Disallows usage of constructor property promotion.
Disallows late static binding for constants.
Disallows using public properties.
Requires use of constructor property promotion.
This sniff provides the following setting:
enable
: either to enable or not this sniff. By default, it is enabled for PHP versions 8.0 or higher.
Reports useless late static binding.
Disallows assignments in if
, elseif
and do-while
loop conditions:
if ($file = findFile($path)) {
}
Assignment in while
loop condition is specifically allowed because it's commonly used.
This is a great addition to already existing SlevomatCodingStandard.ControlStructures.DisallowYodaComparison
because it prevents the danger of assigning something by mistake instead of using comparison operator like ===
.
Sniff provides the following settings:
ignoreAssignmentsInsideFunctionCalls
: ignores assignment inside function calls, like this:
if (in_array(1, $haystack, $strict = true)) {
}
Disallows use of continue
without integer operand in switch
because it emits a warning in PHP 7.3 and higher.
Disallows use of empty()
.
Requires use of null coalesce operator when possible.
Requires use of null coalesce equal operator when possible.
This sniff provides the following setting:
enable
: either to enable or not this sniff. By default, it is enabled for PHP versions 7.4 or higher.
Requires use of early exit.
Sniff provides the following settings:
ignoreStandaloneIfInScope
: ignoresif
that is standalone in scope, like this:
foreach ($values as $value) {
if ($value) {
doSomething();
}
}
ignoreOneLineTrailingIf
: ignoresif
that has one line content and is on the last position in scope, like this:
foreach ($values as $value) {
$value .= 'whatever';
if ($value) {
doSomething();
}
}
ignoreTrailingIfWithOneInstruction
: ignoresif
that has only one instruction and is on the last position in scope, like this:
foreach ($values as $value) {
$value .= 'whatever';
if ($value) {
doSomething(function () {
// Anything
});
}
}
Some functions have $strict
parameter. This sniff reports calls to these functions without the parameter or with $strict = false
.
Reports closures not using $this
that are not declared static
.
Disallows long functions. This sniff provides the following setting:
maxLinesLength
: specifies max allowed function lines length (default value is 20).
Disallows direct call of __invoke()
.
Disallows using loose ==
and !=
comparison operators. Use ===
and !==
instead, they are much more secure and predictable.
Disallows using ++
and --
operators.
Reports ++
and --
operators not used standalone.
Requires using combined assignment operators, eg +=
, .=
etc.
Looks for unused inherited variables passed to closure via use
.
Looks for unused parameters.
Looks for useless parameter default value.
Looks for unused imports from other namespaces.
Sniff provides the following settings:
searchAnnotations
(defaults tofalse
): enables searching for class names in annotations.ignoredAnnotationNames
: case sensitive list of annotation names that the sniff should ignore (only the name is ignored, annotation content is still searched). Useful for name collisions like@testCase
annotation andTestCase
class.ignoredAnnotations
: case sensitive list of annotation names that the sniff ignore completely (both name and content are ignored). Useful for name collisions like@group Cache
annotation andCache
class.
Prohibits uses from the same namespace:
namespace Foo;
use Foo\Bar;
Looks for use
alias that is same as unqualified name.
Disallows references.
Reports usage of forbidden classes, interfaces, parent classes and traits. And provide the following settings:
forbiddenClasses
: forbids creating instances withnew
keyword or accessing with::
operatorforbiddenExtends
: forbids extending withextends
keywordforbiddenInterfaces
: forbids usage inimplements
sectionforbiddenTraits
: forbids imports withuse
keyword
Optionally can be passed as an alternative for auto fixes. See phpcs.xml
file example:
<rule ref="SlevomatCodingStandard.PHP.ForbiddenClasses">
<properties>
<property name="forbiddenClasses" type="array">
<element key="Validator" value="Illuminate\Support\Facades\Validator"/>
</property>
<property name="forbiddenTraits" type="array">
<element key="\AuthTrait" value="null"/>
</property>
</properties>
</rule>
Requires assertion via assert
instead of inline documentation comments.
Requires nowdoc syntax instead of heredoc when possible.
Looks for useless parentheses.
Sniff provides the following settings:
ignoreComplexTernaryConditions
(defaults tofalse
): ignores complex ternary conditions - condition must contain&&
,||
etc or end of line.
PHP optimizes some internal functions into special opcodes on VM level. Such optimization results in much faster execution compared to calling standard function. This only works when these functions are not invoked with argument unpacking (...
).
The list of these functions varies across PHP versions, but is the same as functions that must be referenced by their global name (either by \
prefix or using use function
), not a fallback name inside namespaced code.
Looks for useless semicolons.
Disallows use of super global variables.
Looks for duplicate assignments to a variable.
Looks for unused variables.
Sniff provides the following settings:
ignoreUnusedValuesWhenOnlyKeysAreUsedInForeach
(defaults tofalse
): ignore unused$value
in foreach when only$key
is used
foreach ($values as $key => $value) {
echo $key;
}
Looks for useless variables.
This sniff finds unreachable catch blocks:
try {
doStuff();
} catch (\Throwable $e) {
log($e);
} catch (\InvalidArgumentException $e) {
// unreachable!
}
Enforces reasonable end bracket placement for multi-line arrays.
Checks whitespace in single line array declarations (whitespace between brackets, around commas, ...).
Sniff provides the following settings:
spacesAroundBrackets
: number of spaces you require to have around array bracketsenableEmptyArrayCheck
(defaults tofalse
): enables check for empty arrays
Commas after last element in an array make adding a new element easier and result in a cleaner versioning diff.
This sniff enforces trailing commas in multi-line arrays and requires short array syntax []
.
Sniff provides the following settings:
enableAfterHeredoc
: enables/disables trailing commas after HEREDOC/NOWDOC, default based on PHP version.
Checks lines count between different class members, eg. between last property and first method.
Sniff provides the following settings:
linesCountBetweenMembers
: lines count between different class members
Checks that there is a certain number of blank lines between constants.
Sniff provides the following settings:
minLinesCountBeforeWithComment
: minimum number of lines before constant with a doc commentmaxLinesCountBeforeWithComment
: maximum number of lines before constant with a doc commentminLinesCountBeforeWithoutComment
: minimum number of lines before constant without a doc commentmaxLinesCountBeforeWithoutComment
: maximum number of lines before constant without a doc comment
Disallows multi constant definition.
Disallows multi property definition.
Checks that there is a certain number of blank lines between methods.
Sniff provides the following settings:
minLinesCount
: minimum number of blank linesmaxLinesCount
: maximum number of blank lines
Reports use of __CLASS__
, get_parent_class()
, get_called_class()
, get_class()
and get_class($this)
.
Class names should be referenced via ::class
constant when possible.
Sniff provides the following settings:
enableOnObjects
: Enable::class
on all objects. It's on by default if you're on PHP 8.0+
Enforces configurable number of lines around parent method call.
Sniff provides the following settings:
linesCountBefore
: allows to configure the number of lines before parent call.linesCountBeforeFirst
: allows to configure the number of lines before first parent call.linesCountAfter
: allows to configure the number of lines after parent call.linesCountAfterLast
: allows to configure the number of lines after last parent call.
Checks that there is a certain number of blank lines between properties.
Sniff provides the following settings:
minLinesCountBeforeWithComment
: minimum number of lines before property with a doc commentmaxLinesCountBeforeWithComment
: maximum number of lines before property with a doc commentminLinesCountBeforeWithoutComment
: minimum number of lines before property without a doc commentmaxLinesCountBeforeWithoutComment
: maximum number of lines before property without a doc comment
Enforces method signature to be splitted to more lines so each parameter is on its own line.
Sniff provides the following settings:
-
minLineLength
: specifies min line length to enforce signature to be splitted. Use 0 value to enforce for all methods, regardless of length. -
includedMethodPatterns
: allows to configure which methods are included in sniff detection. This is an array of regular expressions (PCRE) with delimiters. You should not use this withexcludedMethodPatterns
, as it will not work properly. -
excludedMethodPatterns
: allows to configure which methods are excluded from sniff detection. This is an array of regular expressions (PCRE) with delimiters. You should not use this withincludedMethodPatterns
, as it will not work properly.
Enforces method signature to be on a single line.
Sniff provides the following settings:
-
maxLineLength
: specifies max allowed line length. If signature would fit on it, it's enforced. Use 0 value to enforce for all methods, regardless of length. -
includedMethodPatterns
: allows to configure which methods are included in sniff detection. This is an array of regular expressions (PCRE) with delimiters. You should not use this withexcludedMethodPatterns
, as it will not work properly. -
excludedMethodPatterns
: allows to configure which methods are excluded from sniff detection. This is an array of regular expressions (PCRE) with delimiters. You should not use this withincludedMethodPatterns
, as it will not work properly.
Reports use of superfluous prefix or suffix "Abstract" for abstract classes.
Reports use of superfluous prefix or suffix "Interface" for interfaces.
Reports use of superfluous suffix "Exception" for exceptions.
Reports use of superfluous suffix "Error" for errors.
Reports use of superfluous suffix "Trait" for traits.
Prohibits multiple traits separated by commas in one use
statement.
Enforces configurable number of lines before first use
, after last use
and between two use
statements.
Sniff provides the following settings:
linesCountBeforeFirstUse
: allows to configure the number of lines before firstuse
.linesCountBeforeFirstUseWhenFirstInClass
: allows to configure the number of lines before firstuse
when theuse
is the first statement in the class.linesCountBetweenUses
: allows to configure the number of lines between twouse
statements.linesCountAfterLastUse
: allows to configure the number of lines after lastuse
.linesCountAfterLastUseWhenLastInClass
: allows to configure the number of lines after lastuse
when theuse
is the last statement in the class.
Enforces configurable number of lines around block control structures (if, foreach, ...).
Sniff provides the following settings:
linesCountBefore
: allows to configure the number of lines before control structure.linesCountBeforeFirst
: allows to configure the number of lines before first control structure.linesCountAfter
: allows to configure the number of lines after control structure.linesCountAfterLast
: allows to configure the number of lines after last control structure.controlStructures
: allows to narrow the list of checked control structures.
For example, with the following setting, only if
and switch
keywords are checked.
<rule ref="SlevomatCodingStandard.ControlStructures.BlockControlStructureSpacing">
<properties>
<property name="controlStructures" type="array">
<element value="if"/>
<element value="switch"/>
</property>
</properties>
</rule>
Enforces configurable number of lines around jump statements (continue, return, ...).
Sniff provides the following settings:
allowSingleLineYieldStacking
: whether or not to allow multiple yield/yield from statements in a row without blank lines.linesCountBefore
: allows to configure the number of lines before jump statement.linesCountBeforeFirst
: allows to configure the number of lines before first jump statement.linesCountBeforeWhenFirstInCaseOrDefault
: allows to configure the number of lines before jump statement that is first incase
ordefault
linesCountAfter
: allows to configure the number of lines after jump statement.linesCountAfterLast
: allows to configure the number of lines after last jump statement.linesCountAfterWhenLastInCaseOrDefault
: allows to configure the number of lines after jump statement that is last incase
ordefault
linesCountAfterWhenLastInLastCaseOrDefault
: allows to configure the number of lines after jump statement that is last in lastcase
ordefault
jumpStatements
: allows to narrow the list of checked jump statements.
For example, with the following setting, only continue
and break
keywords are checked.
<rule ref="SlevomatCodingStandard.ControlStructures.JumpStatementsSpacing">
<properties>
<property name="jumpStatements" type="array">
<element value="continue"/>
<element value="break"/>
</property>
</properties>
</rule>
LanguageConstructWithParenthesesSniff
checks and fixes language construct used with parentheses.
Requires new
with parentheses.
Reports new
with useless parentheses.
Disallows using ?->
operator.
Disallows short ternary operator ?:
.
Sniff provides the following settings:
fixable
: the sniff is fixable by default, however in strict code it makes sense to forbid this weakly typed form of ternary altogether, you can disable fixability with this option.
Ternary operator has to be reformatted to more lines when the line length exceeds the given limit.
Sniff provides the following settings:
lineLengthLimit
(defaults to0
)minExpressionsLength
(defaults tonull
): when the expressions after?
are shorter than this length, the ternary operator does not has to be reformatted.
Requires using ?->
operator.
Sniff provides the following settings:
enable
: either to enable or not this sniff. By default, it is enabled for PHP versions 8.0 or higher.
Enforces conditions of if
, elseif
, while
and do-while
to be on a single line.
Sniff provides the following settings:
maxLineLength
: specifies max allowed line length. If conditition (and the rest of the line) would fit on it, it's enforced. Use 0 value to enforce for all conditions, regardless of length.alwaysForSimpleConditions
: allows to enforce single line for all simple conditions (i.e no&&
,||
orxor
), regardless of length.
Enforces conditions of if
, elseif
, while
and do-while
with one or more boolean operators to be splitted to more lines
so each condition part is on its own line.
Sniff provides the following settings:
minLineLength
: specifies mininum line length to enforce condition to be splitted. Use 0 value to enforce for all conditions, regardless of length.booleanOperatorOnPreviousLine
: boolean operator is placed at the end of previous line when fixing.alwaysSplitAllConditionParts
: require all condition parts to be on its own line - it reports error even if condition is already multi-line but there are some condition parts on the same line.
Requires short ternary operator ?:
when possible.
Requires ternary operator when possible.
Sniff provides the following settings:
ignoreMultiLine
(defaults tofalse
): ignores multi-line statements.
Yoda conditions decrease code comprehensibility and readability by switching operands around comparison operators forcing the reader to read the code in an unnatural way.
Sniff provides the following settings:
alwaysVariableOnRight
(defaults tofalse
): moves variables always to right.
DisallowYodaComparisonSniff
looks for and fixes such comparisons not only in if
statements but in the whole code.
However, if you prefer Yoda conditions, you can use RequireYodaComparisonSniff
.
Enforces maximum length of a single line of code.
Sniff provides the following settings:
lineLengthLimit
: actual limit of the line lengthignoreComments
: whether or not to ignore line length of commentsignoreImports
: whether or not to ignore line length of import (use) statements
Checks fn
declaration.
Sniff provides the following settings:
spacesCountAfterKeyword
: the number of spaces afterfn
.spacesCountBeforeArrow
: the number of spaces before=>
.spacesCountAfterArrow
: the number of spaces after=>
.allowMultiLine
: allows multi-line declaration.
Reports empty functions body and requires at least a comment inside.
Disallows arrow functions.
Requires arrow functions.
Sniff provides the following settings:
allowNested
(defaults totrue
)enable
: either to enable or not this sniff. By default, it is enabled for PHP versions 7.4 or higher.
Enforces function call to be splitted to more lines so each parameter is on its own line.
Sniff provides the following settings:
minLineLength
: specifies min line length to enforce call to be splitted. Use 0 value to enforce for all calls, regardless of length.
Enforces function call to be on a single line.
Sniff provides the following settings:
maxLineLength
: specifies max allowed line length. If call would fit on it, it's enforced. Use 0 value to enforce for all calls, regardless of length.ignoreWithComplexParameter
(defaults totrue
): ignores calls with arrays, closures, arrow functions and nested calls.
This sniff disallows usage of named arguments.
This sniff disallows trailing commas in multi-line calls.
Commas after the last parameter in function or method call make adding a new parameter easier and result in a cleaner versioning diff.
This sniff enforces trailing commas in multi-line calls.
This sniff provides the following setting:
enable
: either to enable or not this sniff. By default, it is enabled for PHP versions 7.3 or higher.
This sniff disallows trailing commas in multi-line declarations.
Commas after the last parameter in function or method declaration make adding a new parameter easier and result in a cleaner versioning diff.
This sniff enforces trailing commas in multi-line declarations.
This sniff provides the following setting:
enable
: either to enable or not this sniff. By default, it is enabled for PHP versions 8.0 or higher.
Checks whether uses at the top of a file are alphabetically sorted. Follows natural sorting and takes edge cases with special symbols into consideration. The following code snippet is an example of correctly sorted uses:
use LogableTrait;
use LogAware;
use LogFactory;
use LoggerInterface;
use LogLevel;
use LogStandard;
Sniff provides the following settings:
psr12Compatible
(defaults totrue
): sets the required order toclasses
,functions
andconstants
.false
sets the required order toclasses
,constants
andfunctions
.caseSensitive
: compare namespaces case sensitively, which makes this order correct:
use LogAware;
use LogFactory;
use LogLevel;
use LogStandard;
use LogableTrait;
use LoggerInterface;
Requires only one namespace in a file.
Enforces one space after namespace
, disallows content between namespace name and semicolon and disallows use of bracketed syntax.
Enforces configurable number of lines before and after namespace
.
Sniff provides the following settings:
linesCountBeforeNamespace
: allows to configure the number of lines beforenamespace
.linesCountAfterNamespace
: allows to configure the number of lines afternamespace
.
Enforces configurable number of lines before first use
, after last use
and between two different types of use
(eg. between use function
and use const
). Also enforces zero number of lines between same types of use
.
Sniff provides the following settings:
linesCountBeforeFirstUse
: allows to configure the number of lines before firstuse
.linesCountBetweenUseTypes
: allows to configure the number of lines between two different types ofuse
.linesCountAfterLastUse
: allows to configure the number of lines after lastuse
.
Disallows numeric literal separators.
Requires use of numeric literal separators.
This sniff provides the following setting:
enable
: either to enable or not this sniff. By default, it is enabled for PHP versions 7.4 or higher.minDigitsBeforeDecimalPoint
: the mininum digits before decimal point to require separator.minDigitsAfterDecimalPoint
: the mininum digits after decimal point to require separator.
Enforces configurable number of spaces after reference.
Sniff provides the following settings:
spacesCountAfterReference
: the number of spaces after&
.
Checks if there is the same number of spaces after negation operator as expected.
Sniff provides the following settings:
spacesCount
: the number of spaces expected after the negation operator
Enforces configurable number of spaces after the ...
operator.
Sniff provides the following settings:
spacesCountAfterOperator
: the number of spaces after the...
operator.
Disallows usage of array type hint syntax (eg. int[]
, bool[][]
) in phpDocs in favour of generic type hint syntax (eg. array<int>
, array<array<bool>>
).
Sniff provides the following settings:
traversableTypeHints
: helps fixer detect traversable type hints so\Traversable|int[]
can be converted to\Traversable<int>
.
Disallows usage of "mixed" type hint in phpDocs.
Enforces using shorthand scalar typehint variants in phpDocs: int
instead of integer
and bool
instead of boolean
. This is for consistency with native scalar typehints which also allow shorthand variants only.
Enforces null
type hint on last position in annotations.
Enforces using short form of list syntax, [...]
instead of list(...)
.
Enforces using shorthand cast operators, forbids use of unset and binary cast operators: (bool)
instead of (boolean)
, (int)
instead of (integer)
, (float)
instead of (double)
or (real)
. (binary)
and (unset)
are forbidden.
Checks duplicate spaces anywhere because there aren't sniffs for every part of code to check formatting.
Sniff provides the following settings:
ignoreSpacesBeforeAssignment
: to allow multiple spaces to align assignments.ignoreSpacesInAnnotation
: to allow multiple spaces to align annotations.ignoreSpacesInComment
: to allow multiple spaces to align content of the comment.ignoreSpacesInParameters
: to allow multiple spaces to align parameters.
For projects not following the PSR-0 or PSR-4 autoloading standards, this sniff checks whether a namespace and a name of a class/interface/trait follows agreed-on way to organize code into directories and files.
Other than enforcing that the type name must match the name of the file it's contained in, this sniff is very configurable. Consider the following sample configuration:
<rule ref="SlevomatCodingStandard.Files.TypeNameMatchesFileName">
<properties>
<property name="rootNamespaces" type="array">
<element key="app/ui" value="Slevomat\UI"/>
<element key="app" value="Slevomat"/>
<element key="build/SlevomatSniffs/Sniffs" value="SlevomatSniffs\Sniffs"/>
<element key="tests/ui" value="Slevomat\UI"/>
<element key="tests" value="Slevomat"/>
</property>
<property name="skipDirs" type="array">
<element value="components"/>
<element value="forms"/>
<element value="model"/>
<element value="models"/>
<element value="services"/>
<element value="stubs"/>
<element value="data"/>
<element value="new"/>
</property>
<property name="ignoredNamespaces" type="array">
<element value="Slevomat\Services"/>
</property>
</properties>
</rule>
Sniff provides the following settings:
rootNamespaces
property expects configuration similar to PSR-4 - project directories mapped to certain namespaces.skipDirs
are not taken into consideration when comparing a path to a namespace. For example, with the above settings, file at pathapp/services/Product/Product.php
is expected to containSlevomat\Product\Product
, notSlevomat\services\Product\Product
.extensions
: allow different file extensions. Default isphp
.ignoredNamespaces
: sniff is not performed on these namespaces.
In PHP 7.1+ it's possible to declare visibility of class constants. In a similar vein to optional declaration of visibility for properties and methods which is actually required in sane coding standards, this sniff also requires declaring visibility for all class constants.
Sniff provides the following settings:
fixable
: the sniff is not fixable by default because we think it's better to decide about each constant one by one however you can enable fixability with this option.
const FOO = 1; // visibility missing!
public const BAR = 2; // correct
Enforces consistent formatting of return typehints, like this:
function foo(): ?int
Sniff provides the following settings:
spacesCountBeforeColon
: the number of spaces expected between closing brace and colon.
Checks whether the nullablity ?
symbol is present before each nullable and optional parameter (which are marked as = null
):
function foo(
int $foo = null, // ? missing
?int $bar = null // correct
) {
}
- Checks that there's a single space between a typehint and a parameter name:
Foo $foo
- Checks that there's no whitespace between a nullability symbol and a typehint:
?Foo
- Checks that there's a single space between a typehint and a property name:
Foo $foo
- Checks that there's no whitespace between a nullability symbol and a typehint:
?Foo
- Checks that there's a single space before nullability symbol or a typehint:
private ?Foo
orprivate Foo
Group use declarations are ugly, make diffs ugly and this sniff prohibits them.
This sniff reduces confusion in the following code snippet:
try {
$this->foo();
} catch (Exception $e) {
// Is this the general exception all exceptions must extend from? Or Exception from the current namespace?
}
All references to types named Exception
or ending with Exception
must be referenced via a fully qualified name:
try {
$this->foo();
} catch (\FooCurrentNamespace\Exception $e) {
} catch (\Exception $e) {
}
Sniff provides the following settings:
- Exceptions with different names can be configured in
specialExceptionNames
property. - If your codebase uses classes that look like exceptions (because they have
Exception
orError
suffixes) but aren't, you can add them toignoredNames
property and the sniff won't enforce them to be fully qualified. Classes withError
suffix has to be added to ignored only if they are in the root namespace (likeLibXMLError
).
All references to global constants must be referenced via a fully qualified name.
Sniff provides the following settings:
include
: list of global constants that must be referenced via FQN. If not set all constants are considered.exclude
: list of global constants that are allowed not to be referenced via FQN.
All references to global functions must be referenced via a fully qualified name.
Sniff provides the following settings:
include
: list of global functions that must be referenced via FQN. If not set all functions are considered.includeSpecialFunctions
: include complete list of PHP internal functions that could be optimized when referenced via FQN.exclude
: list of global functions that are allowed not to be referenced via FQN.
Prohibits multiple uses separated by commas:
use Foo, Bar;
Sniff provides the following settings:
searchAnnotations
(defaults tofalse
): enables searching for mentions in annotations.namespacesRequiredToUse
: if not set, all namespaces are required to be used. When set, only mentioned namespaces are required to be used. Useful in tandem with UseOnlyWhitelistedNamespaces sniff.allowFullyQualifiedExceptions
,specialExceptionNames
&ignoredNames
: allows fully qualified exceptions. Useful in tandem with FullyQualifiedExceptions sniff.allowFullyQualifiedNameForCollidingClasses
: allow fully qualified name for a class with a colliding use statement.allowFullyQualifiedNameForCollidingFunctions
: allow fully qualified name for a function with a colliding use statement.allowFullyQualifiedNameForCollidingConstants
: allow fully qualified name for a constant with a colliding use statement.allowFullyQualifiedGlobalClasses
: allows using fully qualified classes from global space (i.e.\DateTimeImmutable
).allowFullyQualifiedGlobalFunctions
: allows using fully qualified functions from global space (i.e.\phpversion()
).allowFullyQualifiedGlobalConstants
: allows using fully qualified constants from global space (i.e.\PHP_VERSION
).allowFallbackGlobalFunctions
: allows using global functions via fallback name withoutuse
(i.e.phpversion()
).allowFallbackGlobalConstants
: allows using global constants via fallback name withoutuse
(i.e.PHP_VERSION
).allowPartialUses
: allows using and referencing whole namespaces:
Disallows uses of other than configured namespaces.
Sniff provides the following settings:
namespacesRequiredToUse
: namespaces in this array are the only ones allowed to be used. E. g. root project namespace.allowUseFromRootNamespace
: also allow using top-level namespace:
use DateTimeImmutable;
Disallows leading backslash in use statement:
use \Foo\Bar;
Enforces one configurable number of lines after opening class/interface/trait brace and one empty line before the closing brace.
Sniff provides the following settings:
linesCountAfterOpeningBrace
: allows to configure the number of lines after opening brace.linesCountBeforeClosingBrace
: allows to configure the number of lines before closing brace.
Enforces fully qualified names of classes and interfaces in phpDocs - in annotations. This results in unambiguous phpDocs.
Reports @deprecated
annotations without description.
Disallows comments after code at the same line.
Reports forbidden annotations. No annotations are forbidden by default, the configuration is completely up to the user. It's recommended to forbid obsolete and inappropriate annotations like:
@author
,@created
,@version
: we have version control systems.@package
: we have namespaces.@copyright
,@license
: it's not necessary to repeat licensing information in each file.@throws
: it's not possible to enforce this annotation and the information can become outdated.
Sniff provides the following settings:
forbiddenAnnotations
: allows to configure which annotations are forbidden to be used.
Reports forbidden comments in descriptions. Nothing is forbidden by default, the configuration is completely up to the user. It's recommended to forbid generated or inappropriate messages like:
Constructor.
Created by PhpStorm.
Sniff provides the following settings:
forbiddenCommentPatterns
: allows to configure which comments are forbidden to be used. This is an array of regular expressions (PCRE) with delimiters.
Enforces configurable number of lines before first content (description or annotation), after last content (description or annotation),
between description and annotations, between two different annotations types (eg. between @param
and @return
).
Sniff provides the following settings:
linesCountBeforeFirstContent
: allows to configure the number of lines before first content (description or annotation).linesCountBetweenDescriptionAndAnnotations
: allows to configure the number of lines between description and annotations.linesCountBetweenDifferentAnnotationsTypes
: allows to configure the number of lines between two different annotations types.linesCountBetweenAnnotationsGroups
: allows to configure the number of lines between annotations groups.linesCountAfterLastContent
: allows to configure the number of lines after last content (description or annotation).annotationsGroups
: allows to configurure order of annotations groups and even order of annotations in every group. Supports prefixes, eg.@ORM\
.
<rule ref="SlevomatCodingStandard.Commenting.DocCommentSpacing">
<properties>
<property name="annotationsGroups" type="array">
<element value="
@ORM\,
"/>
<element value="
@var,
@param,
@return,
"/>
</property>
</properties>
</rule>
If annotationsGroups
is set, linesCountBetweenDifferentAnnotationsTypes
is ignored and linesCountBetweenAnnotationsGroups
is applied.
If annotationsGroups
is not set, linesCountBetweenAnnotationsGroups
is ignored and linesCountBetweenDifferentAnnotationsTypes
is applied.
Annotations not in any group are placed to automatically created last group.
Reports empty comments.
Reports invalid inline phpDocs with @var
.
Sniff provides the following settings:
allowDocCommentAboveReturn
: Allows documentation comments without variable name abovereturn
statemens.allowAboveNonAssignment
: Allows documentation comments above non-assigment if the line contains the right variable name.
Requires property comments with single-line content to be written as one-liners.
Requires comments with single-line content to be written as one-liners.
Requires comments with single-line content to be written as multi-liners.
- Checks for useless doc comments. If the native method declaration contains everything and the phpDoc does not add anything useful, it's reported as useless and can optionally be automatically removed with
phpcbf
. - Some phpDocs might still be useful even if they do not add any typehint information. They can contain textual descriptions of code elements and also some meaningful annotations like
@expectException
or@dataProvider
.
Sniff provides the following settings:
traversableTypeHints
: enforces which typehints must have specified contained type. E. g. if you set this to\Doctrine\Common\Collections\Collection
, then\Doctrine\Common\Collections\Collection
must always be supplied with the contained type:\Doctrine\Common\Collections\Collection|Foo[]
.
This sniff can cause an error if you're overriding or implementing a parent method which does not have typehints. In such cases add @phpcsSuppress SlevomatCodingStandard.TypeHints.ParameterTypeHint.MissingNativeTypeHint
annotation to the method to have this sniff skip it.
Reports documentation comments containing only {@inheritDoc}
annotation because inheritance is automatic and it's not needed to use a special annotation for it.
Reports useless conditions where both branches return true
or false
.
Sniff provides the following settings:
assumeAllConditionExpressionsAreAlreadyBoolean
(defaults tofalse
).
Reports useless ternary operator where both branches return true
or false
.
Sniff provides the following settings:
assumeAllConditionExpressionsAreAlreadyBoolean
(defaults tofalse
).
The recommended way to install Slevomat Coding Standard is through Composer.
{
"require-dev": {
"slevomat/coding-standard": "~7.0"
}
}
It's also recommended to install php-parallel-lint/php-parallel-lint which checks source code for syntax errors. Sniffs count on the processed code to be syntactically valid (no parse errors), otherwise they can behave unexpectedly. It is advised to run PHP-Parallel-Lint
in your build tool before running PHP_CodeSniffer
and exiting the build process early if PHP-Parallel-Lint
fails.
You can choose one of two ways to run only selected sniffs from the standard on your codebase:
The recommended way is to write your own ruleset.xml by referencing only the selected sniffs. This is a sample ruleset.xml:
<?xml version="1.0"?>
<ruleset name="AcmeProject">
<config name="installed_paths" value="../../slevomat/coding-standard"/><!-- relative path from PHPCS source location -->
<rule ref="SlevomatCodingStandard.Arrays.TrailingArrayComma"/>
<!-- other sniffs to include -->
</ruleset>
Then run the phpcs
executable the usual way:
vendor/bin/phpcs --standard=ruleset.xml --extensions=php --tab-width=4 -sp src tests
You can also mention Slevomat Coding Standard in your project's ruleset.xml
and exclude only some sniffs:
<?xml version="1.0"?>
<ruleset name="AcmeProject">
<rule ref="vendor/slevomat/coding-standard/SlevomatCodingStandard/ruleset.xml"><!-- relative path to your ruleset.xml -->
<!-- sniffs to exclude -->
</rule>
</ruleset>
However it is not a recommended way to use Slevomat Coding Standard, because your build can break when moving between minor versions of the standard (which can happen if you use ^
or ~
version constraint in composer.json
). We regularly add new sniffs even in minor versions meaning your code won't most likely comply with new minor versions of the package.
Sniffs in this standard marked by the π§ symbol support automatic fixing of coding standard violations. To fix your code automatically, run phpcbf instead of phpcs:
vendor/bin/phpcbf --standard=ruleset.xml --extensions=php --tab-width=4 -sp src tests
Always remember to back up your code before performing automatic fixes and check the results with your own eyes as the automatic fixer can sometimes produce unwanted results.
Selected sniffs in this standard marked by the π§ symbol can be suppressed for a specific piece of code using an annotation. Consider the following example:
/**
* @param int $max
*/
public function createProgressBar($max = 0): ProgressBar
{
}
The parameter $max
could have a native int
scalar typehint. But because the method in the parent class does not have this typehint, so this one cannot have it either. PHP_CodeSniffer shows a following error:
----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
67 | ERROR | [x] Method ErrorsConsoleStyle::createProgressBar()
| | does not have native type hint for its parameter $max
| | but it should be possible to add it based on @param
| | annotation "int".
| | (SlevomatCodingStandard.TypeHints.ParameterTypeHint.MissingNativeTypeHint)
If we want to suppress this error instead of fixing it, we can take the error code (SlevomatCodingStandard.TypeHints.ParameterTypeHint.MissingNativeTypeHint
) and use it with a @phpcsSuppress
annotation like this:
/**
* @phpcsSuppress SlevomatCodingStandard.TypeHints.ParameterTypeHint.MissingNativeTypeHint
* @param int $max
*/
public function createProgressBar($max = 0): ProgressBar
{
}
To make this repository work on your machine, clone it and run these two commands in the root directory of the repository:
composer install
bin/phing
After writing some code and editing or adding unit tests, run phing again to check that everything is OK:
bin/phing
We are always looking forward for your bugreports, feature requests and pull requests. Thank you.
This project adheres to a Contributor Code of Conduct. By participating in this project and its community, you are expected to uphold this code.