From 92b8b0e66ec0c29c0e020c9b4aa7680ba4732f9d Mon Sep 17 00:00:00 2001
From: Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Tue, 16 Jul 2013 07:57:52 +0000
Subject: [PATCH] Respected the Single Responsibility Principle by splitting findAndSetMatchingDN() method into findMatchedDN() and setMatchedDN() methods.
---
opends/src/server/org/opends/server/workflowelement/localbackend/LocalBackendDeleteOperation.java | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/opends/src/server/org/opends/server/workflowelement/localbackend/LocalBackendDeleteOperation.java b/opends/src/server/org/opends/server/workflowelement/localbackend/LocalBackendDeleteOperation.java
index 47e67f3..2e2d0cd 100644
--- a/opends/src/server/org/opends/server/workflowelement/localbackend/LocalBackendDeleteOperation.java
+++ b/opends/src/server/org/opends/server/workflowelement/localbackend/LocalBackendDeleteOperation.java
@@ -245,7 +245,7 @@
appendErrorMessage(ERR_DELETE_NO_SUCH_ENTRY
.get(String.valueOf(entryDN)));
- findAndSetMatchingDN(entryDN);
+ setMatchedDN(findMatchedDN(entryDN));
return;
}
@@ -416,7 +416,7 @@
}
}
- private void findAndSetMatchingDN(DN entryDN)
+ private DN findMatchedDN(DN entryDN)
{
try
{
@@ -425,8 +425,7 @@
{
if (DirectoryServer.entryExists(matchedDN))
{
- setMatchedDN(matchedDN);
- return;
+ return matchedDN;
}
matchedDN = matchedDN.getParentDNInSuffix();
@@ -439,6 +438,7 @@
TRACER.debugCaught(DebugLogLevel.ERROR, e);
}
}
+ return null;
}
/**
--
Gitblit v1.10.0