Author: remm
Date: Tue Dec 11 16:46:24 2018
New Revision: 1848696

URL: http://svn.apache.org/viewvc?rev=1848696&view=rev
Log:
Add i18n for modeler package, and fix package name for mbeans strings.

Added:
    tomcat/trunk/java/org/apache/tomcat/util/modeler/LocalStrings.properties   
(with props)
Modified:
    tomcat/trunk/java/org/apache/catalina/mbeans/ConnectorMBean.java
    tomcat/trunk/java/org/apache/catalina/mbeans/ContextResourceLinkMBean.java
    tomcat/trunk/java/org/apache/catalina/mbeans/ContextResourceMBean.java
    tomcat/trunk/java/org/apache/catalina/mbeans/GroupMBean.java
    tomcat/trunk/java/org/apache/catalina/mbeans/MemoryUserDatabaseMBean.java
    tomcat/trunk/java/org/apache/catalina/mbeans/NamingResourcesMBean.java
    tomcat/trunk/java/org/apache/catalina/mbeans/UserMBean.java
    tomcat/trunk/java/org/apache/tomcat/util/modeler/BaseModelMBean.java
    tomcat/trunk/java/org/apache/tomcat/util/modeler/ManagedBean.java
    tomcat/trunk/java/org/apache/tomcat/util/modeler/Registry.java
    
tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsDigesterSource.java
    
tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsIntrospectionSource.java
    tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/ModelerSource.java

Modified: tomcat/trunk/java/org/apache/catalina/mbeans/ConnectorMBean.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/mbeans/ConnectorMBean.java?rev=1848696&r1=1848695&r2=1848696&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/mbeans/ConnectorMBean.java (original)
+++ tomcat/trunk/java/org/apache/catalina/mbeans/ConnectorMBean.java Tue Dec 11 
16:46:24 2018
@@ -24,6 +24,7 @@ import javax.management.RuntimeOperation
 
 import org.apache.catalina.connector.Connector;
 import org.apache.tomcat.util.IntrospectionUtils;
+import org.apache.tomcat.util.res.StringManager;
 
 /**
  * <p>A <strong>ModelMBean</strong> implementation for the
@@ -33,6 +34,8 @@ import org.apache.tomcat.util.Introspect
  */
 public class ConnectorMBean extends ClassNameMBean<Connector> {
 
+    private static final StringManager sm = 
StringManager.getManager(ConnectorMBean.class);
+
     /**
      * Obtain and return the value of a specific attribute of this MBean.
      *

Modified: 
tomcat/trunk/java/org/apache/catalina/mbeans/ContextResourceLinkMBean.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/mbeans/ContextResourceLinkMBean.java?rev=1848696&r1=1848695&r2=1848696&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/mbeans/ContextResourceLinkMBean.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/mbeans/ContextResourceLinkMBean.java 
Tue Dec 11 16:46:24 2018
@@ -24,6 +24,7 @@ import javax.management.RuntimeOperation
 
 import org.apache.tomcat.util.descriptor.web.ContextResourceLink;
 import org.apache.tomcat.util.descriptor.web.NamingResources;
+import org.apache.tomcat.util.res.StringManager;
 
 /**
  * <p>A <strong>ModelMBean</strong> implementation for the
@@ -33,6 +34,8 @@ import org.apache.tomcat.util.descriptor
  */
 public class ContextResourceLinkMBean extends 
BaseCatalinaMBean<ContextResourceLink> {
 
+    private static final StringManager sm = 
StringManager.getManager(ContextResourceLinkMBean.class);
+
     /**
      * Obtain and return the value of a specific attribute of this MBean.
      *

Modified: tomcat/trunk/java/org/apache/catalina/mbeans/ContextResourceMBean.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/mbeans/ContextResourceMBean.java?rev=1848696&r1=1848695&r2=1848696&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/mbeans/ContextResourceMBean.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/mbeans/ContextResourceMBean.java Tue 
Dec 11 16:46:24 2018
@@ -24,6 +24,7 @@ import javax.management.RuntimeOperation
 
 import org.apache.tomcat.util.descriptor.web.ContextResource;
 import org.apache.tomcat.util.descriptor.web.NamingResources;
+import org.apache.tomcat.util.res.StringManager;
 
 /**
  * <p>A <strong>ModelMBean</strong> implementation for the
@@ -33,6 +34,8 @@ import org.apache.tomcat.util.descriptor
  */
 public class ContextResourceMBean extends BaseCatalinaMBean<ContextResource> {
 
+    private static final StringManager sm = 
StringManager.getManager(ContextResourceMBean.class);
+
     /**
      * Obtain and return the value of a specific attribute of this MBean.
      *

Modified: tomcat/trunk/java/org/apache/catalina/mbeans/GroupMBean.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/mbeans/GroupMBean.java?rev=1848696&r1=1848695&r2=1848696&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/mbeans/GroupMBean.java (original)
+++ tomcat/trunk/java/org/apache/catalina/mbeans/GroupMBean.java Tue Dec 11 
16:46:24 2018
@@ -29,6 +29,7 @@ import org.apache.catalina.User;
 import org.apache.tomcat.util.modeler.BaseModelMBean;
 import org.apache.tomcat.util.modeler.ManagedBean;
 import org.apache.tomcat.util.modeler.Registry;
+import org.apache.tomcat.util.res.StringManager;
 
 /**
  * <p>A <strong>ModelMBean</strong> implementation for the
@@ -38,6 +39,8 @@ import org.apache.tomcat.util.modeler.Re
  */
 public class GroupMBean extends BaseModelMBean {
 
+    private static final StringManager sm = 
StringManager.getManager(GroupMBean.class);
+
     /**
      * The configuration information registry for our managed beans.
      */

Modified: 
tomcat/trunk/java/org/apache/catalina/mbeans/MemoryUserDatabaseMBean.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/mbeans/MemoryUserDatabaseMBean.java?rev=1848696&r1=1848695&r2=1848696&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/mbeans/MemoryUserDatabaseMBean.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/mbeans/MemoryUserDatabaseMBean.java 
Tue Dec 11 16:46:24 2018
@@ -30,6 +30,7 @@ import org.apache.catalina.UserDatabase;
 import org.apache.tomcat.util.modeler.BaseModelMBean;
 import org.apache.tomcat.util.modeler.ManagedBean;
 import org.apache.tomcat.util.modeler.Registry;
+import org.apache.tomcat.util.res.StringManager;
 
 /**
  * <p>A <strong>ModelMBean</strong> implementation for the
@@ -39,6 +40,8 @@ import org.apache.tomcat.util.modeler.Re
  */
 public class MemoryUserDatabaseMBean extends BaseModelMBean {
 
+    private static final StringManager sm = 
StringManager.getManager(MemoryUserDatabaseMBean.class);
+
     // ----------------------------------------------------- Instance Variables
 
     /**

Modified: tomcat/trunk/java/org/apache/catalina/mbeans/NamingResourcesMBean.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/mbeans/NamingResourcesMBean.java?rev=1848696&r1=1848695&r2=1848696&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/mbeans/NamingResourcesMBean.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/mbeans/NamingResourcesMBean.java Tue 
Dec 11 16:46:24 2018
@@ -29,6 +29,7 @@ import org.apache.tomcat.util.descriptor
 import org.apache.tomcat.util.modeler.BaseModelMBean;
 import org.apache.tomcat.util.modeler.ManagedBean;
 import org.apache.tomcat.util.modeler.Registry;
+import org.apache.tomcat.util.res.StringManager;
 
 /**
  * <p>A <strong>ModelMBean</strong> implementation for the
@@ -38,6 +39,8 @@ import org.apache.tomcat.util.modeler.Re
  */
 public class NamingResourcesMBean extends BaseModelMBean {
 
+    private static final StringManager sm = 
StringManager.getManager(NamingResourcesMBean.class);
+
     // ----------------------------------------------------- Instance Variables
 
     /**

Modified: tomcat/trunk/java/org/apache/catalina/mbeans/UserMBean.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/mbeans/UserMBean.java?rev=1848696&r1=1848695&r2=1848696&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/mbeans/UserMBean.java (original)
+++ tomcat/trunk/java/org/apache/catalina/mbeans/UserMBean.java Tue Dec 11 
16:46:24 2018
@@ -29,6 +29,7 @@ import org.apache.catalina.User;
 import org.apache.tomcat.util.modeler.BaseModelMBean;
 import org.apache.tomcat.util.modeler.ManagedBean;
 import org.apache.tomcat.util.modeler.Registry;
+import org.apache.tomcat.util.res.StringManager;
 
 /**
  * <p>A <strong>ModelMBean</strong> implementation for the
@@ -38,6 +39,7 @@ import org.apache.tomcat.util.modeler.Re
  */
 public class UserMBean extends BaseModelMBean {
 
+    private static final StringManager sm = 
StringManager.getManager(UserMBean.class);
 
     // ----------------------------------------------------- Instance Variables
 

Modified: tomcat/trunk/java/org/apache/tomcat/util/modeler/BaseModelMBean.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/modeler/BaseModelMBean.java?rev=1848696&r1=1848695&r2=1848696&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/modeler/BaseModelMBean.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/modeler/BaseModelMBean.java Tue 
Dec 11 16:46:24 2018
@@ -105,7 +105,7 @@ public class BaseModelMBean implements D
         ModelMBeanNotificationBroadcaster {
 
     private static final Log log = LogFactory.getLog(BaseModelMBean.class);
-    protected static final StringManager sm = 
StringManager.getManager(BaseModelMBean.class);
+    private static final StringManager sm = 
StringManager.getManager(BaseModelMBean.class);
 
     // ----------------------------------------------------- Instance Variables
 
@@ -158,8 +158,8 @@ public class BaseModelMBean implements D
         // Validate the input parameters
         if (name == null)
             throw new RuntimeOperationsException
-                (new IllegalArgumentException("Attribute name is null"),
-                 "Attribute name is null");
+                (new 
IllegalArgumentException(sm.getString("baseModelMBean.nullAttributeName")),
+                        sm.getString("baseModelMBean.nullAttributeName"));
 
         if( (resource instanceof DynamicMBean) &&
              ! ( resource instanceof BaseModelMBean )) {
@@ -183,16 +183,16 @@ public class BaseModelMBean implements D
                 t = e;
             if (t instanceof RuntimeException)
                 throw new RuntimeOperationsException
-                    ((RuntimeException) t, "Exception invoking method " + 
name);
+                    ((RuntimeException) t, 
sm.getString("baseModelMBean.invokeError", name));
             else if (t instanceof Error)
                 throw new RuntimeErrorException
-                    ((Error) t, "Error invoking method " + name);
+                    ((Error) t, sm.getString("baseModelMBean.invokeError", 
name));
             else
                 throw new MBeanException
-                    (e, "Exception invoking method " + name);
+                    (e, sm.getString("baseModelMBean.invokeError", name));
         } catch (Exception e) {
             throw new MBeanException
-                (e, "Exception invoking method " + name);
+                (e, sm.getString("baseModelMBean.invokeError", name));
         }
 
         // Return the results of this method invocation
@@ -212,8 +212,8 @@ public class BaseModelMBean implements D
         // Validate the input parameters
         if (names == null)
             throw new RuntimeOperationsException
-                (new IllegalArgumentException("Attribute names list is null"),
-                 "Attribute names list is null");
+                (new 
IllegalArgumentException(sm.getString("baseModelMBean.nullAttributeNameList")),
+                        sm.getString("baseModelMBean.nullAttributeNameList"));
 
         // Prepare our response, eating all exceptions
         AttributeList response = new AttributeList();
@@ -273,8 +273,8 @@ public class BaseModelMBean implements D
         // Validate the input parameters
         if (name == null)
             throw new RuntimeOperationsException
-                (new IllegalArgumentException("Method name is null"),
-                 "Method name is null");
+                (new 
IllegalArgumentException(sm.getString("baseModelMBean.nullMethodName")),
+                        sm.getString("baseModelMBean.nullMethodName"));
 
         if( log.isDebugEnabled()) log.debug("Invoke " + name);
 
@@ -290,22 +290,22 @@ public class BaseModelMBean implements D
             }
         } catch (InvocationTargetException e) {
             Throwable t = e.getTargetException();
-            log.error("Exception invoking method " + name , t );
+            log.error(sm.getString("baseModelMBean.invokeError", name), t );
             if (t == null)
                 t = e;
             if (t instanceof RuntimeException)
                 throw new RuntimeOperationsException
-                    ((RuntimeException) t, "Exception invoking method " + 
name);
+                    ((RuntimeException) t, 
sm.getString("baseModelMBean.invokeError", name));
             else if (t instanceof Error)
                 throw new RuntimeErrorException
-                    ((Error) t, "Error invoking method " + name);
+                    ((Error) t, sm.getString("baseModelMBean.invokeError", 
name));
             else
                 throw new MBeanException
-                    ((Exception)t, "Exception invoking method " + name);
+                    ((Exception)t, sm.getString("baseModelMBean.invokeError", 
name));
         } catch (Exception e) {
-            log.error("Exception invoking method " + name , e );
+            log.error(sm.getString("baseModelMBean.invokeError", name), e );
             throw new MBeanException
-                (e, "Exception invoking method " + name);
+                (e, sm.getString("baseModelMBean.invokeError", name));
         }
 
         // Return the results of this method invocation
@@ -343,8 +343,7 @@ public class BaseModelMBean implements D
             try {
                 return Class.forName(signature);
             } catch (ClassNotFoundException e) {
-                throw new ReflectionException
-                    (e, "Cannot find Class for " + signature);
+                throw new ReflectionException(e, 
sm.getString("baseModelMBean.cnfeForSignature", signature));
             }
         }
     }
@@ -383,16 +382,16 @@ public class BaseModelMBean implements D
         // Validate the input parameters
         if (attribute == null)
             throw new RuntimeOperationsException
-                (new IllegalArgumentException("Attribute is null"),
-                 "Attribute is null");
+                (new 
IllegalArgumentException(sm.getString("baseModelMBean.nullAttribute")),
+                        sm.getString("baseModelMBean.nullAttribute"));
 
         String name = attribute.getName();
         Object value = attribute.getValue();
 
         if (name == null)
             throw new RuntimeOperationsException
-                (new IllegalArgumentException("Attribute name is null"),
-                 "Attribute name is null");
+                (new 
IllegalArgumentException(sm.getString("baseModelMBean.nullAttributeName")),
+                        sm.getString("baseModelMBean.nullAttributeName"));
 
         Object oldValue=null;
         //if( getAttMap.get(name) != null )
@@ -412,23 +411,23 @@ public class BaseModelMBean implements D
                 t = e;
             if (t instanceof RuntimeException)
                 throw new RuntimeOperationsException
-                    ((RuntimeException) t, "Exception invoking method " + 
name);
+                    ((RuntimeException) t, 
sm.getString("baseModelMBean.invokeError", name));
             else if (t instanceof Error)
                 throw new RuntimeErrorException
-                    ((Error) t, "Error invoking method " + name);
+                    ((Error) t, sm.getString("baseModelMBean.invokeError", 
name));
             else
                 throw new MBeanException
-                    (e, "Exception invoking method " + name);
+                    (e, sm.getString("baseModelMBean.invokeError", name));
         } catch (Exception e) {
-            log.error("Exception invoking method " + name , e );
+            log.error(sm.getString("baseModelMBean.invokeError", name) , e );
             throw new MBeanException
-                (e, "Exception invoking method " + name);
+                (e, sm.getString("baseModelMBean.invokeError", name));
         }
         try {
             sendAttributeChangeNotification(new Attribute( name, oldValue),
                     attribute);
         } catch(Exception ex) {
-            log.error("Error sending notification " + name, ex);
+            log.error(sm.getString("baseModelMBean.notificationError", name), 
ex);
         }
         //attributes.put( name, value );
 //        if( source != null ) {
@@ -501,8 +500,8 @@ public class BaseModelMBean implements D
 
         if (resource == null)
             throw new RuntimeOperationsException
-                (new IllegalArgumentException("Managed resource is null"),
-                 "Managed resource is null");
+                (new 
IllegalArgumentException(sm.getString("baseModelMBean.nullResource")),
+                        sm.getString("baseModelMBean.nullResource"));
 
         return resource;
 
@@ -534,8 +533,8 @@ public class BaseModelMBean implements D
     {
         if (resource == null)
             throw new RuntimeOperationsException
-                (new IllegalArgumentException("Managed resource is null"),
-                 "Managed resource is null");
+                (new 
IllegalArgumentException(sm.getString("baseModelMBean.nullResource")),
+                        sm.getString("baseModelMBean.nullResource"));
 
 //        if (!"objectreference".equalsIgnoreCase(type))
 //            throw new InvalidTargetObjectTypeException(type);
@@ -578,7 +577,7 @@ public class BaseModelMBean implements D
         throws IllegalArgumentException {
 
         if (listener == null)
-            throw new IllegalArgumentException("Listener is null");
+            throw new 
IllegalArgumentException(sm.getString("baseModelMBean.nullListener"));
         if (attributeBroadcaster == null)
             attributeBroadcaster = new BaseNotificationBroadcaster();
 
@@ -609,7 +608,7 @@ public class BaseModelMBean implements D
         throws ListenerNotFoundException {
 
         if (listener == null)
-            throw new IllegalArgumentException("Listener is null");
+            throw new 
IllegalArgumentException(sm.getString("baseModelMBean.nullListener"));
 
         // FIXME - currently this removes *all* notifications for this listener
         if (attributeBroadcaster != null) {
@@ -638,8 +637,8 @@ public class BaseModelMBean implements D
 
         if (notification == null)
             throw new RuntimeOperationsException
-                (new IllegalArgumentException("Notification is null"),
-                 "Notification is null");
+                (new 
IllegalArgumentException(sm.getString("baseModelMBean.nullNotification")),
+                        sm.getString("baseModelMBean.nullNotification"));
         if (attributeBroadcaster == null)
             return; // This means there are no registered listeners
         if( log.isDebugEnabled() )
@@ -703,8 +702,8 @@ public class BaseModelMBean implements D
 
         if (notification == null)
             throw new RuntimeOperationsException
-                (new IllegalArgumentException("Notification is null"),
-                 "Notification is null");
+                (new 
IllegalArgumentException(sm.getString("baseModelMBean.nullNotification")),
+                        sm.getString("baseModelMBean.nullNotification"));
         if (generalBroadcaster == null)
             return; // This means there are no registered listeners
         generalBroadcaster.sendNotification(notification);
@@ -729,8 +728,8 @@ public class BaseModelMBean implements D
 
         if (message == null)
             throw new RuntimeOperationsException
-                (new IllegalArgumentException("Message is null"),
-                 "Message is null");
+                (new 
IllegalArgumentException(sm.getString("baseModelMBean.nullMessage")),
+                        sm.getString("baseModelMBean.nullMessage"));
         Notification notification = new Notification
             ("jmx.modelmbean.generic", this, 1, message);
         sendNotification(notification);
@@ -759,7 +758,7 @@ public class BaseModelMBean implements D
         throws IllegalArgumentException {
 
         if (listener == null)
-            throw new IllegalArgumentException("Listener is null");
+            throw new 
IllegalArgumentException(sm.getString("baseModelMBean.nullListener"));
 
         if( log.isDebugEnabled() ) log.debug("addNotificationListener " + 
listener);
 
@@ -843,7 +842,7 @@ public class BaseModelMBean implements D
         throws ListenerNotFoundException {
 
         if (listener == null)
-            throw new IllegalArgumentException("Listener is null");
+            throw new 
IllegalArgumentException(sm.getString("baseModelMBean.nullListener"));
 
         if (generalBroadcaster != null) {
             generalBroadcaster.removeNotificationListener(listener);

Added: tomcat/trunk/java/org/apache/tomcat/util/modeler/LocalStrings.properties
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/modeler/LocalStrings.properties?rev=1848696&view=auto
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/modeler/LocalStrings.properties 
(added)
+++ tomcat/trunk/java/org/apache/tomcat/util/modeler/LocalStrings.properties 
Tue Dec 11 16:46:24 2018
@@ -0,0 +1,48 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+baseModelMBean.cnfeForSignature=Cannot find class for [{0}]
+baseModelMBean.invokeError=Exception invoking method [{0}]
+baseModelMBean.notificationError=Error sending notification [{0}]
+baseModelMBean.nullAttribute=The attribute is null
+baseModelMBean.nullAttributeName=The attribute name is null
+baseModelMBean.nullAttributeNameList=The attribute name list is null
+baseModelMBean.nullListener=The listener is null
+baseModelMBean.nullMessage=The message is null
+baseModelMBean.nullMethodName=The method name is null
+baseModelMBean.nullNotification=The notification is null
+baseModelMBean.nullResource=The managed resource is null
+
+managedMBean.cannotInstantiateClass=Cannot instantiate ModelMBean of class 
[{0}]
+managedMBean.cannotLoadClass=Cannot load ModelMBean class [{0}]
+managedMBean.inconsistentArguments=Inconsistent arguments and signature
+managedMBean.noAttribute=Cannot find attribute [{0}] on resource [{1}]
+managedMBean.noGet=Cannot find getter method [{0}] on resource [{1}]
+managedMBean.noMethod=Cannot find method [{0}] with this signature
+managedMBean.noOperation=Cannot find operation [{0}]
+managedMBean.noSet=Cannot find setter method [{0}] on resource [{1}]
+
+modules.digesterParseError=Error parsing registry data
+modules.nullMethod=No method argument for [{0}]
+modules.readDescriptorsError=Error reading descriptors
+
+registry.initError=Error initializing [{0}]
+registry.loadError=Error loading desriptors from [{0}]
+registry.noMetadata=Cannot find metadata for object [{0}]
+registry.noTypeMetadata=Cannot find metadata for type [{0}]
+registry.nullBean=Cannot register null bean for [{0}]
+registry.objectNameCreateError=Error creating object name
+registry.registerError=Error registering MBean
+registry.unregisterError=Error unregistering MBean

Propchange: 
tomcat/trunk/java/org/apache/tomcat/util/modeler/LocalStrings.properties
------------------------------------------------------------------------------
    svn:eol-style = native

Modified: tomcat/trunk/java/org/apache/tomcat/util/modeler/ManagedBean.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/modeler/ManagedBean.java?rev=1848696&r1=1848695&r2=1848696&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/modeler/ManagedBean.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/modeler/ManagedBean.java Tue Dec 
11 16:46:24 2018
@@ -39,6 +39,7 @@ import javax.management.RuntimeOperation
 import javax.management.ServiceNotFoundException;
 
 import org.apache.tomcat.util.buf.StringUtils;
+import org.apache.tomcat.util.res.StringManager;
 
 
 /**
@@ -50,6 +51,7 @@ import org.apache.tomcat.util.buf.String
 public class ManagedBean implements java.io.Serializable {
 
     private static final long serialVersionUID = 1L;
+    private static final StringManager sm = 
StringManager.getManager(ManagedBean.class);
 
     private static final String BASE_MBEAN = 
"org.apache.tomcat.util.modeler.BaseModelMBean";
     // ----------------------------------------------------- Instance Variables
@@ -314,7 +316,7 @@ public class ManagedBean implements java
 
             if( clazz==null) {
                 throw new MBeanException
-                    (ex, "Cannot load ModelMBean class " + getClassName());
+                    (ex, sm.getString("managedMBean.cannotLoadClass", 
getClassName()));
             }
             try {
                 // Stupid - this will set the default minfo first....
@@ -323,8 +325,7 @@ public class ManagedBean implements java
                 throw e;
             } catch (Exception e) {
                 throw new MBeanException
-                    (e, "Cannot instantiate ModelMBean of class " +
-                     getClassName());
+                    (e, sm.getString("managedMBean.cannotInstantiateClass", 
getClassName()));
             }
         }
 
@@ -431,7 +432,7 @@ public class ManagedBean implements java
         AttributeInfo attrInfo = attributes.get(aname);
         // Look up the actual operation to be used
         if (attrInfo == null)
-            throw new AttributeNotFoundException(" Cannot find attribute " + 
aname + " for " + resource);
+            throw new 
AttributeNotFoundException(sm.getString("managedMBean.noAttribute", aname, 
resource));
 
         String getMethod = attrInfo.getGetMethod();
 
@@ -453,8 +454,7 @@ public class ManagedBean implements java
             }
         }
         if (exception != null) {
-            throw new ReflectionException(exception,
-                                          "Cannot find getter method " + 
getMethod);
+            throw new ReflectionException(exception, 
sm.getString("managedMBean.noGet", getMethod, resource));
         }
 
         return m;
@@ -467,7 +467,7 @@ public class ManagedBean implements java
 
         AttributeInfo attrInfo = attributes.get(aname);
         if (attrInfo == null) {
-            throw new AttributeNotFoundException(" Cannot find attribute " + 
aname);
+            throw new 
AttributeNotFoundException(sm.getString("managedMBean.noAttribute", aname, 
resource));
         }
 
         // Look up the actual operation to be used
@@ -494,8 +494,7 @@ public class ManagedBean implements java
             }
         }
         if (exception != null) {
-            throw new ReflectionException(exception,
-                    "Cannot find setter method " + setMethod + " " + resource);
+            throw new ReflectionException(exception, 
sm.getString("managedMBean.noSet", setMethod, resource));
         }
 
         return m;
@@ -512,18 +511,16 @@ public class ManagedBean implements java
             signature = new String[0];
         if (params.length != signature.length)
             throw new RuntimeOperationsException(
-                    new IllegalArgumentException(
-                            "Inconsistent arguments and signature"),
-                    "Inconsistent arguments and signature");
+                    new 
IllegalArgumentException(sm.getString("managedMBean.inconsistentArguments")),
+                    sm.getString("managedMBean.inconsistentArguments"));
 
         // Acquire the ModelMBeanOperationInfo information for
         // the requested operation
         OperationInfo opInfo =
                 operations.get(createOperationKey(aname, signature));
         if (opInfo == null)
-            throw new MBeanException(new ServiceNotFoundException(
-                    "Cannot find operation " + aname),
-                    "Cannot find operation " + aname);
+            throw new MBeanException(new 
ServiceNotFoundException(sm.getString("managedMBean.noOperation", aname)),
+                    sm.getString("managedMBean.noOperation", aname));
 
         // Prepare the signature required by Java reflection APIs
         // FIXME - should we use the signature from opInfo?
@@ -552,8 +549,7 @@ public class ManagedBean implements java
             exception = e;
         }
         if (method == null) {
-            throw new ReflectionException(exception, "Cannot find method "
-                    + aname + " with this signature");
+            throw new ReflectionException(exception, 
sm.getString("managedMBean.noMethod", aname));
         }
 
         return method;

Modified: tomcat/trunk/java/org/apache/tomcat/util/modeler/Registry.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/modeler/Registry.java?rev=1848696&r1=1848695&r2=1848696&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/modeler/Registry.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/modeler/Registry.java Tue Dec 11 
16:46:24 2018
@@ -39,6 +39,7 @@ import javax.management.ObjectName;
 import org.apache.juli.logging.Log;
 import org.apache.juli.logging.LogFactory;
 import org.apache.tomcat.util.modeler.modules.ModelerSource;
+import org.apache.tomcat.util.res.StringManager;
 
 /*
    Issues:
@@ -70,6 +71,7 @@ public class Registry implements Registr
      * The Log instance to which we will write our log messages.
      */
     private static final Log log = LogFactory.getLog(Registry.class);
+    private static final StringManager sm = 
StringManager.getManager(Registry.class);
 
     // Support for the factory methods
 
@@ -237,7 +239,7 @@ public class Registry implements Registr
         try {
             unregisterComponent(new ObjectName(oname));
         } catch (MalformedObjectNameException e) {
-            log.info("Error creating object name " + e);
+            log.info(sm.getString("registry.objectNameCreateError"), e);
         }
     }
 
@@ -272,7 +274,7 @@ public class Registry implements Registr
             } catch (Exception t) {
                 if (failFirst)
                     throw t;
-                log.info("Error initializing " + current + " " + t.toString());
+                log.info(sm.getString("registry.initError"), t);
             }
         }
     }
@@ -372,7 +374,7 @@ public class Registry implements Registr
         try {
             info = getMBeanServer().getMBeanInfo(oname);
         } catch (Exception e) {
-            log.info("Can't find metadata for object" + oname);
+            log.info(sm.getString("registry.noMetadata", oname));
             return null;
         }
 
@@ -399,7 +401,7 @@ public class Registry implements Registr
         try {
             info = getMBeanServer().getMBeanInfo(oname);
         } catch (Exception e) {
-            log.info("Can't find metadata " + oname);
+            log.info(sm.getString("registry.noMetadata", oname));
             return null;
         }
         MBeanOperationInfo attInfo[] = info.getOperations();
@@ -424,7 +426,7 @@ public class Registry implements Registr
                 getMBeanServer().unregisterMBean(oname);
             }
         } catch (Throwable t) {
-            log.error("Error unregistering mbean", t);
+            log.error(sm.getString("registry.unregisterError"), t);
         }
     }
 
@@ -503,7 +505,7 @@ public class Registry implements Registr
 
             managed = findManagedBean(type);
             if (managed == null) {
-                log.warn("No metadata found for " + type);
+                log.warn(sm.getString("registry.noTypeMetadata", type));
                 return null;
             }
             managed.setName(type);
@@ -613,7 +615,7 @@ public class Registry implements Registr
         }
 
         if (bean == null) {
-            log.error("Null component " + oname);
+            log.error(sm.getString("registry.nullBean", oname));
             return;
         }
 
@@ -636,7 +638,7 @@ public class Registry implements Registr
 
             getMBeanServer().registerMBean(mbean, oname);
         } catch (Exception ex) {
-            log.error("Error registering " + oname, ex);
+            log.error(sm.getString("registry.registerError", oname), ex);
             throw ex;
         }
     }
@@ -672,7 +674,7 @@ public class Registry implements Registr
         try {
             load("MbeansDescriptorsDigesterSource", dURL, null);
         } catch (Exception ex) {
-            log.error("Error loading " + dURL);
+            log.error(sm.getString("registry.loadError", dURL));
         }
     }
 

Modified: 
tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsDigesterSource.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsDigesterSource.java?rev=1848696&r1=1848695&r2=1848696&view=diff
==============================================================================
--- 
tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsDigesterSource.java
 (original)
+++ 
tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsDigesterSource.java
 Tue Dec 11 16:46:24 2018
@@ -168,7 +168,7 @@ public class MbeansDescriptorsDigesterSo
                 digester.push(loadedMbeans);
                 digester.parse(stream);
             } catch (Exception e) {
-                log.error("Error digesting Registry data", e);
+                log.error(sm.getString("modules.digesterParseError"), e);
                 throw e;
             } finally {
                 digester.reset();

Modified: 
tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsIntrospectionSource.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsIntrospectionSource.java?rev=1848696&r1=1848695&r2=1848696&view=diff
==============================================================================
--- 
tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsIntrospectionSource.java
 (original)
+++ 
tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsIntrospectionSource.java
 Tue Dec 11 16:46:24 2018
@@ -83,7 +83,7 @@ public class MbeansDescriptorsIntrospect
             registry.addManagedBean(managed);
 
         } catch( Exception ex ) {
-            log.error( "Error reading descriptors ", ex);
+            log.error(sm.getString("modules.readDescriptorsError"), ex);
         }
     }
 
@@ -351,7 +351,7 @@ public class MbeansDescriptorsIntrospect
                     }
                     mbean.addOperation(op);
                 } else {
-                    log.error("Null arg method for [" + name + "]");
+                    log.error(sm.getString("modules.nullMethod", name));
                 }
             }
 

Modified: 
tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/ModelerSource.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/ModelerSource.java?rev=1848696&r1=1848695&r2=1848696&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/ModelerSource.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/ModelerSource.java 
Tue Dec 11 16:46:24 2018
@@ -21,11 +21,13 @@ import java.util.List;
 import javax.management.ObjectName;
 
 import org.apache.tomcat.util.modeler.Registry;
+import org.apache.tomcat.util.res.StringManager;
 
 /**
  * Source for descriptor data. More sources can be added.
  */
 public abstract class ModelerSource {
+    protected static final StringManager sm = 
StringManager.getManager(Registry.class);
     protected Object source;
 
     /**



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to