Added similar text suggestion Code Action(s) for UndefinedObjext and …

…UndefinedNamespace Signed-off-by: Alexander Chen <alchen@redhat.com>
redhat-developer · Jul 29, 2022 · 444cc78 · 444cc78
1 parent 8a52980
commit 444cc78
Show file tree

Hide file tree

Showing 10 changed files with 1,022 additions and 184 deletions.
diff --git a/...qute.jdt.test/projects/maven/qute-quickstart/src/main/resources/templates/hello.qute.html b/...qute.jdt.test/projects/maven/qute-quickstart/src/main/resources/templates/hello.qute.html
@@ -2,9 +2,14 @@
 <html>
 <head>
 <meta charset="UTF-8">
-<title>Qute Hello World</title> 
+<title>Qute Hello World</title>
 </head>
 <body>
     <p>Hello {name ?: "world"}!</p>
 </body>
-</html>
+</html>
+
+{@java.util.List<org.acme.qute.Item> items}
+"{#for item in itemsXXX}
+"	{item.name}
+"{/for}";
diff --git a/qute.ls/com.redhat.qute.ls/src/main/java/com/redhat/qute/parser/template/Expression.java b/qute.ls/com.redhat.qute.ls/src/main/java/com/redhat/qute/parser/template/Expression.java
@@ -21,15 +21,15 @@
 
 /**
  * Simple Qute expression
- * 
+ *
  * <p>
  * {foo}
  * </p>
- * 
+ *
  * <p>
  * {foo ?: foo : 'bar'}
  * </p>
- * 
+ *
  * @author Angelo ZERR
  *
  */
@@ -51,13 +51,14 @@ public NodeKind getKind() {
 		return NodeKind.Expression;
 	}
 
+	@Override
 	public String getNodeName() {
 		return "#expression";
 	}
 
 	/**
 	 * Returns the start offset of the expression content (after '{')
-	 * 
+	 *
 	 * @return the start offset of the expression content (after '{')
 	 */
 	public int getStartContentOffset() {
@@ -66,7 +67,7 @@ public int getStartContentOffset() {
 
 	/**
 	 * Returns the end offset of the expression content (before '}')
-	 * 
+	 *
 	 * @return the end offset of the expression content (before '}')
 	 */
 	public int getEndContentOffset() {
@@ -96,7 +97,7 @@ public List<Node> getExpressionContent() {
 
 	/**
 	 * Returns the object part of the expression and null otherwise.
-	 * 
+	 *
 	 * @return the object part of the expression and null otherwise.
 	 */
 	public ObjectPart getObjectPart() {
@@ -110,7 +111,7 @@ public ObjectPart getObjectPart() {
 
 	/**
 	 * Returns the namespace part of the expression and null otherwise.
-	 * 
+	 *
 	 * @return the namespace part of the expression and null otherwise.
 	 */
 	public NamespacePart getNamespacePart() {
@@ -124,7 +125,7 @@ public NamespacePart getNamespacePart() {
 
 	/**
 	 * Returns the last part of the expression and null otherwise.
-	 * 
+	 *
 	 * @return the last part of the expression and null otherwise.
 	 */
 	public Part getLastPart() {
@@ -138,7 +139,7 @@ public Part getLastPart() {
 
 	/**
 	 * Returns the Java type of the expression if it's a literal and null otherwise.
-	 * 
+	 *
 	 * @return the Java type of the expression if it's a literal and null otherwise.
 	 */
 	public String getLiteralJavaType() {
@@ -174,11 +175,11 @@ public boolean canSupportInfixNotation() {
 	/**
 	 * Returns true if the last part of the expression is optional (ends with ??)
 	 * and false otherwise.
-	 * 
+	 *
 	 * <p>
 	 * {foo??}
 	 * </p>
-	 * 
+	 *
 	 * @return true if the last part of the expression is optional (ends with ??)
 	 *         and false otherwise.
 	 */
@@ -189,9 +190,9 @@ public boolean isOptional() {
 
 	/**
 	 * Returns the owner parameter of the expression and null otherwise.
-	 * 
+	 *
 	 * For expression like {foo}, the owner parameter is null.
-	 * 
+	 *
 	 * @return the owner parameter of the expression and null otherwise.
 	 */
 	public Parameter getOwnerParameter() {
@@ -200,9 +201,9 @@ public Parameter getOwnerParameter() {
 
 	/**
 	 * Returns the owner section of the expression and null otherwise.
-	 * 
+	 *
 	 * For expression like {foo}, the owner section is null.
-	 * 
+	 *
 	 * @return the owner section of the expression and null otherwise.
 	 */
 	public Section getOwnerSection() {

diff --git a/qute.ls/com.redhat.qute.ls/src/main/java/com/redhat/qute/project/QuteProjectRegistry.java b/qute.ls/com.redhat.qute.ls/src/main/java/com/redhat/qute/project/QuteProjectRegistry.java
@@ -347,7 +347,7 @@ public CompletableFuture<List<UserTagInfo>> getUserTags(QuteUserTagParams params
 	/**
 	 * Returns the Java member (field or method) from the given Java base type with
 	 * the given property and null otherwise.
-	 * 
+	 *
 	 * @param baseType   the Java base type.
 	 * @param property   the member property (field name or getter method name).
 	 * @param projectUri the project Uri used to search in the extended Java type.
@@ -419,11 +419,11 @@ public JavaMemberResult findProperty(ResolvedJavaTypeInfo baseType, String prope
 	/**
 	 * Returns the Java member from the given base type wich matches the given
 	 * property by using Java reflection and null otherwise.
-	 * 
+	 *
 	 * @param baseType   the Java type.
 	 * @param property   the property member.
 	 * @param projectUri the project Uri.
-	 * 
+	 *
 	 * @return the Java member from the given base type wich matches the given
 	 *         property by using Java reflection and null otherwise.
 	 */
@@ -504,7 +504,7 @@ protected static JavaFieldInfo findField(ResolvedJavaTypeInfo baseType, String f
 	/**
 	 * Returns the member method retrieved by the given property or method name and
 	 * null otherwise.
-	 * 
+	 *
 	 * @param baseType          the Java base type.
 	 * @param methodName        property or method name.
 	 * @param getterMethodName  the getter method name.
@@ -668,7 +668,9 @@ private boolean isMatchParameters(JavaMethodInfo method, List<ResolvedJavaTypeIn
 		boolean virtualMethod = method.isVirtual();
 		int nbParameters = method.getParameterslength();
 		int declaredNbParameters = parameterTypes.size();
-		JavaParameterInfo lastParameter = method.hasParameters() ? method.getParameterAt(method.getParameters().size() - 1) : null;
+		JavaParameterInfo lastParameter = method.hasParameters()
+				? method.getParameterAt(method.getParameters().size() - 1)
+				: null;
 		boolean varargs = lastParameter != null && lastParameter.isVarargs();
 		if (varargs) {
 			if (declaredNbParameters == 0) {
@@ -821,7 +823,14 @@ public boolean hasNamespace(String namespace, String projectUri) {
 		return getAllNamespaces(projectUri).contains(namespace);
 	}
 
-	private Set<String> getAllNamespaces(String projectUri) {
+	/**
+	 * Return all namespaces from the given Qute project Uri.
+	 *
+	 * @param projectUri the Qute project Uri
+	 *
+	 * @return all namespace from the given Qute project Uri.
+	 */
+	public Set<String> getAllNamespaces(String projectUri) {
 		if (StringUtils.isEmpty(projectUri)) {
 			return Collections.emptySet();
 		}
@@ -1104,10 +1113,10 @@ private CompletableFuture<List<ValueResolver>> getGlobalVariablesValueResolvers(
 	/**
 	 * Returns the java type filter according the given root java type and the
 	 * native mode.
-	 * 
+	 *
 	 * @param rootJavaType         the Java root type.
 	 * @param nativeImagesSettings the native images settings.
-	 * 
+	 *
 	 * @return the java type filter according the given root java type and the
 	 *         native mode.
 	 */

diff --git a/...om.redhat.qute.ls/src/main/java/com/redhat/qute/project/datamodel/JavaDataModelCache.java b/...om.redhat.qute.ls/src/main/java/com/redhat/qute/project/datamodel/JavaDataModelCache.java
@@ -12,6 +12,7 @@
 package com.redhat.qute.project.datamodel;
 
 import java.util.List;
+import java.util.Set;
 import java.util.concurrent.CompletableFuture;
 import java.util.concurrent.CompletionStage;
 
@@ -89,7 +90,7 @@ private CompletableFuture<ResolvedJavaTypeInfo> resolveJavaType(Parts parts, int
 			boolean nullIfDontMatchWithIterable) {
 		CompletableFuture<ResolvedJavaTypeInfo> future = null;
 		for (int i = 0; i < partIndex + 1; i++) {
-			Part current = ((Part) parts.getChild(i));
+			Part current = (parts.getChild(i));
 			switch (current.getPartKind()) {
 			case Object:
 				ObjectPart objectPart = (ObjectPart) current;
@@ -272,6 +273,10 @@ public boolean hasNamespace(String namespace, String projectUri) {
 		return projectRegistry.hasNamespace(namespace, projectUri);
 	}
 
+	public Set<String> getAllNamespaces(String projectUri) {
+		return projectRegistry.getAllNamespaces(projectUri);
+	}
+
 	public CompletableFuture<JavaElementInfo> findJavaElementWithNamespace(String namespace, String partName,
 			String projectUri) {
 		return projectRegistry.findJavaElementWithNamespace(namespace, partName, projectUri);
@@ -297,10 +302,10 @@ public JavaMemberResult findMethod(ResolvedJavaTypeInfo baseType, String namespa
 	/**
 	 * Returns the java type filter according the given root java type and the
 	 * native mode.
-	 * 
+	 *
 	 * @param rootJavaType         the Java root type.
 	 * @param nativeImagesSettings the native images settings.
-	 * 
+	 *
 	 * @return the java type filter according the given root java type and the
 	 *         native mode.
 	 */