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

[Antoine Herzog (JIRA)]

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
            Priority: Minor
             Fix For: 2.6.5 Final


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