sirbrillig / phpcs-variable-analysis
A PHPCS sniff to detect problems with variables.
Installs: 31 206 543
Dependents: 86
Suggesters: 0
Security: 0
Stars: 135
Watchers: 6
Forks: 14
Open Issues: 21
Type:phpcodesniffer-standard
Requires
- php: >=5.4.0
- phpcsstandards/phpcsutils: ^1.0
- squizlabs/php_codesniffer: ^3.5
Requires (Dev)
- limedeck/phpunit-detailed-printer: ^3.1 || ^4.0 || ^5.0
- phpstan/phpstan: ^0.11.8
- phpunit/phpunit: ^5.0 || ^6.5 || ^7.0 || ^8.0
- sirbrillig/phpcs-import-detection: ^1.1
- dev-master
- v3.0.0-beta.8
- v3.0.0-beta.7
- v3.0.0-beta.6
- v3.0.0-beta.5
- v3.0.0-beta.4
- v3.0.0-beta.3
- v3.0.0-beta.2
- v3.0.0-beta.1
- 2.x-dev
- v2.11.19
- v2.11.18
- v2.11.18-beta.2
- v2.11.18-beta.1
- v2.11.17
- v2.11.16
- v2.11.15
- v2.11.14
- v2.11.13
- v2.11.12
- v2.11.11
- v2.11.10
- v2.11.9
- v2.11.8
- v2.11.7
- v2.11.6
- v2.11.5
- v2.11.4
- v2.11.3
- v2.11.2
- v2.11.1
- v2.11.0
- 2.10.x-dev
- v2.10.3-beta.1
- v2.10.2
- v2.10.2-beta.1
- v2.10.1
- v2.10.0
- v2.10.0-beta.3
- v2.10.0-beta.1
- v2.9.0
- v2.8.3
- 2.8.2.x-dev
- v2.8.2
- v2.8.2-beta.1
- v2.8.1
- v2.8.0
- v2.7.0
- v2.6.4
- v2.6.3
- v2.6.2
- v2.6.1
- v2.6.0
- v2.5.0
- v2.4.0
- v2.3.0
- v2.2.0
- v2.1.3
- v2.1.2
- v2.1.1
- v2.1.0
- v2.0.7
- v2.0.6
- v2.0.5
- v2.0.4
- v2.0.3
- v2.0.2
- v2.0.1
- v2.0
- v1.0
- dev-2.x-legacy
- dev-test-branch-for-failures
- dev-warn-reassign
This package is auto-updated.
Last update: 2024-10-10 14:29:27 UTC
README
Plugin for PHP_CodeSniffer static analysis tool that adds analysis of problematic variable use.
- Warns if variables are used without being defined. (Sniff code:
VariableAnalysis.CodeAnalysis.VariableAnalysis.UndefinedVariable
) - Warns if variables are used inside
unset()
without being defined. (Sniff code:VariableAnalysis.CodeAnalysis.VariableAnalysis.UndefinedUnsetVariable
) - Warns if variables are set or declared but never used. (Sniff code:
VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
) - Warns if
$this
,self::$static_member
,static::$static_member
is used outside class scope. (Sniff codes:VariableAnalysis.CodeAnalysis.VariableAnalysis.SelfOutsideClass
orVariableAnalysis.CodeAnalysis.VariableAnalysis.StaticOutsideClass
)
Installation
Requirements
VariableAnalysis requires PHP 5.4 or higher and PHP CodeSniffer version 3.5.6 or higher.
With PHPCS Composer Installer
This is the easiest method.
First, install phpcodesniffer-composer-installer for your project if you have not already. This will also install PHPCS.
composer config allow-plugins.dealerdirect/phpcodesniffer-composer-installer true
composer require --dev dealerdirect/phpcodesniffer-composer-installer
Then install these standards.
composer require --dev sirbrillig/phpcs-variable-analysis
You can then include the sniffs by adding a line like the following to your phpcs.xml file.
<rule ref="VariableAnalysis"/>
It should just work after that!
Standalone
-
Install PHP_CodeSniffer (PHPCS) by following its installation instructions (via Composer, Phar file, PEAR, or Git checkout).
Do ensure that PHP_CodeSniffer's version matches our requirements.
-
Install VariableAnalysis. Download either the zip or tar.gz file from the VariableAnalysis latest release page. Expand the file and rename the resulting directory to
phpcs-variable-analysis
. Move the directory to a place where you'd like to keep all your PHPCS standards. -
Add the paths of the newly installed standards to the PHP_CodeSniffer installed_paths configuration. The following command should append the new standards to your existing standards (be sure to supply the actual paths to the directories you created above).
phpcs --config-set installed_paths "$(phpcs --config-show|grep installed_paths|awk '{ print $2 }'),/path/to/phpcs-variable-analysis"
If you do not have any other standards installed, you can do this more easily (again, be sure to supply the actual paths):
phpcs --config-set installed_paths /path/to/phpcs-variable-analysis
Customization
There's a variety of options to customize the behaviour of VariableAnalysis, take a look at the included ruleset.xml.example for commented examples of a configuration.
The available options are as follows:
allowUnusedFunctionParameters
(bool, defaultfalse
): if set to true, function arguments will never be marked as unused.allowUnusedCaughtExceptions
(bool, defaulttrue
): if set to true, caught Exception variables will never be marked as unused.allowUnusedParametersBeforeUsed
(bool, defaulttrue
): if set to true, unused function arguments will be ignored if they are followed by used function arguments.allowUnusedVariablesBeforeRequire
(bool, defaultfalse
): if set to true, variables defined before arequire
,require_once
,include
, orinclude_once
will not be marked as unused. They may be intended for the required file.allowUndefinedVariablesInFileScope
(bool, defaultfalse
): if set to true, undefined variables in the file's top-level scope will never be marked as undefined. This can be useful for template files which use many global variables defined elsewhere.allowUnusedVariablesInFileScope
(bool, defaultfalse
): if set to true, unused variables in the file's top-level scope will never be marked as unused. This can be helpful when defining a lot of global variables to be used elsewhere.validUnusedVariableNames
(string, defaultnull
): a space-separated list of names of placeholder variables that you want to ignore from unused variable warnings. For example, to ignore the variables$junk
and$unused
, this could be set to'junk unused'
.ignoreUnusedRegexp
(string, defaultnull
): a PHP regexp string (note that this requires explicit delimiters) for variables that you want to ignore from unused variable warnings. For example, to ignore the variables$_junk
and$_unused
, this could be set to'/^_/'
.validUndefinedVariableNames
(string, defaultnull
): a space-separated list of names of placeholder variables that you want to ignore from undefined variable warnings. For example, to ignore the variables$post
and$undefined
, this could be set to'post undefined'
. This can be used in combination withvalidUndefinedVariableRegexp
.validUndefinedVariableRegexp
(string, defaultnull
): a PHP regexp string (note that this requires explicit delimiters) for variables that you want to ignore from undefined variable warnings. For example, to ignore the variables$post
and$undefined
, this could be set to'/^(post|undefined)$/'
. This can be used in combination withvalidUndefinedVariableNames
.allowUnusedForeachVariables
(bool, defaulttrue
): if set to true, unused values from thekey => value
syntax in aforeach
loop will never be marked as unused.sitePassByRefFunctions
(string, defaultnull
): a list of custom functions which pass in variables to be initialized by reference (egpreg_match()
) and therefore should not require those variables to be defined ahead of time. The list is space separated and each entry is of the formfunctionName:1,2
. The function name comes first followed by a colon and a comma-separated list of argument numbers (starting from 1) which should be considered variable definitions. The special value...
in the arguments list will cause all arguments after the last number to be considered variable definitions.allowWordPressPassByRefFunctions
(bool, defaultfalse
): if set to true, a list of common WordPress pass-by-reference functions will be added to the list of PHP ones so that passing undefined variables to these functions (to be initialized by reference) will be allowed.
To set these these options, you must use XML in your ruleset. For details, see the phpcs customizable sniff properties page. Here is an example that ignores all variables that start with an underscore:
<rule ref="VariableAnalysis.CodeAnalysis.VariableAnalysis"> <properties> <property name="ignoreUnusedRegexp" value="/^_/"/> </properties> </rule>
See Also
- ImportDetection: A set of phpcs sniffs to look for unused or unimported symbols.
- phpcs-changed: Run phpcs on files, but only report warnings/errors from lines which were changed.
Original
This was forked from the excellent work in https://github.com/illusori/PHP_Codesniffer-VariableAnalysis
Contributing
Please open issues or PRs on this repository.
Any changes should be accompanied by tests and should pass linting and static analysis. Please use phpdoc (rather than actual types) for declaring types since this must run in PHP 5.4.
To run tests, make sure composer is installed, then run:
composer install # you only need to do this once
composer test
To run linting, use:
composer lint
To run static analysis, use:
composer phpstan