Author: rahul
Date: Mon Nov 6 22:25:12 2006
New Revision: 472002
URL: http://svn.apache.org/viewvc?view=rev&rev=472002
Log:
Slightly better error reporting when resource key cannot be resolved. Log and
rethrow exception as MissingResourceException (contains problem key in
message), rather than ignoring it (which induces a harder to debug NPE down the
road). Fallout of SHALE-325.
Modified:
shale/framework/trunk/shale-validator/src/main/java/org/apache/shale/validator/CommonsValidator.java
shale/framework/trunk/shale-validator/src/main/resources/org/apache/shale/validator/resources/Bundle.properties
Modified:
shale/framework/trunk/shale-validator/src/main/java/org/apache/shale/validator/CommonsValidator.java
URL:
http://svn.apache.org/viewvc/shale/framework/trunk/shale-validator/src/main/java/org/apache/shale/validator/CommonsValidator.java?view=diff&rev=472002&r1=472001&r2=472002
==============================================================================
---
shale/framework/trunk/shale-validator/src/main/java/org/apache/shale/validator/CommonsValidator.java
(original)
+++
shale/framework/trunk/shale-validator/src/main/java/org/apache/shale/validator/CommonsValidator.java
Mon Nov 6 22:25:12 2006
@@ -956,7 +956,9 @@
try {
msg = bundle.getString(msgkey);
} catch (MissingResourceException ex) {
- ; // Ignore this
+ log.error(messages.getMessage("commonsValidator.msgerror",
+ new Object[] {msgkey}), ex);
+ throw ex;
}
}
}
Modified:
shale/framework/trunk/shale-validator/src/main/resources/org/apache/shale/validator/resources/Bundle.properties
URL:
http://svn.apache.org/viewvc/shale/framework/trunk/shale-validator/src/main/resources/org/apache/shale/validator/resources/Bundle.properties?view=diff&rev=472002&r1=472001&r2=472002
==============================================================================
---
shale/framework/trunk/shale-validator/src/main/resources/org/apache/shale/validator/resources/Bundle.properties
(original)
+++
shale/framework/trunk/shale-validator/src/main/resources/org/apache/shale/validator/resources/Bundle.properties
Mon Nov 6 22:25:12 2006
@@ -22,3 +22,4 @@
commonsValidator.loadresource=Loading validation rules file from {0}
commonsValidator.skipresource=Skipping validation rules file from "{0}". No
url could be located.
commonsValidator.loaderror=Can't initialize resources
+commonsValidator.msgerror=Cannot find error message for key "{0}"
