[exo-jcr-commits] exo-jcr SVN: r5592 - in core/trunk: exo.core.component.organization.jdbc/src/main/java/org/exoplatform/services/organization/jdbc and 5 other directories.

Author: andrew.plotnikov Date: 2012-02-09 06:02:51 -0500 (Thu, 09 Feb 2012) New Revision: 5592 Modified: core/trunk/exo.core.component.database/src/main/java/org/exoplatform/services/database/DAO.java core/trunk/exo.core.component.database/src/main/java/org/exoplatform/services/database/DBPageList.java core/trunk/exo.core.component.organization.jdbc/src/main/java/org/exoplatform/services/organization/jdbc/GroupDAOImpl.java core/trunk/exo.core.component.organization.jdbc/src/main/java/org/exoplatform/services/organization/jdbc/MembershipDAOImpl.java core/trunk/exo.core.component.organization.ldap/src/main/java/org/exoplatform/services/organization/ldap/OrganizationLdapInitializer.java core/trunk/exo.core.component.organization.ldap/src/main/java/org/exoplatform/services/organization/ldap/UserDAOImpl.java core/trunk/exo.core.component.security.core/src/main/java/org/exoplatform/services/security/ConversationRegistry.java core/trunk/exo.core.component.security.core/src/main/java/org/exoplatform/services/security/IdentityRegistry.java core/trunk/exo.core.component.security.core/src/main/java/org/exoplatform/services/security/j2ee/DigestAuthenticationJettyLoginModule.java core/trunk/exo.core.component.security.core/src/main/java/org/exoplatform/services/security/web/JAASConversationStateListener.java core/trunk/exo.core.component.xml-processing/src/main/java/org/exoplatform/services/xml/transform/impl/trax/TRAXTemplatesServiceImpl.java Log: EXOJCR-1725: Fixed the Blocker/Critical violations and the easiest Major ones raised by eXo Quality Level 1.2 Modified: core/trunk/exo.core.component.database/src/main/java/org/exoplatform/services/database/DAO.java =================================================================== --- core/trunk/exo.core.component.database/src/main/java/org/exoplatform/services/database/DAO.java 2012-02-09 10:56:35 UTC (rev 5591) +++ core/trunk/exo.core.component.database/src/main/java/org/exoplatform/services/database/DAO.java 2012-02-09 11:02:51 UTC (rev 5592) @@ -92,10 +92,6 @@ { return loadUnique(connection, query); } - catch (Exception e) - { - throw e; - } finally { eXoDS_.closeConnection(connection); @@ -137,10 +133,6 @@ { loadInstances(connection, loadQuery, list); } - catch (Exception e) - { - throw e; - } finally { eXoDS_.closeConnection(connection); @@ -172,10 +164,6 @@ { execute(connection, query, bean); } - catch (Exception e) - { - throw e; - } finally { eXoDS_.closeConnection(connection); @@ -204,10 +192,6 @@ { return this.<E> loadDBField(connection, query); } - catch (Exception e) - { - throw e; - } finally { eXoDS_.closeConnection(connection); @@ -238,10 +222,6 @@ { execute(connection, template, beans); } - catch (Exception e) - { - throw e; - } finally { eXoDS_.closeConnection(connection); Modified: core/trunk/exo.core.component.database/src/main/java/org/exoplatform/services/database/DBPageList.java =================================================================== --- core/trunk/exo.core.component.database/src/main/java/org/exoplatform/services/database/DBPageList.java 2012-02-09 10:56:35 UTC (rev 5591) +++ core/trunk/exo.core.component.database/src/main/java/org/exoplatform/services/database/DBPageList.java 2012-02-09 11:02:51 UTC (rev 5592) @@ -120,10 +120,6 @@ resultSet.close(); statement.close(); } - catch (Exception e) - { - throw e; - } finally { dao_.getExoDatasource().closeConnection(connection); @@ -151,10 +147,6 @@ statement.close(); return list; } - catch (Exception e) - { - throw e; - } finally { dao_.getExoDatasource().closeConnection(connection); Modified: core/trunk/exo.core.component.organization.jdbc/src/main/java/org/exoplatform/services/organization/jdbc/GroupDAOImpl.java =================================================================== --- core/trunk/exo.core.component.organization.jdbc/src/main/java/org/exoplatform/services/organization/jdbc/GroupDAOImpl.java 2012-02-09 10:56:35 UTC (rev 5591) +++ core/trunk/exo.core.component.organization.jdbc/src/main/java/org/exoplatform/services/organization/jdbc/GroupDAOImpl.java 2012-02-09 11:02:51 UTC (rev 5592) @@ -122,12 +122,10 @@ long id = childImpl.getDBObjectId(); execute(connection, eXoDS_.getQueryBuilder().createInsertQuery(type_, id), childImpl); if (broadcast) + { listenerService_.broadcast("organization.group.postSave", this, childImpl); + } } - catch (Exception e) - { - throw e; - } finally { eXoDS_.closeConnection(connection); Modified: core/trunk/exo.core.component.organization.jdbc/src/main/java/org/exoplatform/services/organization/jdbc/MembershipDAOImpl.java =================================================================== --- core/trunk/exo.core.component.organization.jdbc/src/main/java/org/exoplatform/services/organization/jdbc/MembershipDAOImpl.java 2012-02-09 10:56:35 UTC (rev 5591) +++ core/trunk/exo.core.component.organization.jdbc/src/main/java/org/exoplatform/services/organization/jdbc/MembershipDAOImpl.java 2012-02-09 11:02:51 UTC (rev 5592) @@ -269,10 +269,6 @@ listenerService_.broadcast("organization.membership.postDelete", this, membershipImpl); return membershipImpl; } - catch (Exception e) - { - throw e; - } finally { eXoDS_.closeConnection(connection); @@ -319,10 +315,6 @@ } return list; } - catch (Exception e) - { - throw e; - } finally { eXoDS_.closeConnection(connection); Modified: core/trunk/exo.core.component.organization.ldap/src/main/java/org/exoplatform/services/organization/ldap/OrganizationLdapInitializer.java =================================================================== --- core/trunk/exo.core.component.organization.ldap/src/main/java/org/exoplatform/services/organization/ldap/OrganizationLdapInitializer.java 2012-02-09 10:56:35 UTC (rev 5591) +++ core/trunk/exo.core.component.organization.ldap/src/main/java/org/exoplatform/services/organization/ldap/OrganizationLdapInitializer.java 2012-02-09 11:02:51 UTC (rev 5592) @@ -30,6 +30,7 @@ import java.util.regex.Pattern; import javax.naming.NamingEnumeration; +import javax.naming.NamingException; import javax.naming.directory.BasicAttributes; import javax.naming.directory.SearchControls; import javax.naming.directory.SearchResult; @@ -115,7 +116,7 @@ return; } } - catch (Exception exp) + catch (NamingException exp) { if (LOG.isTraceEnabled()) { Modified: core/trunk/exo.core.component.organization.ldap/src/main/java/org/exoplatform/services/organization/ldap/UserDAOImpl.java =================================================================== --- core/trunk/exo.core.component.organization.ldap/src/main/java/org/exoplatform/services/organization/ldap/UserDAOImpl.java 2012-02-09 10:56:35 UTC (rev 5591) +++ core/trunk/exo.core.component.organization.ldap/src/main/java/org/exoplatform/services/organization/ldap/UserDAOImpl.java 2012-02-09 11:02:51 UTC (rev 5592) @@ -443,7 +443,7 @@ { return ldapService.authenticate(userDN, password); } - catch (Exception exp) + catch (NamingException exp) { return false; } Modified: core/trunk/exo.core.component.security.core/src/main/java/org/exoplatform/services/security/ConversationRegistry.java =================================================================== --- core/trunk/exo.core.component.security.core/src/main/java/org/exoplatform/services/security/ConversationRegistry.java 2012-02-09 10:56:35 UTC (rev 5591) +++ core/trunk/exo.core.component.security.core/src/main/java/org/exoplatform/services/security/ConversationRegistry.java 2012-02-09 11:02:51 UTC (rev 5592) @@ -94,7 +94,7 @@ return DEFAULT_CONCURRENCY_LEVEL; } - catch (Exception e) + catch (NumberFormatException e) { LOG.error("Can't parse parameter " + INIT_PARAM_CONCURRENCY_LEVEL, e); return DEFAULT_CONCURRENCY_LEVEL; Modified: core/trunk/exo.core.component.security.core/src/main/java/org/exoplatform/services/security/IdentityRegistry.java =================================================================== --- core/trunk/exo.core.component.security.core/src/main/java/org/exoplatform/services/security/IdentityRegistry.java 2012-02-09 10:56:35 UTC (rev 5591) +++ core/trunk/exo.core.component.security.core/src/main/java/org/exoplatform/services/security/IdentityRegistry.java 2012-02-09 11:02:51 UTC (rev 5592) @@ -82,7 +82,7 @@ return DEFAULT_CONCURRENCY_LEVEL; } - catch (Exception e) + catch (NumberFormatException e) { LOG.error("Can't parse parameter " + INIT_PARAM_CONCURRENCY_LEVEL, e); return DEFAULT_CONCURRENCY_LEVEL; Modified: core/trunk/exo.core.component.security.core/src/main/java/org/exoplatform/services/security/j2ee/DigestAuthenticationJettyLoginModule.java =================================================================== --- core/trunk/exo.core.component.security.core/src/main/java/org/exoplatform/services/security/j2ee/DigestAuthenticationJettyLoginModule.java 2012-02-09 10:56:35 UTC (rev 5591) +++ core/trunk/exo.core.component.security.core/src/main/java/org/exoplatform/services/security/j2ee/DigestAuthenticationJettyLoginModule.java 2012-02-09 11:02:51 UTC (rev 5592) @@ -187,13 +187,34 @@ field.setAccessible(true); password = (String)field.get(objectFromCallback); } - catch (Exception e) + catch (IllegalArgumentException e) { if (LOG.isErrorEnabled()) { LOG.error("Could not get credentials.", e); } } + catch (NoSuchFieldException e) + { + if (LOG.isErrorEnabled()) + { + LOG.error("Could not get credentials.", e); + } + } + catch (SecurityException e) + { + if (LOG.isErrorEnabled()) + { + LOG.error("Could not get credentials.", e); + } + } + catch (IllegalAccessException e) + { + if (LOG.isErrorEnabled()) + { + LOG.error("Could not get credentials.", e); + } + } if (username == null || password == null) { @@ -207,19 +228,41 @@ { String username = null; Class<?> digestAuthenticatorClazz = DigestAuthenticator.class.getDeclaredClasses()[0]; + try { Field field = digestAuthenticatorClazz.getDeclaredField("username"); field.setAccessible(true); username = (String)field.get((((ObjectCallback)objectCallback[0]).getObject())); } - catch (Exception e) + catch (IllegalArgumentException e) { if (LOG.isErrorEnabled()) { LOG.error("Could not get username.", e); } } + catch (IllegalAccessException e) + { + if (LOG.isErrorEnabled()) + { + LOG.error("Could not get username.", e); + } + } + catch (NoSuchFieldException e) + { + if (LOG.isErrorEnabled()) + { + LOG.error("Could not get username.", e); + } + } + catch (SecurityException e) + { + if (LOG.isErrorEnabled()) + { + LOG.error("Could not get username.", e); + } + } return new UsernameCredential(username); } @@ -229,19 +272,41 @@ { String password = null; Class<?> digestAuthenticatorClazz = DigestAuthenticator.class.getDeclaredClasses()[0]; + try { Field field = digestAuthenticatorClazz.getDeclaredField("response"); field.setAccessible(true); password = (String)field.get((((ObjectCallback)objectCallback[0]).getObject())); } - catch (Exception e) + catch (IllegalArgumentException e) { if (LOG.isErrorEnabled()) { LOG.error("Could not get password.", e); } } + catch (IllegalAccessException e) + { + if (LOG.isErrorEnabled()) + { + LOG.error("Could not get password.", e); + } + } + catch (NoSuchFieldException e) + { + if (LOG.isErrorEnabled()) + { + LOG.error("Could not get password.", e); + } + } + catch (SecurityException e) + { + if (LOG.isErrorEnabled()) + { + LOG.error("Could not get password.", e); + } + } return password; } Modified: core/trunk/exo.core.component.security.core/src/main/java/org/exoplatform/services/security/web/JAASConversationStateListener.java =================================================================== --- core/trunk/exo.core.component.security.core/src/main/java/org/exoplatform/services/security/web/JAASConversationStateListener.java 2012-02-09 10:56:35 UTC (rev 5591) +++ core/trunk/exo.core.component.security.core/src/main/java/org/exoplatform/services/security/web/JAASConversationStateListener.java 2012-02-09 11:02:51 UTC (rev 5592) @@ -26,6 +26,7 @@ import javax.security.auth.Subject; import javax.security.auth.login.LoginContext; +import javax.security.auth.login.LoginException; import javax.servlet.http.HttpSession; import javax.servlet.http.HttpSessionEvent; @@ -81,10 +82,14 @@ } } } - catch (Exception e) + catch (LoginException e) { LOG.error("Can't remove conversation state " + httpSession.getId()); } + catch (SecurityException e) + { + LOG.error("Can't remove conversation state " + httpSession.getId()); + } } } Modified: core/trunk/exo.core.component.xml-processing/src/main/java/org/exoplatform/services/xml/transform/impl/trax/TRAXTemplatesServiceImpl.java =================================================================== --- core/trunk/exo.core.component.xml-processing/src/main/java/org/exoplatform/services/xml/transform/impl/trax/TRAXTemplatesServiceImpl.java 2012-02-09 10:56:35 UTC (rev 5591) +++ core/trunk/exo.core.component.xml-processing/src/main/java/org/exoplatform/services/xml/transform/impl/trax/TRAXTemplatesServiceImpl.java 2012-02-09 11:02:51 UTC (rev 5592) @@ -126,10 +126,18 @@ else LOG.error("XSLT schema not found: " + xsltSchema); } - catch (Exception e) + catch (IllegalArgumentException e) { LOG.error("Add new TRAXTemplates failed : " + e.getMessage()); } + catch (TransformerException e) + { + LOG.error("Add new TRAXTemplates failed : " + e.getMessage()); + } + catch (NotSupportedIOTypeException e) + { + LOG.error("Add new TRAXTemplates failed : " + e.getMessage()); + } } } }