Not sure if you caught the thread on the users list, or the bug recently
posted by Kjeld Froberg:

 http://nagoya.apache.org/bugzilla/show_bug.cgi?id=14332


Here's the proposed code change:

RequestUtils.java
-----------------------------------------------
From:
    public static Class applicationClass(String className)
        throws ClassNotFoundException {

        // Look up the class loader to be used
        ClassLoader classLoader =
            Thread.currentThread().getContextClassLoader();
        if (classLoader == null) {
            classLoader = RequestUtils.class.getClassLoader();
        }

        // Attempt to load the specified class
        return (classLoader.loadClass(className));
    }
-----------------------------------------------
To:
    public static Class applicationClass(String className)
        throws ClassNotFoundException {
                ClassLoader ctxLoader = null;
                try {
                        ctxLoader = Thread.currentThread().getContextClassLoader();
                        return Class.forName(className, true, ctxLoader);

                } catch(ClassNotFoundException ex) {
                        if(ctxLoader == null) { throw ex; }
                } catch(SecurityException ex) {

                }
                return Class.forName(className);
        }

By changing these few lines, I am able to run the full test suite without
error. (And that includes the new test.tomcat.33)

I'm not sure if this should be a [Vote] or a [commit and wait till they
complain], so I opted for the former.

Even though I'm not truly sure of the consequences of this change, I'm +1.


James Mitchell
Software Engineer/Struts Evangelist
http://www.open-tools.org

"Only two things are infinite, the universe and human stupidity, and I'm not
sure about the former."
- Albert Einstein (1879-1955)


--
To unsubscribe, e-mail:   <mailto:struts-dev-unsubscribe@;jakarta.apache.org>
For additional commands, e-mail: <mailto:struts-dev-help@;jakarta.apache.org>

Reply via email to