From de19fe2956d5a4b3817834191c51989664f8c47c Mon Sep 17 00:00:00 2001
From: matthew_swift <matthew_swift@localhost>
Date: Mon, 03 Sep 2007 20:36:20 +0000
Subject: [PATCH] Fix issue 1443: Admin framework: improve error logging in server

---
 opends/src/server/org/opends/server/admin/IllegalPropertyValueStringException.java |   29 +++++++++++++++--------------
 1 files changed, 15 insertions(+), 14 deletions(-)

diff --git a/opends/src/server/org/opends/server/admin/IllegalPropertyValueStringException.java b/opends/src/server/org/opends/server/admin/IllegalPropertyValueStringException.java
index 6c062ea..bf8db42 100644
--- a/opends/src/server/org/opends/server/admin/IllegalPropertyValueStringException.java
+++ b/opends/src/server/org/opends/server/admin/IllegalPropertyValueStringException.java
@@ -29,9 +29,15 @@
 
 
 
+import static org.opends.messages.AdminMessages.*;
+
+import org.opends.messages.Message;
+
+
+
 /**
- * Thrown to indicate that a property value string was invalid according to its
- * associated property definition.
+ * Thrown to indicate that a property value string was invalid
+ * according to its associated property definition.
  */
 public class IllegalPropertyValueStringException extends PropertyException {
 
@@ -48,14 +54,14 @@
   /**
    * Create a new illegal property value string exception.
    *
-   * @param d
+   * @param pd
    *          The property definition.
    * @param value
    *          The illegal property value string.
    */
-  public IllegalPropertyValueStringException(PropertyDefinition<?> d,
+  public IllegalPropertyValueStringException(PropertyDefinition<?> pd,
       String value) {
-    super(d);
+    super(pd, createMessage(pd, value));
     this.value = value;
   }
 
@@ -72,17 +78,12 @@
 
 
 
-  /**
-   * {@inheritDoc}
-   */
-  @Override
-  public String getMessage() {
-    String msg = "The string value \"%s\" is not a valid value for the "
-        + "property \"%s\", which must have the following syntax: %s";
-    PropertyDefinition<?> pd = getPropertyDefinition();
+  // Create the message.
+  private static Message createMessage(PropertyDefinition<?> pd, String value) {
     PropertyDefinitionUsageBuilder builder = new PropertyDefinitionUsageBuilder(
         true);
-    return String.format(msg, value, pd.getName(), builder.getUsage(pd));
+    return ERR_ILLEGAL_PROPERTY_VALUE_STRING_EXCEPTION.get(value, pd.getName(),
+        builder.getUsage(pd));
   }
 
 }

--
Gitblit v1.10.0