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 2015/05/22 15:21:12 UTC

directory-fortress-core git commit: FC-105 - [fortress-core] - utility classes should not have public or default constructor

Repository: directory-fortress-core
Updated Branches:
  refs/heads/master ce90d0bfc -> 017ed30e3


FC-105 - [fortress-core] - utility classes should not have public or default constructor


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/017ed30e
Tree: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/tree/017ed30e
Diff: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/diff/017ed30e

Branch: refs/heads/master
Commit: 017ed30e36d526ed52f76b3214695678fc12655e
Parents: ce90d0b
Author: Shawn McKinney <sm...@apache.org>
Authored: Fri May 22 08:21:07 2015 -0500
Committer: Shawn McKinney <sm...@apache.org>
Committed: Fri May 22 08:21:07 2015 -0500

----------------------------------------------------------------------
 .../apache/directory/fortress/core/cfg/ConfigMgrFactory.java   | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/017ed30e/src/main/java/org/apache/directory/fortress/core/cfg/ConfigMgrFactory.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/cfg/ConfigMgrFactory.java b/src/main/java/org/apache/directory/fortress/core/cfg/ConfigMgrFactory.java
index 9969b80..1662903 100755
--- a/src/main/java/org/apache/directory/fortress/core/cfg/ConfigMgrFactory.java
+++ b/src/main/java/org/apache/directory/fortress/core/cfg/ConfigMgrFactory.java
@@ -40,6 +40,11 @@ public class ConfigMgrFactory
     private final static String ENABLE_REST = "enable.mgr.impl.rest";
     private static final boolean IS_REST = ((Config.getProperty(ENABLE_REST) != null) && (Config.getProperty(ENABLE_REST).equalsIgnoreCase("true")));
 
+    private ConfigMgrFactory()
+    {
+
+    }
+
     /**
      * Create and return a reference to {@link ConfigMgr} object.
      *
@@ -50,7 +55,6 @@ public class ConfigMgrFactory
         throws SecurityException
     {
         // TODO: Don't reuse {@link VUtil#isNotNullOrEmpty} here until it is determined why it forces different execution path through GlobalIds.IS_OPENLDAP:
-        //if (!VUtil.isNotNullOrEmpty(configClassName))
         if (configClassName == null || configClassName.compareTo("") == 0)
         {
             if(IS_REST)