UnusedUsesSniff - fixed multiple implemented interfaces marked as unused

Author: ondrejmirtes

SlevomatCodingStandard/Helpers/ReferencedNameHelper.php

@@ -91,7 +91,18 @@ private static function createAllReferencedNames(PHP_CodeSniffer_File $phpcsFile
 							T_DOUBLE_COLON,
 						], true)
 					) {
-						$type = ReferencedName::TYPE_CONSTANT;
+						if ($tokens[$previousTokenBeforeStartPointer]['code'] === T_COMMA) {
+							$implementsPointer = TokenHelper::findPreviousExcluding(
+								$phpcsFile,
+								array_merge(TokenHelper::$nameTokenCodes, TokenHelper::$ineffectiveTokenCodes),
+								$previousTokenBeforeStartPointer - 1
+							);
+							if ($tokens[$implementsPointer]['code'] !== T_IMPLEMENTS) {
+								$type = ReferencedName::TYPE_CONSTANT;
+							}
+						} else {
+							$type = ReferencedName::TYPE_CONSTANT;
+						}
 					}
 				}
 			}

tests/Sniffs/Namespaces/UnusedUsesSniffTest.php

@@ -77,6 +77,12 @@ public function testUsedConstant()
 		$this->assertNoSniffError($this->getFileReport(), 15);
 	}
 
+	public function testUsedClassesInImplements()
+	{
+		$this->assertNoSniffError($this->getFileReport(), 17);
+		$this->assertNoSniffError($this->getFileReport(), 18);
+	}
+
 	public function testReturnTypehint()
 	{
 		if (PHP_VERSION_ID < 70000) {

tests/Sniffs/Namespaces/data/unusedUses.php

@@ -14,8 +14,10 @@
 use const FooBar\UNUSED_CONSTANT;
 use const LoremIpsum\USED_CONSTANT;
 use X;
+use Lorem\FirstInterface;
+use Ipsum\SecondInterface;
 
-class TestClass
+class TestClass implements FirstInterface, SecondInterface
 {
 
 	/**