You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@struts.apache.org by "David Evans (JIRA)" <ji...@apache.org> on 2006/04/25 01:04:10 UTC

[jira] Reopened: (STR-111) Digester ignoring parse errors

     [ http://issues.apache.org/struts/browse/STR-111?page=all ]
     
David Evans reopened STR-111:
-----------------------------


> Digester ignoring parse errors
> ------------------------------
>
>          Key: STR-111
>          URL: http://issues.apache.org/struts/browse/STR-111
>      Project: Struts Action 1
>         Type: Bug

>   Components: Action
>     Versions: Nightly Build
>  Environment: Operating System: All
> Platform: PC
>     Reporter: Hal Deadman
>     Assignee: Craig McClanahan
>      Fix For: 1.0.0

>
> The Digester ignores parsing errors by overriding fatalError(), error() and 
> warning() and just logging the parse exception without throwing it. The default 
> behavior for the HandlerBase fatalError() method is to re-throw the exception 
> so it seems like Digester should re-throw it after it logs it. Extending 
> Digester and overriding those methods to throw exceptions doesn't work because 
> Digester is a "final" class. Digester should throw the parse exception in at 
> least fatalError() so as to act like a normal handler. It should not be a final 
> class so the behaviour can be changed by Digester users. If ignoring the 
> exceptions during the parse is intended, how is one supposed to know that the 
> xml was invalid?

-- 
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


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