UselessParenthesesSniff: Fixed false positive

Author: kukulich

SlevomatCodingStandard/Sniffs/PHP/UselessParenthesesSniff.php

@@ -73,8 +73,12 @@ private function checkParenthesesAroundConditionInTernaryOperator(File $phpcsFil
 	{
 		$tokens = $phpcsFile->getTokens();
 
-		$ternaryOperatorPointer = TokenHelper::findNextEffective($phpcsFile, $tokens[$parenthesisOpenerPointer]['parenthesis_closer'] + 1);
+		$pointerBeforeParenthesisOpener = TokenHelper::findPreviousEffective($phpcsFile, $parenthesisOpenerPointer - 1);
+		if ($tokens[$pointerBeforeParenthesisOpener]['code'] === T_BOOLEAN_NOT) {
+			return;
+		}
 
+		$ternaryOperatorPointer = TokenHelper::findNextEffective($phpcsFile, $tokens[$parenthesisOpenerPointer]['parenthesis_closer'] + 1);
 		if ($tokens[$ternaryOperatorPointer]['code'] !== T_INLINE_THEN) {
 			return;
 		}

tests/Sniffs/PHP/data/uselessParenthesesNoErrors.php

@@ -47,6 +47,7 @@ function doSomething($parameter) {
 
 $z = ($a ? '0' : '1') ? '2' : '3';
 $zz = ($a + $b);
+$zzz = !($a) ? true : false;
 
 exit($a);
 die($a);