DO NOT REPLY [Bug 28671] New: - RequestUtils.lookup() / TagUtils.lookup()

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=28671>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=28671

RequestUtils.lookup() / TagUtils.lookup()

           Summary: RequestUtils.lookup() / TagUtils.lookup()
           Product: Struts
           Version: 1.1 Final
          Platform: All
        OS/Version: Other
            Status: NEW
          Severity: Major
          Priority: Other
         Component: Utilities
        AssignedTo: dev@struts.apache.org
        ReportedBy: sascha.gross@mathema.de


The following lookup()-method from 
org.apache.struts.util.RequestUtils
and since Struts 1.2 
org.apache.struts.taglib.TagUtils

    public static Object lookup( 
        PageContext pageContext, 
        String name, 
        String property,
        String scope)
        throws JspException

has documented in the comment that 
<snip>
     * @exception JspException if accessing this property causes an
     *  IllegalAccessException, IllegalArgumentException,
     *  InvocationTargetException, or NoSuchMethodException

</snip

but the IllegalArgumentException is never catch in this method and so it comes.

Please catch the IllegalArgumentException and throw a JspException (my favored
solution)
or change the comment (unfavored).

I think this is a major-bug, because this method is used in some taglib e.g.
<logic:present/> (with attribute property) and so the tags are useless.


Thanks

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org