You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2014/11/24 12:16:05 UTC

[4/6] directory-fortress-enmasse git commit: Same formating and changes than for the other managers

Same formating and changes than for the other managers

Project: http://git-wip-us.apache.org/repos/asf/directory-fortress-enmasse/repo
Commit: http://git-wip-us.apache.org/repos/asf/directory-fortress-enmasse/commit/f88d1abb
Tree: http://git-wip-us.apache.org/repos/asf/directory-fortress-enmasse/tree/f88d1abb
Diff: http://git-wip-us.apache.org/repos/asf/directory-fortress-enmasse/diff/f88d1abb

Branch: refs/heads/master
Commit: f88d1abbef29668935184af38f5340e1fa4e065e
Parents: f35bf52
Author: Emmanuel Lécharny <el...@symas.com>
Authored: Sat Nov 22 10:11:46 2014 +0100
Committer: Emmanuel Lécharny <el...@symas.com>
Committed: Sat Nov 22 11:03:59 2014 +0100

----------------------------------------------------------------------
 .../fortress/rest/AbstractMgrImpl.java          |  62 ++
 .../directory/fortress/rest/AccessMgrImpl.java  |   2 +-
 .../directory/fortress/rest/AdminMgrImpl.java   |   2 +-
 .../directory/fortress/rest/AuditMgrImpl.java   |   2 +-
 .../directory/fortress/rest/ConfigMgrImpl.java  |   2 +-
 .../fortress/rest/DelegatedAccessMgrImpl.java   |   2 +-
 .../fortress/rest/DelegatedAdminMgrImpl.java    | 343 +++++-----
 .../fortress/rest/DelegatedReviewMgrImpl.java   | 136 ++--
 .../fortress/rest/PswdPolicyMgrImpl.java        | 157 ++---
 .../directory/fortress/rest/ReviewMgrImpl.java  | 649 ++++++++++---------
 10 files changed, 717 insertions(+), 640 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/directory-fortress-enmasse/blob/f88d1abb/src/main/java/org/apache/directory/fortress/rest/AbstractMgrImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/rest/AbstractMgrImpl.java b/src/main/java/org/apache/directory/fortress/rest/AbstractMgrImpl.java
new file mode 100644
index 0000000..94ba1ac
--- /dev/null
+++ b/src/main/java/org/apache/directory/fortress/rest/AbstractMgrImpl.java
@@ -0,0 +1,62 @@
+/*
+ *   Licensed to the Apache Software Foundation (ASF) under one
+ *   or more contributor license agreements.  See the NOTICE file
+ *   distributed with this work for additional information
+ *   regarding copyright ownership.  The ASF licenses this file
+ *   to you under the Apache License, Version 2.0 (the
+ *   "License"); you may not use this file except in compliance
+ *   with the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *   Unless required by applicable law or agreed to in writing,
+ *   software distributed under the License is distributed on an
+ *   "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ *   KIND, either express or implied.  See the License for the
+ *   specific language governing permissions and limitations
+ *   under the License.
+ *
+ */
+package org.apache.directory.fortress.rest;
+
+import org.apache.log4j.Logger;
+import org.apache.directory.fortress.core.GlobalErrIds;
+import org.apache.directory.fortress.core.SecurityException;
+import org.apache.directory.fortress.core.rest.FortResponse;
+
+/**
+ * An abstract class containing some methods shared by all the implementations.
+ * 
+ * @author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>
+
+ */
+abstract class AbstractMgrImpl
+{
+    /**
+     * Create an error message and log it.
+     * 
+     * @param response The {@link FortResponse} instance in which we will store the error message and ID
+     * @param LOG The Logger
+     * @param se The exception
+     */
+    protected void createError( FortResponse response, Logger LOG, SecurityException se )
+    {
+        LOG.info( "Caught " + se );
+        response.setErrorCode( se.getErrorId() );
+        response.setErrorMessage( se.getMessage() );
+    }
+    
+    
+    /**
+     * Creates a {@link FortResponse} instance where the error code is set with a default value.
+     * 
+     * @return The created instancd
+     */
+    protected FortResponse createResponse()
+    {
+        FortResponse response = new FortResponse();
+        response.setErrorCode( GlobalErrIds.NO_ERROR );
+
+        return response;
+    }
+}

http://git-wip-us.apache.org/repos/asf/directory-fortress-enmasse/blob/f88d1abb/src/main/java/org/apache/directory/fortress/rest/AccessMgrImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/rest/AccessMgrImpl.java b/src/main/java/org/apache/directory/fortress/rest/AccessMgrImpl.java
index 9763f56..6665389 100644
--- a/src/main/java/org/apache/directory/fortress/rest/AccessMgrImpl.java
+++ b/src/main/java/org/apache/directory/fortress/rest/AccessMgrImpl.java
@@ -39,7 +39,7 @@ import java.util.Set;
  *
  * @author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>
  */
-class AccessMgrImpl extends AbstractImpl
+class AccessMgrImpl extends AbstractMgrImpl
 {
     /** A logger for this class */
     private static final Logger LOG = Logger.getLogger( AccessMgrImpl.class.getName() );

http://git-wip-us.apache.org/repos/asf/directory-fortress-enmasse/blob/f88d1abb/src/main/java/org/apache/directory/fortress/rest/AdminMgrImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/rest/AdminMgrImpl.java b/src/main/java/org/apache/directory/fortress/rest/AdminMgrImpl.java
index a4c27bb..8856c5b 100644
--- a/src/main/java/org/apache/directory/fortress/rest/AdminMgrImpl.java
+++ b/src/main/java/org/apache/directory/fortress/rest/AdminMgrImpl.java
@@ -44,7 +44,7 @@ import org.apache.log4j.Logger;
  *
  * @author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>
  */
-class AdminMgrImpl extends AbstractImpl
+class AdminMgrImpl extends AbstractMgrImpl
 {
     /** A logger for this class */
     private static final Logger log = Logger.getLogger( AdminMgrImpl.class.getName() );

http://git-wip-us.apache.org/repos/asf/directory-fortress-enmasse/blob/f88d1abb/src/main/java/org/apache/directory/fortress/rest/AuditMgrImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/rest/AuditMgrImpl.java b/src/main/java/org/apache/directory/fortress/rest/AuditMgrImpl.java
index 2f49506..0ef3230 100644
--- a/src/main/java/org/apache/directory/fortress/rest/AuditMgrImpl.java
+++ b/src/main/java/org/apache/directory/fortress/rest/AuditMgrImpl.java
@@ -37,7 +37,7 @@ import java.util.List;
  *
  * @author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>
  */
-class AuditMgrImpl extends AbstractImpl
+class AuditMgrImpl extends AbstractMgrImpl
 {
     /** A logger for this class */
     private static final Logger log = Logger.getLogger( AuditMgrImpl.class.getName() );

http://git-wip-us.apache.org/repos/asf/directory-fortress-enmasse/blob/f88d1abb/src/main/java/org/apache/directory/fortress/rest/ConfigMgrImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/rest/ConfigMgrImpl.java b/src/main/java/org/apache/directory/fortress/rest/ConfigMgrImpl.java
index 3b35a5e..66ddcf6 100644
--- a/src/main/java/org/apache/directory/fortress/rest/ConfigMgrImpl.java
+++ b/src/main/java/org/apache/directory/fortress/rest/ConfigMgrImpl.java
@@ -35,7 +35,7 @@ import java.util.Properties;
  *
  * @author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>
  */
-class ConfigMgrImpl extends AbstractImpl
+class ConfigMgrImpl extends AbstractMgrImpl
 {
     private static final Logger log = Logger.getLogger( ConfigMgrImpl.class.getName() );
 

http://git-wip-us.apache.org/repos/asf/directory-fortress-enmasse/blob/f88d1abb/src/main/java/org/apache/directory/fortress/rest/DelegatedAccessMgrImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/rest/DelegatedAccessMgrImpl.java b/src/main/java/org/apache/directory/fortress/rest/DelegatedAccessMgrImpl.java
index 01af49e..2f63376 100644
--- a/src/main/java/org/apache/directory/fortress/rest/DelegatedAccessMgrImpl.java
+++ b/src/main/java/org/apache/directory/fortress/rest/DelegatedAccessMgrImpl.java
@@ -41,7 +41,7 @@ import java.util.Set;
  *
  * @author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>
  */
-class DelegatedAccessMgrImpl extends AbstractImpl
+class DelegatedAccessMgrImpl extends AbstractMgrImpl
 {
     /** A logger for this class */
     private static final Logger LOG = Logger.getLogger( DelegatedAccessMgrImpl.class.getName() );

http://git-wip-us.apache.org/repos/asf/directory-fortress-enmasse/blob/f88d1abb/src/main/java/org/apache/directory/fortress/rest/DelegatedAdminMgrImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/rest/DelegatedAdminMgrImpl.java b/src/main/java/org/apache/directory/fortress/rest/DelegatedAdminMgrImpl.java
index 3debbbe..4615a4c 100644
--- a/src/main/java/org/apache/directory/fortress/rest/DelegatedAdminMgrImpl.java
+++ b/src/main/java/org/apache/directory/fortress/rest/DelegatedAdminMgrImpl.java
@@ -37,10 +37,10 @@ import org.apache.log4j.Logger;
  *
  * @author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>
  */
-class DelegatedAdminMgrImpl
+class DelegatedAdminMgrImpl extends AbstractMgrImpl
 {
-    private static final String CLS_NM = DelegatedAdminMgrImpl.class.getName();
-    private static final Logger log = Logger.getLogger(CLS_NM);
+    /** A logger for this class */
+    private static final Logger LOG = Logger.getLogger( DelegatedAdminMgrImpl.class.getName() );
 
     /**
      * ************************************************************************************************************************************
@@ -48,339 +48,342 @@ class DelegatedAdminMgrImpl
      * **************************************************************************************************************************************
      */
 
-    FortResponse addAdminRole(FortRequest request)
+    /* No qualifier */ FortResponse addAdminRole( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             AdminRole inRole = (AdminRole) request.getEntity();
-            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance(request.getContextId());
-            delegatedAdminMgr.setAdmin(request.getSession());
-            AdminRole retRole = delegatedAdminMgr.addRole(inRole);
+            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance( request.getContextId() );
+            delegatedAdminMgr.setAdmin( request.getSession() );
+            AdminRole retRole = delegatedAdminMgr.addRole( inRole );
             response.setEntity(retRole);
-            response.setErrorCode(0);
         }
-        catch (org.apache.directory.fortress.core.SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se + " warnId=" + se.getErrorId());
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            LOG.info( "Caught " + se + " warnId=" + se.getErrorId() );
+            response.setErrorCode( se.getErrorId() );
+            response.setErrorMessage( se.getMessage() );
         }
+        
         return response;
     }
 
-    FortResponse deleteAdminRole(FortRequest request)
+    
+    /* No qualifier */ FortResponse deleteAdminRole( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             AdminRole inRole = (AdminRole) request.getEntity();
-            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance(request.getContextId());
-            delegatedAdminMgr.setAdmin(request.getSession());
-            delegatedAdminMgr.deleteRole(inRole);
+            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance( request.getContextId() );
+            delegatedAdminMgr.setAdmin( request.getSession() );
+            delegatedAdminMgr.deleteRole( inRole );
             response.setEntity(inRole);
-            response.setErrorCode(0);
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse updateAdminRole(FortRequest request)
+    
+    /* No qualifier */ FortResponse updateAdminRole( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             AdminRole inRole = (AdminRole) request.getEntity();
-            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance(request.getContextId());
-            delegatedAdminMgr.setAdmin(request.getSession());
-            AdminRole retRole = delegatedAdminMgr.updateRole(inRole);
+            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance( request.getContextId() );
+            delegatedAdminMgr.setAdmin( request.getSession() );
+            AdminRole retRole = delegatedAdminMgr.updateRole( inRole );
             response.setEntity(retRole);
-            response.setErrorCode(0);
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se + " errorId=" + se.getErrorId());
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            LOG.info( "Caught " + se + " errorId=" + se.getErrorId() );
+            response.setErrorCode( se.getErrorId() );
+            response.setErrorMessage( se.getMessage() );
         }
+        
         return response;
     }
 
-    FortResponse assignAdminUser(FortRequest request)
+    
+    /* No qualifier */ FortResponse assignAdminUser( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             UserAdminRole inRole = (UserAdminRole) request.getEntity();
-            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance(request.getContextId());
-            delegatedAdminMgr.setAdmin(request.getSession());
-            delegatedAdminMgr.assignUser(inRole);
+            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance( request.getContextId() );
+            delegatedAdminMgr.setAdmin( request.getSession() );
+            delegatedAdminMgr.assignUser( inRole );
             response.setEntity(inRole);
-            response.setErrorCode(0);
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse deassignAdminUser(FortRequest request)
+    
+    /* No qualifier */ FortResponse deassignAdminUser( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             UserAdminRole inRole = (UserAdminRole) request.getEntity();
-            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance(request.getContextId());
-            delegatedAdminMgr.setAdmin(request.getSession());
-            delegatedAdminMgr.deassignUser(inRole);
+            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance( request.getContextId() );
+            delegatedAdminMgr.setAdmin( request.getSession() );
+            delegatedAdminMgr.deassignUser( inRole );
             response.setEntity(inRole);
-            response.setErrorCode(0);
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+
         return response;
     }
 
-    FortResponse addAdminDescendant(FortRequest request)
+    
+    /* No qualifier */ FortResponse addAdminDescendant( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             AdminRoleRelationship relationship = (AdminRoleRelationship) request.getEntity();
-            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance(request.getContextId());
-            delegatedAdminMgr.setAdmin(request.getSession());
-            delegatedAdminMgr.addDescendant(relationship.getParent(), relationship.getChild());
-            response.setEntity(relationship);
-            response.setErrorCode(0);
+            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance( request.getContextId() );
+            delegatedAdminMgr.setAdmin( request.getSession() );
+            delegatedAdminMgr.addDescendant( relationship.getParent(), relationship.getChild() );
+            response.setEntity( relationship );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse addAdminAscendant(FortRequest request)
+    
+    /* No qualifier */ FortResponse addAdminAscendant( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             AdminRoleRelationship relationship = (AdminRoleRelationship) request.getEntity();
-            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance(request.getContextId());
-            delegatedAdminMgr.setAdmin(request.getSession());
-            delegatedAdminMgr.addAscendant(relationship.getChild(), relationship.getParent());
-            response.setEntity(relationship);
-            response.setErrorCode(0);
+            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance( request.getContextId() );
+            delegatedAdminMgr.setAdmin( request.getSession() );
+            delegatedAdminMgr.addAscendant( relationship.getChild(), relationship.getParent() );
+            response.setEntity( relationship );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+
         return response;
     }
 
-    FortResponse addAdminInheritance(FortRequest request)
+    
+    /* No qualifier */ FortResponse addAdminInheritance( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             AdminRoleRelationship relationship = (AdminRoleRelationship) request.getEntity();
-            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance(request.getContextId());
-            delegatedAdminMgr.setAdmin(request.getSession());
-            delegatedAdminMgr.addInheritance(relationship.getParent(), relationship.getChild());
-            response.setEntity(relationship);
-            response.setErrorCode(0);
+            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance( request.getContextId() );
+            delegatedAdminMgr.setAdmin( request.getSession() );
+            delegatedAdminMgr.addInheritance( relationship.getParent(), relationship.getChild() );
+            response.setEntity( relationship );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse deleteAdminInheritance(FortRequest request)
+    
+    /* No qualifier */ FortResponse deleteAdminInheritance( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             AdminRoleRelationship relationship = (AdminRoleRelationship) request.getEntity();
-            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance(request.getContextId());
-            delegatedAdminMgr.setAdmin(request.getSession());
-            delegatedAdminMgr.deleteInheritance(relationship.getParent(), relationship.getChild());
-            response.setEntity(relationship);
-            response.setErrorCode(0);
+            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance( request.getContextId() );
+            delegatedAdminMgr.setAdmin( request.getSession() );
+            delegatedAdminMgr.deleteInheritance( relationship.getParent(), relationship.getChild() );
+            response.setEntity( relationship );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse addOrg(FortRequest request)
+    
+    /* No qualifier */ FortResponse addOrg( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             OrgUnit inOrg = (OrgUnit) request.getEntity();
-            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance(request.getContextId());
-            delegatedAdminMgr.setAdmin(request.getSession());
-            OrgUnit retOrg = delegatedAdminMgr.add(inOrg);
+            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance( request.getContextId() );
+            delegatedAdminMgr.setAdmin( request.getSession() );
+            OrgUnit retOrg = delegatedAdminMgr.add( inOrg );
             response.setEntity(retOrg);
-            response.setErrorCode(0);
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse updateOrg(FortRequest request)
+    
+    /* No qualifier */ FortResponse updateOrg( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             OrgUnit inOrg = (OrgUnit) request.getEntity();
-            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance(request.getContextId());
-            delegatedAdminMgr.setAdmin(request.getSession());
-            OrgUnit retOrg = delegatedAdminMgr.update(inOrg);
+            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance( request.getContextId() );
+            delegatedAdminMgr.setAdmin( request.getSession() );
+            OrgUnit retOrg = delegatedAdminMgr.update( inOrg );
             response.setEntity(retOrg);
-            response.setErrorCode(0);
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse deleteOrg(FortRequest request)
+    
+    /* No qualifier */ FortResponse deleteOrg( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             OrgUnit inOrg = (OrgUnit) request.getEntity();
-            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance(request.getContextId());
-            delegatedAdminMgr.setAdmin(request.getSession());
-            OrgUnit retOrg = delegatedAdminMgr.delete(inOrg);
+            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance( request.getContextId() );
+            delegatedAdminMgr.setAdmin( request.getSession() );
+            OrgUnit retOrg = delegatedAdminMgr.delete( inOrg );
             response.setEntity(retOrg);
-            response.setErrorCode(0);
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse addOrgDescendant(FortRequest request)
+    
+    /* No qualifier */ FortResponse addOrgDescendant( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             OrgUnitRelationship relationship = (OrgUnitRelationship) request.getEntity();
-            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance(request.getContextId());
-            delegatedAdminMgr.setAdmin(request.getSession());
-            delegatedAdminMgr.addDescendant(relationship.getParent(), relationship.getChild());
-            response.setEntity(relationship);
-            response.setErrorCode(0);
+            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance( request.getContextId() );
+            delegatedAdminMgr.setAdmin( request.getSession() );
+            delegatedAdminMgr.addDescendant( relationship.getParent(), relationship.getChild() );
+            response.setEntity( relationship );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse addOrgAscendant(FortRequest request)
+    
+    /* No qualifier */ FortResponse addOrgAscendant( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             OrgUnitRelationship relationship = (OrgUnitRelationship) request.getEntity();
-            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance(request.getContextId());
-            delegatedAdminMgr.setAdmin(request.getSession());
-            delegatedAdminMgr.addAscendant(relationship.getChild(), relationship.getParent());
-            response.setEntity(relationship);
-            response.setErrorCode(0);
+            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance( request.getContextId() );
+            delegatedAdminMgr.setAdmin( request.getSession() );
+            delegatedAdminMgr.addAscendant( relationship.getChild(), relationship.getParent() );
+            response.setEntity( relationship );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse addOrgInheritance(FortRequest request)
+    
+    /* No qualifier */ FortResponse addOrgInheritance( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             OrgUnitRelationship relationship = (OrgUnitRelationship) request.getEntity();
-            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance(request.getContextId());
-            delegatedAdminMgr.setAdmin(request.getSession());
+            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance( request.getContextId() );
+            delegatedAdminMgr.setAdmin( request.getSession() );
             delegatedAdminMgr.addInheritance(relationship.getParent(), relationship.getChild());
-            response.setEntity(relationship);
-            response.setErrorCode(0);
+            response.setEntity( relationship );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+            
         return response;
     }
 
-    FortResponse deleteOrgInheritance(FortRequest request)
+    
+    /* No qualifier */ FortResponse deleteOrgInheritance( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             OrgUnitRelationship relationship = (OrgUnitRelationship) request.getEntity();
-            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance(request.getContextId());
-            delegatedAdminMgr.setAdmin(request.getSession());
-            delegatedAdminMgr.deleteInheritance(relationship.getParent(), relationship.getChild());
-            response.setEntity(relationship);
-            response.setErrorCode(0);
+            DelAdminMgr delegatedAdminMgr = DelAdminMgrFactory.createInstance( request.getContextId() );
+            delegatedAdminMgr.setAdmin( request.getSession() );
+            delegatedAdminMgr.deleteInheritance( relationship.getParent(), relationship.getChild() );
+            response.setEntity( relationship );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 }
\ No newline at end of file

http://git-wip-us.apache.org/repos/asf/directory-fortress-enmasse/blob/f88d1abb/src/main/java/org/apache/directory/fortress/rest/DelegatedReviewMgrImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/rest/DelegatedReviewMgrImpl.java b/src/main/java/org/apache/directory/fortress/rest/DelegatedReviewMgrImpl.java
index a228d72..dc3d8ae 100644
--- a/src/main/java/org/apache/directory/fortress/rest/DelegatedReviewMgrImpl.java
+++ b/src/main/java/org/apache/directory/fortress/rest/DelegatedReviewMgrImpl.java
@@ -37,10 +37,10 @@ import java.util.List;
  *
  * @author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>
  */
-class DelegatedReviewMgrImpl
+class DelegatedReviewMgrImpl extends AbstractMgrImpl
 {
-    private static final String CLS_NM = DelegatedReviewMgrImpl.class.getName();
-    private static final Logger log = Logger.getLogger(CLS_NM);
+    /** A logger for this class */
+    private static final Logger LOG = Logger.getLogger( DelegatedReviewMgrImpl.class.getName() );
 
     /**
      * ************************************************************************************************************************************
@@ -48,129 +48,127 @@ class DelegatedReviewMgrImpl
      * **************************************************************************************************************************************
      */
 
-    FortResponse readAdminRole(FortRequest request)
+    /* No qualifier */ FortResponse readAdminRole( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             AdminRole inRole = (AdminRole) request.getEntity();
-            DelReviewMgr delegatedReviewMgr = DelReviewMgrFactory.createInstance(request.getContextId());
-            AdminRole outRole = delegatedReviewMgr.readRole(inRole);
-            response.setEntity(outRole);
-            response.setErrorCode(0);
+            DelReviewMgr delegatedReviewMgr = DelReviewMgrFactory.createInstance( request.getContextId() );
+            AdminRole outRole = delegatedReviewMgr.readRole( inRole );
+            response.setEntity( outRole );
         }
-        catch (org.apache.directory.fortress.core.SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse findAdminRoles(FortRequest request)
+    
+    /* No qualifier */ FortResponse findAdminRoles( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             String searchVal = request.getValue();
-            DelReviewMgr delegatedReviewMgr = DelReviewMgrFactory.createInstance(request.getContextId());
-            delegatedReviewMgr.setAdmin(request.getSession());
-            List<AdminRole> outRoles = delegatedReviewMgr.findRoles(searchVal);
-            response.setEntities(outRoles);
-            response.setErrorCode(0);
-
+            DelReviewMgr delegatedReviewMgr = DelReviewMgrFactory.createInstance( request.getContextId() );
+            delegatedReviewMgr.setAdmin( request.getSession() );
+            List<AdminRole> outRoles = delegatedReviewMgr.findRoles( searchVal );
+            response.setEntities( outRoles );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse assignedAdminRoles(FortRequest request)
+    
+    /* No qualifier */ FortResponse assignedAdminRoles( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             User inUser = (User)request.getEntity();
-            DelReviewMgr delegatedReviewMgr = DelReviewMgrFactory.createInstance(request.getContextId());
-            delegatedReviewMgr.setAdmin(request.getSession());
-            List<UserAdminRole> uRoles = delegatedReviewMgr.assignedRoles(inUser);
-            response.setEntities(uRoles);
-            response.setErrorCode(0);
+            DelReviewMgr delegatedReviewMgr = DelReviewMgrFactory.createInstance( request.getContextId() );
+            delegatedReviewMgr.setAdmin( request.getSession() );
+            List<UserAdminRole> uRoles = delegatedReviewMgr.assignedRoles( inUser );
+            response.setEntities( uRoles );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse assignedAdminUsers(FortRequest request)
+    
+    /* No qualifier */ FortResponse assignedAdminUsers( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             AdminRole inRole = (AdminRole) request.getEntity();
-            DelReviewMgr delegatedReviewMgr = DelReviewMgrFactory.createInstance(request.getContextId());
-            delegatedReviewMgr.setAdmin(request.getSession());
-            List<User> users = delegatedReviewMgr.assignedUsers(inRole);
-            response.setEntities(users);
-            response.setErrorCode(0);
+            DelReviewMgr delegatedReviewMgr = DelReviewMgrFactory.createInstance( request.getContextId() );
+            delegatedReviewMgr.setAdmin( request.getSession() );
+            List<User> users = delegatedReviewMgr.assignedUsers( inRole );
+            response.setEntities( users );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse readOrg(FortRequest request)
+    
+    /* No qualifier */ FortResponse readOrg( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             OrgUnit inOrg = (OrgUnit) request.getEntity();
-            DelReviewMgr delegatedReviewMgr = DelReviewMgrFactory.createInstance(request.getContextId());
-            delegatedReviewMgr.setAdmin(request.getSession());
-            OrgUnit returnOrg = delegatedReviewMgr.read(inOrg);
-            response.setEntity(returnOrg);
-            response.setErrorCode(0);
+            DelReviewMgr delegatedReviewMgr = DelReviewMgrFactory.createInstance( request.getContextId() );
+            delegatedReviewMgr.setAdmin( request.getSession() );
+            OrgUnit returnOrg = delegatedReviewMgr.read( inOrg );
+            response.setEntity( returnOrg );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse searchOrg(FortRequest request)
+    
+    /* No qualifier */ FortResponse searchOrg( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             OrgUnit inOrg = (OrgUnit) request.getEntity();
-            DelReviewMgr delegatedReviewMgr = DelReviewMgrFactory.createInstance(request.getContextId());
-            delegatedReviewMgr.setAdmin(request.getSession());
-            List<OrgUnit> orgs = delegatedReviewMgr.search(inOrg.getType(), inOrg.getName());
-            response.setEntities(orgs);
-            response.setErrorCode(0);
+            DelReviewMgr delegatedReviewMgr = DelReviewMgrFactory.createInstance( request.getContextId() );
+            delegatedReviewMgr.setAdmin( request.getSession() );
+            List<OrgUnit> orgs = delegatedReviewMgr.search( inOrg.getType(), inOrg.getName() );
+            response.setEntities( orgs );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 }
\ No newline at end of file

http://git-wip-us.apache.org/repos/asf/directory-fortress-enmasse/blob/f88d1abb/src/main/java/org/apache/directory/fortress/rest/PswdPolicyMgrImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/rest/PswdPolicyMgrImpl.java b/src/main/java/org/apache/directory/fortress/rest/PswdPolicyMgrImpl.java
index 183297e..5dce451 100644
--- a/src/main/java/org/apache/directory/fortress/rest/PswdPolicyMgrImpl.java
+++ b/src/main/java/org/apache/directory/fortress/rest/PswdPolicyMgrImpl.java
@@ -34,159 +34,162 @@ import java.util.List;
  *
  * @author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>
  */
-class PswdPolicyMgrImpl
+class PswdPolicyMgrImpl extends AbstractMgrImpl
 {
-    private static final String CLS_NM = PswdPolicyMgrImpl.class.getName();
-    private static final Logger log = Logger.getLogger(CLS_NM);
+    /** A logger for this class */
+    private static final Logger LOG = Logger.getLogger( PswdPolicyMgrImpl.class.getName() );
 
     /**
      * ************************************************************************************************************************************
      * BEGIN PSWDPOLICYMGR
      * **************************************************************************************************************************************
      */
-    FortResponse addPolicy(FortRequest request)
+    /* No qualifier */ FortResponse addPolicy( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             PwPolicy inPolicy = (PwPolicy) request.getEntity();
-            PwPolicyMgr policyMgr = PwPolicyMgrFactory.createInstance(request.getContextId());
-            policyMgr.setAdmin(request.getSession());
-            policyMgr.add(inPolicy);
-            response.setEntity(inPolicy);
-            response.setErrorCode(0);
+            PwPolicyMgr policyMgr = PwPolicyMgrFactory.createInstance( request.getContextId() );
+            policyMgr.setAdmin( request.getSession() );
+            policyMgr.add( inPolicy );
+            response.setEntity( inPolicy );
         }
-        catch (org.apache.directory.fortress.core.SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse updatePolicy(FortRequest request)
+    
+    /* No qualifier */ FortResponse updatePolicy( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             PwPolicy inPolicy = (PwPolicy) request.getEntity();
-            PwPolicyMgr policyMgr = PwPolicyMgrFactory.createInstance(request.getContextId());
-            policyMgr.setAdmin(request.getSession());
-            policyMgr.update(inPolicy);
-            response.setEntity(inPolicy);
-            response.setErrorCode(0);
+            PwPolicyMgr policyMgr = PwPolicyMgrFactory.createInstance( request.getContextId() );
+            policyMgr.setAdmin( request.getSession() );
+            policyMgr.update( inPolicy );
+            response.setEntity( inPolicy );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse deletePolicy(FortRequest request)
+    
+    /* No qualifier */ FortResponse deletePolicy( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             PwPolicy inPolicy = (PwPolicy) request.getEntity();
-            PwPolicyMgr policyMgr = PwPolicyMgrFactory.createInstance(request.getContextId());
-            policyMgr.setAdmin(request.getSession());
-            policyMgr.delete(inPolicy);
-            response.setEntity(inPolicy);
-            response.setErrorCode(0);
+            PwPolicyMgr policyMgr = PwPolicyMgrFactory.createInstance( request.getContextId() );
+            policyMgr.setAdmin( request.getSession() );
+            policyMgr.delete( inPolicy );
+            response.setEntity( inPolicy );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse readPolicy(FortRequest request)
+    
+    /* No qualifier */ FortResponse readPolicy( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
         PwPolicy outPolicy;
+        
         try
         {
             PwPolicy inPolicy = (PwPolicy) request.getEntity();
-            PwPolicyMgr policyMgr = PwPolicyMgrFactory.createInstance(request.getContextId());
-            policyMgr.setAdmin(request.getSession());
-            outPolicy = policyMgr.read(inPolicy.getName());
-            response.setEntity(outPolicy);
-            response.setErrorCode(0);
+            PwPolicyMgr policyMgr = PwPolicyMgrFactory.createInstance( request.getContextId() );
+            policyMgr.setAdmin( request.getSession() );
+            outPolicy = policyMgr.read( inPolicy.getName() );
+            response.setEntity( outPolicy );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            response.setErrorCode( se.getErrorId() );
+            response.setErrorMessage( se.getMessage() );
         }
+        
         return response;
     }
 
-    FortResponse searchPolicy(FortRequest request)
+    
+    /* No qualifier */ FortResponse searchPolicy( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
         List<PwPolicy> policyList;
+        
         try
         {
             PwPolicy inPolicy = (PwPolicy) request.getEntity();
-            PwPolicyMgr policyMgr = PwPolicyMgrFactory.createInstance(request.getContextId());
-            policyMgr.setAdmin(request.getSession());
-            policyList = policyMgr.search(inPolicy.getName());
-            response.setEntities(policyList);
-            response.setErrorCode(0);
+            PwPolicyMgr policyMgr = PwPolicyMgrFactory.createInstance( request.getContextId() );
+            policyMgr.setAdmin( request.getSession() );
+            policyList = policyMgr.search( inPolicy.getName() );
+            response.setEntities( policyList );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            response.setErrorCode( se.getErrorId() );
+            response.setErrorMessage( se.getMessage() );
         }
+        
         return response;
     }
-
-    FortResponse updateUserPolicy(FortRequest request)
+    
+    
+    /* No qualifier */ FortResponse updateUserPolicy( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             PwPolicy inPolicy = (PwPolicy) request.getEntity();
-            PwPolicyMgr policyMgr = PwPolicyMgrFactory.createInstance(request.getContextId());
-            policyMgr.setAdmin(request.getSession());
+            PwPolicyMgr policyMgr = PwPolicyMgrFactory.createInstance( request.getContextId() );
+            policyMgr.setAdmin( request.getSession() );
             String userId = request.getValue();
-            policyMgr.updateUserPolicy(userId, inPolicy.getName());
-            response.setErrorCode(0);
+            policyMgr.updateUserPolicy( userId, inPolicy.getName() );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse deleteUserPolicy(FortRequest request)
+    
+    /* No qualifier */ FortResponse deleteUserPolicy( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            PwPolicyMgr policyMgr = PwPolicyMgrFactory.createInstance(request.getContextId());
-            policyMgr.setAdmin(request.getSession());
+            PwPolicyMgr policyMgr = PwPolicyMgrFactory.createInstance( request.getContextId() );
+            policyMgr.setAdmin( request.getSession() );
             String userId = request.getValue();
-            policyMgr.deletePasswordPolicy(userId);
-            response.setErrorCode(0);
+            policyMgr.deletePasswordPolicy( userId );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 }
\ No newline at end of file

http://git-wip-us.apache.org/repos/asf/directory-fortress-enmasse/blob/f88d1abb/src/main/java/org/apache/directory/fortress/rest/ReviewMgrImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/rest/ReviewMgrImpl.java b/src/main/java/org/apache/directory/fortress/rest/ReviewMgrImpl.java
index e6d98f8..40609b5 100644
--- a/src/main/java/org/apache/directory/fortress/rest/ReviewMgrImpl.java
+++ b/src/main/java/org/apache/directory/fortress/rest/ReviewMgrImpl.java
@@ -42,638 +42,649 @@ import java.util.Set;
  *
  * @author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>
  */
-class ReviewMgrImpl
+class ReviewMgrImpl extends AbstractMgrImpl
 {
-    private static final String CLS_NM = ReviewMgrImpl.class.getName();
-    private static final Logger log = Logger.getLogger(CLS_NM);
+    /** A logger for this class */
+    private static final Logger LOG = Logger.getLogger( ReviewMgrImpl.class.getName() );
 
-    FortResponse readPermission(FortRequest request)
+    /* No qualifier */  FortResponse readPermission( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             Permission inPerm = (Permission) request.getEntity();
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
-            Permission retPerm = reviewMgr.readPermission(inPerm);
-            response.setEntity(retPerm);
-            response.setErrorCode(0);
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
+            Permission retPerm = reviewMgr.readPermission( inPerm );
+            response.setEntity( retPerm );
         }
-        catch (org.apache.directory.fortress.core.SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse readPermObj(FortRequest request)
+    
+    /* No qualifier */  FortResponse readPermObj( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
             PermObj inObj = (PermObj) request.getEntity();
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
-            PermObj retObj = reviewMgr.readPermObj(inObj);
-            response.setEntity(retObj);
-            response.setErrorCode(0);
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
+            PermObj retObj = reviewMgr.readPermObj( inObj );
+            response.setEntity( retObj );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse findPermissions(FortRequest request)
+    
+    /* No qualifier */  FortResponse findPermissions( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             Permission inPerm = (Permission) request.getEntity();
-            List<Permission> perms = reviewMgr.findPermissions(inPerm);
-            response.setEntities(perms);
-            response.setErrorCode(0);
+            List<Permission> perms = reviewMgr.findPermissions( inPerm );
+            response.setEntities( perms );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse findPermObjs(FortRequest request)
+    
+    /* No qualifier */  FortResponse findPermObjs( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             PermObj inObj = (PermObj) request.getEntity();
             List<PermObj> objs = null;
-            if (VUtil.isNotNullOrEmpty(inObj.getOu()))
+            
+            if ( VUtil.isNotNullOrEmpty( inObj.getOu() ) )
             {
-                objs = reviewMgr.findPermObjs(new OrgUnit(inObj.getOu(), OrgUnit.Type.PERM));
+                objs = reviewMgr.findPermObjs( new OrgUnit( inObj.getOu(), OrgUnit.Type.PERM ) );
             }
             else
             {
-                objs = reviewMgr.findPermObjs(inObj);
+                objs = reviewMgr.findPermObjs( inObj );
             }
+            
             response.setEntities(objs);
-            response.setErrorCode(0);
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse readRole(FortRequest request)
+    
+    /* No qualifier */  FortResponse readRole( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             Role inRole = (Role) request.getEntity();
-            Role outRole = reviewMgr.readRole(inRole);
-            response.setEntity(outRole);
-            response.setErrorCode(0);
+            Role outRole = reviewMgr.readRole( inRole );
+            response.setEntity( outRole );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse findRoles(FortRequest request)
+    
+    /* No qualifier */  FortResponse findRoles( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             String searchValue = request.getValue();
-            if (request.getLimit() != null)
+            
+            if ( request.getLimit() != null )
             {
-                List<String> retRoles = reviewMgr.findRoles(searchValue, request.getLimit());
-                response.setValues(retRoles);
+                List<String> retRoles = reviewMgr.findRoles( searchValue, request.getLimit() );
+                response.setValues( retRoles );
             }
             else
             {
-                List<Role> roles = reviewMgr.findRoles(searchValue);
-                response.setEntities(roles);
+                List<Role> roles = reviewMgr.findRoles( searchValue );
+                response.setEntities( roles );
             }
-            response.setErrorCode(0);
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse readUserM(FortRequest request)
+    
+    /* No qualifier */  FortResponse readUserM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             User inUser = (User) request.getEntity();
-            User outUser = reviewMgr.readUser(inUser);
-            response.setEntity(outUser);
-            response.setErrorCode(0);
+            User outUser = reviewMgr.readUser( inUser );
+            response.setEntity( outUser );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse findUsersM(FortRequest request)
+    
+    /* No qualifier */  FortResponse findUsersM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             User inUser = (User) request.getEntity();
-            if (request.getLimit() != null)
+            
+            if ( request.getLimit() != null )
             {
-                List<String> retUsers = reviewMgr.findUsers(inUser, request.getLimit());
-                response.setValues(retUsers);
+                List<String> retUsers = reviewMgr.findUsers( inUser, request.getLimit() );
+                response.setValues( retUsers );
             }
             else
             {
                 List<User> retUsers;
-                if (VUtil.isNotNullOrEmpty(inUser.getOu()))
+                
+                if ( VUtil.isNotNullOrEmpty( inUser.getOu() ) )
                 {
-                    retUsers = reviewMgr.findUsers(new OrgUnit(inUser.getOu(), OrgUnit.Type.USER));
+                    retUsers = reviewMgr.findUsers( new OrgUnit( inUser.getOu(), OrgUnit.Type.USER ) );
                 }
                 else
                 {
-                    retUsers = reviewMgr.findUsers(inUser);
+                    retUsers = reviewMgr.findUsers( inUser );
                 }
-                response.setEntities(retUsers);
+                
+                response.setEntities( retUsers );
             }
-            response.setErrorCode(0);
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse assignedUsersM(FortRequest request)
+    
+    /* No qualifier */  FortResponse assignedUsersM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             Role inRole = (Role) request.getEntity();
-            if (request.getLimit() != null)
+            
+            if ( request.getLimit() != null )
             {
-                List<String> retUsers = reviewMgr.assignedUsers(inRole, request.getLimit());
-                response.setValues(retUsers);
+                List<String> retUsers = reviewMgr.assignedUsers( inRole, request.getLimit() );
+                response.setValues( retUsers );
             }
             else
             {
-                List<User> users = reviewMgr.assignedUsers(inRole);
-                response.setEntities(users);
-                response.setEntities(users);
+                List<User> users = reviewMgr.assignedUsers( inRole );
+                response.setEntities( users );
+                response.setEntities( users );
             }
-            response.setErrorCode(0);
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse assignedRolesM(FortRequest request)
+    
+    /* No qualifier */  FortResponse assignedRolesM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
-            if (VUtil.isNotNullOrEmpty(request.getValue()))
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
+            
+            if ( VUtil.isNotNullOrEmpty( request.getValue() ) )
             {
                 String userId = request.getValue();
-                List<String> retRoles = reviewMgr.assignedRoles(userId);
-                response.setValues(retRoles);
+                List<String> retRoles = reviewMgr.assignedRoles( userId );
+                response.setValues( retRoles );
             }
             else
             {
                 User inUser = (User) request.getEntity();
-                List<UserRole> uRoles = reviewMgr.assignedRoles(inUser);
-                response.setEntities(uRoles);
+                List<UserRole> uRoles = reviewMgr.assignedRoles( inUser );
+                response.setEntities( uRoles );
             }
-            response.setErrorCode(0);
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse authorizedUsersM(FortRequest request)
+    
+    /* No qualifier */  FortResponse authorizedUsersM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             Role inRole = (Role) request.getEntity();
-            List<User> users = reviewMgr.authorizedUsers(inRole);
-            response.setEntities(users);
-            response.setErrorCode(0);
+            List<User> users = reviewMgr.authorizedUsers( inRole );
+            response.setEntities( users );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse authorizedRoleM(FortRequest request)
+    
+    /* No qualifier */  FortResponse authorizedRoleM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             User inUser = (User) request.getEntity();
-            Set<String> outSet = reviewMgr.authorizedRoles(inUser);
-            response.setValueSet(outSet);
-            response.setErrorCode(0);
+            Set<String> outSet = reviewMgr.authorizedRoles( inUser );
+            response.setValueSet( outSet );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse permissionRolesM(FortRequest request)
+    
+    /* No qualifier */  FortResponse permissionRolesM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             Permission inPerm = (Permission) request.getEntity();
-            List<String> outList = reviewMgr.permissionRoles(inPerm);
-            response.setValues(outList);
-            response.setErrorCode(0);
+            List<String> outList = reviewMgr.permissionRoles( inPerm );
+            response.setValues( outList );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse authorizedPermissionRolesM(FortRequest request)
+    
+    /* No qualifier */  FortResponse authorizedPermissionRolesM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             Permission inPerm = (Permission) request.getEntity();
-            Set<String> outSet = reviewMgr.authorizedPermissionRoles(inPerm);
-            response.setValueSet(outSet);
-            response.setErrorCode(0);
+            Set<String> outSet = reviewMgr.authorizedPermissionRoles( inPerm );
+            response.setValueSet( outSet );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse permissionUsersM(FortRequest request)
+    
+    /* No qualifier */  FortResponse permissionUsersM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
             Permission inPerm = (Permission) request.getEntity();
-            List<String> outList = reviewMgr.permissionUsers(inPerm);
-            response.setValues(outList);
-            response.setErrorCode(0);
+            List<String> outList = reviewMgr.permissionUsers( inPerm );
+            response.setValues( outList );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse authorizedPermissionUsersM(FortRequest request)
+    
+    /* No qualifier */  FortResponse authorizedPermissionUsersM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
             Permission inPerm = (Permission) request.getEntity();
-            Set<String> outSet = reviewMgr.authorizedPermissionUsers(inPerm);
-            response.setValueSet(outSet);
-            response.setErrorCode(0);
+            Set<String> outSet = reviewMgr.authorizedPermissionUsers( inPerm );
+            response.setValueSet( outSet );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse userPermissionsM(FortRequest request)
+    
+    /* No qualifier */  FortResponse userPermissionsM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             User inUser = (User) request.getEntity();
-            List<Permission> perms = reviewMgr.userPermissions(inUser);
-            response.setEntities(perms);
-            response.setErrorCode(0);
+            List<Permission> perms = reviewMgr.userPermissions( inUser );
+            response.setEntities( perms );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse rolePermissionsM(FortRequest request)
+    
+    /* No qualifier */  FortResponse rolePermissionsM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             Role inRole = (Role) request.getEntity();
-            List<Permission> perms = reviewMgr.rolePermissions(inRole);
-            response.setEntities(perms);
-            response.setErrorCode(0);
+            List<Permission> perms = reviewMgr.rolePermissions( inRole );
+            response.setEntities( perms );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse ssdRoleSetsM(FortRequest request)
+    
+    /* No qualifier */  FortResponse ssdRoleSetsM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             Role inRole = (Role) request.getEntity();
-            List<SDSet> outSets = reviewMgr.ssdRoleSets(inRole);
-            response.setEntities(outSets);
-            response.setErrorCode(0);
+            List<SDSet> outSets = reviewMgr.ssdRoleSets( inRole );
+            response.setEntities( outSets );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse ssdRoleSetM(FortRequest request)
+    
+    /* No qualifier */  FortResponse ssdRoleSetM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             SDSet inSet = (SDSet) request.getEntity();
-            SDSet outSet = reviewMgr.ssdRoleSet(inSet);
-            response.setEntity(outSet);
-            response.setErrorCode(0);
+            SDSet outSet = reviewMgr.ssdRoleSet( inSet );
+            response.setEntity( outSet );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse ssdRoleSetRolesM(FortRequest request)
+    
+    /* No qualifier */  FortResponse ssdRoleSetRolesM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
             SDSet inSet = (SDSet) request.getEntity();
-            Set<String> outSet = reviewMgr.ssdRoleSetRoles(inSet);
-            response.setValueSet(outSet);
-            response.setErrorCode(0);
+            Set<String> outSet = reviewMgr.ssdRoleSetRoles( inSet );
+            response.setValueSet( outSet );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse ssdRoleSetCardinalityM(FortRequest request)
+    
+    /* No qualifier */  FortResponse ssdRoleSetCardinalityM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
             SDSet inSet = (SDSet) request.getEntity();
-            int cardinality = reviewMgr.ssdRoleSetCardinality(inSet);
-            inSet.setCardinality(cardinality);
-            response.setEntity(inSet);
-            response.setErrorCode(0);
+            int cardinality = reviewMgr.ssdRoleSetCardinality( inSet );
+            inSet.setCardinality( cardinality );
+            response.setEntity( inSet );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
+            LOG.info( "Caught " + se );
         }
+        
         return response;
     }
 
-    FortResponse ssdSetsM(FortRequest request)
+    
+    /* No qualifier */  FortResponse ssdSetsM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             SDSet inSdSet = (SDSet) request.getEntity();
-            List<SDSet> outSets = reviewMgr.ssdSets(inSdSet);
-            response.setEntities(outSets);
-            response.setErrorCode(0);
+            List<SDSet> outSets = reviewMgr.ssdSets( inSdSet );
+            response.setEntities( outSets );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse dsdRoleSetsM(FortRequest request)
+    
+    /* No qualifier */  FortResponse dsdRoleSetsM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             Role inRole = (Role) request.getEntity();
-            List<SDSet> outSets = reviewMgr.dsdRoleSets(inRole);
-            response.setEntities(outSets);
-            response.setErrorCode(0);
+            List<SDSet> outSets = reviewMgr.dsdRoleSets( inRole );
+            response.setEntities( outSets );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse dsdRoleSetM(FortRequest request)
+    
+    /* No qualifier */  FortResponse dsdRoleSetM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             SDSet inSet = (SDSet) request.getEntity();
-            SDSet outSet = reviewMgr.dsdRoleSet(inSet);
-            response.setEntity(outSet);
-            response.setErrorCode(0);
+            SDSet outSet = reviewMgr.dsdRoleSet( inSet );
+            response.setEntity( outSet );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse dsdRoleSetRolesM(FortRequest request)
+    
+    /* No qualifier */  FortResponse dsdRoleSetRolesM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
             SDSet inSet = (SDSet) request.getEntity();
-            Set<String> outSet = reviewMgr.dsdRoleSetRoles(inSet);
-            response.setValueSet(outSet);
-            response.setErrorCode(0);
+            Set<String> outSet = reviewMgr.dsdRoleSetRoles( inSet );
+            response.setValueSet( outSet );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 
-    FortResponse dsdRoleSetCardinalityM(FortRequest request)
+    
+    /* No qualifier */  FortResponse dsdRoleSetCardinalityM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+        
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
             SDSet inSet = (SDSet) request.getEntity();
-            int cardinality = reviewMgr.dsdRoleSetCardinality(inSet);
-            inSet.setCardinality(cardinality);
-            response.setEntity(inSet);
-            response.setErrorCode(0);
+            int cardinality = reviewMgr.dsdRoleSetCardinality( inSet );
+            inSet.setCardinality( cardinality );
+            response.setEntity( inSet );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
+            LOG.info( "Caught " + se );
         }
+        
         return response;
     }
 
-    FortResponse dsdSetsM(FortRequest request)
+    
+    /* No qualifier */  FortResponse dsdSetsM( FortRequest request )
     {
-        FortResponse response = new FortResponse();
+        FortResponse response = createResponse();
+
         try
         {
-            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance(request.getContextId());
-            reviewMgr.setAdmin(request.getSession());
+            ReviewMgr reviewMgr = ReviewMgrFactory.createInstance( request.getContextId() );
+            reviewMgr.setAdmin( request.getSession() );
             SDSet inSdSet = (SDSet) request.getEntity();
-            List<SDSet> outSets = reviewMgr.dsdSets(inSdSet);
-            response.setEntities(outSets);
-            response.setErrorCode(0);
+            List<SDSet> outSets = reviewMgr.dsdSets( inSdSet );
+            response.setEntities( outSets );
         }
-        catch (SecurityException se)
+        catch ( SecurityException se )
         {
-            log.info(CLS_NM + " caught " + se);
-            response.setErrorCode(se.getErrorId());
-            response.setErrorMessage(se.getMessage());
+            createError( response, LOG, se );
         }
+        
         return response;
     }
 }
\ No newline at end of file