Phpinspectionsea: if-else workflow: wrong with !empty()

Created on 21 May 2019  路  3Comments  路  Source: kalessil/phpinspectionsea

| Subject | Details |
| :------------- | :---------------------------------------------------------------------------- |
| Plugin | Php Inspections (EA Ultimate), 3.0.1 |
| Language level |PHP 7.0 |

Example Code:

        if (!empty($headerEntry->tooltip)) {
            $title = FormHelpers::getToolTipText(
                $headerEntry->name,
                $headerEntry->tooltip
            );
        } else {
            $title = $headerEntry->name;
        }

Current behaviour

        if (empty($headerEntry->tooltip)) {
            $title = $headerEntry->name;
        } else {
            $title = FormHelpers::getToolTipText(
                $headerEntry->name,
                $headerEntry->tooltip
            );
        }

Expected behaviour

I think we can keep !empty because it's !false === true

Environment details

PhpStorm 2019.1.2
Build #PS-191.7141.52, built on May 8, 2019
JRE: 1.8.0_202-release-1483-b49 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Linux 4.4.0-148-generic

bug / false-positive fixed

All 3 comments

Agree, will make adjustments.

Fixed!

Thanks for the quick fix.

Was this page helpful?
0 / 5 - 0 ratings