You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by sm...@apache.org on 2016/10/10 15:07:43 UTC
directory-fortress-core git commit: FC-191 - make constructors private
Repository: directory-fortress-core
Updated Branches:
refs/heads/master 50422f735 -> 0e38f216b
FC-191 - make constructors private
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/0e38f216
Tree: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/tree/0e38f216
Diff: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/diff/0e38f216
Branch: refs/heads/master
Commit: 0e38f216b923ecd60599b08bc557f0c26e21e80b
Parents: 50422f7
Author: Shawn McKinney <sm...@apache.org>
Authored: Mon Oct 10 09:46:17 2016 -0500
Committer: Shawn McKinney <sm...@apache.org>
Committed: Mon Oct 10 09:46:17 2016 -0500
----------------------------------------------------------------------
.../java/org/apache/directory/fortress/core/impl/HierUtil.java | 2 +-
.../java/org/apache/directory/fortress/core/impl/PsoUtil.java | 2 +-
.../java/org/apache/directory/fortress/core/impl/RoleUtil.java | 2 +-
.../java/org/apache/directory/fortress/core/impl/UserDAO.java | 6 +++---
4 files changed, 6 insertions(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0e38f216/src/main/java/org/apache/directory/fortress/core/impl/HierUtil.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/impl/HierUtil.java b/src/main/java/org/apache/directory/fortress/core/impl/HierUtil.java
index a608778..3381741 100755
--- a/src/main/java/org/apache/directory/fortress/core/impl/HierUtil.java
+++ b/src/main/java/org/apache/directory/fortress/core/impl/HierUtil.java
@@ -75,7 +75,7 @@ final class HierUtil
/**
* The 'Type' attribute corresponds to what type of hierarchy is being referred to.
*/
- static enum Type
+ enum Type
{
ROLE,
ARLE,
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0e38f216/src/main/java/org/apache/directory/fortress/core/impl/PsoUtil.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/impl/PsoUtil.java b/src/main/java/org/apache/directory/fortress/core/impl/PsoUtil.java
index 4678e7a..c2b9467 100755
--- a/src/main/java/org/apache/directory/fortress/core/impl/PsoUtil.java
+++ b/src/main/java/org/apache/directory/fortress/core/impl/PsoUtil.java
@@ -84,7 +84,7 @@ final class PsoUtil
return sINSTANCE;
}
- public PsoUtil(){
+ private PsoUtil(){
init();
}
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0e38f216/src/main/java/org/apache/directory/fortress/core/impl/RoleUtil.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/impl/RoleUtil.java b/src/main/java/org/apache/directory/fortress/core/impl/RoleUtil.java
index be2f45b..e9edf66 100755
--- a/src/main/java/org/apache/directory/fortress/core/impl/RoleUtil.java
+++ b/src/main/java/org/apache/directory/fortress/core/impl/RoleUtil.java
@@ -86,7 +86,7 @@ final class RoleUtil implements ParentUtil
return sINSTANCE;
}
- RoleUtil(){
+ private RoleUtil(){
init();
}
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0e38f216/src/main/java/org/apache/directory/fortress/core/impl/UserDAO.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/impl/UserDAO.java b/src/main/java/org/apache/directory/fortress/core/impl/UserDAO.java
index 483f463..052dd95 100755
--- a/src/main/java/org/apache/directory/fortress/core/impl/UserDAO.java
+++ b/src/main/java/org/apache/directory/fortress/core/impl/UserDAO.java
@@ -2318,7 +2318,7 @@ final class UserDAO extends LdapDataProvider
for ( String raw : roles )
{
UserAdminRole ure = new ObjectFactory().createUserAdminRole();
- ure.load( raw, contextId, new RoleUtil() );
+ ure.load( raw, contextId, RoleUtil.getInstance() );
ure.setSequenceId( sequence++ );
ure.setUserId( userId );
uRoles.add( ure );
@@ -2367,12 +2367,12 @@ final class UserDAO extends LdapDataProvider
//if already found, add to user role
if(uRoles.containsKey(roleName)){
UserRole ure = uRoles.get(roleName);
- ure.load( raw, contextId, new RoleUtil() );
+ ure.load( raw, contextId, RoleUtil.getInstance() );
}
//else create new
else{
UserRole ure = new ObjectFactory().createUserRole();
- ure.load( raw, contextId, new RoleUtil() );
+ ure.load( raw, contextId, RoleUtil.getInstance() );
ure.setUserId( userId );
ure.setSequenceId( sequence++ );
uRoles.put(roleName, ure );