From c2d0d212b510d8e82b9b123b4d06a80b835c8cd4 Mon Sep 17 00:00:00 2001
From: Nicolas Capponi <nicolas.capponi@forgerock.com>
Date: Fri, 06 Dec 2013 16:52:01 +0000
Subject: [PATCH] OpenDJ 3 : config framework

---
 opendj-admin/src/main/java/org/opends/server/admin/AggregationPropertyDefinition.java |   37 ++++++++++++++++++++++---------------
 1 files changed, 22 insertions(+), 15 deletions(-)

diff --git a/opendj-admin/src/main/java/org/opends/server/admin/AggregationPropertyDefinition.java b/opendj-admin/src/main/java/org/opends/server/admin/AggregationPropertyDefinition.java
index 33d3c7b..651b95e 100644
--- a/opendj-admin/src/main/java/org/opends/server/admin/AggregationPropertyDefinition.java
+++ b/opendj-admin/src/main/java/org/opends/server/admin/AggregationPropertyDefinition.java
@@ -26,6 +26,10 @@
  */
 package org.opends.server.admin;
 
+import static com.forgerock.opendj.ldap.AdminMessages.*;
+import static com.forgerock.opendj.util.StaticUtils.*;
+import static com.forgerock.opendj.util.Validator.*;
+
 import java.util.Collection;
 import java.util.Collections;
 import java.util.EnumSet;
@@ -53,7 +57,11 @@
 import org.opends.server.admin.server.ServerManagementContext;
 import org.opends.server.config.ConfigException;
 import org.opends.server.types.ConfigChangeResult;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.forgerock.i18n.LocalizableMessage;
+import org.forgerock.i18n.slf4j.LocalizedLogger;
+import org.forgerock.opendj.admin.meta.RootCfgDefn;
 import org.forgerock.opendj.ldap.DN;
 import org.forgerock.opendj.ldap.ResultCode;
 
@@ -260,13 +268,13 @@
                 }
             } catch (ConfigException e) {
                 // The condition could not be evaluated.
-                if (debugEnabled()) {
-                    TRACER.debugCaught(DebugLogLevel.ERROR, e);
-                }
-
-                LocalizableMessage message = ERR_REFINT_UNABLE_TO_EVALUATE_TARGET_CONDITION.get(mo.getManagedObjectDefinition()
-                        .getUserFriendlyName(), String.valueOf(mo.getDN()), StaticUtils.getExceptionLocalizableMessage(e));
-                ErrorLogger.logError(message);
+                debugLogger.trace("Unable to perform post add", e);
+                LocalizableMessage message = ERR_REFINT_UNABLE_TO_EVALUATE_TARGET_CONDITION.get(mo
+                        .getManagedObjectDefinition().getUserFriendlyName(), String.valueOf(mo.getDN()),
+                        getExceptionMessage(e));
+                LocalizedLogger logger = LocalizedLogger
+                        .getLocalizedLogger(ERR_REFINT_UNABLE_TO_EVALUATE_TARGET_CONDITION.resourceName());
+                logger.error(message);
                 unacceptableReasons.add(message);
                 return false;
             }
@@ -504,12 +512,14 @@
                 try {
                     ref = context.getManagedObject(path);
                 } catch (DefinitionDecodingException e) {
-                    LocalizableMessage msg = ERR_CLIENT_REFINT_TARGET_INVALID.get(ufn, name, getName(), e.getLocalizableMessageObject());
+                    LocalizableMessage msg = ERR_CLIENT_REFINT_TARGET_INVALID.get(ufn, name, getName(),
+                            e.getMessageObject());
                     unacceptableReasons.add(msg);
                     isAcceptable = false;
                     continue;
                 } catch (ManagedObjectDecodingException e) {
-                    LocalizableMessage msg = ERR_CLIENT_REFINT_TARGET_INVALID.get(ufn, name, getName(), e.getLocalizableMessageObject());
+                    LocalizableMessage msg = ERR_CLIENT_REFINT_TARGET_INVALID.get(ufn, name, getName(),
+                            e.getMessageObject());
                     unacceptableReasons.add(msg);
                     isAcceptable = false;
                     continue;
@@ -688,11 +698,6 @@
     }
 
     /**
-     * The tracer object for the debug logger.
-     */
-    private static final DebugTracer TRACER = getTracer();
-
-    /**
      * Creates an aggregation property definition builder.
      *
      * @param <C>
@@ -713,6 +718,8 @@
         return new Builder<C, S>(d, propertyName);
     }
 
+    private static final Logger debugLogger = LoggerFactory.getLogger(AggregationPropertyDefinition.class);
+
     // The active server-side referential integrity change listeners
     // associated with this property.
     private final Map<DN, List<ReferentialIntegrityChangeListener>> changeListeners = new HashMap<DN, List<ReferentialIntegrityChangeListener>>();
@@ -902,7 +909,7 @@
         ManagedObjectDefinitionI18NResource resource = ManagedObjectDefinitionI18NResource.getInstance();
         String property = "property." + getName() + ".syntax.aggregation.constraint-synopsis";
         try {
-            return resource.getLocalizableMessage(getManagedObjectDefinition(), property, locale);
+            return resource.getMessage(getManagedObjectDefinition(), property, locale);
         } catch (MissingResourceException e) {
             return null;
         }

--
Gitblit v1.10.0