Factorized a lot of String joining code into Utils.joinAsString(). Removed ...

Jean-Noel Rouvignac

04.49.2014 4a87d0ed96753944d40e2a7bc4fc64f342767649

 opendj3-server-dev/src/server/org/opends/server/extensions/ConfigFileHandler.java

@@ -1220,19 +1220,8 @@

      if (resultCode != ResultCode.SUCCESS)
      {
        LocalizableMessageBuilder buffer = new LocalizableMessageBuilder();
        if (! messages.isEmpty())
        {
          Iterator<LocalizableMessage> iterator = messages.iterator();
          buffer.append(iterator.next());
          while (iterator.hasNext())
          {
            buffer.append(".  ");
            buffer.append(iterator.next());
          }
        }

        LocalizableMessage message = ERR_CONFIG_FILE_ADD_APPLY_FAILED.get(buffer);
        String reasons = Utils.joinAsString(".  ", messages);
        LocalizableMessage message = ERR_CONFIG_FILE_ADD_APPLY_FAILED.get(reasons);
        throw new DirectoryException(resultCode, message);
      }
    }
@@ -1518,19 +1507,8 @@

      if (resultCode != ResultCode.SUCCESS)
      {
        LocalizableMessageBuilder buffer = new LocalizableMessageBuilder();
        if (! messages.isEmpty())
        {
          Iterator<LocalizableMessage> iterator = messages.iterator();
          buffer.append(iterator.next());
          while (iterator.hasNext())
          {
            buffer.append(".  ");
            buffer.append(iterator.next());
          }
        }

        LocalizableMessage message = ERR_CONFIG_FILE_MODIFY_APPLY_FAILED.get(buffer);
        String reasons = Utils.joinAsString(".  ", messages);
        LocalizableMessage message = ERR_CONFIG_FILE_MODIFY_APPLY_FAILED.get(reasons);
        throw new DirectoryException(resultCode, message);
      }
    }
@@ -3225,20 +3203,7 @@
    boolean       adminActionRequired = result.adminActionRequired();
    List<LocalizableMessage> messages            = result.getMessages();

    LocalizableMessageBuilder messageBuffer = new LocalizableMessageBuilder();
    if (messages != null)
    {
      for (LocalizableMessage s : messages)
      {
        if (messageBuffer.length() > 0)
        {
          messageBuffer.append("  ");
        }
        messageBuffer.append(s);
      }
    }


    String messageBuffer = Utils.joinAsString("  ", messages);
    if (resultCode != ResultCode.SUCCESS)
    {
      logger.error(ERR_CONFIG_CHANGE_RESULT_ERROR, className, methodName,