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 2019/04/16 23:22:21 UTC

[directory-fortress-core] branch master updated: move get connection to just before it's needed

This is an automated email from the ASF dual-hosted git repository.

smckinney pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/directory-fortress-core.git


The following commit(s) were added to refs/heads/master by this push:
     new 5594a43  move get connection to just before it's needed
5594a43 is described below

commit 5594a43ba436280064b48b8c3a22c938847fe72e
Author: Shawn McKinney <sm...@apache.org>
AuthorDate: Tue Apr 16 18:22:11 2019 -0500

    move get connection to just before it's needed
---
 src/main/java/org/apache/directory/fortress/core/impl/ConfigDAO.java | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/main/java/org/apache/directory/fortress/core/impl/ConfigDAO.java b/src/main/java/org/apache/directory/fortress/core/impl/ConfigDAO.java
index 73f44ac..808bdcb 100755
--- a/src/main/java/org/apache/directory/fortress/core/impl/ConfigDAO.java
+++ b/src/main/java/org/apache/directory/fortress/core/impl/ConfigDAO.java
@@ -125,7 +125,6 @@ final class ConfigDAO extends LdapDataProvider
         {
             Entry myEntry = new DefaultEntry( dn );
             myEntry.add( SchemaConstants.OBJECT_CLASS_AT, CONFIG_OBJ_CLASS );
-            ld = getAdminConnection();
             myEntry.add( SchemaConstants.CN_AT, cfg.getName() );
             loadProperties( cfg.getProperties(), myEntry, GlobalIds.PROPS );
             // These attributes hold sequence numbers:
@@ -145,7 +144,7 @@ final class ConfigDAO extends LdapDataProvider
             {
                 myEntry.add( GlobalIds.GID_NUMBER, "0" );
             }
-
+            ld = getAdminConnection();
             add( ld, myEntry, cfg );
         }
         catch ( LdapEntryAlreadyExistsException e )