UnusedParameterSniff: Fixed false positive

Author: kukulich

SlevomatCodingStandard/Sniffs/Functions/UnusedParameterSniff.php

@@ -129,7 +129,16 @@ private function isParameter(File $phpcsFile, int $variablePointer): bool
 
 	private function isUsedInString(string $stringContent, string $variableName): bool
 	{
-		return preg_match('~(?<!\\\\)' . preg_quote($variableName, '~') . '\b(?!\()~', $stringContent) > 0;
+		if (preg_match('~(?<!\\\\)' . preg_quote($variableName, '~') . '\b(?!\()~', $stringContent) > 0) {
+			return true;
+		}
+
+		$variableNameWithoutDollar = substr($variableName, 1);
+		if (preg_match('~\$\{' . preg_quote($variableNameWithoutDollar, '~') . '\}~', $stringContent) > 0) {
+			return true;
+		}
+
+		return false;
 	}
 
 	private function isUsedInCompactFunction(File $phpcsFile, int $compactFunctionPointer, string $variableName): bool

tests/Sniffs/Functions/data/unusedParameterNoErrors.php

@@ -33,9 +33,10 @@ function usedParameterInAnotherScope($a)
 	}
 }
 
-function parameterInString($a)
+function parameterInString($a, $b)
 {
 	echo "$a";
+	echo "${b}";
 }
 
 function parameterInHeredoc($a)