voku/phpstan-rules

Provides additional rules for phpstan/phpstan.

3.2.0 2023-06-09 14:59 UTC

Requires

• php: ^7.2 || ^8.0
• phpstan/phpstan: ^1.9.0

Requires (Dev)

• phpstan/phpstan-phpunit: ^1.1.0
• phpunit/phpunit: ~7.0 || ~9.0

MIT

• Lars Moelleken

• PHPStan
• phpstan-rules
• phpstan-extreme-rules

README

phpstan-rules

Provides additional rules for phpstan/phpstan.

Installation

Run

$ composer require --dev voku/phpstan-rules

Usage

All the rules provided (and used) by this library are included in rules.neon.

When you are using phpstan/extension-installer, rules.neon will be automatically included.

Otherwise, you need to include rules.neon in your phpstan.neon:

includes:
	- vendor/voku/phpstan-rules/rules.neon

Rules

IfConditionHelper

This helper is used by different "condition"-rules: if - and - or - not - ternary

💡 We use this "hack" (helper) to run the check for all kind of conditions.

• double negative string conditions. e.g. (string)$foo != '' is the same as (string)$foo
  • https://3v4l.org/oDMie
• double negative integer conditions. e.g. (int)$foo != 0 is the same as (int)$foo
  • https://3v4l.org/OWhrc
• double negative boolean conditions. e.g. (bool)$foo != false is the same as (bool)$foo
  • https://3v4l.org/SHoQP
• double negative null conditions. Use "!==" instead if needed
  • https://3v4l.org/a4VdC
• check 0 vs '' conditions, the behavior was changed in PHP 8
  • https://3v4l.org/lBFHI
• check possible insane comparisons. e.g. 0 == '0foo', the behavior was changed in PHP 8
  • https://3v4l.org/BJ6b8
• check insane comparisons. e.g. 0 === '0' or false && true
• check non-empty string is never empty
• check non-empty string is always empty
• check non-empty array is never empty
• do not compare objects with another type
• do not use magic string-concat for objects with "__toString()"
• do not allow assignments. e.g. if ($a = 0) (see "checkForAssignments")
• do not allow Yoda conditions. e.g. ìf (0 == $a) (see "checkYodaConditions")

Configuration

If you want to configure a list of classes / subclasses that can NOT be used in conditions directly:

e.g.:

• ok: if ($emailValueObject->isValid())
• error: if ($emailValueObject != '')

parameters:
    voku:
        classesNotInIfConditions: [
            AbstractValueObject
        ]

If you want to check assignments e.g. in "if"-conditions you can use this:

parameters:
    voku:
        checkForAssignments: true

If you want to check Yoda conditions can use this:

parameters:
    voku:
        checkYodaConditions: true

ExtendedBinaryOpRule

This rule will check "+", "*", "/", "-", ... (operators) and "." (concatenation) for compatible types.

It's included in the default rules.neon so that you don't need to add it manually.

DisallowedCallMethodOnNull

This code is copy&pasted from [phpstan/phpstan-src] and I used it to prevent Call to a member function on null errors while I wasn't already on level 8 where all kind of "NULL" checks are already covered by default.

e.g.

rules:
    - voku\PHPStan\Rules\DisallowedCallMethodOnNullRule

Support

For support and donations please visit Github | Issues | PayPal | Patreon.

For status updates and release announcements please visit Releases | Twitter | Patreon.

For professional support please contact me.

Thanks

• Thanks to GitHub (Microsoft) for hosting the code and a good infrastructure including Issues-Managment, etc.
• Thanks to IntelliJ as they make the best IDEs for PHP and they gave me an open source license for PhpStorm!
• Thanks to Travis CI for being the most awesome, easiest continous integration tool out there!
• Thanks to StyleCI for the simple but powerfull code style check.
• Thanks to PHPStan && Psalm for relly great Static analysis tools and for discover bugs in the code!