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

Jean-Noel Rouvignac

03.46.2014 88cdfff756887b6d8d1cedc3bb6d5db82ecdac48

 opendj3-server-dev/src/ads/org/opends/admin/ads/ADSContext.java

@@ -27,6 +27,7 @@

package org.opends.admin.ads;

import static org.forgerock.util.Utils.*;
import static org.opends.messages.QuickSetupMessages.*;

import java.io.File;
@@ -65,10 +66,8 @@

import org.opends.admin.ads.util.ConnectionUtils;
import org.opends.quicksetup.Constants;
import org.opends.quicksetup.util.Utils;
import org.opends.server.schema.SchemaConstants;


/**
 * Class used to update and read the contents of the Administration Data.
 */
@@ -2642,8 +2641,7 @@
      LocalizableMessage msg = ERR_ADS_ADMINISTRATOR_MERGE.get(
          ConnectionUtils.getHostPort(adsCtx.getDirContext()),
          ConnectionUtils.getHostPort(getDirContext()),
          Utils.getStringFromCollection(notDefinedAdmins,
              Constants.LINE_SEPARATOR),
          joinAsString(Constants.LINE_SEPARATOR, notDefinedAdmins),
          ConnectionUtils.getHostPort(getDirContext()));
      throw new ADSContextException(ADSContextException.ErrorType.ERROR_MERGING,
          msg, null);