You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@struts.apache.org by bu...@apache.org on 2004/07/23 11:22:21 UTC
DO NOT REPLY [Bug 30202] -
No exception handling when exposing all request params using ImportAttribute
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=30202>.
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=30202
No exception handling when exposing all request params using ImportAttribute
husted@apache.org changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
------- Additional Comments From husted@apache.org 2004-07-23 09:22 -------
Thanks, Peter.
For those of you following along, here's the diff:
171c171,186
< pageContext.setAttribute(name, compContext.getAttribute(name), scope);
---
> if(name == null ) {
> if(!isErrorIgnored)
> throw new JspException ( "Error - tag importAttribute :
encountered an attribute with key 'null'" );
> else
> return SKIP_BODY;
> }
>
> Object value = compContext.getAttribute(name);
> // Check if value exist and if we must send a runtime exception
> if( value == null ) {
> if(!isErrorIgnored)
> throw new JspException ( "Error - tag importAttribute : property
'"+ name + "' has a value of 'null'" );
> else
> return SKIP_BODY;
> }
> pageContext.setAttribute(name, value, scope);
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org