[jira] Resolved: (STR-2809) ImportAttribute does not import all attribute with ignore=true

["Don Brown (JIRA)" <ji...@apache.org>]

     [ http://issues.apache.org/struts/browse/STR-2809?page=all ]
     
Don Brown resolved STR-2809:
----------------------------

    Fix Version: 1.3.3
     Resolution: Fixed
      Assign To: Don Brown  (was: Struts Developer Mailing List)

Makes sense to me.  Please close if this fixes your problem.

> ImportAttribute does not  import all attribute with ignore=true
> ---------------------------------------------------------------
>
>          Key: STR-2809
>          URL: http://issues.apache.org/struts/browse/STR-2809
>      Project: Struts Action 1
>         Type: Bug

>   Components: Tiles
>     Versions: 1.2.8
>  Environment: Operating System: other
> Platform: Other
>     Reporter: Andrzej Turowski
>     Assignee: Don Brown
>      Fix For: 1.3.3

>
> If we have a list of attribute and a null value is encountered SKIP_BODY is
> returned and other attributes are not imported into the pageContext.
> I think we need to take out the else clause?
> public int doStartTag() throws JspException
>  ....
>  ....
> if( value == null ) {
>           if(!isErrorIgnored)
>             throw new JspException ( "Error - tag importAttribute : property '"+
> name + "' has a value of 'null'" );
>           else
>             return SKIP_BODY;
>         }
> should become 
> if( value == null ) {
>           if(!isErrorIgnored)
>             throw new JspException ( "Error - tag importAttribute : property '"+
> name + "' has a value of 'null'" );
>         }

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira