Skip to content

Commit

Permalink
Micronaut: Method parameters should be resolvable in expression langu…
Browse files Browse the repository at this point in the history 
…age.
  • Loading branch information
@dbalek
dbalek committed Apr 23, 2024
1 parent f01e76e commit 224f972
Show file tree
Hide file tree
Showing 11 changed files with 152 additions and 57 deletions.
2 changes: 1 addition & 1 deletion enterprise/micronaut/nbproject/project.xml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -273,7 +273,7 @@
<build-prerequisite/>
<compile-dependency/>
<run-dependency>
<specification-version>2.49</specification-version>
<specification-version>2.68</specification-version>
</run-dependency>
</dependency>
<dependency>
Expand Down
2 changes: 1 addition & 1 deletion enterprise/micronaut/src/org/netbeans/modules/micronaut/MicronautConfigUtilities.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@
*/
public class MicronautConfigUtilities {

private static final Pattern REGEXP = Pattern.compile("^(application|bootstrap)(-\\w*)*\\.(yml|properties)$", Pattern.CASE_INSENSITIVE);
private static final Pattern REGEXP = Pattern.compile("^(application|bootstrap)(-\\w*)*\\.(yml|yaml|properties)$", Pattern.CASE_INSENSITIVE);

public static final String YAML_MIME = "text/x-yaml";
public static final String PROPERTIES_MIME = "text/x-properties";
Expand Down
8 changes: 8 additions & 0 deletions ...onaut/src/org/netbeans/modules/micronaut/completion/MicronautDataCompletionCollector.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -309,6 +309,14 @@ public Completion createJavaElementItem(CompilationInfo info, Element element, i
}
Builder builder = CompletionCollector.newBuilder(simpleName);
switch (element.getKind()) {
case PARAMETER:
builder.kind(Completion.Kind.Variable).sortText(String.format("%04d%s", 90, simpleName))
.labelDescription(Utils.getTypeName(info, element.asType(), false, false).toString());
break;
case RECORD_COMPONENT:
builder.kind(Completion.Kind.Field).sortText(String.format("%04d%s", 90, simpleName))
.labelDescription(Utils.getTypeName(info, element.asType(), false, false).toString());
break;
case ENUM:
builder.kind(Completion.Kind.Enum).sortText(String.format("%04d%s", 300, simpleName));
break;
Expand Down
12 changes: 12 additions & 0 deletions ...ronaut/src/org/netbeans/modules/micronaut/completion/MicronautDataCompletionProvider.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -107,12 +107,16 @@ private static class MicronautDataCompletionQuery extends AsyncCompletionQuery {
private static final String RECORD_ICON = "org/netbeans/modules/editor/resources/completion/record.png";
private static final String METHOD_PUBLIC = "org/netbeans/modules/editor/resources/completion/method_16.png"; //NOI18N
private static final String METHOD_ST_PUBLIC = "org/netbeans/modules/editor/resources/completion/method_static_16.png";
private static final String FIELD_PUBLIC = "org/netbeans/modules/editor/resources/completion/field_16.png"; //NOI18N
private static final String LOCAL_VARIABLE = "org/netbeans/modules/editor/resources/completion/localVariable.gif"; //NOI18N
private static final String ATTRIBUTE_VALUE = "org/netbeans/modules/java/editor/resources/attribute_value_16.png"; // NOI18N
private static final String PROPERTY = "org/netbeans/modules/beans/resources/propertyRO.gif";
private static final String KEYWORD_COLOR = getHTMLColor(64, 64, 217);
private static final String PACKAGE_COLOR = getHTMLColor(64, 150, 64);
private static final String CLASS_COLOR = getHTMLColor(150, 64, 64);
private static final String INTERFACE_COLOR = getHTMLColor(128, 128, 128);
private static final String FIELD_COLOR = getHTMLColor(64, 198, 88);
private static final String PARAMETER_COLOR = getHTMLColor(64, 64, 188);
private static final String PARAMETERS_COLOR = getHTMLColor(192, 192, 192);
private static final String PARAMETER_NAME_COLOR = getHTMLColor(224, 160, 65);
private static final String ATTRIBUTE_VALUE_COLOR = getHTMLColor(128, 128, 128);
Expand Down Expand Up @@ -496,6 +500,14 @@ public CompletionItem createJavaElementItem(CompilationInfo info, Element elemen
}
CompletionUtilities.CompletionItemBuilder builder = CompletionUtilities.newCompletionItemBuilder(simpleName).startOffset(offset);
switch (element.getKind()) {
case PARAMETER:
builder.iconResource(LOCAL_VARIABLE).leftHtmlText(PARAMETER_COLOR + simpleName + COLOR_END).sortPriority(90)
.rightHtmlText(Utils.getTypeName(info, element.asType(), false, false).toString());
break;
case RECORD_COMPONENT:
builder.iconResource(FIELD_PUBLIC).leftHtmlText(FIELD_COLOR + simpleName + COLOR_END).sortPriority(90)
.rightHtmlText(Utils.getTypeName(info, element.asType(), false, false).toString());
break;
case ENUM:
builder.iconResource(ENUM_ICON).leftHtmlText(CLASS_COLOR + simpleName + COLOR_END).sortPriority(300);
break;
Expand Down
35 changes: 22 additions & 13 deletions .../src/org/netbeans/modules/micronaut/completion/MicronautExpressionLanguageCompletion.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ public <T> Result<T> query(int offset, ItemFactory<T> factory) {
if (tree == null) {
kws = ctx.getScope().getEnclosingMethod() != null ? Arrays.asList("true", "false", "null", "this", "empty", "not") : Arrays.asList("true", "false", "null", "empty", "not");
builtins = Arrays.asList("T", "()", "ctx", "[]", "env", "[]");
elements = ctx.getContextMethods();
elements = ctx.getContextElements();
anchorOffset = startOffset + offset;
} else {
String tokenText = ts.token().text().subSequence(0, offset - ts.offset()).toString().trim();
Expand All @@ -105,7 +105,7 @@ public <T> Result<T> query(int offset, ItemFactory<T> factory) {
if (offset <= tree.getStartPosition()) {
kws = ctx.getScope().getEnclosingMethod() != null ? Arrays.asList("true", "false", "null", "this", "empty", "not") : Arrays.asList("true", "false", "null", "empty", "not");
builtins = Arrays.asList("T", "()", "ctx", "[]", "env", "[]");
elements = ctx.getContextMethods();
elements = ctx.getContextElements();
} else {
ExpressionTree lastTree = tree;
if (tree.getKind() == ExpressionTree.Kind.ERRONEOUS) {
Expand Down Expand Up @@ -149,7 +149,7 @@ public <T> Result<T> query(int offset, ItemFactory<T> factory) {
case NONE:
String prev = prevNonWSTokenText(prefix);
if ("#".equals(prev)) {
elements = ctx.getContextMethods();
elements = ctx.getContextElements();
}
break;
}
Expand Down Expand Up @@ -214,7 +214,7 @@ public <T> Result<T> query(int offset, ItemFactory<T> factory) {
kws = Arrays.asList("this");
}
builtins = Arrays.asList("T", "()", "ctx", "[]", "env", "[]");
elements = ctx.getContextMethods();
elements = ctx.getContextElements();
}
break;
case EQUAL_TO:
Expand All @@ -225,7 +225,7 @@ public <T> Result<T> query(int offset, ItemFactory<T> factory) {
} else {
kws = ctx.getScope().getEnclosingMethod() != null ? Arrays.asList("null", "this"): Arrays.asList("null");
builtins = Arrays.asList("T", "()", "ctx", "[]", "env", "[]");
elements = ctx.getContextMethods();
elements = ctx.getContextElements();
}
break;
case AND:
Expand All @@ -238,11 +238,11 @@ public <T> Result<T> query(int offset, ItemFactory<T> factory) {
case NOT:
kws = ctx.getScope().getEnclosingMethod() != null ? Arrays.asList("true", "false", "not", "empty", "this"): Arrays.asList("true", "false", "not", "empty");
builtins = Arrays.asList("T", "()", "ctx", "[]", "env", "[]");
elements = ctx.getContextMethods();
elements = ctx.getContextElements();
break;
case EMPTY:
builtins = Arrays.asList("T", "()", "ctx", "[]", "env", "[]");
elements = ctx.getContextMethods();
elements = ctx.getContextElements();
break;
case INSTANCE_OF:
ExpressionTree.InstanceOf instanceOf = (ExpressionTree.InstanceOf) path.getLeaf();
Expand All @@ -262,7 +262,7 @@ public <T> Result<T> query(int offset, ItemFactory<T> factory) {
if ("?". equals(prev) || ":".equals(prev)) {
kws = ctx.getScope().getEnclosingMethod() != null ? Arrays.asList("true", "false", "null", "this", "empty", "not") : Arrays.asList("true", "false", "null", "empty", "not");
builtins = Arrays.asList("T", "()", "ctx", "[]", "env", "[]");
elements = ctx.getContextMethods();
elements = ctx.getContextElements();
}
}
break;
Expand All @@ -274,12 +274,12 @@ public <T> Result<T> query(int offset, ItemFactory<T> factory) {
if (callee != null) {
TypeMirror pacTM = callee.getTypeMirror(ctx);
if (pacTM.getKind() == TypeKind.DECLARED) {
elements = ElementFilter.methodsIn(((DeclaredType) pacTM).asElement().getEnclosedElements()).stream()
elements = ((DeclaredType) pacTM).asElement().getEnclosedElements().stream()
.filter(ee -> callee.getKind() != ExpressionTree.Kind.TYPE_REFERENCE || ee.getModifiers().contains(Modifier.STATIC))
.collect(Collectors.toList());
}
} else {
elements = ctx.getContextMethods();
elements = ctx.getContextElements();
}
}
break;
Expand All @@ -296,12 +296,12 @@ public <T> Result<T> query(int offset, ItemFactory<T> factory) {
.collect(Collectors.toList());
}
} else {
elements = ctx.getContextMethods();
elements = ctx.getContextElements();
}
} else if ("(".equals(prev) || ",".equals(prev)) {
kws = ctx.getScope().getEnclosingMethod() != null ? Arrays.asList("true", "false", "null", "this", "empty", "not") : Arrays.asList("true", "false", "null", "empty", "not");
builtins = Arrays.asList("T", "()", "ctx", "[]", "env", "[]");
elements = ctx.getContextMethods();
elements = ctx.getContextElements();
}
break;
}
Expand Down Expand Up @@ -381,7 +381,16 @@ public <T> Result<T> query(int offset, ItemFactory<T> factory) {
items.add(factory.createBeanPropertyItem(propertyName, returnType, anchorOffset));
}
}
} else {
} else if (element.getKind() == ElementKind.RECORD_COMPONENT) {
TypeMirror enclType = element.getEnclosingElement().asType();
if (enclType.getKind() == TypeKind.DECLARED && Utils.startsWith(name, prefix) && info.getTrees().isAccessible(ctx.getScope(), element, (DeclaredType) enclType)) {
items.add(factory.createJavaElementItem(info, element, anchorOffset));
}
} else if (element.getKind() == ElementKind.PARAMETER) {
if (Utils.startsWith(name, prefix)) {
items.add(factory.createJavaElementItem(info, element, anchorOffset));
}
} else if (element.getKind().isClass() || element.getKind().isInterface()) {
if (Utils.startsWith(name, prefix) && info.getTrees().isAccessible(ctx.getScope(), (TypeElement) element)) {
items.add(factory.createJavaElementItem(info, element, anchorOffset));
}
Expand Down
12 changes: 8 additions & 4 deletions enterprise/micronaut/src/org/netbeans/modules/micronaut/expression/EvaluationContext.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -107,15 +107,19 @@ public Scope getScope() {
return scope;
}

public List<ExecutableElement> getContextMethods() {
public List<? extends Element> getContextElements() {
if (contextClasses == null) {
initializeContextClasses();
}
List<ExecutableElement> methods = new ArrayList<>();
List<Element> elements = new ArrayList<>();
for (TypeElement contextClass : contextClasses) {
methods.addAll(ElementFilter.methodsIn(contextClass.getEnclosedElements()));
elements.addAll(ElementFilter.methodsIn(contextClass.getEnclosedElements()));
}
return methods;
ExecutableElement enclosingMethod = scope.getEnclosingMethod();
if (enclosingMethod != null && !enclosingMethod.getParameters().isEmpty()) {
elements.addAll(enclosingMethod.getParameters());
}
return elements;
}

private void initializeContextClasses() {
Expand Down
Loading

0 comments on commit 224f972

Please sign in to comment.