From 1a44dbda59f95c7170bbd9e2693a3ec1ff347cb8 Mon Sep 17 00:00:00 2001
From: neil_a_wilson <neil_a_wilson@localhost>
Date: Fri, 31 Aug 2007 22:49:33 +0000
Subject: [PATCH] Create configuration definitions for "generic" objects in the server configuration.  This also includes allocating object classes for each of these types.  This will help make it easier for users to create new instances of these kinds of configuration objects because they will not need to know the fully-qualified name of the Java class that implements the associated logic.

---
 opendj-sdk/opends/src/server/org/opends/server/extensions/JMXAlertHandler.java |   17 ++++++++++-------
 1 files changed, 10 insertions(+), 7 deletions(-)

diff --git a/opendj-sdk/opends/src/server/org/opends/server/extensions/JMXAlertHandler.java b/opendj-sdk/opends/src/server/org/opends/server/extensions/JMXAlertHandler.java
index bee4e4a..f8c5a2c 100644
--- a/opendj-sdk/opends/src/server/org/opends/server/extensions/JMXAlertHandler.java
+++ b/opendj-sdk/opends/src/server/org/opends/server/extensions/JMXAlertHandler.java
@@ -51,6 +51,7 @@
 
 import org.opends.server.admin.server.ConfigurationChangeListener;
 import org.opends.server.admin.std.server.AlertHandlerCfg;
+import org.opends.server.admin.std.server.JMXAlertHandlerCfg;
 import org.opends.server.api.AlertGenerator;
 import org.opends.server.api.AlertHandler;
 import org.opends.server.api.DirectoryServerMBean;
@@ -78,8 +79,8 @@
  */
 public class JMXAlertHandler
        extends NotificationBroadcasterSupport
-       implements AlertHandler<AlertHandlerCfg>,
-                  ConfigurationChangeListener<AlertHandlerCfg>, DynamicMBean,
+       implements AlertHandler<JMXAlertHandlerCfg>,
+                  ConfigurationChangeListener<JMXAlertHandlerCfg>, DynamicMBean,
                   DirectoryServerMBean
 {
   /**
@@ -125,7 +126,7 @@
   /**
    * {@inheritDoc}
    */
-  public void initializeAlertHandler(AlertHandlerCfg configuration)
+  public void initializeAlertHandler(JMXAlertHandlerCfg configuration)
        throws ConfigException, InitializationException
   {
     sequenceNumber = new AtomicLong(1);
@@ -168,7 +169,7 @@
 
     if (configuration != null)
     {
-      configuration.addChangeListener(this);
+      configuration.addJMXChangeListener(this);
       currentConfig = configuration;
     }
   }
@@ -191,7 +192,8 @@
   public boolean isConfigurationAcceptable(AlertHandlerCfg configuration,
                                            List<Message> unacceptableReasons)
   {
-    return true;
+    JMXAlertHandlerCfg cfg = (JMXAlertHandlerCfg) configuration;
+    return isConfigurationChangeAcceptable(cfg, unacceptableReasons);
   }
 
 
@@ -401,7 +403,8 @@
   /**
    * {@inheritDoc}
    */
-  public boolean isConfigurationChangeAcceptable(AlertHandlerCfg configuration,
+  public boolean isConfigurationChangeAcceptable(
+                      JMXAlertHandlerCfg configuration,
                       List<Message> unacceptableReasons)
   {
     return true;
@@ -413,7 +416,7 @@
    * {@inheritDoc}
    */
   public ConfigChangeResult applyConfigurationChange(
-                                        AlertHandlerCfg configuration)
+                                        JMXAlertHandlerCfg configuration)
   {
     currentConfig = configuration;
 

--
Gitblit v1.10.0