UselessParenthesesSniff: Fixed false positive

Author: kukulich

SlevomatCodingStandard/Sniffs/PHP/UselessParenthesesSniff.php

@@ -155,8 +155,8 @@ private function checkParenthesesAroundVariableOrFunctionCall(File $phpcsFile, i
 			return;
 		}
 
-		$ternaryOperatorPointer = TokenHelper::findNextEffective($phpcsFile, $tokens[$parenthesisOpenerPointer]['parenthesis_closer'] + 1);
-		if ($tokens[$ternaryOperatorPointer]['code'] === T_INLINE_THEN) {
+		$pointerAfterParenthesisCloser = TokenHelper::findNextEffective($phpcsFile, $tokens[$parenthesisOpenerPointer]['parenthesis_closer'] + 1);
+		if (in_array($tokens[$pointerAfterParenthesisCloser]['code'], [T_INLINE_THEN, T_OPEN_PARENTHESIS], true)) {
 			return;
 		}
 

tests/Sniffs/PHP/data/uselessParenthesesNoErrors.php

@@ -63,3 +63,17 @@ function doSomething($parameter) {
 switch (true) {
 	case !($boo !== null):
 }
+
+class ClassWithClosure
+{
+	private $closure;
+
+	public function __construct()
+	{
+		$this->closure = function () {
+			echo 123;
+		};
+
+		($this->closure)();
+	}
+}