GROOVY-9966

groovy · Mar 5, 2021 · ccc3896 · ccc3896
1 parent 783164f
commit ccc3896
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 0 deletions.
diff --git a/...vy.core.tests.compiler/src/org/eclipse/jdt/groovy/core/tests/basic/GroovySimpleTests.java b/...vy.core.tests.compiler/src/org/eclipse/jdt/groovy/core/tests/basic/GroovySimpleTests.java
@@ -4655,6 +4655,21 @@ public void testGroovy9906() {
         runConformTest(sources, "");
     }
 
+    @Test // https://issues.apache.org/jira/browse/GROOVY-9966
+    public void testGroovy9966() {
+        //@formatter:off
+        String[] sources = {
+            "Main.groovy",
+            "import static java.util.Arrays.*\n" +
+            "def m(x) { return x }\n" +
+            "final value = 123.456\n" +
+            "m(\"$value\").missing\n",
+        };
+        //@formatter:on
+
+        runConformTest(sources, "", "groovy.lang.MissingPropertyException: No such property: missing for class: org.codehaus.groovy.runtime.GStringImpl");
+    }
+
     // was worried <clinit> would surface in list of methods used to build the type declaration, but that doesn't appear to be the case
     @Test
     public void testExtendingGroovyObjects_clinit() {

diff --git a/base/org.codehaus.groovy30/src/org/codehaus/groovy/control/StaticImportVisitor.java b/base/org.codehaus.groovy30/src/org/codehaus/groovy/control/StaticImportVisitor.java
@@ -132,13 +132,19 @@ public Expression transform(Expression exp) {
         }
         if (clazz == ArgumentListExpression.class) {
             Expression result = exp.transformExpression(this);
+            // GRECLIPSE add -- GROOVY-9966
+            if (foundArgs == null)
+            // GRECLIPSE end
             if (inPropertyExpression) {
                 foundArgs = result;
             }
             return result;
         }
         if (exp instanceof ConstantExpression) {
             Expression result = exp.transformExpression(this);
+            // GRECLIPSE add -- GROOVY-9966
+            if (foundConstant == null)
+            // GRECLIPSE end
             if (inPropertyExpression) {
                 foundConstant = result;
             }

diff --git a/base/org.codehaus.groovy40/src/org/codehaus/groovy/control/StaticImportVisitor.java b/base/org.codehaus.groovy40/src/org/codehaus/groovy/control/StaticImportVisitor.java
@@ -135,13 +135,19 @@ public Expression transform(Expression exp) {
         }
         if (clazz == ArgumentListExpression.class) {
             Expression result = exp.transformExpression(this);
+            // GRECLIPSE add -- GROOVY-9966
+            if (foundArgs == null)
+            // GRECLIPSE end
             if (inPropertyExpression) {
                 foundArgs = result;
             }
             return result;
         }
         if (exp instanceof ConstantExpression) {
             Expression result = exp.transformExpression(this);
+            // GRECLIPSE add -- GROOVY-9966
+            if (foundConstant == null)
+            // GRECLIPSE end
             if (inPropertyExpression) {
                 foundConstant = result;
             }