Merge pull request #89 from VincentLanglet/fixUnused

🐛 Fix false unused

Author: VincentLanglet

SymfonyCustom/Sniffs/Namespaces/UnusedUseSniff.php

@@ -311,7 +311,7 @@ private function isClassUsed(File $phpcsFile, int $usePtr, int $classPtr): bool
                 || ('class' === $type
                     && ((T_DOC_COMMENT_STRING === $tokens[$classUsed]['code']
                             && preg_match(
-                                '/(\s|\||\(|^)'.preg_quote($searchName, '/').'(\s|\||\\\\|$|\[\])/i',
+                                '/(\s|\||\(|^)'.preg_quote($searchName, '/').'(\s|\||\\\\|<|$|\[\])/i',
                                 $tokens[$classUsed]['content']
                             ))
                         || (T_DOC_COMMENT_TAG === $tokens[$classUsed]['code']

SymfonyCustom/Tests/Namespaces/UnusedUseUnitTest.inc

@@ -131,3 +131,17 @@ use /* comment */DateTime;
 new MyClass1();
 new MyClass2();
 new MyClass3();
+
+namespace BugCollection;
+
+use Doctrine\Collection;
+
+/**
+ * @param Collection<int, \Client> $a
+ *
+ * @return null
+ */
+function ($a)
+{
+    return null;
+}

SymfonyCustom/Tests/Namespaces/UnusedUseUnitTest.inc.fixed

@@ -118,3 +118,17 @@ use \Hello\MyClass2;
 new MyClass1();
 new MyClass2();
 new MyClass3();
+
+namespace BugCollection;
+
+use Doctrine\Collection;
+
+/**
+ * @param Collection<int, \Client> $a
+ *
+ * @return null
+ */
+function ($a)
+{
+    return null;
+}