[CodeQuality] Handle not identical return false then true on SimplifyIfReturnBoolRector

[samsonasik]

Currently, the following code is skipped:

        $docToken = new Token([]);
        if (strpos($docToken->getContent(), "\n") !== false) {
            return false;
        }

        return true;

which should be transformed to:

-        if (strpos($docToken->getContent(), "\n") !== false) {
-            return false;
-        }

-        return true;
+        return strpos($docToken->getContent(), "\n") === false;

[samsonasik]

implemented 🎉

[samsonasik]

It seems make invalid for the following condition:

        if ($firstChar !== $lastChar) {
            return false;
        }

        return in_array($firstChar, self::POSSIBLE_DELIMITERS, true);

I will check.

[samsonasik]

Fixed 🎉

[samsonasik]

All checks have passed 🎉 @TomasVotruba it is ready for review.

[TomasVotruba]

Thank you 👍