From 288f117ffff3921da19f509c13fca924797515bb Mon Sep 17 00:00:00 2001
From: Violette Roche-Montane <violette.roche-montane@forgerock.com>
Date: Thu, 24 Apr 2014 13:38:24 +0000
Subject: [PATCH] OPENDJ-1343 Migrate dsconfig - Removed some printVerbose() and replaced them by println to fix a minor bug in which the final action - e.g: modifying an element and apply; did not display a success (or failure) message in interactive mode.
---
opendj3-server-dev/src/server/org/opends/server/tools/dsconfig/CreateSubCommandHandler.java | 20 ++++++++++----------
1 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/opendj3-server-dev/src/server/org/opends/server/tools/dsconfig/CreateSubCommandHandler.java b/opendj3-server-dev/src/server/org/opends/server/tools/dsconfig/CreateSubCommandHandler.java
index f222595..f0f6f67 100644
--- a/opendj3-server-dev/src/server/org/opends/server/tools/dsconfig/CreateSubCommandHandler.java
+++ b/opendj3-server-dev/src/server/org/opends/server/tools/dsconfig/CreateSubCommandHandler.java
@@ -739,8 +739,8 @@
if (result.isQuit()) {
if (!app.isMenuDrivenMode()) {
// User chose to cancel any changes.
- LocalizableMessage msg = INFO_DSCFG_CONFIRM_CREATE_FAIL.get(ufn);
- app.printVerboseMessage(msg);
+ app.println();
+ app.println(INFO_DSCFG_CONFIRM_CREATE_FAIL.get(ufn));
}
return MenuResult.quit();
} else if (result.isCancel()) {
@@ -756,9 +756,8 @@
if (app.isInteractive() || app.isVerbose())
{
app.println();
+ app.println(INFO_DSCFG_CONFIRM_CREATE_SUCCESS.get(ufn));
}
- LocalizableMessage msg = INFO_DSCFG_CONFIRM_CREATE_SUCCESS.get(ufn);
- app.printVerboseMessage(msg);
if (handler != null) {
for (PropertyEditorModification<?> mod : editor.getModifications()) {
@@ -810,16 +809,16 @@
.getMessageObject(), e);
}
} catch (ErrorResultException e) {
- LocalizableMessage msg = ERR_DSCFG_ERROR_CREATE_CE.get(ufn, e.getMessage());
+ final LocalizableMessage msg = ERR_DSCFG_ERROR_CREATE_CE.get(ufn, e.getMessage());
if (app.isInteractive()) {
app.println();
app.printVerboseMessage(msg);
return MenuResult.cancel();
} else {
- throw new ClientException(ReturnCode.OTHER, msg);
+ throw new ClientException(ReturnCode.CLIENT_SIDE_SERVER_DOWN, msg);
}
} catch (ManagedObjectAlreadyExistsException e) {
- LocalizableMessage msg = ERR_DSCFG_ERROR_CREATE_MOAEE.get(ufn);
+ final LocalizableMessage msg = ERR_DSCFG_ERROR_CREATE_MOAEE.get(ufn);
if (app.isInteractive()) {
app.println();
app.printVerboseMessage(msg);
@@ -1194,8 +1193,8 @@
if (result.isQuit()) {
if (!app.isMenuDrivenMode()) {
// User chose to cancel creation.
- LocalizableMessage msg = INFO_DSCFG_CONFIRM_CREATE_FAIL.get(ufn);
- app.printVerboseMessage(msg);
+ app.println();
+ app.println(INFO_DSCFG_CONFIRM_CREATE_FAIL.get(ufn));
}
return MenuResult.quit();
} else if (result.isCancel()) {
@@ -1257,7 +1256,8 @@
} else {
// Must be quit.
if (!app.isMenuDrivenMode()) {
- app.printVerboseMessage(INFO_DSCFG_CONFIRM_CREATE_FAIL.get(ufn));
+ app.println();
+ app.println(INFO_DSCFG_CONFIRM_CREATE_FAIL.get(ufn));
}
return MenuResult.quit();
}
--
Gitblit v1.10.0