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/05 18:33:15 UTC
[08/20] git commit: Added constants for some method name
Added constants for some method name
Project: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/repo
Commit: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/commit/8fbf3614
Tree: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/tree/8fbf3614
Diff: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/diff/8fbf3614
Branch: refs/heads/master
Commit: 8fbf3614ef803c2fa3b6419bafffb18202d96a4b
Parents: c1d87e8
Author: Emmanuel Lécharny <el...@symas.com>
Authored: Sun Nov 2 17:00:41 2014 +0100
Committer: Emmanuel Lécharny <el...@symas.com>
Committed: Sun Nov 2 17:00:41 2014 +0100
----------------------------------------------------------------------
.../fortress/core/rbac/DelAdminMgrImpl.java | 18 +++++++++---------
1 file changed, 9 insertions(+), 9 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/8fbf3614/src/main/java/org/apache/directory/fortress/core/rbac/DelAdminMgrImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/rbac/DelAdminMgrImpl.java b/src/main/java/org/apache/directory/fortress/core/rbac/DelAdminMgrImpl.java
index ea6291c..ab6f3a3 100755
--- a/src/main/java/org/apache/directory/fortress/core/rbac/DelAdminMgrImpl.java
+++ b/src/main/java/org/apache/directory/fortress/core/rbac/DelAdminMgrImpl.java
@@ -51,6 +51,8 @@ import java.util.Set;
public final class DelAdminMgrImpl extends Manageable implements DelAdminMgr
{
private static final String CLS_NM = DelAdminMgrImpl.class.getName();
+ private static final String ADD_OU_METHOD = ".addOU";
+ private static final String FULL_ADD_OU_METHOD = CLS_NM + "." + ADD_OU_METHOD;
private static final OrgUnitP ouP = new OrgUnitP();
private static final AdminRoleP admRP = new AdminRoleP();
private static final PermP permP = new PermP();
@@ -332,18 +334,16 @@ public final class DelAdminMgrImpl extends Manageable implements DelAdminMgr
*
* @param entity contains OrgUnit name and type.
* @return OrgUnit contains reference to entity added.
- * @throws org.apache.directory.fortress.core.SecurityException
- * in the event of data validation or system error.
+ * @throws SecurityException in the event of data validation or system error.
*/
@Override
- public OrgUnit add(OrgUnit entity)
- throws SecurityException
+ public OrgUnit add(OrgUnit entity) throws SecurityException
{
- String methodName = "addOU";
- assertContext(CLS_NM, methodName, entity, GlobalErrIds.ORG_NULL);
- setEntitySession(CLS_NM, methodName, entity);
- VUtil.assertNotNull(entity.getType(), GlobalErrIds.ORG_TYPE_NULL, CLS_NM + "." + methodName);
- return ouP.add(entity);
+ assertContext( ADD_OU_METHOD, entity, GlobalErrIds.ORG_NULL );
+ setEntitySession( CLS_NM, ADD_OU_METHOD, entity);
+ VUtil.assertNotNull( entity.getType(), GlobalErrIds.ORG_TYPE_NULL, FULL_ADD_OU_METHOD );
+
+ return ouP.add( entity );
}
/**