You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by re...@apache.org on 2018/12/11 11:48:54 UTC
svn commit: r1848669 - in /tomcat/trunk/java/org/apache/tomcat/util:
IntrospectionUtils.java LocalStrings.properties
Author: remm
Date: Tue Dec 11 11:48:54 2018
New Revision: 1848669
URL: http://svn.apache.org/viewvc?rev=1848669&view=rev
Log:
Add i18n for util package.
Modified:
tomcat/trunk/java/org/apache/tomcat/util/IntrospectionUtils.java
tomcat/trunk/java/org/apache/tomcat/util/LocalStrings.properties
Modified: tomcat/trunk/java/org/apache/tomcat/util/IntrospectionUtils.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/IntrospectionUtils.java?rev=1848669&r1=1848668&r2=1848669&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/IntrospectionUtils.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/IntrospectionUtils.java Tue Dec 11 11:48:54 2018
@@ -25,14 +25,15 @@ import java.util.Hashtable;
import org.apache.juli.logging.Log;
import org.apache.juli.logging.LogFactory;
+import org.apache.tomcat.util.res.StringManager;
/**
* Utils for introspection and reflection
*/
public final class IntrospectionUtils {
-
private static final Log log = LogFactory.getLog(IntrospectionUtils.class);
+ private static final StringManager sm = StringManager.getManager(IntrospectionUtils.class);
/**
* Find a method with the right name If found, call the method ( if param is
@@ -165,18 +166,11 @@ public final class IntrospectionUtils {
}
}
- } catch (IllegalArgumentException ex2) {
- log.warn("IAE " + o + " " + name + " " + value, ex2);
- } catch (SecurityException ex1) {
- log.warn("IntrospectionUtils: SecurityException for " +
- o.getClass() + " " + name + "=" + value + ")", ex1);
- } catch (IllegalAccessException iae) {
- log.warn("IntrospectionUtils: IllegalAccessException for " +
- o.getClass() + " " + name + "=" + value + ")", iae);
- } catch (InvocationTargetException ie) {
- ExceptionUtils.handleThrowable(ie.getCause());
- log.warn("IntrospectionUtils: InvocationTargetException for " +
- o.getClass() + " " + name + "=" + value + ")", ie);
+ } catch (IllegalArgumentException | SecurityException | IllegalAccessException e) {
+ log.warn(sm.getString("introspectionUtils.setPropertyError", name, value, o.getClass()), e);
+ } catch (InvocationTargetException e) {
+ ExceptionUtils.handleThrowable(e.getCause());
+ log.warn(sm.getString("introspectionUtils.setPropertyError", name, value, o.getClass()), e);
}
return false;
}
@@ -211,22 +205,15 @@ public final class IntrospectionUtils {
return getPropertyMethod.invoke(o, params);
}
- } catch (IllegalArgumentException ex2) {
- log.warn("IAE " + o + " " + name, ex2);
- } catch (SecurityException ex1) {
- log.warn("IntrospectionUtils: SecurityException for " +
- o.getClass() + " " + name + ")", ex1);
- } catch (IllegalAccessException iae) {
- log.warn("IntrospectionUtils: IllegalAccessException for " +
- o.getClass() + " " + name + ")", iae);
- } catch (InvocationTargetException ie) {
- if (ie.getCause() instanceof NullPointerException) {
+ } catch (IllegalArgumentException | SecurityException | IllegalAccessException e) {
+ log.warn(sm.getString("introspectionUtils.getPropertyError", name, o.getClass()), e);
+ } catch (InvocationTargetException e) {
+ if (e.getCause() instanceof NullPointerException) {
// Assume the underlying object uses a storage to represent an unset property
return null;
}
- ExceptionUtils.handleThrowable(ie.getCause());
- log.warn("IntrospectionUtils: InvocationTargetException for " +
- o.getClass() + " " + name + ")", ie);
+ ExceptionUtils.handleThrowable(e.getCause());
+ log.warn(sm.getString("introspectionUtils.getPropertyError", name, o.getClass()), e);
}
return null;
}
@@ -350,10 +337,8 @@ public final class IntrospectionUtils {
public static Object callMethod1(Object target, String methodN,
Object param1, String typeParam1, ClassLoader cl) throws Exception {
- if (target == null || param1 == null) {
- throw new IllegalArgumentException(
- "IntrospectionUtils: Assert: Illegal params " +
- target + " " + param1);
+ if (target == null || methodN == null || param1 == null) {
+ throw new IllegalArgumentException(sm.getString("introspectionUtils.nullParameter"));
}
if (log.isDebugEnabled())
log.debug("IntrospectionUtils: callMethod1 " +
@@ -443,7 +428,7 @@ public final class IntrospectionUtils {
paramType.getName());
}
if (result == null) {
- throw new IllegalArgumentException("Can't convert argument: " + object);
+ throw new IllegalArgumentException(sm.getString("introspectionUtils.conversionError", object, paramType.getName()));
}
return result;
}
Modified: tomcat/trunk/java/org/apache/tomcat/util/LocalStrings.properties
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/LocalStrings.properties?rev=1848669&r1=1848668&r2=1848669&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/LocalStrings.properties [UTF-8] (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/LocalStrings.properties [UTF-8] Tue Dec 11 11:48:54 2018
@@ -28,3 +28,8 @@ diagnostics.vmInfoStartup=Startup argume
diagnostics.vmInfoSystem=System properties
diagnostics.vmInfoThreadCounts=Thread counts
diagnostics.vmInfoThreadMxBean=ThreadMXBean capabilities
+
+introspectionUtils.conversionError=Error converting [{0}] to type [{1}]
+introspectionUtils.getPropertyError=Error getting property [{0}] on class [{1}]
+introspectionUtils.nullParameter=Method name, parameter and object target arguments must not be null
+introspectionUtils.setPropertyError=Error setting property [{0}] to [{1}] on class [{2}]
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org