Add Str (literal string) as a knonw type, so that it appears as so in parsed vars

acardona · acardona · commit 7f00a1a94c52 · 2022-02-20T15:47:35.000Z
rather than the latter appearing as an empty.
diff --git a/src/main/java/org/scijava/plugins/scripteditor/jython/JythonScriptParser.java b/src/main/java/org/scijava/plugins/scripteditor/jython/JythonScriptParser.java
@@ -51,6 +51,7 @@
 import org.python.antlr.ast.Name;
 import org.python.antlr.ast.Num;
 import org.python.antlr.ast.Return;
+import org.python.antlr.ast.Str;
 import org.python.antlr.ast.TryExcept;
 import org.python.antlr.ast.TryFinally;
 import org.python.antlr.ast.Tuple;
@@ -352,9 +353,13 @@ static public DotAutocompletions parseRight(final PyObject right, final Scope sc
 		if (right instanceof Num) {
 			// e.g. return 10
 			// e.g. n = 42
-			final Class<?> c = ((Num)right).getInternalN().getClass() == PyInteger.class ? Long.TYPE : Double.TYPE;
+			final Class<?> c = ((Num)right).getInternalN().getClass() == PyInteger.class ? Long.TYPE : Double.TYPE; // TODO this creates trouble for Scope.findVarsByType and would be perhaps easier as Long and Double (non-primitive).
 			return new VarDotAutocompletions(c.toString());
 		}
+		if (right instanceof Str) {
+			// Literal String
+			return new VarDotAutocompletions(String.class.getCanonicalName());
+		}
 		if (right instanceof Attribute) {
 			// e.g. a field or a method
 			// gray8 = IJ.getImage().GRAY8
