From 1fe92b6d4d1f2b4e3e9cb124278717afc0f85dcc Mon Sep 17 00:00:00 2001
From: Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Thu, 05 Feb 2015 13:17:11 +0000
Subject: [PATCH] OPENDJ-1242 (CR-5982) Enable dsconfig to generate doc for properties changed through subcommand options
---
opendj-config/src/main/java/org/forgerock/opendj/config/dsconfig/DSConfig.java | 35 +++++++++++++++++------------------
1 files changed, 17 insertions(+), 18 deletions(-)
diff --git a/opendj-config/src/main/java/org/forgerock/opendj/config/dsconfig/DSConfig.java b/opendj-config/src/main/java/org/forgerock/opendj/config/dsconfig/DSConfig.java
index 3a403fd..4d04547 100644
--- a/opendj-config/src/main/java/org/forgerock/opendj/config/dsconfig/DSConfig.java
+++ b/opendj-config/src/main/java/org/forgerock/opendj/config/dsconfig/DSConfig.java
@@ -157,23 +157,20 @@
@Override
public void appendUsage(StringBuilder sb, SubCommand sc, String argLongID) {
final SubCommandHandler sch = handlers.get(sc);
- final RelationDefinition<?, ?> rd = getRelationDefinition(sch);
- if (rd instanceof InstantiableRelationDefinition) {
- final PropertyDefinition<?> pd =
- ((InstantiableRelationDefinition<?, ?>) rd).getNamingPropertyDefinition();
- if (pd != null) {
- final AbstractManagedObjectDefinition<?, ?> defn = pd.getManagedObjectDefinition();
- final List<PropertyDefinition<?>> props =
- new ArrayList<PropertyDefinition<?>>(defn.getAllPropertyDefinitions());
- Collections.sort(props);
-
- final String propPrefix = getScriptName() + "-" + sc.getName() + "-" + argLongID + "-";
- sb.append(EOL);
- toSimpleList(props, propPrefix, sb);
- sb.append(EOL);
- toVariableList(props, defn, propPrefix, sb);
- }
+ if (sch instanceof HelpSubCommandHandler) {
+ return;
}
+ final RelationDefinition<?, ?> rd = getRelationDefinition(sch);
+ final AbstractManagedObjectDefinition<?, ?> defn = rd.getChildDefinition();
+ final List<PropertyDefinition<?>> props =
+ new ArrayList<PropertyDefinition<?>>(defn.getAllPropertyDefinitions());
+ Collections.sort(props);
+
+ final String propPrefix = getScriptName() + "-" + sc.getName() + "-" + argLongID + "-";
+ sb.append(EOL);
+ toSimpleList(props, propPrefix, sb);
+ sb.append(EOL);
+ toVariableList(props, defn, propPrefix, sb);
}
private RelationDefinition<?, ?> getRelationDefinition(final SubCommandHandler sch) {
@@ -245,8 +242,10 @@
b.append(indent).append(" </variablelist>").append(EOL);
b.append(indent).append(" </listitem>").append(EOL);
} else if (prop instanceof BooleanPropertyDefinition) {
- b.append(indent).append(" <listitem><para>true</para></listitem>").append(EOL);
- b.append(indent).append(" <listitem><para>false</para></listitem>").append(EOL);
+ b.append(indent).append(" <listitem>").append(EOL);
+ b.append(indent).append(" <para>true</para>").append(EOL);
+ b.append(indent).append(" <para>false</para>").append(EOL);
+ b.append(indent).append(" </listitem>").append(EOL);
} else {
b.append(indent).append(" <listitem>").append(EOL);
b.append(indent).append(" <para>");
--
Gitblit v1.10.0