Respected the Single Responsibility Principle by splitting findAndSetMatchi...

Jean-Noel Rouvignac

16.57.2013 92b8b0e66ec0c29c0e020c9b4aa7680ba4732f9d

 opends/src/server/org/opends/server/extensions/PasswordModifyExtendedOperation.java

@@ -124,7 +124,7 @@
  /** The reference to the identity mapper. */
  private IdentityMapper<?> identityMapper;

  /** The default set of supported control OIDs for this extended */
  /** The default set of supported control OIDs for this extended. */
  private Set<String> supportedControlOIDs = new HashSet<String>(0);

  /**
@@ -1242,7 +1242,7 @@
                        String.valueOf(entryDN)));

        // See if one of the entry's ancestors exists.
        findAndSetMatchingDN(operation, entryDN);
        operation.setMatchedDN(findMatchedDN(entryDN));
        return null;
      }

@@ -1264,7 +1264,7 @@
    }
  }

  private void findAndSetMatchingDN(Operation operation, DN entryDN)
  private DN findMatchedDN(DN entryDN)
  {
    try
    {
@@ -1273,8 +1273,7 @@
      {
        if (DirectoryServer.entryExists(matchedDN))
        {
          operation.setMatchedDN(matchedDN);
          return;
          return matchedDN;
        }

        matchedDN = matchedDN.getParentDNInSuffix();
@@ -1287,6 +1286,7 @@
        TRACER.debugCaught(DebugLogLevel.ERROR, e);
      }
    }
    return null;
  }

  /**