[jboss-jira] [JBoss JIRA] Resolved: (JBPORTAL-1927) IdentityUIConfigurationServiceImpl : better logs of exception, for better config debugging

[Thomas Heute (JIRA)]

     [ http://jira.jboss.com/jira/browse/JBPORTAL-1927?page=all ]

Thomas Heute resolved JBPORTAL-1927.
------------------------------------

    Fix Version/s: 2.7 Final
       Resolution: Done
         Assignee: Emanuel Muckenhuber

Merci Antoine
Thank you Emanuel

:)

> IdentityUIConfigurationServiceImpl : better logs of exception, for better config debugging
> ------------------------------------------------------------------------------------------
>
>                 Key: JBPORTAL-1927
>                 URL: http://jira.jboss.com/jira/browse/JBPORTAL-1927
>             Project: JBoss Portal
>          Issue Type: Task
>      Security Level: Public(Everyone can see) 
>          Components: Portal Identity
>    Affects Versions: 2.6.4 Final
>         Environment: JBP 2.6.4
>            Reporter: Antoine Herzog
>         Assigned To: Emanuel Muckenhuber
>            Priority: Minor
>             Fix For: 2.7 Final, 2.6.5 Final
>
>   Original Estimate: 20 minutes
>  Remaining Estimate: 20 minutes
>
> In class : IdentityUIConfigurationServiceImpl 
> the exception handling is not re throwing the cause of the exception.
> hence, the error (in config) are quite difficult to understant and debug.
> would be great to add the cause exception in the "throw new xxxException('bla bla');"
> example :
> line 337
>       catch (JBossXBException e)
>       {
>          throw new CoreIdentityConfigurationException("Could not parse configuration file.");
>       }
> would be better :
>       catch (JBossXBException e)
>       {
>          throw new CoreIdentityConfigurationException("Could not parse configuration file.", e);
>       }
> everywhere in the file : add the ", e);" in all the new exception throw 
> Thanks,

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