#1366 mark classed used if its tagged via "kernel.event_listener" server defintion

Author: Haehnchen

src/main/java/fr/adrienbrault/idea/symfony2plugin/codeInsight/SymfonyImplicitUsageProvider.java

@@ -38,12 +38,21 @@ public boolean isImplicitUsage(@NotNull PsiElement element) {
                 || isVoter((PhpClass) element)
                 || isTwigExtension((PhpClass) element)
                 || isEntityRepository((PhpClass) element)
-                || isConstraint((PhpClass) element);
+                || isConstraint((PhpClass) element)
+                || isKernelEventListener((PhpClass) element);
         }
 
         return false;
     }
 
+    private boolean isKernelEventListener(@NotNull PhpClass phpClass) {
+        if (!ServiceUtil.isPhpClassAService(phpClass)) {
+            return false;
+        }
+
+        return ServiceUtil.isPhpClassTaggedWith(phpClass, "kernel.event_listener");
+    }
+
     private boolean isConstraint(@NotNull PhpClass phpClass) {
         if(!PhpElementsUtil.isInstanceOf(phpClass, "\\Symfony\\Component\\Validator\\Constraint")) {
             return false;

src/main/java/fr/adrienbrault/idea/symfony2plugin/util/dict/ServiceUtil.java

@@ -339,6 +339,11 @@ public static Collection<PhpClass> getTaggedClasses(@NotNull Project project, @N
         return phpClasses;
     }
 
+    public static boolean isPhpClassTaggedWith(@NotNull PhpClass phpClass, @NotNull String tagName) {
+        return ServiceUtil.getTaggedClasses(phpClass.getProject(), tagName).stream()
+            .anyMatch(phpClass1 -> phpClass1.getFQN().equalsIgnoreCase(phpClass1.getFQN()));
+    }
+
     @NotNull
     public static Collection<PhpClass> getTaggedClassesWithCompiled(@NotNull Project project, @NotNull String tagName) {
 

src/test/java/fr/adrienbrault/idea/symfony2plugin/tests/codeInsight/SymfonyImplicitUsageProviderTest.java

@@ -167,6 +167,16 @@ public void testConstraintValidatorReferenceIsMarkedAsUsed() {
         assertTrue(new SymfonyImplicitUsageProvider().isImplicitUsage(firstClassFromFile));
     }
 
+    public void testClassWithTaggedEventListenerIsMarkedAsUsed() {
+        PsiFile psiFile = myFixture.configureByText(PhpFileType.INSTANCE, "<?php\n" +
+            "namespace App\\EventListener;\n" +
+            "class ExceptionListener {}\n"
+        );
+
+        PhpClass firstClassFromFile = PhpElementsUtil.getFirstClassFromFile((PhpFile) psiFile.getContainingFile());
+        assertTrue(new SymfonyImplicitUsageProvider().isImplicitUsage(firstClassFromFile));
+    }
+
     public void testEventSubscriberGetSubscribedEventsArray() {
         PsiFile psiFile = myFixture.configureByText(PhpFileType.INSTANCE, "<?php\n" +
             "<?php\n" +

src/test/java/fr/adrienbrault/idea/symfony2plugin/tests/codeInsight/fixtures/services.yml

@@ -12,4 +12,9 @@ services:
     class: App\Voter\MyFoobarVoter
 
   foo_command.twig_extension:
-    class: App\TwigExtension\MyTwigExtension
+    class: App\TwigExtension\MyTwigExtension
+
+  foo_command.eventListener_exception:
+    class: App\EventListener\ExceptionListener
+    tags:
+        - { name: kernel.event_listener, event: kernel.exception }