Author: kkolinko Date: Mon Jan 4 16:00:49 2016 New Revision: 1722905 URL: http://svn.apache.org/viewvc?rev=1722905&view=rev Log: Simplify. Fix naming of message keys (s/Resources/Resource/). Log name of a missing resource.
Modified: tomcat/trunk/java/org/apache/catalina/core/DefaultInstanceManager.java tomcat/trunk/java/org/apache/catalina/core/LocalStrings.properties Modified: tomcat/trunk/java/org/apache/catalina/core/DefaultInstanceManager.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/DefaultInstanceManager.java?rev=1722905&r1=1722904&r2=1722905&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/core/DefaultInstanceManager.java (original) +++ tomcat/trunk/java/org/apache/catalina/core/DefaultInstanceManager.java Mon Jan 4 16:00:49 2016 @@ -88,19 +88,16 @@ public class DefaultInstanceManager impl privileged = catalinaContext.getPrivileged(); this.containerClassLoader = containerClassLoader; ignoreAnnotations = catalinaContext.getIgnoreAnnotations(); - StringManager sm = StringManager.getManager(Constants.Package); + Log log = catalinaContext.getLogger(); restrictedServlets = loadProperties( "org/apache/catalina/core/RestrictedServlets.properties", - sm.getString("defaultInstanceManager.restrictedServletsResource"), - catalinaContext.getLogger()); + "defaultInstanceManager.restrictedServletsResource", log); restrictedListeners = loadProperties( "org/apache/catalina/core/RestrictedListeners.properties", - sm.getString("defaultInstanceManager.restrictedListenersResources"), - catalinaContext.getLogger()); + "defaultInstanceManager.restrictedListenersResource", log); restrictedFilters = loadProperties( "org/apache/catalina/core/RestrictedFilters.properties", - sm.getString("defaultInstanceManager.restrictedFiltersResource"), - catalinaContext.getLogger()); + "defaultInstanceManager.restrictedFiltersResource", log); this.context = context; this.injectionMap = injectionMap; this.postConstructMethods = catalinaContext.findPostConstructMethods(); @@ -624,17 +621,17 @@ public class DefaultInstanceManager impl } } - private static Properties loadProperties(String resourceName, String errorString, Log log) { + private static Properties loadProperties(String resourceName, String messageKey, Log log) { Properties result = new Properties(); ClassLoader cl = DefaultInstanceManager.class.getClassLoader(); try (InputStream is = cl.getResourceAsStream(resourceName)) { - if (is ==null) { - log.error(errorString); + if (is == null) { + log.error(sm.getString(messageKey, resourceName)); } else { result.load(is); } } catch (IOException ioe) { - log.error(errorString, ioe); + log.error(sm.getString(messageKey, resourceName), ioe); } return result; } Modified: tomcat/trunk/java/org/apache/catalina/core/LocalStrings.properties URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/LocalStrings.properties?rev=1722905&r1=1722904&r2=1722905&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/core/LocalStrings.properties (original) +++ tomcat/trunk/java/org/apache/catalina/core/LocalStrings.properties Mon Jan 4 16:00:49 2016 @@ -222,6 +222,6 @@ threadLocalLeakPreventionListener.lifecy threadLocalLeakPreventionListener.containerEvent.error=Exception processing container event {0} defaultInstanceManager.invalidInjection=Invalid method resource injection annotation -defaultInstanceManager.restrictedServletsResource=Restricted servlets property file not found -defaultInstanceManager.restrictedFiltersResource=Restricted filters property file not found -defaultInstanceManager.restrictedListenersResources=Restricted listeners property file not found +defaultInstanceManager.restrictedFiltersResource=Restricted filters property file not found [{0}] +defaultInstanceManager.restrictedListenersResource=Restricted listeners property file not found [{0}] +defaultInstanceManager.restrictedServletsResource=Restricted servlets property file not found [{0}] --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org