OPENDJ-1980 CR-6915 Cannot enable replication using dsreplication and Persi...

Fabio Pistolesi

15.22.2015 c16b8a7b8896914a3ab382fc835aefcf88c81e78

OPENDJ-1980 CR-6915 Cannot enable replication using dsreplication and Persistit

When looking for the list of suffixes to propose for replication, scan the entire list instead of stopping at first "private administrative" suffix.

 opendj-server-legacy/src/main/java/org/opends/server/tools/dsreplication/ReplicationCliMain.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2007-2010 Sun Microsystems, Inc.
 *      Portions Copyright 2011-2014 ForgeRock AS
 *      Portions Copyright 2011-2015 ForgeRock AS
 *      Portions Copyright 2012 profiq s.r.o.
 */
package org.opends.server.tools.dsreplication;
@@ -1684,17 +1684,16 @@
      Collection<String> availableSuffixes, Arg0 noSuffixAvailableMsg,
      Arg0 noSuffixSelectedMsg, Arg1<Object> confirmationMsgPromt)
  {
    while (suffixes.isEmpty())
    if (containsOnlySchemaOrAdminSuffix(availableSuffixes))
    {
      if (containsSchemaOrAdminSuffix(availableSuffixes))
      {
        // In interactive mode we do not propose to manage the
        // administration suffix.
      // In interactive mode we do not propose to manage the administration suffix.
        println();
        println(noSuffixAvailableMsg.get());
        break;
      return;
      }

    while (suffixes.isEmpty())
    {
      println();
      println(noSuffixSelectedMsg.get());
      boolean confirmationLimitReached = askConfirmations(confirmationMsgPromt, availableSuffixes, suffixes);
@@ -1706,17 +1705,17 @@
    }
  }

  private boolean containsSchemaOrAdminSuffix(Collection<String> suffixes)
  private boolean containsOnlySchemaOrAdminSuffix(Collection<String> suffixes)
  {
    for (String suffix : suffixes)
    {
      if (isSchemaOrAdminSuffix(suffix))
      if (!isSchemaOrAdminSuffix(suffix))
      {
        return true;
      }
    }
    return false;
  }
    }
    return true;
  }

  private boolean isSchemaOrAdminSuffix(String suffix)
  {
@@ -4687,10 +4686,9 @@
      {
        while (suffixes.isEmpty())
        {
          if (containsSchemaOrAdminSuffix(availableSuffixes))
          if (containsOnlySchemaOrAdminSuffix(availableSuffixes))
          {
            // In interactive mode we do not propose to manage the
            // administration suffix.
            // In interactive mode we do not propose to manage the administration suffix.
            if (displayErrors)
            {
              println();
@@ -4843,7 +4841,7 @@
        while (suffixes.isEmpty())
        {
          println();
          if (containsSchemaOrAdminSuffix(availableSuffixes))
          if (containsOnlySchemaOrAdminSuffix(availableSuffixes))
          {
            // In interactive mode we do not propose to manage the administration suffix.
            if (argParser.isInitializeAllReplicationSubcommand())