Fixed problem with TPS profile default status.

The base class of ProfileDatabase (i.e. CSCfgDatabase) has been
modified to return the correct default value (i.e. Enabled) if the
status parameter doesn't exist. The TPSProcessor has been modified
to use ProfileDatabase and other TPS codes have also been changed
to use constants instead of string literals to ensure consistency.

https://fedorahosted.org/pki/ticket/1270

1 files changed, 2 insertions, 1 deletions

diff --git a/base/server/cmscore/src/com/netscape/cmscore/dbs/CSCfgDatabase.java b/base/server/cmscore/src/com/netscape/cmscore/dbs/CSCfgDatabase.java
index 4dee392e2..38f542ffb 100644
--- a/base/server/cmscore/src/com/netscape/cmscore/dbs/CSCfgDatabase.java
+++ b/base/server/cmscore/src/com/netscape/cmscore/dbs/CSCfgDatabase.java
@@ -26,6 +26,7 @@ import org.apache.commons.lang.StringUtils;
 import com.netscape.certsrv.apps.CMS;
 import com.netscape.certsrv.base.EBaseException;
 import com.netscape.certsrv.base.IConfigStore;
+import com.netscape.certsrv.common.Constants;
 import com.netscape.cms.realm.PKIPrincipal;
 
 
@@ -59,7 +60,7 @@ public class CSCfgDatabase<E extends CSCfgRecord> extends Database<E> {
     }
 
     public String getRecordStatus(String recordID) throws EBaseException {
-        return configStore.getString("config." + substoreName + "." + recordID + ".state", "Disabled");
+        return configStore.getString("config." + substoreName + "." + recordID + ".state", Constants.CFG_ENABLED);
     }
 
     public void setRecordStatus(String recordID, String status) throws EBaseException {