[jboss-jira] [JBoss JIRA] (WFLY-4023) javax.el.ImportHandler#importClass & #importPackage doesn't import WEB-INF/classes

[Patrick Garner (JIRA)]

     [ https://issues.jboss.org/browse/WFLY-4023?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Patrick Garner updated WFLY-4023:
---------------------------------
    Attachment: ImportHandlerDebug.zip


foo.bar.EnumConfig is where you want to set the break point.

Go to http://localhost:8080/ImportHandlerDebug/index.xhtml

Take a look at ImportHandler.notAClass and notice that it contains the entry "foo.bar.enumeration.YesOrNo" which indicates that the class was not found.

> javax.el.ImportHandler#importClass & #importPackage doesn't import WEB-INF/classes
> ----------------------------------------------------------------------------------
>
>                 Key: WFLY-4023
>                 URL: https://issues.jboss.org/browse/WFLY-4023
>             Project: WildFly
>          Issue Type: Bug
>          Components: JSF
>    Affects Versions: 8.1.0.Final
>         Environment: Win7, Wildfly 8.1.0.Final
>            Reporter: Patrick Garner
>            Assignee: Farah Juma
>         Attachments: ImportHandlerDebug.zip
>
>   Original Estimate: 4 hours
>  Remaining Estimate: 4 hours
>
> ImportHandler is not importing classes in WEB-INF/classes and, thus, developers are unable to import classes into the EL evaluation environment per EL 3.0 spec. section 1.22.2 "Imports of Packages, Classes, and Static Fields"
> Here's the ImportHandler relevant code (https://github.com/jboss/jboss-el-api_spec/blob/master/src/main/java/javax/el/ImportHandler.java#L176)
> {code:java}
> private Class<?> getClassFor(String className) {
>     if (!notAClass.contains(className)) {
>         try {
>             return Class.forName(className, false, getClass().getClassLoader());
>         } catch (ClassNotFoundException ex) {
>             notAClass.add(className);
>         }
>     }
>     return null;
> }
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.1#6329)