A better fix for https://java.net/jira/browse/GLASSFISH-20778

Kin Man Chung · Kin Man Chung · commit c042ad71b961 · 2013-08-27T05:03:33.000Z
svn path=/trunk/; revision=1460
diff --git a/api/src/main/java/javax/servlet/jsp/el/ScopedAttributeELResolver.java b/api/src/main/java/javax/servlet/jsp/el/ScopedAttributeELResolver.java
@@ -129,17 +129,19 @@ public Object getValue(ELContext context,
                 // To support reference of static fields for imported class in
                 // EL 3.0, if a scoped attribute returns null, this attribute
                 // is further checked to see if it is the name of an imported
-                // class.  If so, the value of the imported field is return.
-                // Note: the JSP spec needs to be updated for this behavior. It is
-                // not backward compatible and a runtime switch may be needed to
-                // force backward compatility.
+                // class.  If so, an ELClass instance is returned.
+                // Note: the JSP spec needs to be updated for this behavior. Note
+                // also that this behavior is not backward compatible with JSP 2.2
+                // and a runtime switch may be needed to force backward
+                // compatility.
                 if (value == null) {
                     // check to see if the property is an imported class
                     if (context.getImportHandler() != null) {
                         Class<?> c = context.getImportHandler().resolveClass(attribute);
                         if (c != null) {
-                            value = context.getELResolver().getValue(
-                                    context, new ELClass(c), attribute);
+                            value = new ELClass(c);
+                            // A possible optimization is to set the ELClass
+                            // instance in an attribute map.
                         }
                     }
                 }
