#1984 detected missing services inside "Autowire" attribute

Author: Haehnchen

src/main/java/fr/adrienbrault/idea/symfony2plugin/dic/inspection/MissingServiceInspection.java

@@ -6,6 +6,7 @@
 import com.intellij.psi.PsiElement;
 import com.intellij.psi.PsiElementVisitor;
 import com.jetbrains.php.lang.PhpLanguage;
+import com.jetbrains.php.lang.lexer.PhpTokenTypes;
 import com.jetbrains.php.lang.psi.elements.MethodReference;
 import com.jetbrains.php.lang.psi.elements.StringLiteralExpression;
 import fr.adrienbrault.idea.symfony2plugin.Symfony2ProjectComponent;
@@ -19,6 +20,8 @@
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.yaml.YAMLLanguage;
 
+import java.util.Arrays;
+
 /**
  * @author Daniel Espendiller <daniel@espendiller.net>
  */
@@ -56,6 +59,22 @@ public void visitElement(PsiElement element) {
                         }
                     }
                 }
+
+                // get leaf element
+                PsiElement leafText = Arrays.stream(YamlHelper.getChildrenFix(element))
+                    .filter(p -> p.getNode().getElementType() == PhpTokenTypes.STRING_LITERAL_SINGLE_QUOTE || p.getNode().getElementType() == PhpTokenTypes.STRING_LITERAL)
+                    .findFirst()
+                    .orElse(null);
+
+                if (leafText != null && PhpElementsUtil.getAttributeNamedArgumentStringPattern(ServiceContainerUtil.AUTOWIRE_ATTRIBUTE_CLASS, "service").accepts(leafText)) {
+                    String serviceName = ((StringLiteralExpression) element).getContents();
+                    if (StringUtils.isNotBlank(serviceName)) {
+                        if (!hasService(serviceName)) {
+                            holder.registerProblem(element, INSPECTION_MESSAGE, ProblemHighlightType.GENERIC_ERROR_OR_WARNING);
+                        }
+                    }
+                }
+
             } else if(element.getLanguage() == YAMLLanguage.INSTANCE) {
                 // yaml
 

src/main/java/fr/adrienbrault/idea/symfony2plugin/util/PsiElementUtils.java

@@ -3,6 +3,7 @@
 import com.intellij.openapi.project.Project;
 import com.intellij.openapi.vfs.VirtualFile;
 import com.intellij.patterns.ElementPattern;
+import com.intellij.patterns.IElementTypePattern;
 import com.intellij.patterns.PlatformPatterns;
 import com.intellij.psi.PsiElement;
 import com.intellij.psi.PsiFile;
@@ -219,6 +220,20 @@ public static <T extends PsiElement> T getChildrenOfType(@Nullable PsiElement el
         return null;
     }
 
+    @Nullable
+    public static <T extends PsiElement> T getChildrenOfType(@Nullable PsiElement element, IElementTypePattern pattern) {
+        if (element == null) return null;
+
+        for (PsiElement child = element.getFirstChild(); child != null; child = child.getNextSibling()) {
+            if (pattern.accepts(child)) {
+                //noinspection unchecked
+                return (T)child;
+            }
+        }
+
+        return null;
+    }
+
     @NotNull
     public static <T extends PsiElement> Collection<T> getChildrenOfTypeAsList(@Nullable PsiElement element, ElementPattern<T> pattern) {
 

src/test/java/fr/adrienbrault/idea/symfony2plugin/tests/dic/inspection/MissingServiceInspectionTest.java

@@ -32,6 +32,20 @@ public void testThatPhpServiceInterfaceForGetMethodIsInspected() {
         );
     }
 
+    public void testThatPhpAttributesForServiceAutowireIsInspected() {
+        assertLocalInspectionContains("test.php", "<?php\n" +
+                "use Symfony\\Component\\DependencyInjection\\Attribute\\Autowire;\n" +
+                "\n" +
+                "class HandlerCollection\n" +
+                "{\n" +
+                "    public function __construct(\n" +
+                "        #[Autowire(service: 'fo<caret>obar')]" +
+                "    ) {}\n" +
+                "}",
+            MissingServiceInspection.INSPECTION_MESSAGE
+        );
+    }
+
     public void testThatYamlServiceInterfaceForGetMethodIsInspected() {
         assertLocalInspectionContains("services.yml", "services:\n   @args<caret>_unknown", MissingServiceInspection.INSPECTION_MESSAGE);
         assertLocalInspectionContains("services.yml", "services:\n   @Args<caret>_unknown", MissingServiceInspection.INSPECTION_MESSAGE);

src/test/java/fr/adrienbrault/idea/symfony2plugin/tests/dic/inspection/fixtures/classes.php

@@ -21,4 +21,9 @@ public function __construct($foobar)
         {
         }
     }
+}
+
+namespace  Symfony\Component\DependencyInjection\Attribute
+{
+    class Autowire {}
 }