support "Symfony\Component\Routing\Attribute\Route" for method comple…

…tion
Haehnchen · Aug 3, 2024 · 2131dfe · 2131dfe
1 parent d5da735
commit 2131dfe
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/src/main/java/de/espend/idea/php/annotation/symfony/SymfonyCompletionProvider.java b/src/main/java/de/espend/idea/php/annotation/symfony/SymfonyCompletionProvider.java
@@ -17,7 +17,11 @@ public void getPropertyValueCompletions(AnnotationPropertyParameter parameter, A
             return;
         }
 
-        if("methods".equals(parameter.getPropertyName()) && PhpLangUtil.equalsClassNames(StringUtils.stripStart(parameter.getPhpClass().getFQN(), "\\"), "Symfony\\Component\\Routing\\Annotation\\Route")) {
+        boolean b = "methods".equals(parameter.getPropertyName()) &&
+            (PhpLangUtil.equalsClassNames(StringUtils.stripStart(parameter.getPhpClass().getFQN(), "\\"), "Symfony\\Component\\Routing\\Annotation\\Route") || PhpLangUtil.equalsClassNames(StringUtils.stripStart(parameter.getPhpClass().getFQN(), "\\"), "Symfony\\Component\\Routing\\Attribute\\Route")
+        );
+
+        if(b) {
             for (String s : new String[]{"HEAD", "GET", "POST", "PUT", "PATCH", "DELETE", "PURGE", "OPTIONS", "TRACE", "CONNECT"}) {
                 completion.getResult().addElement(LookupElementBuilder.create(s));
             }

diff --git a/src/main/java/de/espend/idea/php/annotation/util/PhpDocTagAnnotationVisitorUtil.java b/src/main/java/de/espend/idea/php/annotation/util/PhpDocTagAnnotationVisitorUtil.java
@@ -25,9 +25,7 @@ public static void visitElement(@NotNull PhpFile psiFile, @NotNull Processor<Pai
             if (topLevelElement instanceof PhpClass clazz) {
                 PhpDocComment docComment = clazz.getDocComment();
                 if (docComment != null) {
-                    PhpDocUtil.consumeTagElementsByName(docComment, null, docTag -> {
-                        visitPhpDocTag(docTag, processor);
-                    });
+                    PhpDocUtil.consumeTagElementsByName(docComment, null, docTag -> visitPhpDocTag(docTag, processor));
                 }
 
                 for (PhpAttribute attribute : clazz.getAttributes()) {

diff --git a/src/main/java/de/espend/idea/php/annotation/util/PhpDocUtil.java b/src/main/java/de/espend/idea/php/annotation/util/PhpDocUtil.java
@@ -156,7 +156,7 @@ private static void addAttribute(@NotNull Document document, @NotNull PhpNamedEl
         if (forElement instanceof PhpClass) {
             // on class scope: class Foobar {}
             List<PhpAttributesList> childrenOfTypeAsList = PsiTreeUtil.getChildrenOfTypeAsList(forElement, PhpAttributesList.class);
-            CodeStyleManager.getInstance(forElement.getProject()).reformatRange(forElement, childrenOfTypeAsList.get(0).getTextRange().getStartOffset(), childrenOfTypeAsList.get(childrenOfTypeAsList.size() - 1).getNextPsiSibling().getTextRange().getEndOffset());
+            CodeStyleManager.getInstance(forElement.getProject()).reformatRange(forElement, childrenOfTypeAsList.getFirst().getTextRange().getStartOffset(), childrenOfTypeAsList.getLast().getNextPsiSibling().getTextRange().getEndOffset());
         } else {
             // on attribute scope: private $foo;
             PhpClassFieldsList f = PsiTreeUtil.getParentOfType(forElement, PhpClassFieldsList.class);