UselessParenthesesSniff: Fixed internal error

slevomat · Oct 5, 2018 · 057f3f1 · 057f3f1
1 parent 06d996b
commit 057f3f1
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/SlevomatCodingStandard/Sniffs/PHP/UselessParenthesesSniff.php b/SlevomatCodingStandard/Sniffs/PHP/UselessParenthesesSniff.php
@@ -269,7 +269,6 @@ private function checkParenthesesAroundVariableOrFunctionCall(File $phpcsFile, i
 
 			do {
 				$nextPointer = TokenHelper::findNextEffective($phpcsFile, $contentEndPointer + 1);
-
 				if ($tokens[$nextPointer]['code'] !== T_OPEN_PARENTHESIS) {
 					break;
 				}
@@ -278,6 +277,10 @@ private function checkParenthesesAroundVariableOrFunctionCall(File $phpcsFile, i
 			} while (true);
 		} else {
 			$nextPointer = TokenHelper::findNext($phpcsFile, T_OPEN_PARENTHESIS, $notBooleanNotOperatorPointer + 1);
+			if ($nextPointer === null) {
+				return;
+			}
+
 			$contentEndPointer = $tokens[$nextPointer]['parenthesis_closer'];
 		}
 

diff --git a/tests/Sniffs/PHP/data/uselessParenthesesNoErrors.php b/tests/Sniffs/PHP/data/uselessParenthesesNoErrors.php
@@ -100,3 +100,7 @@ function () {
 include_once($file);
 require($file);
 require_once($file);
+
+return true
+    ? 100
+    : (int) ((100 / 50) * 100);