Fix for issue #2574 (properties file do not take bindPasswordFile into acco...

lutoff

09.25.2007 0d3aa40c87adbaeaf549e4b0df81048f15e984d0

Fix for issue #2574 (properties file do not take bindPasswordFile into account)
In some cases, "addValue" method is not 'self-contained'. The
"valueIsAcceptable' method has to be called before "addValue" (and has
some mandatory side effect, in FileBaedArgument at least).

 opends/src/server/org/opends/server/util/args/ArgumentParser.java

@@ -1173,7 +1173,8 @@
        {
          String value = argumentProperties.getProperty(a.getPropertyName()
              .toLowerCase());
          if (value != null)
          MessageBuilder invalidReason =  new MessageBuilder();
          if ((value != null) && (a.valueIsAcceptable(value, invalidReason)))
          {
            a.addValue(value);
            if (a.needsValue())

 opends/src/server/org/opends/server/util/args/SubCommandArgumentParser.java

@@ -1265,7 +1265,8 @@
        {
          String value = argumentProperties.getProperty(a.getPropertyName()
              .toLowerCase());
          if (value != null)
          MessageBuilder invalidReason =  new MessageBuilder();
          if ( (value != null) && (a.valueIsAcceptable(value, invalidReason)))
          {
            a.addValue(value);
            if (a.needsValue())
@@ -1310,7 +1311,8 @@
          {
            String value = argumentProperties.getProperty(a.getPropertyName()
                .toLowerCase());
            if (value != null)
            MessageBuilder invalidReason =  new MessageBuilder();
            if ((value != null) && (a.valueIsAcceptable(value, invalidReason)))
            {
              a.addValue(value);
              if (a.needsValue())