From 0bc851bff69ef174b11cf147aeb1289c43de0666 Mon Sep 17 00:00:00 2001 From: Endi Sukma Dewata Date: Mon, 27 Feb 2012 09:36:14 -0600 Subject: Refactored NameValuePairs. The NameValuePairs class has been modified to extend the Linked- HashMap which preserves the order of elements as in the original code. Some methods are renamed to match Java Map interface. The NameValuePair class is no longer needed and has been removed. Ticket #78 --- .../ca/src/com/netscape/ca/CMSCRLExtensions.java | 23 ++- .../ca/src/com/netscape/ca/CRLIssuingPoint.java | 7 +- pki/base/common/src/CMakeLists.txt | 1 - .../com/netscape/certsrv/common/NameValuePair.java | 68 --------- .../netscape/certsrv/common/NameValuePairs.java | 121 +--------------- .../cms/crl/CMSAuthInfoAccessExtension.java | 14 +- .../cms/crl/CMSCertificateIssuerExtension.java | 14 +- .../netscape/cms/crl/CMSFreshestCRLExtension.java | 10 +- .../cms/crl/CMSHoldInstructionExtension.java | 2 +- .../cms/crl/CMSIssuerAlternativeNameExtension.java | 14 +- .../crl/CMSIssuingDistributionPointExtension.java | 16 +- .../src/com/netscape/cms/logging/LogFile.java | 2 +- .../com/netscape/cms/logging/RollingLogFile.java | 2 +- .../common/src/com/netscape/cms/ocsp/DefStore.java | 15 +- .../src/com/netscape/cms/ocsp/LDAPStore.java | 29 ++-- .../netscape/cms/profile/common/BasicProfile.java | 18 +-- .../cms/profile/def/AuthInfoAccessExtDefault.java | 32 ++-- .../def/CRLDistributionPointsExtDefault.java | 60 ++++---- .../netscape/cms/profile/def/EnrollDefault.java | 10 +- .../cms/profile/def/FreshestCRLExtDefault.java | 48 +++--- .../cms/profile/def/NameConstraintsExtDefault.java | 24 ++- .../cms/profile/def/PolicyMappingsExtDefault.java | 24 ++- .../def/SubjectDirAttributesExtDefault.java | 25 ++-- .../profile/def/SubjectInfoAccessExtDefault.java | 28 ++-- .../cms/servlet/admin/ACLAdminServlet.java | 12 +- .../netscape/cms/servlet/admin/AdminServlet.java | 22 +-- .../cms/servlet/admin/AuthAdminServlet.java | 33 ++--- .../netscape/cms/servlet/admin/CAAdminServlet.java | 50 +++---- .../cms/servlet/admin/CMSAdminServlet.java | 78 +++++----- .../cms/servlet/admin/JobsAdminServlet.java | 33 ++--- .../cms/servlet/admin/KRAAdminServlet.java | 2 +- .../cms/servlet/admin/LogAdminServlet.java | 31 ++-- .../cms/servlet/admin/OCSPAdminServlet.java | 8 +- .../cms/servlet/admin/PolicyAdminServlet.java | 16 +- .../cms/servlet/admin/ProfileAdminServlet.java | 44 +++--- .../cms/servlet/admin/PublisherAdminServlet.java | 161 ++++++++++----------- .../netscape/cms/servlet/admin/RAAdminServlet.java | 10 +- .../cms/servlet/admin/RegistryAdminServlet.java | 6 +- .../cms/servlet/admin/UsrGrpAdminServlet.java | 26 ++-- .../netscape/cmscore/security/JssSubsystem.java | 35 ++--- pki/base/console/src/CMakeLists.txt | 1 - .../com/netscape/admin/certsrv/CMSCAUILoader.java | 4 +- .../netscape/admin/certsrv/CMSKernelUILoader.java | 3 +- .../com/netscape/admin/certsrv/CMSServerInfo.java | 11 +- .../com/netscape/admin/certsrv/CMSUIFramework.java | 9 +- .../netscape/admin/certsrv/config/ACIDialog.java | 12 +- .../admin/certsrv/config/ACLEditDialog.java | 16 +- .../netscape/admin/certsrv/config/ACLImplTab.java | 7 +- .../netscape/admin/certsrv/config/ACLPanel.java | 7 +- .../netscape/admin/certsrv/config/CACertsTab.java | 34 ++--- .../admin/certsrv/config/CMSAccessLogPanel.java | 65 ++++----- .../admin/certsrv/config/CMSAuditLogPanel.java | 65 ++++----- .../admin/certsrv/config/CMSAutoRecovery.java | 10 +- .../admin/certsrv/config/CMSBaseConfigDialog.java | 43 +++--- .../admin/certsrv/config/CMSBaseLDAPPanel.java | 123 ++++++++-------- .../certsrv/config/CMSCACertSettingPanel.java | 16 +- .../admin/certsrv/config/CMSCAConnectorPanel.java | 19 +-- .../admin/certsrv/config/CMSCAGeneralPanel.java | 47 +++--- .../admin/certsrv/config/CMSCRLCachePanel.java | 29 ++-- .../admin/certsrv/config/CMSCRLFormatPanel.java | 50 +++---- .../admin/certsrv/config/CMSCRLIPPanel.java | 16 +- .../admin/certsrv/config/CMSCRLSettingPanel.java | 55 ++++--- .../admin/certsrv/config/CMSEAGeneralPanel.java | 16 +- .../admin/certsrv/config/CMSEncryptionPanel.java | 31 ++-- .../admin/certsrv/config/CMSErrorLogPanel.java | 37 +++-- .../admin/certsrv/config/CMSKRAAutoPanel.java | 13 +- .../admin/certsrv/config/CMSKRAPasswdPanel.java | 15 +- .../admin/certsrv/config/CMSKRASchemePanel.java | 15 +- .../admin/certsrv/config/CMSLDAPSettingPanel.java | 42 +++--- .../admin/certsrv/config/CMSNetworkPanel.java | 39 +++-- .../admin/certsrv/config/CMSOCSPGeneralPanel.java | 17 +-- .../admin/certsrv/config/CMSPasswordDialog.java | 6 +- .../admin/certsrv/config/CMSPluginInstanceTab.java | 29 ++-- .../admin/certsrv/config/CMSRACLMPanel.java | 49 +++---- .../admin/certsrv/config/CMSRAConnectorPanel.java | 19 +-- .../admin/certsrv/config/CMSRAGeneralPanel.java | 15 +- .../admin/certsrv/config/CMSSMTPPanel.java | 17 +-- .../admin/certsrv/config/CMSSNMPPanel.java | 53 ++++--- .../admin/certsrv/config/CMSSelfTestsPanel.java | 16 +- .../certsrv/config/CMSUserCertSettingPanel.java | 16 +- .../certsrv/config/CRLExtensionsRuleDataModel.java | 9 +- .../netscape/admin/certsrv/config/CRLIPEditor.java | 22 +-- .../admin/certsrv/config/ConnectorEditor.java | 51 +++---- .../admin/certsrv/config/GeneralLogPanel.java | 24 ++- .../admin/certsrv/config/JobsImplDataModel.java | 5 +- .../netscape/admin/certsrv/config/JobsImplTab.java | 31 ++-- .../admin/certsrv/config/JobsRuleDataModel.java | 9 +- .../admin/certsrv/config/JobsSettingPanel.java | 23 ++- .../admin/certsrv/config/KeyCreateDialog.java | 7 +- .../admin/certsrv/config/LogImplDataModel.java | 5 +- .../netscape/admin/certsrv/config/LogImplTab.java | 25 ++-- .../admin/certsrv/config/LogRuleDataModel.java | 5 +- .../admin/certsrv/config/MNSchemeWizardInfo.java | 13 +- .../admin/certsrv/config/MapperImplDataModel.java | 5 +- .../admin/certsrv/config/MapperImplTab.java | 25 ++-- .../admin/certsrv/config/MapperRuleDataModel.java | 5 +- .../certsrv/config/OCSPStoresInstanceTab.java | 9 +- .../certsrv/config/OCSPStoresRuleDataModel.java | 9 +- .../certsrv/config/PanelMapperConfigDialog.java | 21 ++- .../certsrv/config/PluginSelectionDialog.java | 11 +- .../admin/certsrv/config/PolicyImplDataModel.java | 5 +- .../admin/certsrv/config/PolicyImplTab.java | 25 ++-- .../admin/certsrv/config/PolicyRuleDataModel.java | 9 +- .../certsrv/config/PolicyRuleOrderDialog.java | 5 +- .../admin/certsrv/config/ProfileConfigDialog.java | 15 +- .../admin/certsrv/config/ProfileEditDialog.java | 71 +++++---- .../admin/certsrv/config/ProfileImplDataModel.java | 7 +- .../admin/certsrv/config/ProfileImplTab.java | 57 ++++---- .../certsrv/config/ProfileNonPolicyNewDialog.java | 23 ++- .../certsrv/config/ProfileNonPolicySelDialog.java | 9 +- .../config/ProfilePluginSelectionDialog.java | 12 +- .../certsrv/config/ProfilePolicyEditDialog.java | 22 ++- .../certsrv/config/ProfilePolicyNewDialog.java | 24 ++- .../config/ProfilePolicySelectionDialog.java | 8 +- .../certsrv/config/ProfileRegisterDialog.java | 8 +- .../admin/certsrv/config/ProfileRuleDataModel.java | 10 +- .../certsrv/config/PublisherImplDataModel.java | 5 +- .../admin/certsrv/config/PublisherImplTab.java | 23 ++- .../certsrv/config/PublisherRuleDataModel.java | 5 +- .../admin/certsrv/config/RegisterDialog.java | 5 +- .../admin/certsrv/config/RuleImplDataModel.java | 5 +- .../netscape/admin/certsrv/config/RuleImplTab.java | 23 ++- .../admin/certsrv/config/RuleRuleDataModel.java | 9 +- .../netscape/admin/certsrv/config/TKSKeysTab.java | 23 +-- .../admin/certsrv/config/UserCertsTab.java | 21 ++- .../config/install/WICertExtensionPage.java | 68 ++++----- .../certsrv/config/install/WIGenKeyCertPage.java | 12 +- .../config/install/WIGenKeyCertReqPage.java | 16 +- .../admin/certsrv/connection/AdminConnection.java | 50 +++---- .../admin/certsrv/keycert/WCertDNPage.java | 16 +- .../admin/certsrv/keycert/WCertExtensionPage.java | 66 ++++----- .../admin/certsrv/keycert/WCertTypePage.java | 9 +- .../admin/certsrv/keycert/WDisplayCertPage.java | 15 +- .../admin/certsrv/keycert/WExecute1Page.java | 9 +- .../admin/certsrv/keycert/WExecutePage.java | 7 +- .../admin/certsrv/keycert/WInstallOpPage.java | 5 +- .../netscape/admin/certsrv/keycert/WIntroPage.java | 9 +- .../netscape/admin/certsrv/keycert/WKeyPage.java | 34 ++--- .../admin/certsrv/keycert/WPasteCertPage.java | 11 +- .../admin/certsrv/keycert/WTokenLogonPage.java | 9 +- .../certsrv/keycert/WWarningExecute1Page.java | 6 +- .../certsrv/managecert/CertificateInfoDialog.java | 8 +- .../admin/certsrv/managecert/ManageCertDialog.java | 17 +-- .../certsrv/notification/RequestCompletePanel.java | 36 ++--- .../certsrv/notification/RequestInQPanel.java | 42 +++--- .../certsrv/notification/RequestRevokedPanel.java | 33 ++--- .../admin/certsrv/status/LogInstancePanel.java | 23 ++- .../netscape/admin/certsrv/status/StatusPanel.java | 17 +-- .../netscape/admin/certsrv/ug/AuthBaseDialog.java | 19 +-- .../admin/certsrv/ug/AuthImplDataModel.java | 9 +- .../com/netscape/admin/certsrv/ug/AuthImplTab.java | 31 ++-- .../netscape/admin/certsrv/ug/AuthInstanceTab.java | 11 +- .../admin/certsrv/ug/AuthRuleDataModel.java | 5 +- .../netscape/admin/certsrv/ug/CertDataModel.java | 5 +- .../admin/certsrv/ug/CertManagementDialog.java | 13 +- .../com/netscape/admin/certsrv/ug/GroupEditor.java | 23 ++- .../netscape/admin/certsrv/ug/GroupListDialog.java | 4 +- .../com/netscape/admin/certsrv/ug/GroupTab.java | 10 +- .../netscape/admin/certsrv/ug/MemberDataModel.java | 19 ++- .../com/netscape/admin/certsrv/ug/UserEditor.java | 52 ++++--- .../netscape/admin/certsrv/ug/UserListDialog.java | 6 +- .../src/com/netscape/admin/certsrv/ug/UserTab.java | 5 +- .../com/netscape/certsrv/common/NameValuePair.java | 56 ------- .../netscape/certsrv/common/NameValuePairs.java | 122 +++------------- 164 files changed, 1639 insertions(+), 2273 deletions(-) delete mode 100644 pki/base/common/src/com/netscape/certsrv/common/NameValuePair.java delete mode 100644 pki/base/console/src/com/netscape/certsrv/common/NameValuePair.java diff --git a/pki/base/ca/src/com/netscape/ca/CMSCRLExtensions.java b/pki/base/ca/src/com/netscape/ca/CMSCRLExtensions.java index 8c59d42d8..94693d69a 100644 --- a/pki/base/ca/src/com/netscape/ca/CMSCRLExtensions.java +++ b/pki/base/ca/src/com/netscape/ca/CMSCRLExtensions.java @@ -49,7 +49,6 @@ import com.netscape.certsrv.ca.ICMSCRLExtensions; import com.netscape.certsrv.ca.ICRLIssuingPoint; import com.netscape.certsrv.ca.ICertificateAuthority; import com.netscape.certsrv.common.Constants; -import com.netscape.certsrv.common.NameValuePair; import com.netscape.certsrv.common.NameValuePairs; import com.netscape.certsrv.logging.ILogger; import com.netscape.cms.crl.CMSIssuingDistributionPointExtension; @@ -521,14 +520,14 @@ public class CMSCRLExtensions implements ICMSCRLExtensions { */ if (mEnabledCRLExtensions.contains(id)) { - nvp.add(PROP_ENABLE, Constants.TRUE); + nvp.put(PROP_ENABLE, Constants.TRUE); } else { - nvp.add(PROP_ENABLE, Constants.FALSE); + nvp.put(PROP_ENABLE, Constants.FALSE); } if (mCriticalCRLExtensions.contains(id)) { - nvp.add(PROP_CRITICAL, Constants.TRUE); + nvp.put(PROP_CRITICAL, Constants.TRUE); } else { - nvp.add(PROP_CRITICAL, Constants.FALSE); + nvp.put(PROP_CRITICAL, Constants.FALSE); } if (mCRLExtensionClassNames.containsKey(id)) { @@ -563,7 +562,7 @@ public class CMSCRLExtensions implements ICMSCRLExtensions { String idName = name.substring(i + 1); if (idName != null) { - nvp.add(Constants.PR_CRLEXT_IMPL_NAME, idName); + nvp.put(Constants.PR_CRLEXT_IMPL_NAME, idName); } } } @@ -574,17 +573,15 @@ public class CMSCRLExtensions implements ICMSCRLExtensions { public void setConfigParams(String id, NameValuePairs nvp, IConfigStore config) { ICertificateAuthority ca = (ICertificateAuthority) CMS.getSubsystem(CMS.SUBSYSTEM_CA); - String ipId = nvp.getValue("id"); + String ipId = nvp.get("id"); ICRLIssuingPoint ip = null; if (ipId != null && ca != null) { ip = ca.getCRLIssuingPoint(ipId); } - for (int i = 0; i < nvp.size(); i++) { - NameValuePair p = nvp.elementAt(i); - String name = p.getName(); - String value = p.getValue(); + for (String name : nvp.keySet()) { + String value = nvp.get(name); if (name.equals(PROP_ENABLE)) { if (!(value.equals(Constants.TRUE) || value.equals(Constants.FALSE))) { @@ -647,7 +644,7 @@ public class CMSCRLExtensions implements ICMSCRLExtensions { if (value.equals(Constants.TRUE) && (issuingDistPointExtEnabled == true)) { if (crlCACertsOnly == false) { CMS.debug(" value = true and CRLCACertsOnly is already false."); - crlIssuingPointPairs.add(Constants.PR_CA_CERTS_ONLY, Constants.TRUE); + crlIssuingPointPairs.put(Constants.PR_CA_CERTS_ONLY, Constants.TRUE); newValue = Constants.TRUE; ip.updateConfig(crlIssuingPointPairs); modifiedCRLConfig = true; @@ -656,7 +653,7 @@ public class CMSCRLExtensions implements ICMSCRLExtensions { //If the CRLCACertsOnly prop is true change it to false to sync. if (value.equals(Constants.FALSE) && (issuingDistPointExtEnabled == true)) { - crlIssuingPointPairs.add(Constants.PR_CA_CERTS_ONLY, Constants.FALSE); + crlIssuingPointPairs.put(Constants.PR_CA_CERTS_ONLY, Constants.FALSE); if (ip != null) { ip.updateConfig(crlIssuingPointPairs); newValue = Constants.FALSE; diff --git a/pki/base/ca/src/com/netscape/ca/CRLIssuingPoint.java b/pki/base/ca/src/com/netscape/ca/CRLIssuingPoint.java index 3de6128b6..dd6194101 100644 --- a/pki/base/ca/src/com/netscape/ca/CRLIssuingPoint.java +++ b/pki/base/ca/src/com/netscape/ca/CRLIssuingPoint.java @@ -58,7 +58,6 @@ import com.netscape.certsrv.ca.ICMSCRLExtensions; import com.netscape.certsrv.ca.ICRLIssuingPoint; import com.netscape.certsrv.ca.ICertificateAuthority; import com.netscape.certsrv.common.Constants; -import com.netscape.certsrv.common.NameValuePair; import com.netscape.certsrv.common.NameValuePairs; import com.netscape.certsrv.dbs.EDBNotAvailException; import com.netscape.certsrv.dbs.IElementProcessor; @@ -931,10 +930,8 @@ public class CRLIssuingPoint implements ICRLIssuingPoint, Runnable { boolean noRestart = true; boolean modifiedSchedule = false; - for (int i = 0; i < params.size(); i++) { - NameValuePair p = params.elementAt(i); - String name = p.getName(); - String value = p.getValue(); + for (String name : params.keySet()) { + String value = params.get(name); // -- Update Schema -- if (name.equals(Constants.PR_ENABLE_CRL)) { diff --git a/pki/base/common/src/CMakeLists.txt b/pki/base/common/src/CMakeLists.txt index ed83bb68e..6dfd322a1 100644 --- a/pki/base/common/src/CMakeLists.txt +++ b/pki/base/common/src/CMakeLists.txt @@ -90,7 +90,6 @@ set(pki-certsrv_java_SRCS com/netscape/certsrv/common/ScopeDef.java com/netscape/certsrv/common/PrefixDef.java com/netscape/certsrv/common/ConfigConstants.java - com/netscape/certsrv/common/NameValuePair.java com/netscape/certsrv/common/OpDef.java com/netscape/certsrv/common/Constants.java com/netscape/certsrv/usrgrp/EUsrGrpException.java diff --git a/pki/base/common/src/com/netscape/certsrv/common/NameValuePair.java b/pki/base/common/src/com/netscape/certsrv/common/NameValuePair.java deleted file mode 100644 index ed1d06149..000000000 --- a/pki/base/common/src/com/netscape/certsrv/common/NameValuePair.java +++ /dev/null @@ -1,68 +0,0 @@ -// --- BEGIN COPYRIGHT BLOCK --- -// This program is free software; you can redistribute it and/or modify -// it under the terms of the GNU General Public License as published by -// the Free Software Foundation; version 2 of the License. -// -// This program is distributed in the hope that it will be useful, -// but WITHOUT ANY WARRANTY; without even the implied warranty of -// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -// GNU General Public License for more details. -// -// You should have received a copy of the GNU General Public License along -// with this program; if not, write to the Free Software Foundation, Inc., -// 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. -// -// (C) 2007 Red Hat, Inc. -// All rights reserved. -// --- END COPYRIGHT BLOCK --- -package com.netscape.certsrv.common; - -/** - * A class represents a name value pair. A name value - * pair consists of a name and a value. - * - * @version $Revision$, $Date$ - */ -public class NameValuePair { - - private String mName = null; - private String mValue = null; - - /** - * Constructs value pair object. - * - * @param name name - * @param value value - */ - public NameValuePair(String name, String value) { - mName = name; - mValue = value; - } - - /** - * Retrieves the name. - * - * @return name - */ - public String getName() { - return mName; - } - - /** - * Retrieves the value. - * - * @return value - */ - public String getValue() { - return mValue; - } - - /** - * Sets the value - * - * @param value value - */ - public void setValue(String value) { - mValue = value; - } -} diff --git a/pki/base/common/src/com/netscape/certsrv/common/NameValuePairs.java b/pki/base/common/src/com/netscape/certsrv/common/NameValuePairs.java index 61d3cad62..0999db7bc 100644 --- a/pki/base/common/src/com/netscape/certsrv/common/NameValuePairs.java +++ b/pki/base/common/src/com/netscape/certsrv/common/NameValuePairs.java @@ -17,10 +17,8 @@ // --- END COPYRIGHT BLOCK --- package com.netscape.certsrv.common; -import java.util.Enumeration; -import java.util.Hashtable; +import java.util.LinkedHashMap; import java.util.StringTokenizer; -import java.util.Vector; /** * A class represents an ordered list of name @@ -28,13 +26,9 @@ import java.util.Vector; * * @version $Revision$, $Date$ */ -public class NameValuePairs { +public class NameValuePairs extends LinkedHashMap { - private Vector mPairs = new Vector(); - - // an index to speed up searching - // The key is the name. The element is the NameValuePair. - private Hashtable index = new Hashtable(); + private static final long serialVersionUID = 1494507857048437440L; /** * Constructs name value pairs. @@ -42,97 +36,6 @@ public class NameValuePairs { public NameValuePairs() { } - /** - * Adds a name value pair into this set. - * if the name already exist, the value will - * be replaced. - * - * @param name name - * @param value value - */ - public void add(String name, String value) { - NameValuePair pair = getPair(name); - - if (pair == null) { - pair = new NameValuePair(name, value); - mPairs.addElement(pair); - index.put(name, pair); - } else { - pair.setValue(value); - } - } - - /** - * Retrieves name value pair from this set. - * - * @param name name - * @return name value pair - */ - public NameValuePair getPair(String name) { - return (NameValuePair) index.get(name); - } - - /** - * Returns number of pairs in this set. - * - * @return size - */ - public int size() { - return mPairs.size(); - } - - /** - * Retrieves name value pairs in specific position. - * - * @param pos position of the value - * @return name value pair - */ - public NameValuePair elementAt(int pos) { - return (NameValuePair) mPairs.elementAt(pos); - } - - /** - * Removes all name value pairs in this set. - */ - public void removeAllPairs() { - mPairs.removeAllElements(); - index.clear(); - } - - /** - * Retrieves value of the name value pairs that matches - * the given name. - * - * @param name name - * @return value - */ - public String getValue(String name) { - NameValuePair p = getPair(name); - - if (p != null) { - return p.getValue(); - } - return null; - } - - /** - * Retrieves a list of names. - * - * @return a list of names - */ - public Enumeration getNames() { - Vector v = new Vector(); - int size = mPairs.size(); - - for (int i = 0; i < size; i++) { - NameValuePair p = (NameValuePair) mPairs.elementAt(i); - - v.addElement(p.getName()); - } - //System.out.println("getNames: "+v.size()); - return v.elements(); - } - /** * Show the content of this name value container as * string representation. @@ -142,12 +45,13 @@ public class NameValuePairs { public String toString() { StringBuffer buf = new StringBuffer(); - for (int i = 0; i < mPairs.size(); i++) { - NameValuePair p = (NameValuePair) mPairs.elementAt(i); + for (String name : keySet()) { + String value = get(name); - buf.append(p.getName() + "=" + p.getValue()); + buf.append(name + "=" + value); buf.append("\n"); } + return buf.toString(); } @@ -171,17 +75,8 @@ public class NameValuePairs { String n = t.substring(0, i); String v = t.substring(i + 1); - nvp.add(n, v); + nvp.put(n, v); } return true; } - - /** - * Returns a list of name value pair object. - * - * @return name value objects - */ - public Enumeration elements() { - return mPairs.elements(); - } } diff --git a/pki/base/common/src/com/netscape/cms/crl/CMSAuthInfoAccessExtension.java b/pki/base/common/src/com/netscape/cms/crl/CMSAuthInfoAccessExtension.java index d070cc6fd..d4cef0148 100644 --- a/pki/base/common/src/com/netscape/cms/crl/CMSAuthInfoAccessExtension.java +++ b/pki/base/common/src/com/netscape/cms/crl/CMSAuthInfoAccessExtension.java @@ -159,7 +159,7 @@ public class CMSAuthInfoAccessExtension } catch (EBaseException e) { log(ILogger.LL_FAILURE, CMS.getLogMessage("CRL_CREATE_AIA_INVALID_NUM_ADS", e.toString())); } - nvp.add(PROP_NUM_ADS, String.valueOf(numberOfAccessDescriptions)); + nvp.put(PROP_NUM_ADS, String.valueOf(numberOfAccessDescriptions)); for (int i = 0; i < numberOfAccessDescriptions; i++) { String accessMethod = null; @@ -175,9 +175,9 @@ public class CMSAuthInfoAccessExtension } if (accessMethod != null && accessMethod.length() > 0) { - nvp.add(PROP_ACCESS_METHOD + i, accessMethod); + nvp.put(PROP_ACCESS_METHOD + i, accessMethod); } else { - nvp.add(PROP_ACCESS_METHOD + i, PROP_ACCESS_METHOD_CAISSUERS); + nvp.put(PROP_ACCESS_METHOD + i, PROP_ACCESS_METHOD_CAISSUERS); } try { @@ -189,9 +189,9 @@ public class CMSAuthInfoAccessExtension } if (accessLocationType != null && accessLocationType.length() > 0) { - nvp.add(PROP_ACCESS_LOCATION_TYPE + i, accessLocationType); + nvp.put(PROP_ACCESS_LOCATION_TYPE + i, accessLocationType); } else { - nvp.add(PROP_ACCESS_LOCATION_TYPE + i, PROP_URINAME); + nvp.put(PROP_ACCESS_LOCATION_TYPE + i, PROP_URINAME); } try { @@ -203,14 +203,14 @@ public class CMSAuthInfoAccessExtension } if (accessLocation != null && accessLocation.length() > 0) { - nvp.add(PROP_ACCESS_LOCATION + i, accessLocation); + nvp.put(PROP_ACCESS_LOCATION + i, accessLocation); } else { String hostname = CMS.getEENonSSLHost(); String port = CMS.getEENonSSLPort(); if (hostname != null && port != null) { accessLocation = "http://" + hostname + ":" + port + "/ca/ee/ca/getCAChain?op=downloadBIN"; } - nvp.add(PROP_ACCESS_LOCATION + i, accessLocation); + nvp.put(PROP_ACCESS_LOCATION + i, accessLocation); } } } diff --git a/pki/base/common/src/com/netscape/cms/crl/CMSCertificateIssuerExtension.java b/pki/base/common/src/com/netscape/cms/crl/CMSCertificateIssuerExtension.java index 67f4736c7..b0bf20856 100644 --- a/pki/base/common/src/com/netscape/cms/crl/CMSCertificateIssuerExtension.java +++ b/pki/base/common/src/com/netscape/cms/crl/CMSCertificateIssuerExtension.java @@ -149,7 +149,7 @@ public class CMSCertificateIssuerExtension } catch (EBaseException e) { log(ILogger.LL_FAILURE, CMS.getLogMessage("CRL_CREATE_INVALID_NUM_NAMES", e.toString())); } - nvp.add("numNames", String.valueOf(numNames)); + nvp.put("numNames", String.valueOf(numNames)); for (int i = 0; i < numNames; i++) { String nameType = null; @@ -164,9 +164,9 @@ public class CMSCertificateIssuerExtension } if (nameType != null && nameType.length() > 0) { - nvp.add("nameType" + i, nameType); + nvp.put("nameType" + i, nameType); } else { - nvp.add("nameType" + i, ""); + nvp.put("nameType" + i, ""); } String name = null; @@ -181,16 +181,16 @@ public class CMSCertificateIssuerExtension } if (name != null && name.length() > 0) { - nvp.add("name" + i, name); + nvp.put("name" + i, name); } else { - nvp.add("name" + i, ""); + nvp.put("name" + i, ""); } } if (numNames < 3) { for (int i = numNames; i < 3; i++) { - nvp.add("nameType" + i, ""); - nvp.add("name" + i, ""); + nvp.put("nameType" + i, ""); + nvp.put("name" + i, ""); } } } diff --git a/pki/base/common/src/com/netscape/cms/crl/CMSFreshestCRLExtension.java b/pki/base/common/src/com/netscape/cms/crl/CMSFreshestCRLExtension.java index edb6494af..72dbe5502 100644 --- a/pki/base/common/src/com/netscape/cms/crl/CMSFreshestCRLExtension.java +++ b/pki/base/common/src/com/netscape/cms/crl/CMSFreshestCRLExtension.java @@ -157,7 +157,7 @@ public class CMSFreshestCRLExtension log(ILogger.LL_FAILURE, "Invalid numPoints property for CRL " + "Freshest CRL extension - " + e); } - nvp.add(PROP_NUM_POINTS, String.valueOf(numPoints)); + nvp.put(PROP_NUM_POINTS, String.valueOf(numPoints)); for (int i = 0; i < numPoints; i++) { String pointType = null; @@ -171,9 +171,9 @@ public class CMSFreshestCRLExtension } if (pointType != null && pointType.length() > 0) { - nvp.add(PROP_POINTTYPE + i, pointType); + nvp.put(PROP_POINTTYPE + i, pointType); } else { - nvp.add(PROP_POINTTYPE + i, ""); + nvp.put(PROP_POINTTYPE + i, ""); } String pointName = null; @@ -187,9 +187,9 @@ public class CMSFreshestCRLExtension } if (pointName != null && pointName.length() > 0) { - nvp.add(PROP_POINTNAME + i, pointName); + nvp.put(PROP_POINTNAME + i, pointName); } else { - nvp.add(PROP_POINTNAME + i, ""); + nvp.put(PROP_POINTNAME + i, ""); } } } diff --git a/pki/base/common/src/com/netscape/cms/crl/CMSHoldInstructionExtension.java b/pki/base/common/src/com/netscape/cms/crl/CMSHoldInstructionExtension.java index df8d058ce..4023e3b2f 100644 --- a/pki/base/common/src/com/netscape/cms/crl/CMSHoldInstructionExtension.java +++ b/pki/base/common/src/com/netscape/cms/crl/CMSHoldInstructionExtension.java @@ -123,7 +123,7 @@ public class CMSHoldInstructionExtension } else { instruction = PROP_INSTR_NONE; } - nvp.add(PROP_INSTR, instruction); + nvp.put(PROP_INSTR, instruction); } public String[] getExtendedPluginInfo(Locale locale) { diff --git a/pki/base/common/src/com/netscape/cms/crl/CMSIssuerAlternativeNameExtension.java b/pki/base/common/src/com/netscape/cms/crl/CMSIssuerAlternativeNameExtension.java index ee656a199..64252a0b9 100644 --- a/pki/base/common/src/com/netscape/cms/crl/CMSIssuerAlternativeNameExtension.java +++ b/pki/base/common/src/com/netscape/cms/crl/CMSIssuerAlternativeNameExtension.java @@ -200,7 +200,7 @@ public class CMSIssuerAlternativeNameExtension log(ILogger.LL_FAILURE, "Invalid numNames property for CRL " + "IssuerAlternativeName extension - " + e); } - nvp.add("numNames", String.valueOf(numNames)); + nvp.put("numNames", String.valueOf(numNames)); for (int i = 0; i < numNames; i++) { String nameType = null; @@ -216,9 +216,9 @@ public class CMSIssuerAlternativeNameExtension } if (nameType != null && nameType.length() > 0) { - nvp.add("nameType" + i, nameType); + nvp.put("nameType" + i, nameType); } else { - nvp.add("nameType" + i, ""); + nvp.put("nameType" + i, ""); } String name = null; @@ -234,16 +234,16 @@ public class CMSIssuerAlternativeNameExtension } if (name != null && name.length() > 0) { - nvp.add("name" + i, name); + nvp.put("name" + i, name); } else { - nvp.add("name" + i, ""); + nvp.put("name" + i, ""); } } if (numNames < 3) { for (int i = numNames; i < 3; i++) { - nvp.add("nameType" + i, ""); - nvp.add("name" + i, ""); + nvp.put("nameType" + i, ""); + nvp.put("name" + i, ""); } } } diff --git a/pki/base/common/src/com/netscape/cms/crl/CMSIssuingDistributionPointExtension.java b/pki/base/common/src/com/netscape/cms/crl/CMSIssuingDistributionPointExtension.java index 21bd86b0a..4253584ce 100644 --- a/pki/base/common/src/com/netscape/cms/crl/CMSIssuingDistributionPointExtension.java +++ b/pki/base/common/src/com/netscape/cms/crl/CMSIssuingDistributionPointExtension.java @@ -229,9 +229,9 @@ public class CMSIssuingDistributionPointExtension log(ILogger.LL_FAILURE, CMS.getLogMessage("CRL_CREATE_DIST_POINT_INVALID", e.toString())); } if (pointType != null && pointType.length() > 0) { - nvp.add("pointType", pointType); + nvp.put("pointType", pointType); } else { - nvp.add("pointType", ""); + nvp.put("pointType", ""); } String pointName = null; @@ -244,9 +244,9 @@ public class CMSIssuingDistributionPointExtension log(ILogger.LL_FAILURE, CMS.getLogMessage("CRL_CREATE_DIST_POINT_INVALID", e.toString())); } if (pointName != null && pointName.length() > 0) { - nvp.add("pointName", pointName); + nvp.put("pointName", pointName); } else { - nvp.add("pointName", ""); + nvp.put("pointName", ""); } String reasons = null; @@ -257,17 +257,17 @@ public class CMSIssuingDistributionPointExtension log(ILogger.LL_FAILURE, CMS.getLogMessage("CRL_INVALID_PROPERTY", PROP_REASONS, e.toString())); } if (reasons != null && reasons.length() > 0) { - nvp.add(PROP_REASONS, reasons); + nvp.put(PROP_REASONS, reasons); } else { - nvp.add(PROP_REASONS, ""); + nvp.put(PROP_REASONS, ""); } try { boolean caCertsOnly = config.getBoolean(PROP_CACERTS, false); - nvp.add(PROP_CACERTS, String.valueOf(caCertsOnly)); + nvp.put(PROP_CACERTS, String.valueOf(caCertsOnly)); } catch (EBaseException e) { - nvp.add(PROP_CACERTS, "false"); + nvp.put(PROP_CACERTS, "false"); log(ILogger.LL_FAILURE, CMS.getLogMessage("CRL_INVALID_PROPERTY", "caCertsOnly", e.toString())); } // Disable these for now unitl we support them fully diff --git a/pki/base/common/src/com/netscape/cms/logging/LogFile.java b/pki/base/common/src/com/netscape/cms/logging/LogFile.java index 0a5f18579..5144bf16d 100644 --- a/pki/base/common/src/com/netscape/cms/logging/LogFile.java +++ b/pki/base/common/src/com/netscape/cms/logging/LogFile.java @@ -1340,7 +1340,7 @@ public class LogFile implements ILogEventListener, IExtendedPluginInfo { try { entries = readEntry(maxLine, level, source, fName); for (int i = 0; i < entries.size(); i++) { - params.add(Integer.toString(i) + + params.put(Integer.toString(i) + ((LogEntry) entries.elementAt(i)).getEntry(), ""); } } catch (Exception e) { diff --git a/pki/base/common/src/com/netscape/cms/logging/RollingLogFile.java b/pki/base/common/src/com/netscape/cms/logging/RollingLogFile.java index e085937e5..93455e9fe 100644 --- a/pki/base/common/src/com/netscape/cms/logging/RollingLogFile.java +++ b/pki/base/common/src/com/netscape/cms/logging/RollingLogFile.java @@ -524,7 +524,7 @@ public class RollingLogFile extends LogFile { files = fileList(); for (int i = 0; i < files.length; i++) { - params.add(files[i], ""); + params.put(files[i], ""); } return params; } diff --git a/pki/base/common/src/com/netscape/cms/ocsp/DefStore.java b/pki/base/common/src/com/netscape/cms/ocsp/DefStore.java index d3b5d545e..2482d69af 100644 --- a/pki/base/common/src/com/netscape/cms/ocsp/DefStore.java +++ b/pki/base/common/src/com/netscape/cms/ocsp/DefStore.java @@ -798,13 +798,13 @@ public class DefStore implements IDefStore, IExtendedPluginInfo { try { NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_OCSPSTORE_IMPL_NAME, + params.put(Constants.PR_OCSPSTORE_IMPL_NAME, mConfig.getString("class")); - params.add(PROP_NOT_FOUND_GOOD, + params.put(PROP_NOT_FOUND_GOOD, mConfig.getString(PROP_NOT_FOUND_GOOD, "true")); - params.add(PROP_BY_NAME, + params.put(PROP_BY_NAME, mConfig.getString(PROP_BY_NAME, "true")); - params.add(PROP_INCLUDE_NEXT_UPDATE, + params.put(PROP_INCLUDE_NEXT_UPDATE, mConfig.getString(PROP_INCLUDE_NEXT_UPDATE, "false")); return params; } catch (Exception e) { @@ -814,12 +814,9 @@ public class DefStore implements IDefStore, IExtendedPluginInfo { public void setConfigParameters(NameValuePairs pairs) throws EBaseException { - Enumeration k = pairs.getNames(); - while (k.hasMoreElements()) { - String key = (String) k.nextElement(); - - mConfig.put(key, pairs.getValue(key)); + for (String key : pairs.keySet()) { + mConfig.put(key, pairs.get(key)); } } diff --git a/pki/base/common/src/com/netscape/cms/ocsp/LDAPStore.java b/pki/base/common/src/com/netscape/cms/ocsp/LDAPStore.java index 697d1bb40..11e91fdeb 100644 --- a/pki/base/common/src/com/netscape/cms/ocsp/LDAPStore.java +++ b/pki/base/common/src/com/netscape/cms/ocsp/LDAPStore.java @@ -531,34 +531,34 @@ public class LDAPStore implements IDefStore, IExtendedPluginInfo { try { NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_OCSPSTORE_IMPL_NAME, + params.put(Constants.PR_OCSPSTORE_IMPL_NAME, mConfig.getString("class")); int num = mConfig.getInteger(PROP_NUM_CONNS, 0); - params.add(PROP_NUM_CONNS, Integer.toString(num)); + params.put(PROP_NUM_CONNS, Integer.toString(num)); for (int i = 0; i < num; i++) { - params.add(PROP_HOST + Integer.toString(i), + params.put(PROP_HOST + Integer.toString(i), mConfig.getString(PROP_HOST + Integer.toString(i), "")); - params.add(PROP_PORT + Integer.toString(i), + params.put(PROP_PORT + Integer.toString(i), mConfig.getString(PROP_PORT + Integer.toString(i), "389")); - params.add(PROP_BASE_DN + Integer.toString(i), + params.put(PROP_BASE_DN + Integer.toString(i), mConfig.getString(PROP_BASE_DN + Integer.toString(i), "")); - params.add(PROP_REFRESH_IN_SEC + Integer.toString(i), + params.put(PROP_REFRESH_IN_SEC + Integer.toString(i), mConfig.getString(PROP_REFRESH_IN_SEC + Integer.toString(i), Integer.toString(DEF_REFRESH_IN_SEC))); } - params.add(PROP_BY_NAME, + params.put(PROP_BY_NAME, mConfig.getString(PROP_BY_NAME, "true")); - params.add(PROP_CA_CERT_ATTR, + params.put(PROP_CA_CERT_ATTR, mConfig.getString(PROP_CA_CERT_ATTR, DEF_CA_CERT_ATTR)); - params.add(PROP_CRL_ATTR, + params.put(PROP_CRL_ATTR, mConfig.getString(PROP_CRL_ATTR, DEF_CRL_ATTR)); - params.add(PROP_NOT_FOUND_GOOD, + params.put(PROP_NOT_FOUND_GOOD, mConfig.getString(PROP_NOT_FOUND_GOOD, "true")); - params.add(PROP_INCLUDE_NEXT_UPDATE, + params.put(PROP_INCLUDE_NEXT_UPDATE, mConfig.getString(PROP_INCLUDE_NEXT_UPDATE, "false")); return params; } catch (Exception e) { @@ -568,12 +568,9 @@ public class LDAPStore implements IDefStore, IExtendedPluginInfo { public void setConfigParameters(NameValuePairs pairs) throws EBaseException { - Enumeration k = pairs.getNames(); - while (k.hasMoreElements()) { - String key = k.nextElement(); - - mConfig.put(key, pairs.getValue(key)); + for (String key : pairs.keySet()) { + mConfig.put(key, pairs.get(key)); } } } diff --git a/pki/base/common/src/com/netscape/cms/profile/common/BasicProfile.java b/pki/base/common/src/com/netscape/cms/profile/common/BasicProfile.java index cab594188..7f624cb8b 100644 --- a/pki/base/common/src/com/netscape/cms/profile/common/BasicProfile.java +++ b/pki/base/common/src/com/netscape/cms/profile/common/BasicProfile.java @@ -615,15 +615,12 @@ public abstract class BasicProfile implements IProfile { outputInfo.getName(Locale.getDefault())); outputStore.putString(prefix + "class_id", outputId); - Enumeration enum1 = nvps.getNames(); + for (String name : nvps.keySet()) { - while (enum1.hasMoreElements()) { - String name = enum1.nextElement(); - - outputStore.putString(prefix + "params." + name, nvps.getValue(name)); + outputStore.putString(prefix + "params." + name, nvps.get(name)); try { if (output != null) { - output.setConfig(name, nvps.getValue(name)); + output.setConfig(name, nvps.get(name)); } } catch (EBaseException e) { CMS.debug(e.toString()); @@ -718,15 +715,12 @@ public abstract class BasicProfile implements IProfile { inputInfo.getName(Locale.getDefault())); inputStore.putString(prefix + "class_id", inputId); - Enumeration enum1 = nvps.getNames(); - - while (enum1.hasMoreElements()) { - String name = enum1.nextElement(); + for (String name : nvps.keySet()) { - inputStore.putString(prefix + "params." + name, nvps.getValue(name)); + inputStore.putString(prefix + "params." + name, nvps.get(name)); try { if (input != null) { - input.setConfig(name, nvps.getValue(name)); + input.setConfig(name, nvps.get(name)); } } catch (EBaseException e) { CMS.debug(e.toString()); diff --git a/pki/base/common/src/com/netscape/cms/profile/def/AuthInfoAccessExtDefault.java b/pki/base/common/src/com/netscape/cms/profile/def/AuthInfoAccessExtDefault.java index f23b7e24c..76ea4e143 100644 --- a/pki/base/common/src/com/netscape/cms/profile/def/AuthInfoAccessExtDefault.java +++ b/pki/base/common/src/com/netscape/cms/profile/def/AuthInfoAccessExtDefault.java @@ -225,7 +225,7 @@ public class AuthInfoAccessExtDefault extends EnrollExtDefault { } boolean critical = ext.isCritical(); - Vector v = parseRecords(value); + Vector v = parseRecords(value); int size = v.size(); ext = new AuthInfoAccessExtension(critical); @@ -235,20 +235,18 @@ public class AuthInfoAccessExtDefault extends EnrollExtDefault { String enable = null; for (int i = 0; i < size; i++) { - NameValuePairs nvps = (NameValuePairs) v.elementAt(i); - Enumeration names = nvps.getNames(); + NameValuePairs nvps = v.elementAt(i); - while (names.hasMoreElements()) { - String name1 = names.nextElement(); + for (String name1 : nvps.keySet()) { if (name1.equals(AD_METHOD)) { - method = nvps.getValue(name1); + method = nvps.get(name1); } else if (name1.equals(AD_LOCATION_TYPE)) { - locationType = nvps.getValue(name1); + locationType = nvps.get(name1); } else if (name1.equals(AD_LOCATION)) { - location = nvps.getValue(name1); + location = nvps.get(name1); } else if (name1.equals(AD_ENABLE)) { - enable = nvps.getValue(name1); + enable = nvps.get(name1); } } @@ -352,18 +350,18 @@ public class AuthInfoAccessExtDefault extends EnrollExtDefault { des = ext.getAccessDescription(i); } if (des == null) { - np.add(AD_METHOD, ""); - np.add(AD_LOCATION_TYPE, ""); - np.add(AD_LOCATION, ""); - np.add(AD_ENABLE, "false"); + np.put(AD_METHOD, ""); + np.put(AD_LOCATION_TYPE, ""); + np.put(AD_LOCATION, ""); + np.put(AD_ENABLE, "false"); } else { ObjectIdentifier methodOid = des.getMethod(); GeneralName gn = des.getLocation(); - np.add(AD_METHOD, methodOid.toString()); - np.add(AD_LOCATION_TYPE, getGeneralNameType(gn)); - np.add(AD_LOCATION, getGeneralNameValue(gn)); - np.add(AD_ENABLE, "true"); + np.put(AD_METHOD, methodOid.toString()); + np.put(AD_LOCATION_TYPE, getGeneralNameType(gn)); + np.put(AD_LOCATION, getGeneralNameValue(gn)); + np.put(AD_ENABLE, "true"); } recs.addElement(np); } diff --git a/pki/base/common/src/com/netscape/cms/profile/def/CRLDistributionPointsExtDefault.java b/pki/base/common/src/com/netscape/cms/profile/def/CRLDistributionPointsExtDefault.java index a95ec6b7d..c7a0f9abd 100644 --- a/pki/base/common/src/com/netscape/cms/profile/def/CRLDistributionPointsExtDefault.java +++ b/pki/base/common/src/com/netscape/cms/profile/def/CRLDistributionPointsExtDefault.java @@ -247,8 +247,7 @@ public class CRLDistributionPointsExtDefault extends EnrollExtDefault { int i = 0; for (; i < size; i++) { - NameValuePairs nvps = (NameValuePairs) v.elementAt(i); - Enumeration names = nvps.getNames(); + NameValuePairs nvps = v.elementAt(i); String pointType = null; String pointValue = null; String issuerType = null; @@ -256,21 +255,20 @@ public class CRLDistributionPointsExtDefault extends EnrollExtDefault { String enable = null; CRLDistributionPoint cdp = new CRLDistributionPoint(); - while (names.hasMoreElements()) { - String name1 = (String) names.nextElement(); + for (String name1 : nvps.keySet()) { if (name1.equals(REASONS)) { - addReasons(locale, cdp, REASONS, nvps.getValue(name1)); + addReasons(locale, cdp, REASONS, nvps.get(name1)); } else if (name1.equals(POINT_TYPE)) { - pointType = nvps.getValue(name1); + pointType = nvps.get(name1); } else if (name1.equals(POINT_NAME)) { - pointValue = nvps.getValue(name1); + pointValue = nvps.get(name1); } else if (name1.equals(ISSUER_TYPE)) { - issuerType = nvps.getValue(name1); + issuerType = nvps.get(name1); } else if (name1.equals(ISSUER_NAME)) { - issuerValue = nvps.getValue(name1); + issuerValue = nvps.get(name1); } else if (name1.equals(ENABLE)) { - enable = nvps.getValue(name1); + enable = nvps.get(name1); } } @@ -466,12 +464,12 @@ public class CRLDistributionPointsExtDefault extends EnrollExtDefault { protected NameValuePairs buildEmptyGeneralNames() { NameValuePairs pairs = new NameValuePairs(); - pairs.add(POINT_TYPE, ""); - pairs.add(POINT_NAME, ""); - pairs.add(REASONS, ""); - pairs.add(ISSUER_TYPE, ""); - pairs.add(ISSUER_NAME, ""); - pairs.add(ENABLE, "false"); + pairs.put(POINT_TYPE, ""); + pairs.put(POINT_NAME, ""); + pairs.put(REASONS, ""); + pairs.put(ISSUER_TYPE, ""); + pairs.put(ISSUER_NAME, ""); + pairs.put(ENABLE, "false"); return pairs; } @@ -483,16 +481,16 @@ public class CRLDistributionPointsExtDefault extends EnrollExtDefault { RDN rdn = null; boolean hasFullName = false; - pairs.add(ENABLE, "true"); + pairs.put(ENABLE, "true"); if (gns == null) { rdn = p.getRelativeName(); if (rdn != null) { hasFullName = true; - pairs.add(POINT_TYPE, RELATIVETOISSUER); - pairs.add(POINT_NAME, rdn.toString()); + pairs.put(POINT_TYPE, RELATIVETOISSUER); + pairs.put(POINT_NAME, rdn.toString()); } else { - pairs.add(POINT_TYPE, ""); - pairs.add(POINT_NAME, ""); + pairs.put(POINT_TYPE, ""); + pairs.put(POINT_NAME, ""); } } else { GeneralName gn = (GeneralName) gns.elementAt(0); @@ -501,30 +499,30 @@ public class CRLDistributionPointsExtDefault extends EnrollExtDefault { hasFullName = true; int type = gn.getType(); - pairs.add(POINT_TYPE, getGeneralNameType(gn)); - pairs.add(POINT_NAME, getGeneralNameValue(gn)); + pairs.put(POINT_TYPE, getGeneralNameType(gn)); + pairs.put(POINT_NAME, getGeneralNameValue(gn)); } } if (!hasFullName) { - pairs.add(POINT_TYPE, GN_DIRECTORY_NAME); - pairs.add(POINT_NAME, ""); + pairs.put(POINT_TYPE, GN_DIRECTORY_NAME); + pairs.put(POINT_NAME, ""); } BitArray reasons = p.getReasons(); String s = convertBitArrayToReasonNames(reasons); if (s.length() > 0) { - pairs.add(REASONS, s); + pairs.put(REASONS, s); } else { - pairs.add(REASONS, ""); + pairs.put(REASONS, ""); } gns = p.getCRLIssuer(); if (gns == null) { - pairs.add(ISSUER_TYPE, GN_DIRECTORY_NAME); - pairs.add(ISSUER_NAME, ""); + pairs.put(ISSUER_TYPE, GN_DIRECTORY_NAME); + pairs.put(ISSUER_NAME, ""); } else { GeneralName gn = (GeneralName) gns.elementAt(0); @@ -532,8 +530,8 @@ public class CRLDistributionPointsExtDefault extends EnrollExtDefault { hasFullName = true; int type = gn.getType(); - pairs.add(ISSUER_TYPE, getGeneralNameType(gn)); - pairs.add(ISSUER_NAME, getGeneralNameValue(gn)); + pairs.put(ISSUER_TYPE, getGeneralNameType(gn)); + pairs.put(ISSUER_NAME, getGeneralNameValue(gn)); } } return pairs; diff --git a/pki/base/common/src/com/netscape/cms/profile/def/EnrollDefault.java b/pki/base/common/src/com/netscape/cms/profile/def/EnrollDefault.java index 855cd92c7..5310986d5 100644 --- a/pki/base/common/src/com/netscape/cms/profile/def/EnrollDefault.java +++ b/pki/base/common/src/com/netscape/cms/profile/def/EnrollDefault.java @@ -641,11 +641,9 @@ public abstract class EnrollDefault implements IPolicyDefault, ICertInfoPolicyDe sb.append("Record #"); sb.append(i); sb.append("\r\n"); - Enumeration e = pairs.getNames(); - while (e.hasMoreElements()) { - String key = e.nextElement(); - String val = pairs.getValue(key); + for (String key : pairs.keySet()) { + String val = pairs.get(key); sb.append(key); sb.append(":"); @@ -691,9 +689,9 @@ public abstract class EnrollDefault implements IPolicyDefault, ICertInfoPolicyDe throw new EPropertyException("Bad Input Format"); } else { if (pos == (token.length() - 1)) { - nvps.add(token.substring(0, pos), ""); + nvps.put(token.substring(0, pos), ""); } else { - nvps.add(token.substring(0, pos), token.substring(pos + 1)); + nvps.put(token.substring(0, pos), token.substring(pos + 1)); } } } diff --git a/pki/base/common/src/com/netscape/cms/profile/def/FreshestCRLExtDefault.java b/pki/base/common/src/com/netscape/cms/profile/def/FreshestCRLExtDefault.java index a2de3f491..739fd3448 100644 --- a/pki/base/common/src/com/netscape/cms/profile/def/FreshestCRLExtDefault.java +++ b/pki/base/common/src/com/netscape/cms/profile/def/FreshestCRLExtDefault.java @@ -230,8 +230,7 @@ public class FreshestCRLExtDefault extends EnrollExtDefault { int i = 0; for (; i < size; i++) { - NameValuePairs nvps = (NameValuePairs) v.elementAt(i); - Enumeration names = nvps.getNames(); + NameValuePairs nvps = v.elementAt(i); String pointType = null; String pointValue = null; String issuerType = null; @@ -239,19 +238,18 @@ public class FreshestCRLExtDefault extends EnrollExtDefault { String enable = null; CRLDistributionPoint cdp = new CRLDistributionPoint(); - while (names.hasMoreElements()) { - String name1 = (String) names.nextElement(); + for (String name1 : nvps.keySet()) { if (name1.equals(POINT_TYPE)) { - pointType = nvps.getValue(name1); + pointType = nvps.get(name1); } else if (name1.equals(POINT_NAME)) { - pointValue = nvps.getValue(name1); + pointValue = nvps.get(name1); } else if (name1.equals(ISSUER_TYPE)) { - issuerType = nvps.getValue(name1); + issuerType = nvps.get(name1); } else if (name1.equals(ISSUER_NAME)) { - issuerValue = nvps.getValue(name1); + issuerValue = nvps.get(name1); } else if (name1.equals(ENABLE)) { - enable = nvps.getValue(name1); + enable = nvps.get(name1); } } @@ -409,11 +407,11 @@ public class FreshestCRLExtDefault extends EnrollExtDefault { protected NameValuePairs buildEmptyGeneralNames() { NameValuePairs pairs = new NameValuePairs(); - pairs.add(POINT_TYPE, ""); - pairs.add(POINT_NAME, ""); - pairs.add(ISSUER_TYPE, ""); - pairs.add(ISSUER_NAME, ""); - pairs.add(ENABLE, "false"); + pairs.put(POINT_TYPE, ""); + pairs.put(POINT_NAME, ""); + pairs.put(ISSUER_TYPE, ""); + pairs.put(ISSUER_NAME, ""); + pairs.put(ENABLE, "false"); return pairs; } @@ -424,10 +422,10 @@ public class FreshestCRLExtDefault extends EnrollExtDefault { boolean hasFullName = false; - pairs.add(ENABLE, "true"); + pairs.put(ENABLE, "true"); if (gns == null) { - pairs.add(POINT_TYPE, ""); - pairs.add(POINT_NAME, ""); + pairs.put(POINT_TYPE, ""); + pairs.put(POINT_NAME, ""); } else { GeneralName gn = (GeneralName) gns.elementAt(0); @@ -435,21 +433,21 @@ public class FreshestCRLExtDefault extends EnrollExtDefault { hasFullName = true; int type = gn.getType(); - pairs.add(POINT_TYPE, getGeneralNameType(gn)); - pairs.add(POINT_NAME, getGeneralNameValue(gn)); + pairs.put(POINT_TYPE, getGeneralNameType(gn)); + pairs.put(POINT_NAME, getGeneralNameValue(gn)); } } if (!hasFullName) { - pairs.add(POINT_TYPE, GN_DIRECTORY_NAME); - pairs.add(POINT_NAME, ""); + pairs.put(POINT_TYPE, GN_DIRECTORY_NAME); + pairs.put(POINT_NAME, ""); } gns = p.getCRLIssuer(); if (gns == null) { - pairs.add(ISSUER_TYPE, GN_DIRECTORY_NAME); - pairs.add(ISSUER_NAME, ""); + pairs.put(ISSUER_TYPE, GN_DIRECTORY_NAME); + pairs.put(ISSUER_NAME, ""); } else { GeneralName gn = (GeneralName) gns.elementAt(0); @@ -457,8 +455,8 @@ public class FreshestCRLExtDefault extends EnrollExtDefault { hasFullName = true; int type = gn.getType(); - pairs.add(ISSUER_TYPE, getGeneralNameType(gn)); - pairs.add(ISSUER_NAME, getGeneralNameValue(gn)); + pairs.put(ISSUER_TYPE, getGeneralNameType(gn)); + pairs.put(ISSUER_NAME, getGeneralNameValue(gn)); } } return pairs; diff --git a/pki/base/common/src/com/netscape/cms/profile/def/NameConstraintsExtDefault.java b/pki/base/common/src/com/netscape/cms/profile/def/NameConstraintsExtDefault.java index c513c332b..e57d04067 100644 --- a/pki/base/common/src/com/netscape/cms/profile/def/NameConstraintsExtDefault.java +++ b/pki/base/common/src/com/netscape/cms/profile/def/NameConstraintsExtDefault.java @@ -366,20 +366,18 @@ public class NameConstraintsExtDefault extends EnrollExtDefault { Vector subtrees = new Vector(); for (int i = 0; i < size; i++) { - NameValuePairs nvps = (NameValuePairs) v.elementAt(i); - Enumeration names = nvps.getNames(); + NameValuePairs nvps = v.elementAt(i); - while (names.hasMoreElements()) { - String name1 = (String) names.nextElement(); + for (String name1 : nvps.keySet()) { if (name1.equals(GENERAL_NAME_CHOICE)) { - choice = nvps.getValue(name1); + choice = nvps.get(name1); } else if (name1.equals(GENERAL_NAME_VALUE)) { - val = nvps.getValue(name1); + val = nvps.get(name1); } else if (name1.equals(MIN_VALUE)) { - minS = nvps.getValue(name1); + minS = nvps.get(name1); } else if (name1.equals(MAX_VALUE)) { - maxS = nvps.getValue(name1); + maxS = nvps.get(name1); } } @@ -527,11 +525,11 @@ public class NameConstraintsExtDefault extends EnrollExtDefault { NameValuePairs pairs = new NameValuePairs(); - pairs.add(GENERAL_NAME_CHOICE, type); - pairs.add(GENERAL_NAME_VALUE, getGeneralNameValue(gn)); - pairs.add(MIN_VALUE, Integer.toString(min)); - pairs.add(MAX_VALUE, Integer.toString(max)); - pairs.add(ENABLE, "true"); + pairs.put(GENERAL_NAME_CHOICE, type); + pairs.put(GENERAL_NAME_VALUE, getGeneralNameValue(gn)); + pairs.put(MIN_VALUE, Integer.toString(min)); + pairs.put(MAX_VALUE, Integer.toString(max)); + pairs.put(ENABLE, "true"); recs.addElement(pairs); } diff --git a/pki/base/common/src/com/netscape/cms/profile/def/PolicyMappingsExtDefault.java b/pki/base/common/src/com/netscape/cms/profile/def/PolicyMappingsExtDefault.java index 183ef87bc..1f05fef3e 100644 --- a/pki/base/common/src/com/netscape/cms/profile/def/PolicyMappingsExtDefault.java +++ b/pki/base/common/src/com/netscape/cms/profile/def/PolicyMappingsExtDefault.java @@ -216,18 +216,16 @@ public class PolicyMappingsExtDefault extends EnrollExtDefault { Vector policyMaps = new Vector(); for (int i = 0; i < size; i++) { - NameValuePairs nvps = (NameValuePairs) v.elementAt(i); - Enumeration names = nvps.getNames(); + NameValuePairs nvps = v.elementAt(i); - while (names.hasMoreElements()) { - String name1 = (String) names.nextElement(); + for (String name1 : nvps.keySet()) { if (name1.equals(ISSUER_POLICY_ID)) { - issuerPolicyId = nvps.getValue(name1); + issuerPolicyId = nvps.get(name1); } else if (name1.equals(SUBJECT_POLICY_ID)) { - subjectPolicyId = nvps.getValue(name1); + subjectPolicyId = nvps.get(name1); } else if (name1.equals(POLICY_ID_ENABLE)) { - enable = nvps.getValue(name1); + enable = nvps.get(name1); } } @@ -327,13 +325,13 @@ public class PolicyMappingsExtDefault extends EnrollExtDefault { CertificatePolicyId i1 = map.getIssuerIdentifier(); CertificatePolicyId s1 = map.getSubjectIdentifier(); - pairs.add(ISSUER_POLICY_ID, i1.getIdentifier().toString()); - pairs.add(SUBJECT_POLICY_ID, s1.getIdentifier().toString()); - pairs.add(POLICY_ID_ENABLE, "true"); + pairs.put(ISSUER_POLICY_ID, i1.getIdentifier().toString()); + pairs.put(SUBJECT_POLICY_ID, s1.getIdentifier().toString()); + pairs.put(POLICY_ID_ENABLE, "true"); } else { - pairs.add(ISSUER_POLICY_ID, ""); - pairs.add(SUBJECT_POLICY_ID, ""); - pairs.add(POLICY_ID_ENABLE, "false"); + pairs.put(ISSUER_POLICY_ID, ""); + pairs.put(SUBJECT_POLICY_ID, ""); + pairs.put(POLICY_ID_ENABLE, "false"); } recs.addElement(pairs); diff --git a/pki/base/common/src/com/netscape/cms/profile/def/SubjectDirAttributesExtDefault.java b/pki/base/common/src/com/netscape/cms/profile/def/SubjectDirAttributesExtDefault.java index a706fb4ad..cca5ab234 100644 --- a/pki/base/common/src/com/netscape/cms/profile/def/SubjectDirAttributesExtDefault.java +++ b/pki/base/common/src/com/netscape/cms/profile/def/SubjectDirAttributesExtDefault.java @@ -220,19 +220,18 @@ public class SubjectDirAttributesExtDefault extends EnrollExtDefault { Vector attrV = new Vector(); for (int i = 0; i < size; i++) { NameValuePairs nvps = v.elementAt(i); - Enumeration names = nvps.getNames(); String attrName = null; String attrValue = null; String enable = "false"; - while (names.hasMoreElements()) { - String name1 = names.nextElement(); + + for (String name1 : nvps.keySet()) { if (name1.equals(ATTR_NAME)) { - attrName = nvps.getValue(name1); + attrName = nvps.get(name1); } else if (name1.equals(ATTR_VALUE)) { - attrValue = nvps.getValue(name1); + attrValue = nvps.get(name1); } else if (name1.equals(ENABLE)) { - enable = nvps.getValue(name1); + enable = nvps.get(name1); } } @@ -316,7 +315,7 @@ public class SubjectDirAttributesExtDefault extends EnrollExtDefault { while (e.hasMoreElements()) { NameValuePairs pairs = new NameValuePairs(); - pairs.add(ENABLE, "true"); + pairs.put(ENABLE, "true"); Attribute attr = e.nextElement(); CMS.debug("SubjectDirAttributesExtDefault: getValue: attribute=" + attr); ObjectIdentifier oid = attr.getOid(); @@ -325,9 +324,9 @@ public class SubjectDirAttributesExtDefault extends EnrollExtDefault { String vv = map.getName(oid); if (vv != null) - pairs.add(ATTR_NAME, vv); + pairs.put(ATTR_NAME, vv); else - pairs.add(ATTR_NAME, oid.toString()); + pairs.put(ATTR_NAME, oid.toString()); Enumeration v = attr.getValues(); // just support single value for now @@ -341,16 +340,16 @@ public class SubjectDirAttributesExtDefault extends EnrollExtDefault { } } - pairs.add(ATTR_VALUE, ss.toString()); + pairs.put(ATTR_VALUE, ss.toString()); recs.addElement(pairs); i++; } for (; i < num; i++) { NameValuePairs pairs = new NameValuePairs(); - pairs.add(ENABLE, "false"); - pairs.add(ATTR_NAME, "GENERATIONQUALIFIER"); - pairs.add(ATTR_VALUE, ""); + pairs.put(ENABLE, "false"); + pairs.put(ATTR_NAME, "GENERATIONQUALIFIER"); + pairs.put(ATTR_VALUE, ""); recs.addElement(pairs); } diff --git a/pki/base/common/src/com/netscape/cms/profile/def/SubjectInfoAccessExtDefault.java b/pki/base/common/src/com/netscape/cms/profile/def/SubjectInfoAccessExtDefault.java index 670728e59..104b29a08 100644 --- a/pki/base/common/src/com/netscape/cms/profile/def/SubjectInfoAccessExtDefault.java +++ b/pki/base/common/src/com/netscape/cms/profile/def/SubjectInfoAccessExtDefault.java @@ -231,19 +231,17 @@ public class SubjectInfoAccessExtDefault extends EnrollExtDefault { for (int i = 0; i < size; i++) { NameValuePairs nvps = v.elementAt(i); - Enumeration names = nvps.getNames(); - while (names.hasMoreElements()) { - String name1 = names.nextElement(); + for (String name1 : nvps.keySet()) { if (name1.equals(AD_METHOD)) { - method = nvps.getValue(name1); + method = nvps.get(name1); } else if (name1.equals(AD_LOCATION_TYPE)) { - locationType = nvps.getValue(name1); + locationType = nvps.get(name1); } else if (name1.equals(AD_LOCATION)) { - location = nvps.getValue(name1); + location = nvps.get(name1); } else if (name1.equals(AD_ENABLE)) { - enable = nvps.getValue(name1); + enable = nvps.get(name1); } } @@ -347,18 +345,18 @@ public class SubjectInfoAccessExtDefault extends EnrollExtDefault { des = ext.getAccessDescription(i); } if (des == null) { - np.add(AD_METHOD, ""); - np.add(AD_LOCATION_TYPE, ""); - np.add(AD_LOCATION, ""); - np.add(AD_ENABLE, "false"); + np.put(AD_METHOD, ""); + np.put(AD_LOCATION_TYPE, ""); + np.put(AD_LOCATION, ""); + np.put(AD_ENABLE, "false"); } else { ObjectIdentifier methodOid = des.getMethod(); GeneralName gn = des.getLocation(); - np.add(AD_METHOD, methodOid.toString()); - np.add(AD_LOCATION_TYPE, getGeneralNameType(gn)); - np.add(AD_LOCATION, getGeneralNameValue(gn)); - np.add(AD_ENABLE, "true"); + np.put(AD_METHOD, methodOid.toString()); + np.put(AD_LOCATION_TYPE, getGeneralNameType(gn)); + np.put(AD_LOCATION, getGeneralNameValue(gn)); + np.put(AD_ENABLE, "true"); } recs.addElement(np); } diff --git a/pki/base/common/src/com/netscape/cms/servlet/admin/ACLAdminServlet.java b/pki/base/common/src/com/netscape/cms/servlet/admin/ACLAdminServlet.java index 29088fc2d..bde075334 100644 --- a/pki/base/common/src/com/netscape/cms/servlet/admin/ACLAdminServlet.java +++ b/pki/base/common/src/com/netscape/cms/servlet/admin/ACLAdminServlet.java @@ -260,9 +260,9 @@ public class ACLAdminServlet extends AdminServlet { String desc = acl.getDescription(); if (desc == null) - params.add(acl.getName(), ""); + params.put(acl.getName(), ""); else - params.add(acl.getName(), desc); + params.put(acl.getName(), desc); } sendResponse(SUCCESS, null, params, resp); @@ -305,7 +305,7 @@ public class ACLAdminServlet extends AdminServlet { } } - params.add(Constants.PR_ACL_OPS, rights.toString()); + params.put(Constants.PR_ACL_OPS, rights.toString()); Enumeration aclEntryEnum; aclEntryEnum = acl.entries(); @@ -323,7 +323,7 @@ public class ACLAdminServlet extends AdminServlet { } } - params.add(Constants.PR_ACI, acis); + params.put(Constants.PR_ACI, acis); sendResponse(SUCCESS, null, params, resp); return; @@ -472,7 +472,7 @@ public class ACLAdminServlet extends AdminServlet { IAccessEvaluator evaluator = res.nextElement(); // params.add(evaluator.getType(), evaluator.getDescription()); - params.add(evaluator.getType(), evaluator.getClass().getName()); + params.put(evaluator.getType(), evaluator.getClass().getName()); } sendResponse(SUCCESS, null, params, resp); @@ -495,7 +495,7 @@ public class ACLAdminServlet extends AdminServlet { str.append(operators[i]); } - params.add(evaluator.getType(), str.toString()); + params.put(evaluator.getType(), str.toString()); } sendResponse(SUCCESS, null, params, resp); diff --git a/pki/base/common/src/com/netscape/cms/servlet/admin/AdminServlet.java b/pki/base/common/src/com/netscape/cms/servlet/admin/AdminServlet.java index 0e3d2c228..c7d7d619e 100644 --- a/pki/base/common/src/com/netscape/cms/servlet/admin/AdminServlet.java +++ b/pki/base/common/src/com/netscape/cms/servlet/admin/AdminServlet.java @@ -21,7 +21,9 @@ import java.io.ByteArrayOutputStream; import java.io.DataOutputStream; import java.io.IOException; import java.security.cert.X509Certificate; +import java.util.Collection; import java.util.Enumeration; +import java.util.Iterator; import java.util.Locale; import java.util.StringTokenizer; @@ -794,17 +796,17 @@ public class AdminServlet extends HttpServlet { StringBuffer buf = new StringBuffer(); if (params != null) { - Enumeration e = params.getNames(); + Collection names = params.keySet(); - if (e.hasMoreElements()) { - while (e.hasMoreElements()) { - String name = e.nextElement(); - String value = java.net.URLEncoder.encode((String) - params.getValue(name)); + if (!names.isEmpty()) { + for (Iterator i = names.iterator(); i.hasNext(); ) { + String name = i.next(); + String value = java.net.URLEncoder.encode( + params.get(name)); buf.append(java.net.URLEncoder.encode(name) + "=" + value); - if (e.hasMoreElements()) + if (i.hasNext()) buf.append("&"); } byte content[] = buf.toString().getBytes(); @@ -875,7 +877,7 @@ public class AdminServlet extends HttpServlet { //System.out.println(name); //System.out.println(name+","+config.getString(name)); - params.add(name, config.getString(name)); + params.put(name, config.getString(name)); } sendResponse(SUCCESS, null, params, resp); } @@ -925,7 +927,7 @@ public class AdminServlet extends HttpServlet { while (e.hasMoreElements()) { String s = e.nextElement(); - params.add(s, config.getString(s)); + params.put(s, config.getString(s)); } sendResponse(SUCCESS, null, params, resp); } @@ -1275,7 +1277,7 @@ public class AdminServlet extends HttpServlet { String paramName = s[i].substring(0, j); String args = s[i].substring(j + 1); - nvps.add(paramName, args); + nvps.put(paramName, args); } return nvps; diff --git a/pki/base/common/src/com/netscape/cms/servlet/admin/AuthAdminServlet.java b/pki/base/common/src/com/netscape/cms/servlet/admin/AuthAdminServlet.java index 6a7ad9599..cacd0b5d0 100644 --- a/pki/base/common/src/com/netscape/cms/servlet/admin/AuthAdminServlet.java +++ b/pki/base/common/src/com/netscape/cms/servlet/admin/AuthAdminServlet.java @@ -168,7 +168,7 @@ public class AuthAdminServlet extends AdminServlet { // no need to authenticate this. if we're alive, return true. NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_PING, Constants.TRUE); + params.put(Constants.PR_PING, Constants.TRUE); sendResponse(SUCCESS, null, params, resp); return; } else { @@ -188,7 +188,7 @@ public class AuthAdminServlet extends AdminServlet { String val = configStore.getString("authType", "pwd"); NameValuePairs params = new NameValuePairs(); - params.add("authType", val); + params.put("authType", val); sendResponse(SUCCESS, null, params, resp); return; } @@ -856,7 +856,7 @@ public class AuthAdminServlet extends AdminServlet { NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_AUTH_IMPL_NAME, implname); + params.put(Constants.PR_AUTH_IMPL_NAME, implname); // store a message in the signed audit log file auditMessage = CMS.getLogMessage( @@ -921,7 +921,7 @@ public class AuthAdminServlet extends AdminServlet { mAuths.getPlugins().get(name); if (value.isVisible()) { - params.add(name, value.getClassPath() + EDIT); + params.put(name, value.getClassPath() + EDIT); } } sendResponse(SUCCESS, null, params, resp); @@ -948,9 +948,9 @@ public class AuthAdminServlet extends AdminServlet { mAuths.getPlugins().get(value.getImplName()); if (!amgrplugin.isVisible()) { - params.add(name, value.getImplName() + ";invisible;" + enableStr); + params.put(name, value.getImplName() + ";invisible;" + enableStr); } else { - params.add(name, value.getImplName() + ";visible;" + enableStr); + params.put(name, value.getImplName() + ";visible;" + enableStr); } } sendResponse(SUCCESS, null, params, resp); @@ -1316,10 +1316,10 @@ public class AuthAdminServlet extends AdminServlet { NameValuePairs params = new NameValuePairs(); // implName is always required so always send it. - params.add(Constants.PR_AUTH_IMPL_NAME, ""); + params.put(Constants.PR_AUTH_IMPL_NAME, ""); if (configParams != null) { for (int i = 0; i < configParams.length; i++) { - params.add(configParams[i], ""); + params.put(configParams[i], ""); } } sendResponse(0, null, params, resp); @@ -1355,7 +1355,7 @@ public class AuthAdminServlet extends AdminServlet { String[] configParams = mgrInst.getConfigParams(); NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_AUTH_IMPL_NAME, mgrInst.getImplName()); + params.put(Constants.PR_AUTH_IMPL_NAME, mgrInst.getImplName()); // implName is always required so always send it. if (configParams != null) { for (int i = 0; i < configParams.length; i++) { @@ -1363,9 +1363,9 @@ public class AuthAdminServlet extends AdminServlet { String val = (String) config.get(key); if (val != null) { - params.add(key, val); + params.put(key, val); } else { - params.add(key, ""); + params.put(key, ""); } } } @@ -1501,7 +1501,7 @@ public class AuthAdminServlet extends AdminServlet { NameValuePairs saveParams = new NameValuePairs(); // implName is always required so always include it it. - saveParams.add(IAuthSubsystem.PROP_PLUGIN, + saveParams.put(IAuthSubsystem.PROP_PLUGIN, (String) oldConfig.get(IAuthSubsystem.PROP_PLUGIN)); if (oldConfigParms != null) { for (int i = 0; i < oldConfigParms.length; i++) { @@ -1509,7 +1509,7 @@ public class AuthAdminServlet extends AdminServlet { Object val = oldConfig.get(key); if (val != null) { - saveParams.add(key, (String) val); + saveParams.put(key, (String) val); } } } @@ -1711,11 +1711,8 @@ public class AuthAdminServlet extends AdminServlet { store.removeSubStore(id); IConfigStore rstore = store.makeSubStore(id); - Enumeration keys = saveParams.getNames(); - - while (keys.hasMoreElements()) { - String key = (String) keys.nextElement(); - String value = saveParams.getValue(key); + for (String key : saveParams.keySet()) { + String value = saveParams.get(key); if (value != null) rstore.put(key, value); diff --git a/pki/base/common/src/com/netscape/cms/servlet/admin/CAAdminServlet.java b/pki/base/common/src/com/netscape/cms/servlet/admin/CAAdminServlet.java index 5bcaa46d2..e7b32e844 100644 --- a/pki/base/common/src/com/netscape/cms/servlet/admin/CAAdminServlet.java +++ b/pki/base/common/src/com/netscape/cms/servlet/admin/CAAdminServlet.java @@ -243,10 +243,10 @@ public class CAAdminServlet extends AdminServlet { continue; if (name.equals(Constants.PR_ENABLE)) continue; - params.add(name, rc.getString(name, "")); + params.put(name, rc.getString(name, "")); } - params.add(Constants.PR_ENABLE, + params.put(Constants.PR_ENABLE, rc.getString(PROP_ENABLED, Constants.FALSE)); sendResponse(SUCCESS, null, params, resp); } @@ -304,10 +304,10 @@ public class CAAdminServlet extends AdminServlet { continue; if (name.equals(Constants.PR_ENABLE)) continue; - params.add(name, riq.getString(name, "")); + params.put(name, riq.getString(name, "")); } - params.add(Constants.PR_ENABLE, + params.put(Constants.PR_ENABLE, riq.getString(PROP_ENABLED, Constants.FALSE)); sendResponse(SUCCESS, null, params, resp); } @@ -462,8 +462,8 @@ public class CAAdminServlet extends AdminServlet { String ipId = ip.getId(); if (ipId != null && ipId.length() > 0) - params.add(ipId, ip.getDescription()); - params.add(ipId + "." + Constants.PR_ENABLED, + params.put(ipId, ip.getDescription()); + params.put(ipId + "." + Constants.PR_ENABLED, (Boolean.valueOf(ip.isCRLIssuingPointEnabled())).toString()); } } @@ -504,7 +504,7 @@ public class CAAdminServlet extends AdminServlet { if (name.equals(Constants.PR_CLASS)) value = ip.getClass().getName(); - params.add(name, value); + params.put(name, value); } } } @@ -552,7 +552,7 @@ public class CAAdminServlet extends AdminServlet { sendResponse(ERROR, "Missing CRL IP name", null, resp); return; } - params.add(Constants.PR_ID, ipId); + params.put(Constants.PR_ID, ipId); String desc = req.getParameter(Constants.PR_DESCRIPTION); @@ -569,7 +569,7 @@ public class CAAdminServlet extends AdminServlet { sendResponse(ERROR, "Missing CRL IP description", null, resp); return; } - params.add(Constants.PR_DESCRIPTION, desc); + params.put(Constants.PR_DESCRIPTION, desc); String sEnable = req.getParameter(Constants.PR_ENABLED); boolean enable = true; @@ -577,9 +577,9 @@ public class CAAdminServlet extends AdminServlet { if (sEnable != null && sEnable.length() > 0 && sEnable.equalsIgnoreCase(Constants.FALSE)) { enable = false; - params.add(Constants.PR_ENABLED, Constants.FALSE); + params.put(Constants.PR_ENABLED, Constants.FALSE); } else { - params.add(Constants.PR_ENABLED, Constants.TRUE); + params.put(Constants.PR_ENABLED, Constants.TRUE); } IConfigStore crlSubStore = @@ -708,7 +708,7 @@ public class CAAdminServlet extends AdminServlet { sendResponse(ERROR, "Missing CRL IP name", null, resp); return; } - params.add(Constants.PR_ID, ipId); + params.put(Constants.PR_ID, ipId); String desc = req.getParameter(Constants.PR_DESCRIPTION); @@ -725,7 +725,7 @@ public class CAAdminServlet extends AdminServlet { sendResponse(ERROR, "Missing CRL IP description", null, resp); return; } - params.add(Constants.PR_DESCRIPTION, desc); + params.put(Constants.PR_DESCRIPTION, desc); String sEnable = req.getParameter(Constants.PR_ENABLED); boolean enable = true; @@ -733,9 +733,9 @@ public class CAAdminServlet extends AdminServlet { if (sEnable != null && sEnable.length() > 0 && sEnable.equalsIgnoreCase(Constants.FALSE)) { enable = false; - params.add(Constants.PR_ENABLED, Constants.FALSE); + params.put(Constants.PR_ENABLED, Constants.FALSE); } else { - params.add(Constants.PR_ENABLED, Constants.TRUE); + params.put(Constants.PR_ENABLED, Constants.TRUE); } IConfigStore crlSubStore = @@ -1036,7 +1036,7 @@ public class CAAdminServlet extends AdminServlet { continue; String value = req.getParameter(name); - params.add(name, value); + params.put(name, value); } crlExts.setConfigParams(id, params, crlExtSubStore); commit(true); @@ -1125,7 +1125,7 @@ public class CAAdminServlet extends AdminServlet { crlExtEnabled = crlExtSubStore.getBoolean(name, false); } } - params.add(extName, extName + ";visible;" + ((crlExtEnabled) ? "enabled" : "disabled")); + params.put(extName, extName + ";visible;" + ((crlExtEnabled) ? "enabled" : "disabled")); } } @@ -1245,7 +1245,7 @@ public class CAAdminServlet extends AdminServlet { continue; String value = req.getParameter(name); - params.add(name, value); + params.put(name, value); crlSubStore.putString(name, value); } boolean noRestart = ip.updateConfig(params); @@ -1334,7 +1334,7 @@ public class CAAdminServlet extends AdminServlet { continue; if (name.equals(Constants.PR_ENABLE)) continue; - params.add(name, crlSubStore.getString(name, "")); + params.put(name, crlSubStore.getString(name, "")); } getSigningAlgConfig(params); @@ -1369,7 +1369,7 @@ public class CAAdminServlet extends AdminServlet { if (name.equals(Constants.OP_TYPE)) continue; - params.add(name, caConnectorConfig.getString(name, "")); + params.put(name, caConnectorConfig.getString(name, "")); } } sendResponse(SUCCESS, null, params, resp); @@ -1488,7 +1488,7 @@ public class CAAdminServlet extends AdminServlet { IConfigStore caConfig = mCA.getConfigStore(); value = caConfig.getString(ICertificateAuthority.PROP_ENABLE_PAST_CATIME, "false"); - params.add(Constants.PR_VALIDITY, value); + params.put(Constants.PR_VALIDITY, value); getSigningAlgConfig(params); getSerialConfig(params); @@ -1498,7 +1498,7 @@ public class CAAdminServlet extends AdminServlet { } private void getSigningAlgConfig(NameValuePairs params) { - params.add(Constants.PR_DEFAULT_ALGORITHM, + params.put(Constants.PR_DEFAULT_ALGORITHM, mCA.getDefaultAlgorithm()); String[] algorithms = mCA.getCASigningAlgorithms(); StringBuffer algorStr = new StringBuffer(); @@ -1511,16 +1511,16 @@ public class CAAdminServlet extends AdminServlet { algorStr.append(algorithms[i]); } } - params.add(Constants.PR_ALL_ALGORITHMS, algorStr.toString()); + params.put(Constants.PR_ALL_ALGORITHMS, algorStr.toString()); } private void getSerialConfig(NameValuePairs params) { - params.add(Constants.PR_SERIAL, + params.put(Constants.PR_SERIAL, mCA.getStartSerial()); } private void getMaxSerialConfig(NameValuePairs params) { - params.add(Constants.PR_MAXSERIAL, + params.put(Constants.PR_MAXSERIAL, mCA.getMaxSerial()); } diff --git a/pki/base/common/src/com/netscape/cms/servlet/admin/CMSAdminServlet.java b/pki/base/common/src/com/netscape/cms/servlet/admin/CMSAdminServlet.java index 58fb1d03f..3a2ae0a11 100644 --- a/pki/base/common/src/com/netscape/cms/servlet/admin/CMSAdminServlet.java +++ b/pki/base/common/src/com/netscape/cms/servlet/admin/CMSAdminServlet.java @@ -320,9 +320,9 @@ public final class CMSAdminServlet extends AdminServlet { NameValuePairs params = new NameValuePairs(); if (File.separator.equals("\\")) - params.add(Constants.PR_NT, Constants.TRUE); + params.put(Constants.PR_NT, Constants.TRUE); else - params.add(Constants.PR_NT, Constants.FALSE); + params.put(Constants.PR_NT, Constants.FALSE); sendResponse(SUCCESS, null, params, resp); } @@ -335,7 +335,7 @@ public final class CMSAdminServlet extends AdminServlet { CMS.getSubsystem(CMS.SUBSYSTEM_CRYPTO); NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_TOKEN_LIST, jssSubSystem.getTokenList()); + params.put(Constants.PR_TOKEN_LIST, jssSubSystem.getTokenList()); sendResponse(SUCCESS, null, params, resp); } @@ -348,7 +348,7 @@ public final class CMSAdminServlet extends AdminServlet { ICryptoSubsystem jssSubSystem = (ICryptoSubsystem) CMS.getSubsystem(CMS.SUBSYSTEM_CRYPTO); - params.add(Constants.PR_ALL_NICKNAMES, jssSubSystem.getAllCerts()); + params.put(Constants.PR_ALL_NICKNAMES, jssSubSystem.getAllCerts()); sendResponse(SUCCESS, null, params, resp); } @@ -405,10 +405,10 @@ public final class CMSAdminServlet extends AdminServlet { NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_CIPHER_VERSION, + params.put(Constants.PR_CIPHER_VERSION, jssSubSystem.getCipherVersion()); - params.add(Constants.PR_CIPHER_FORTEZZA, jssSubSystem.isCipherFortezza()); - params.add(Constants.PR_CIPHER_PREF, jssSubSystem.getCipherPreferences()); + params.put(Constants.PR_CIPHER_FORTEZZA, jssSubSystem.isCipherFortezza()); + params.put(Constants.PR_CIPHER_PREF, jssSubSystem.getCipherPreferences()); String tokenList = jssSubSystem.getTokenList(); @@ -426,10 +426,10 @@ public final class CMSAdminServlet extends AdminServlet { else tokenNewList = tokenNewList + "," + tokenName; tokenName = escapeString(tokenName); - params.add(Constants.PR_TOKEN_PREFIX + tokenName, certs); + params.put(Constants.PR_TOKEN_PREFIX + tokenName, certs); } - params.add(Constants.PR_TOKEN_LIST, tokenNewList); + params.put(Constants.PR_TOKEN_LIST, tokenNewList); if (isCAInstalled) { ICertificateAuthority ca = (ICertificateAuthority) CMS.getSubsystem(CMS.SUBSYSTEM_CA); @@ -443,7 +443,7 @@ public final class CMSAdminServlet extends AdminServlet { String caNickName = signingUnit.getNickname(); //params.add(Constants.PR_CERT_CA, caTokenName+","+caNickName); - params.add(Constants.PR_CERT_CA, getCertNickname(caNickName)); + params.put(Constants.PR_CERT_CA, getCertNickname(caNickName)); } if (isRAInstalled) { @@ -451,7 +451,7 @@ public final class CMSAdminServlet extends AdminServlet { CMS.getSubsystem(CMS.SUBSYSTEM_RA); String raNickname = ra.getNickname(); - params.add(Constants.PR_CERT_RA, getCertNickname(raNickname)); + params.put(Constants.PR_CERT_RA, getCertNickname(raNickname)); } if (isKRAInstalled) { @@ -459,12 +459,12 @@ public final class CMSAdminServlet extends AdminServlet { CMS.getSubsystem(CMS.SUBSYSTEM_KRA); String kraNickname = kra.getNickname(); - params.add(Constants.PR_CERT_TRANS, getCertNickname(kraNickname)); + params.put(Constants.PR_CERT_TRANS, getCertNickname(kraNickname)); } String nickName = CMS.getServerCertNickname(); - params.add(Constants.PR_CERT_SERVER, getCertNickname(nickName)); + params.put(Constants.PR_CERT_SERVER, getCertNickname(nickName)); sendResponse(SUCCESS, null, params, resp); } @@ -795,7 +795,7 @@ public final class CMSAdminServlet extends AdminServlet { if (sys instanceof ITKSAuthority) type = Constants.PR_TKS_INSTANCE; if (!type.trim().equals("")) - params.add(sys.getId(), type); + params.put(sys.getId(), type); } sendResponse(SUCCESS, null, params, resp); @@ -811,25 +811,25 @@ public final class CMSAdminServlet extends AdminServlet { IConfigStore cs = CMS.getConfigStore(); try { String installdate = cs.getString(Constants.PR_STAT_INSTALLDATE, ""); - params.add(Constants.PR_STAT_INSTALLDATE, installdate); + params.put(Constants.PR_STAT_INSTALLDATE, installdate); } catch (Exception e) { } try { String version = cs.getString(Constants.PR_STAT_VERSION, ""); - params.add(Constants.PR_STAT_VERSION, version); + params.put(Constants.PR_STAT_VERSION, version); } catch (Exception e) { } try { String instanceId = cs.getString(Constants.PR_STAT_INSTANCEID, ""); - params.add(Constants.PR_STAT_INSTANCEID, instanceId); + params.put(Constants.PR_STAT_INSTANCEID, instanceId); } catch (Exception e) { } - params.add(Constants.PR_STAT_STARTUP, + params.put(Constants.PR_STAT_STARTUP, (new Date(CMS.getStartupTime())).toString()); - params.add(Constants.PR_STAT_TIME, + params.put(Constants.PR_STAT_TIME, (new Date(System.currentTimeMillis())).toString()); sendResponse(SUCCESS, null, params, resp); } @@ -890,8 +890,8 @@ public final class CMSAdminServlet extends AdminServlet { String masterKeyPrefix = CMS.getConfigStore().getString("tks.master_key_prefix", null); SessionKey.SetDefaultPrefix(masterKeyPrefix); - params.add(Constants.PR_KEY_LIST, newKeyName); - params.add(Constants.PR_TOKEN_LIST, selectedToken); + params.put(Constants.PR_KEY_LIST, newKeyName); + params.put(Constants.PR_TOKEN_LIST, selectedToken); } sendResponse(SUCCESS, null, params, resp); } @@ -937,7 +937,7 @@ public final class CMSAdminServlet extends AdminServlet { } // String symKeys = new String("key1,key2"); String symKeys = SessionKey.ListSymmetricKeys(selectedToken); - params.add(Constants.PR_TOKEN_LIST, symKeys); + params.put(Constants.PR_TOKEN_LIST, symKeys); } } @@ -966,9 +966,9 @@ public final class CMSAdminServlet extends AdminServlet { if (name.equals(Constants.OP_SCOPE)) continue; if (name.equals(Constants.PR_SECURE_PORT_ENABLED)) - params.add(name, ldapConfig.getString(name, "Constants.FALSE")); + params.put(name, ldapConfig.getString(name, "Constants.FALSE")); else - params.add(name, ldapConfig.getString(name, "")); + params.put(name, ldapConfig.getString(name, "")); } sendResponse(SUCCESS, null, params, resp); } @@ -1006,9 +1006,9 @@ public final class CMSAdminServlet extends AdminServlet { IConfigStore dbConfig = mConfig.getSubStore(PROP_SMTP); NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_SERVER_NAME, + params.put(Constants.PR_SERVER_NAME, dbConfig.getString("host")); - params.add(Constants.PR_PORT, + params.put(Constants.PR_PORT, dbConfig.getString("port")); sendResponse(SUCCESS, null, params, resp); } @@ -1064,7 +1064,7 @@ public final class CMSAdminServlet extends AdminServlet { NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_LOGGED_IN, "" + status); + params.put(Constants.PR_LOGGED_IN, "" + status); sendResponse(SUCCESS, null, params, resp); } @@ -1210,8 +1210,8 @@ public final class CMSAdminServlet extends AdminServlet { } String certReq = jssSubSystem.getCertRequest(subjectName, keypair); - params.add(Constants.PR_CSR, certReq); - params.add(Constants.PR_CERT_REQUEST_DIR, dir); + params.put(Constants.PR_CSR, certReq); + params.put(Constants.PR_CERT_REQUEST_DIR, dir); PrintStream ps = new PrintStream(new FileOutputStream(pathname)); ps.println(certReq); @@ -2432,8 +2432,8 @@ public final class CMSAdminServlet extends AdminServlet { String content = jssSubSystem.getCertPrettyPrint(b64Cert, super.getLocale(req)); - results.add(Constants.PR_NICKNAME, "FBCA cross-signed cert"); - results.add(Constants.PR_CERT_CONTENT, content); + results.put(Constants.PR_NICKNAME, "FBCA cross-signed cert"); + results.put(Constants.PR_CERT_CONTENT, content); // store a message in the signed audit log file auditMessage = CMS.getLogMessage( @@ -2618,8 +2618,8 @@ public final class CMSAdminServlet extends AdminServlet { super.getLocale(req)); if (nickname != null && !nickname.equals("")) - results.add(Constants.PR_NICKNAME, nickname); - results.add(Constants.PR_CERT_CONTENT, content); + results.put(Constants.PR_NICKNAME, nickname); + results.put(Constants.PR_CERT_CONTENT, content); //results = jssSubSystem.getCertInfo(value); sendResponse(SUCCESS, null, results, resp); @@ -2664,7 +2664,7 @@ public final class CMSAdminServlet extends AdminServlet { String print = jssSubSystem.getCertPrettyPrintAndFingerPrint(nickname, serialno, issuername, locale); - pairs.add(nickname, print); + pairs.put(nickname, print); sendResponse(SUCCESS, null, pairs, resp); } @@ -2707,7 +2707,7 @@ public final class CMSAdminServlet extends AdminServlet { String trustbit = jssSubSystem.getRootCertTrustBit(nickname, serialno, issuername); - pairs.add(nickname, trustbit); + pairs.put(nickname, trustbit); sendResponse(SUCCESS, null, pairs, resp); } @@ -2924,7 +2924,7 @@ public final class CMSAdminServlet extends AdminServlet { CMS.getSubsystem(CMS.SUBSYSTEM_CRYPTO); String subjectName = jssSubSystem.getSubjectDN(nickname); - params.add(Constants.PR_SUBJECT_NAME, subjectName); + params.put(Constants.PR_SUBJECT_NAME, subjectName); sendResponse(SUCCESS, null, params, resp); } @@ -2950,7 +2950,7 @@ public final class CMSAdminServlet extends AdminServlet { CMS.getSubsystem(CMS.SUBSYSTEM_CRYPTO); String subjectName = jssSubSystem.getSubjectDN(nickname); - params.add(Constants.PR_SUBJECT_NAME, subjectName); + params.put(Constants.PR_SUBJECT_NAME, subjectName); sendResponse(SUCCESS, null, params, resp); } @@ -3342,9 +3342,9 @@ public final class CMSAdminServlet extends AdminServlet { audit(auditMessage); // notify console of SUCCESS - results.add(Constants.PR_RUN_SELFTESTS_ON_DEMAND_CLASS, + results.put(Constants.PR_RUN_SELFTESTS_ON_DEMAND_CLASS, CMSAdminServlet.class.getName()); - results.add(Constants.PR_RUN_SELFTESTS_ON_DEMAND_CONTENT, + results.put(Constants.PR_RUN_SELFTESTS_ON_DEMAND_CONTENT, content); sendResponse(SUCCESS, null, results, resp); diff --git a/pki/base/common/src/com/netscape/cms/servlet/admin/JobsAdminServlet.java b/pki/base/common/src/com/netscape/cms/servlet/admin/JobsAdminServlet.java index a9baa874c..42ff32ebe 100644 --- a/pki/base/common/src/com/netscape/cms/servlet/admin/JobsAdminServlet.java +++ b/pki/base/common/src/com/netscape/cms/servlet/admin/JobsAdminServlet.java @@ -510,7 +510,7 @@ public class JobsAdminServlet extends AdminServlet { NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_JOBS_IMPL_NAME, implname); + params.put(Constants.PR_JOBS_IMPL_NAME, implname); sendResponse(SUCCESS, null, params, resp); return; } @@ -526,7 +526,7 @@ public class JobsAdminServlet extends AdminServlet { String name = e.nextElement(); JobPlugin value = mJobsSched.getPlugins().get(name); - params.add(name, value.getClassPath()); + params.put(name, value.getClassPath()); // params.add(name, value.getClassPath()+EDIT); } sendResponse(SUCCESS, null, params, resp); @@ -544,9 +544,9 @@ public class JobsAdminServlet extends AdminServlet { IJob value = mJobsSched.getInstances().get((Object) name); // params.add(name, value.getImplName()); - params.add(name, value.getImplName() + VISIBLE + + params.put(name, value.getImplName() + VISIBLE + (value.isEnabled() ? ENABLED : DISABLED) - ); + ); } sendResponse(SUCCESS, null, params, resp); return; @@ -691,10 +691,10 @@ public class JobsAdminServlet extends AdminServlet { NameValuePairs params = new NameValuePairs(); // implName is always required so always send it. - params.add(Constants.PR_JOBS_IMPL_NAME, ""); + params.put(Constants.PR_JOBS_IMPL_NAME, ""); if (configParams != null) { for (int i = 0; i < configParams.length; i++) { - params.add(configParams[i], ""); + params.put(configParams[i], ""); } } sendResponse(0, null, params, resp); @@ -728,7 +728,7 @@ public class JobsAdminServlet extends AdminServlet { String[] configParams = jobInst.getConfigParams(); NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_JOBS_IMPL_NAME, jobInst.getImplName()); + params.put(Constants.PR_JOBS_IMPL_NAME, jobInst.getImplName()); // implName is always required so always send it. if (configParams != null) { @@ -738,9 +738,9 @@ public class JobsAdminServlet extends AdminServlet { String val = (String) config.get(key); if (val != null && !val.equals("")) { - params.add(key, val); + params.put(key, val); } else { - params.add(key, ""); + params.put(key, ""); } } } @@ -813,7 +813,7 @@ public class JobsAdminServlet extends AdminServlet { NameValuePairs saveParams = new NameValuePairs(); // implName is always required so always include it it. - saveParams.add(IJobsScheduler.PROP_PLUGIN, + saveParams.put(IJobsScheduler.PROP_PLUGIN, (String) oldConfig.get(IJobsScheduler.PROP_PLUGIN)); if (oldConfigParms != null) { for (int i = 0; i < oldConfigParms.length; i++) { @@ -821,7 +821,7 @@ public class JobsAdminServlet extends AdminServlet { Object val = oldConfig.get(key); if (val != null) { - saveParams.add(key, (String) val); + saveParams.put(key, (String) val); } } } @@ -945,11 +945,11 @@ public class JobsAdminServlet extends AdminServlet { NameValuePairs params = new NameValuePairs(); IConfigStore config = mConfig.getSubStore(DestDef.DEST_JOBS_ADMIN); - params.add(Constants.PR_ENABLE, + params.put(Constants.PR_ENABLE, config.getString(IJobsScheduler.PROP_ENABLED, Constants.FALSE)); // default 1 minute - params.add(Constants.PR_JOBS_FREQUENCY, + params.put(Constants.PR_JOBS_FREQUENCY, config.getString(IJobsScheduler.PROP_INTERVAL, "1")); //System.out.println("Send: "+params.toString()); @@ -997,11 +997,8 @@ public class JobsAdminServlet extends AdminServlet { store.removeSubStore(id); IConfigStore rstore = store.makeSubStore(id); - Enumeration keys = saveParams.getNames(); - - while (keys.hasMoreElements()) { - String key = keys.nextElement(); - String value = saveParams.getValue(key); + for (String key : saveParams.keySet()) { + String value = saveParams.get(key); if (!value.equals("")) rstore.put(key, value); diff --git a/pki/base/common/src/com/netscape/cms/servlet/admin/KRAAdminServlet.java b/pki/base/common/src/com/netscape/cms/servlet/admin/KRAAdminServlet.java index 0d3648c98..eaa5a95c4 100644 --- a/pki/base/common/src/com/netscape/cms/servlet/admin/KRAAdminServlet.java +++ b/pki/base/common/src/com/netscape/cms/servlet/admin/KRAAdminServlet.java @@ -180,7 +180,7 @@ public class KRAAdminServlet extends AdminServlet { int value = 1; value = mKRA.getNoOfRequiredAgents(); - params.add(Constants.PR_NO_OF_REQUIRED_RECOVERY_AGENTS, Integer.toString(value)); + params.put(Constants.PR_NO_OF_REQUIRED_RECOVERY_AGENTS, Integer.toString(value)); sendResponse(SUCCESS, null, params, resp); } diff --git a/pki/base/common/src/com/netscape/cms/servlet/admin/LogAdminServlet.java b/pki/base/common/src/com/netscape/cms/servlet/admin/LogAdminServlet.java index ff70a5c53..be723fcd6 100644 --- a/pki/base/common/src/com/netscape/cms/servlet/admin/LogAdminServlet.java +++ b/pki/base/common/src/com/netscape/cms/servlet/admin/LogAdminServlet.java @@ -355,7 +355,7 @@ public class LogAdminServlet extends AdminServlet { // not show ntEventlog here if (all || (!all && !c.endsWith("NTEventLog"))) - params.add(name, pName + ";visible"); + params.put(name, pName + ";visible"); } sendResponse(SUCCESS, null, params, resp); return; @@ -976,7 +976,7 @@ public class LogAdminServlet extends AdminServlet { NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_LOG_IMPL_NAME, implname); + params.put(Constants.PR_LOG_IMPL_NAME, implname); // store a message in the signed audit log file if (logType.equals(SIGNED_AUDIT_LOG_TYPE)) { @@ -1054,7 +1054,7 @@ public class LogAdminServlet extends AdminServlet { resp); return; } - params.add(name, value.getClassPath() + "," + desc); + params.put(name, value.getClassPath() + "," + desc); } sendResponse(SUCCESS, null, params, resp); return; @@ -1569,13 +1569,13 @@ public class LogAdminServlet extends AdminServlet { NameValuePairs saveParams = new NameValuePairs(); // implName is always required so always include it it. - saveParams.add("pluginName", implname); + saveParams.put("pluginName", implname); if (oldConfigParms != null) { for (int i = 0; i < oldConfigParms.size(); i++) { String kv = (String) oldConfigParms.elementAt(i); int index = kv.indexOf('='); - saveParams.add(kv.substring(0, index), + saveParams.put(kv.substring(0, index), kv.substring(index + 1)); } } @@ -2214,16 +2214,16 @@ public class LogAdminServlet extends AdminServlet { NameValuePairs params = new NameValuePairs(); // implName is always required so always send it. - params.add(Constants.PR_LOG_IMPL_NAME, ""); + params.put(Constants.PR_LOG_IMPL_NAME, ""); if (configParams != null) { for (int i = 0; i < configParams.size(); i++) { String kv = (String) configParams.elementAt(i); int index = kv.indexOf('='); if (index == -1) { - params.add(kv, ""); + params.put(kv, ""); } else { - params.add(kv.substring(0, index), + params.put(kv.substring(0, index), kv.substring(index + 1)); } } @@ -2259,7 +2259,7 @@ public class LogAdminServlet extends AdminServlet { Vector configParams = logInst.getInstanceParams(); NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_LOG_IMPL_NAME, + params.put(Constants.PR_LOG_IMPL_NAME, getLogPluginName(logInst)); // implName is always required so always send it. if (configParams != null) { @@ -2267,7 +2267,7 @@ public class LogAdminServlet extends AdminServlet { String kv = (String) configParams.elementAt(i); int index = kv.indexOf('='); - params.add(kv.substring(0, index), + params.put(kv.substring(0, index), kv.substring(index + 1)); } } @@ -2282,11 +2282,8 @@ public class LogAdminServlet extends AdminServlet { store.removeSubStore(id); IConfigStore rstore = store.makeSubStore(id); - Enumeration keys = saveParams.getNames(); - - while (keys.hasMoreElements()) { - String key = (String) keys.nextElement(); - String value = saveParams.getValue(key); + for (String key : saveParams.keySet()) { + String value = saveParams.get(key); if (value != null) rstore.put(key, value); @@ -2323,10 +2320,10 @@ public class LogAdminServlet extends AdminServlet { String value = "false"; value = mConfig.getString(Constants.PR_DEBUG_LOG_ENABLE, "false"); - params.add(Constants.PR_DEBUG_LOG_ENABLE, value); + params.put(Constants.PR_DEBUG_LOG_ENABLE, value); value = mConfig.getString(Constants.PR_DEBUG_LOG_LEVEL, "0"); - params.add(Constants.PR_DEBUG_LOG_LEVEL, value); + params.put(Constants.PR_DEBUG_LOG_LEVEL, value); sendResponse(SUCCESS, null, params, resp); } diff --git a/pki/base/common/src/com/netscape/cms/servlet/admin/OCSPAdminServlet.java b/pki/base/common/src/com/netscape/cms/servlet/admin/OCSPAdminServlet.java index 317a50d68..59eead9df 100644 --- a/pki/base/common/src/com/netscape/cms/servlet/admin/OCSPAdminServlet.java +++ b/pki/base/common/src/com/netscape/cms/servlet/admin/OCSPAdminServlet.java @@ -362,7 +362,7 @@ public class OCSPAdminServlet extends AdminServlet { continue; String value = req.getParameter(name); - params.add(name, value); + params.put(name, value); } store.setConfigParameters(params); commit(true); @@ -432,7 +432,7 @@ public class OCSPAdminServlet extends AdminServlet { if (storeName.equals(defStore)) { storeEnabled = true; } - params.add(storeName, storeName + ";visible;" + ((storeEnabled) ? "enabled" : "disabled")); + params.put(storeName, storeName + ";visible;" + ((storeEnabled) ? "enabled" : "disabled")); } sendResponse(SUCCESS, null, params, resp); } @@ -449,7 +449,7 @@ public class OCSPAdminServlet extends AdminServlet { } private void getSigningAlgConfig(NameValuePairs params) { - params.add(Constants.PR_DEFAULT_ALGORITHM, + params.put(Constants.PR_DEFAULT_ALGORITHM, mOCSP.getDefaultAlgorithm()); String[] algorithms = mOCSP.getOCSPSigningAlgorithms(); StringBuffer algorStr = new StringBuffer(); @@ -461,7 +461,7 @@ public class OCSPAdminServlet extends AdminServlet { algorStr.append(":"); algorStr.append(algorithms[i]); } - params.add(Constants.PR_ALL_ALGORITHMS, algorStr.toString()); + params.put(Constants.PR_ALL_ALGORITHMS, algorStr.toString()); } /** diff --git a/pki/base/common/src/com/netscape/cms/servlet/admin/PolicyAdminServlet.java b/pki/base/common/src/com/netscape/cms/servlet/admin/PolicyAdminServlet.java index 4dbde7e06..9eb5c3455 100644 --- a/pki/base/common/src/com/netscape/cms/servlet/admin/PolicyAdminServlet.java +++ b/pki/base/common/src/com/netscape/cms/servlet/admin/PolicyAdminServlet.java @@ -298,11 +298,11 @@ public class PolicyAdminServlet extends AdminServlet { /* make sure policy rules have 'enable' and 'predicate' */ if (ext_info instanceof IPolicyRule) { - if (nvps.getPair(IPolicyRule.PROP_ENABLE) == null) { - nvps.add(IPolicyRule.PROP_ENABLE, "boolean;Enable this policy rule"); + if (nvps.get(IPolicyRule.PROP_ENABLE) == null) { + nvps.put(IPolicyRule.PROP_ENABLE, "boolean;Enable this policy rule"); } - if (nvps.getPair(PROP_PREDICATE) == null) { - nvps.add(PROP_PREDICATE, "string;Rules describing when this policy should run."); + if (nvps.get(PROP_PREDICATE) == null) { + nvps.put(PROP_PREDICATE, "string;Rules describing when this policy should run."); } } } @@ -448,7 +448,7 @@ public class PolicyAdminServlet extends AdminServlet { impl.getClass().getName(); String desc = impl.getDescription(); - nvp.add(id, className + "," + desc); + nvp.put(id, className + "," + desc); } sendResponse(SUCCESS, null, nvp, resp); } @@ -471,7 +471,7 @@ public class PolicyAdminServlet extends AdminServlet { String info = instancesInfo.nextElement(); int i = info.indexOf(";"); - nvp.add(info.substring(0, i), info.substring(i + 1)); + nvp.put(info.substring(0, i), info.substring(i + 1)); } sendResponse(SUCCESS, null, nvp, resp); @@ -594,7 +594,7 @@ public class PolicyAdminServlet extends AdminServlet { String nv = e.nextElement(); int index = nv.indexOf("="); - nvp.add(nv.substring(0, index), nv.substring(index + 1)); + nvp.put(nv.substring(0, index), nv.substring(index + 1)); } sendResponse(SUCCESS, null, nvp, resp); } @@ -829,7 +829,7 @@ public class PolicyAdminServlet extends AdminServlet { value = ""; } - nvp.add(name, value); + nvp.put(name, value); } sendResponse(SUCCESS, null, nvp, resp); diff --git a/pki/base/common/src/com/netscape/cms/servlet/admin/ProfileAdminServlet.java b/pki/base/common/src/com/netscape/cms/servlet/admin/ProfileAdminServlet.java index 8e67c3402..94235f532 100644 --- a/pki/base/common/src/com/netscape/cms/servlet/admin/ProfileAdminServlet.java +++ b/pki/base/common/src/com/netscape/cms/servlet/admin/ProfileAdminServlet.java @@ -387,7 +387,7 @@ public class ProfileAdminServlet extends AdminServlet { String id = (String) impls.nextElement(); IPluginInfo info = mRegistry.getPluginInfo("profile", id); - nvp.add(id, info.getClassName() + "," + + nvp.put(id, info.getClassName() + "," + info.getDescription(getLocale(req))); } sendResponse(SUCCESS, null, nvp, resp); @@ -599,7 +599,7 @@ public class ProfileAdminServlet extends AdminServlet { continue; if (name.equals("RS_ID")) continue; - nvps.add(name, req.getParameter(name)); + nvps.put(name, req.getParameter(name)); } try { @@ -725,7 +725,7 @@ public class ProfileAdminServlet extends AdminServlet { continue; if (name.equals("RS_ID")) continue; - nvps.add(name, req.getParameter(name)); + nvps.put(name, req.getParameter(name)); } try { @@ -1968,9 +1968,9 @@ public class ProfileAdminServlet extends AdminServlet { IDescriptor desc = rule.getConfigDescriptor(getLocale(req), name); if (desc == null) { - nvp.add(name, ";" + ";" + rule.getConfig(name)); + nvp.put(name, ";" + ";" + rule.getConfig(name)); } else { - nvp.add(name, + nvp.put(name, desc.getSyntax() + ";" + ";" + getNonNull(desc.getConstraint()) + ";" + desc.getDescription(getLocale(req)) + ";" + rule.getConfig(name)); @@ -2019,9 +2019,9 @@ public class ProfileAdminServlet extends AdminServlet { IDescriptor desc = rule.getConfigDescriptor(getLocale(req), name); if (desc == null) { - nvp.add(name, ";" + rule.getConfig(name)); + nvp.put(name, ";" + rule.getConfig(name)); } else { - nvp.add(name, + nvp.put(name, desc.getSyntax() + ";" + getNonNull(desc.getConstraint()) + ";" + desc.getDescription(getLocale(req)) + ";" + rule.getConfig(name)); @@ -2064,7 +2064,7 @@ public class ProfileAdminServlet extends AdminServlet { IPolicyDefault def = policy.getDefault(); IPolicyConstraint con = policy.getConstraint(); - nvp.add(setId + ":" + policy.getId(), + nvp.put(setId + ":" + policy.getId(), def.getName(getLocale(req)) + ";" + con.getName(getLocale(req))); } @@ -2094,7 +2094,7 @@ public class ProfileAdminServlet extends AdminServlet { String outputId = (String) outputs.nextElement(); IProfileOutput output = profile.getProfileOutput(outputId); - nvp.add(outputId, output.getName(getLocale(req))); + nvp.put(outputId, output.getName(getLocale(req))); } sendResponse(SUCCESS, null, nvp, resp); @@ -2121,7 +2121,7 @@ public class ProfileAdminServlet extends AdminServlet { String inputId = (String) inputs.nextElement(); IProfileInput input = profile.getProfileInput(inputId); - nvp.add(inputId, input.getName(getLocale(req))); + nvp.put(inputId, input.getName(getLocale(req))); } sendResponse(SUCCESS, null, nvp, resp); @@ -2156,9 +2156,9 @@ public class ProfileAdminServlet extends AdminServlet { IDescriptor desc = profileInput.getConfigDescriptor( getLocale(req), name); if (desc == null) { - nvp.add(name, ";" + ";" + profileInput.getConfig(name)); + nvp.put(name, ";" + ";" + profileInput.getConfig(name)); } else { - nvp.add(name, desc.getSyntax() + ";" + + nvp.put(name, desc.getSyntax() + ";" + getNonNull(desc.getConstraint()) + ";" + desc.getDescription(getLocale(req)) + ";" + profileInput.getConfig(name)); @@ -2197,9 +2197,9 @@ public class ProfileAdminServlet extends AdminServlet { IDescriptor desc = profileOutput.getConfigDescriptor( getLocale(req), name); if (desc == null) { - nvp.add(name, ";" + ";" + profileOutput.getConfig(name)); + nvp.put(name, ";" + ";" + profileOutput.getConfig(name)); } else { - nvp.add(name, desc.getSyntax() + ";" + + nvp.put(name, desc.getSyntax() + ";" + getNonNull(desc.getConstraint()) + ";" + desc.getDescription(getLocale(req)) + ";" + profileOutput.getConfig(name)); @@ -2228,7 +2228,7 @@ public class ProfileAdminServlet extends AdminServlet { } // mInstanceId + ";visible;" + enabled - nvp.add(profileId, profileId + ";visible;" + status); + nvp.put(profileId, profileId + ";visible;" + status); } sendResponse(SUCCESS, null, nvp, resp); } @@ -2250,21 +2250,21 @@ public class ProfileAdminServlet extends AdminServlet { NameValuePairs nvp = new NameValuePairs(); - nvp.add("name", profile.getName(getLocale(req))); - nvp.add("desc", profile.getDescription(getLocale(req))); - nvp.add("visible", Boolean.toString(profile.isVisible())); - nvp.add("enable", Boolean.toString( + nvp.put("name", profile.getName(getLocale(req))); + nvp.put("desc", profile.getDescription(getLocale(req))); + nvp.put("visible", Boolean.toString(profile.isVisible())); + nvp.put("enable", Boolean.toString( mProfileSub.isProfileEnable(id))); String authid = profile.getAuthenticatorId(); if (authid == null) { - nvp.add("auth", ""); + nvp.put("auth", ""); } else { - nvp.add("auth", authid); + nvp.put("auth", authid); } CMS.debug("ProfileAdminServlet: authid=" + authid); - nvp.add("plugin", mProfileSub.getProfileClassId(id)); + nvp.put("plugin", mProfileSub.getProfileClassId(id)); sendResponse(SUCCESS, null, nvp, resp); } diff --git a/pki/base/common/src/com/netscape/cms/servlet/admin/PublisherAdminServlet.java b/pki/base/common/src/com/netscape/cms/servlet/admin/PublisherAdminServlet.java index 799e1bb37..3774d6e6f 100644 --- a/pki/base/common/src/com/netscape/cms/servlet/admin/PublisherAdminServlet.java +++ b/pki/base/common/src/com/netscape/cms/servlet/admin/PublisherAdminServlet.java @@ -468,7 +468,7 @@ public class PublisherAdminServlet extends AdminServlet { if (name.equals(Constants.PR_CERT_NAMES)) { ICryptoSubsystem jss = (ICryptoSubsystem) CMS.getSubsystem(CMS.SUBSYSTEM_CRYPTO); - params.add(name, jss.getAllCerts()); + params.put(name, jss.getAllCerts()); } else { String value = ldap.getString(name, ""); @@ -481,22 +481,22 @@ public class PublisherAdminServlet extends AdminServlet { value = ILdapAuthInfo.PROP_BINDDN_DEFAULT; } } - params.add(name, value); + params.put(name, value); } } - params.add(Constants.PR_PUBLISHING_ENABLE, + params.put(Constants.PR_PUBLISHING_ENABLE, publishcfg.getString(IPublisherProcessor.PROP_ENABLE, Constants.FALSE)); - params.add(Constants.PR_PUBLISHING_QUEUE_ENABLE, + params.put(Constants.PR_PUBLISHING_QUEUE_ENABLE, publishcfg.getString(Constants.PR_PUBLISHING_QUEUE_ENABLE, Constants.TRUE)); - params.add(Constants.PR_PUBLISHING_QUEUE_THREADS, + params.put(Constants.PR_PUBLISHING_QUEUE_THREADS, publishcfg.getString(Constants.PR_PUBLISHING_QUEUE_THREADS, "3")); - params.add(Constants.PR_PUBLISHING_QUEUE_PAGE_SIZE, + params.put(Constants.PR_PUBLISHING_QUEUE_PAGE_SIZE, publishcfg.getString(Constants.PR_PUBLISHING_QUEUE_PAGE_SIZE, "40")); - params.add(Constants.PR_PUBLISHING_QUEUE_PRIORITY, + params.put(Constants.PR_PUBLISHING_QUEUE_PRIORITY, publishcfg.getString(Constants.PR_PUBLISHING_QUEUE_PRIORITY, "0")); - params.add(Constants.PR_PUBLISHING_QUEUE_STATUS, + params.put(Constants.PR_PUBLISHING_QUEUE_STATUS, publishcfg.getString(Constants.PR_PUBLISHING_QUEUE_STATUS, "200")); - params.add(Constants.PR_ENABLE, + params.put(Constants.PR_ENABLE, ldapcfg.getString(IPublisherProcessor.PROP_ENABLE, Constants.FALSE)); sendResponse(SUCCESS, null, params, resp); } @@ -694,7 +694,7 @@ public class PublisherAdminServlet extends AdminServlet { // test before commit if (publishcfg.getBoolean(IPublisherProcessor.PROP_ENABLE) && ldapcfg.getBoolean(IPublisherProcessor.PROP_ENABLE)) { - params.add("title", + params.put("title", "You've attempted to configure CMS to connect" + " to a LDAP directory. The connection status is" + " as follows:\n \n"); @@ -725,16 +725,16 @@ public class PublisherAdminServlet extends AdminServlet { conn = new LDAPConnection(CMS.getLdapJssSSLSocketFactory( certNickName)); CMS.debug("Publishing Test certNickName=" + certNickName); - params.add(Constants.PR_CONN_INITED, + params.put(Constants.PR_CONN_INITED, "Create ssl LDAPConnection with certificate: " + certNickName + dashes(70 - 44 - certNickName.length()) + " Success"); } catch (Exception ex) { - params.add(Constants.PR_CONN_INIT_FAIL, + params.put(Constants.PR_CONN_INIT_FAIL, "Create ssl LDAPConnection with certificate: " + certNickName + dashes(70 - 44 - certNickName.length()) + " failure\n" + " exception: " + ex); - params.add(Constants.PR_SAVE_NOT, + params.put(Constants.PR_SAVE_NOT, "\n \nIf the problem is not fixed then LDAP publishing will fail.\n" + "Do you want to save the configuration anyway?"); sendResponse(SUCCESS, null, params, resp); @@ -742,13 +742,13 @@ public class PublisherAdminServlet extends AdminServlet { } try { conn.connect(host, port); - params.add(Constants.PR_CONN_OK, + params.put(Constants.PR_CONN_OK, "Connect to directory server " + host + " at port " + port + dashes(70 - 37 - host.length() - (Integer.valueOf(port)).toString().length()) + " Success"); - params.add(Constants.PR_AUTH_OK, + params.put(Constants.PR_AUTH_OK, "Authentication: SSL client authentication" + dashes(70 - 41) + " Success" + "\nBind to the directory as: " + certNickName + @@ -757,20 +757,20 @@ public class PublisherAdminServlet extends AdminServlet { if (ex.getLDAPResultCode() == LDAPException.UNAVAILABLE) { // need to intercept this because message from LDAP is // "DSA is unavailable" which confuses with DSA PKI. - params.add(Constants.PR_CONN_FAIL, + params.put(Constants.PR_CONN_FAIL, "Connect to directory server " + host + " at port " + port + dashes(70 - 37 - host.length() - (Integer.valueOf(port)).toString().length()) + " Failure\n" + " error: server unavailable"); } else { - params.add(Constants.PR_CONN_FAIL, + params.put(Constants.PR_CONN_FAIL, "Connect to directory server " + host + " at port " + port + dashes(70 - 37 - host.length() - (Integer.valueOf(port)).toString().length()) + " Failure"); } - params.add(Constants.PR_SAVE_NOT, + params.put(Constants.PR_SAVE_NOT, "\n \nIf the problem is not fixed then " + "LDAP publishing will fail.\n" + "Do you want to save the configuration anyway?"); @@ -782,21 +782,21 @@ public class PublisherAdminServlet extends AdminServlet { if (secure) { conn = new LDAPConnection( CMS.getLdapJssSSLSocketFactory()); - params.add(Constants.PR_CONN_INITED, + params.put(Constants.PR_CONN_INITED, "Create ssl LDAPConnection" + dashes(70 - 25) + " Success"); } else { conn = new LDAPConnection(); - params.add(Constants.PR_CONN_INITED, + params.put(Constants.PR_CONN_INITED, "Create LDAPConnection" + dashes(70 - 21) + " Success"); } } catch (Exception ex) { - params.add(Constants.PR_CONN_INIT_FAIL, + params.put(Constants.PR_CONN_INIT_FAIL, "Create LDAPConnection" + dashes(70 - 21) + " Failure\n" + "exception: " + ex); - params.add(Constants.PR_SAVE_NOT, + params.put(Constants.PR_SAVE_NOT, "\n \nIf the problem is not fixed then " + "LDAP publishing will fail.\n" + "Do you want to save the configuration anyway?"); @@ -805,7 +805,7 @@ public class PublisherAdminServlet extends AdminServlet { } try { conn.connect(host, port); - params.add(Constants.PR_CONN_OK, + params.put(Constants.PR_CONN_OK, "Connect to directory server " + host + " at port " + port + @@ -815,7 +815,7 @@ public class PublisherAdminServlet extends AdminServlet { if (ex.getLDAPResultCode() == LDAPException.UNAVAILABLE) { // need to intercept this because message from LDAP is // "DSA is unavailable" which confuses with DSA PKI. - params.add(Constants.PR_CONN_FAIL, + params.put(Constants.PR_CONN_FAIL, "Connect to directory server " + host + " at port " + port + @@ -823,7 +823,7 @@ public class PublisherAdminServlet extends AdminServlet { + " Failure" + "\nerror: server unavailable"); } else { - params.add(Constants.PR_CONN_FAIL, + params.put(Constants.PR_CONN_FAIL, "Connect to directory server " + host + " at port " + port + @@ -831,7 +831,7 @@ public class PublisherAdminServlet extends AdminServlet { + " Failure" + "\nexception: " + ex); } - params.add(Constants.PR_SAVE_NOT, + params.put(Constants.PR_SAVE_NOT, "\n \nIf the problem is not fixed then " + "LDAP publishing will fail.\n" + "Do you want to save the configuration anyway?"); @@ -843,14 +843,14 @@ public class PublisherAdminServlet extends AdminServlet { bindAs = ldap.getSubStore( ILdapBoundConnFactory.PROP_LDAPAUTHINFO).getString(ILdapAuthInfo.PROP_BINDDN); conn.authenticate(version, bindAs, pwd); - params.add(Constants.PR_AUTH_OK, + params.put(Constants.PR_AUTH_OK, "Authentication: Basic authentication" + dashes(70 - 36) + " Success" + "\nBind to the directory as: " + bindAs + dashes(70 - 26 - bindAs.length()) + " Success"); } catch (LDAPException ex) { if (ex.getLDAPResultCode() == LDAPException.NO_SUCH_OBJECT) { - params.add(Constants.PR_AUTH_FAIL, + params.put(Constants.PR_AUTH_FAIL, "Authentication: Basic authentication" + dashes(70 - 36) + "Failure" + "\nBind to the directory as: " + bindAs + @@ -859,7 +859,7 @@ public class PublisherAdminServlet extends AdminServlet { "Please correct the value assigned in the" + " \"Directory manager DN\" field."); } else if (ex.getLDAPResultCode() == LDAPException.INVALID_CREDENTIALS) { - params.add(Constants.PR_AUTH_FAIL, + params.put(Constants.PR_AUTH_FAIL, "Authentication: Basic authentication" + dashes(70 - 36) + " Failure" + "\nBind to the directory as: " + bindAs + @@ -868,14 +868,14 @@ public class PublisherAdminServlet extends AdminServlet { "Please correct the value assigned in the" + " \"Password\" field."); } else { - params.add(Constants.PR_AUTH_FAIL, + params.put(Constants.PR_AUTH_FAIL, "Authentication: Basic authentication" + dashes(70 - 36) + " Failure" + "\nBind to the directory as: " + bindAs + dashes(70 - 26 - bindAs.length()) + " Failure"); } - params.add(Constants.PR_SAVE_NOT, + params.put(Constants.PR_SAVE_NOT, "\n \nIf the problem is not fixed then " + "LDAP publishing will fail.\n" + "Do you want to save the configuration anyway?"); @@ -942,23 +942,23 @@ public class PublisherAdminServlet extends AdminServlet { try { mProcessor.publishCACert(ca.getCACert()); CMS.debug("PublisherAdminServlet: " + CMS.getLogMessage("ADMIN_SRVLT_PUB_CA_CERT")); - params.add("publishCA", + params.put("publishCA", "CA certificate is published."); } catch (Exception ex) { // exception not thrown - not seen as a fatal error. log(ILogger.LL_FAILURE, CMS.getLogMessage("ADMIN_SRVLT_NO_PUB_CA_CERT", ex.toString())); - params.add("publishCA", + params.put("publishCA", "Failed to publish CA certificate."); int index = ex.toString().indexOf("Failed to create CA"); if (index > -1) { - params.add("createError", + params.put("createError", ex.toString().substring(index)); } mProcessor.shutdown(); // Do you want to enable LDAP publishing anyway - params.add(Constants.PR_SAVE_NOT, + params.put(Constants.PR_SAVE_NOT, "\n \nIf the problem is not fixed then " + "the CA certificate won't be published.\n" + "Do you want to enable LDAP publishing anyway?"); @@ -971,17 +971,17 @@ public class PublisherAdminServlet extends AdminServlet { CMS.debug("PublisherAdminServlet: about to update CRL"); ca.publishCRLNow(); CMS.debug(CMS.getLogMessage("ADMIN_SRVLT_PUB_CRL")); - params.add("publishCRL", + params.put("publishCRL", "CRL is published."); } catch (Exception ex) { // exception not thrown - not seen as a fatal error. log(ILogger.LL_FAILURE, "Could not publish crl " + ex.toString()); - params.add("publishCRL", + params.put("publishCRL", "Failed to publish CRL."); mProcessor.shutdown(); // Do you want to enable LDAP publishing anyway - params.add(Constants.PR_SAVE_NOT, + params.put(Constants.PR_SAVE_NOT, "\n \nIf the problem is not fixed then " + "the CRL won't be published.\n" + "Do you want to enable LDAP publishing anyway?"); @@ -990,14 +990,14 @@ public class PublisherAdminServlet extends AdminServlet { } } commit(true); - params.add(Constants.PR_SAVE_OK, + params.put(Constants.PR_SAVE_OK, "\n \nConfiguration changes are now committed."); - params.add("restarted", "Publishing is restarted."); + params.put("restarted", "Publishing is restarted."); } else { commit(true); - params.add(Constants.PR_SAVE_OK, + params.put(Constants.PR_SAVE_OK, "\n \nConfiguration changes are now committed."); - params.add("stopped", + params.put("stopped", "Publishing is stopped."); } @@ -1236,7 +1236,7 @@ public class PublisherAdminServlet extends AdminServlet { NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_MAPPER_IMPL_NAME, implname); + params.put(Constants.PR_MAPPER_IMPL_NAME, implname); sendResponse(SUCCESS, null, params, resp); return; } @@ -1265,7 +1265,7 @@ public class PublisherAdminServlet extends AdminServlet { resp); return; } - params.add(name, value.getClassPath() + "," + desc); + params.put(name, value.getClassPath() + "," + desc); } sendResponse(SUCCESS, null, params, resp); return; @@ -1292,7 +1292,7 @@ public class PublisherAdminServlet extends AdminServlet { String name = e.nextElement(); ILdapMapper value = mProcessor.getMapperInstance(name); - params.add(name, getMapperPluginName(value) + ";visible"); + params.put(name, getMapperPluginName(value) + ";visible"); } sendResponse(SUCCESS, null, params, resp); return; @@ -1427,13 +1427,13 @@ public class PublisherAdminServlet extends AdminServlet { NameValuePairs params = new NameValuePairs(); // implName is always required so always send it. - params.add(Constants.PR_MAPPER_IMPL_NAME, ""); + params.put(Constants.PR_MAPPER_IMPL_NAME, ""); if (configParams != null) { for (int i = 0; i < configParams.size(); i++) { String kv = configParams.elementAt(i); int index = kv.indexOf('='); - params.add(kv.substring(0, index), + params.put(kv.substring(0, index), kv.substring(index + 1)); } } @@ -1468,7 +1468,7 @@ public class PublisherAdminServlet extends AdminServlet { Vector configParams = mapperInst.getInstanceParams(); NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_MAPPER_IMPL_NAME, + params.put(Constants.PR_MAPPER_IMPL_NAME, getMapperPluginName(mapperInst)); // implName is always required so always send it. if (configParams != null) { @@ -1476,7 +1476,7 @@ public class PublisherAdminServlet extends AdminServlet { String kv = configParams.elementAt(i); int index = kv.indexOf('='); - params.add(kv.substring(0, index), + params.put(kv.substring(0, index), kv.substring(index + 1)); } } @@ -1534,13 +1534,13 @@ public class PublisherAdminServlet extends AdminServlet { NameValuePairs saveParams = new NameValuePairs(); // implName is always required so always include it it. - saveParams.add("pluginName", implname); + saveParams.put("pluginName", implname); if (oldConfigParms != null) { for (int i = 0; i < oldConfigParms.size(); i++) { String kv = oldConfigParms.elementAt(i); int index = kv.indexOf('='); - saveParams.add(kv.substring(0, index), + saveParams.put(kv.substring(0, index), kv.substring(index + 1)); } } @@ -1875,7 +1875,7 @@ public class PublisherAdminServlet extends AdminServlet { NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_RULE_IMPL_NAME, implname); + params.put(Constants.PR_RULE_IMPL_NAME, implname); sendResponse(SUCCESS, null, params, resp); return; } @@ -1901,7 +1901,7 @@ public class PublisherAdminServlet extends AdminServlet { desc = lp.getDescription(); } catch (Exception exp) { } - params.add(name, value.getClassPath() + "," + desc); + params.put(name, value.getClassPath() + "," + desc); } sendResponse(SUCCESS, null, params, resp); return; @@ -1919,7 +1919,7 @@ public class PublisherAdminServlet extends AdminServlet { mProcessor.getRuleInsts().get((Object) name); String enabled = value.enabled() ? "enabled" : "disabled"; - params.add(name, value.getInstanceName() + ";visible;" + enabled); + params.put(name, value.getInstanceName() + ";visible;" + enabled); } sendResponse(SUCCESS, null, params, resp); return; @@ -2062,13 +2062,13 @@ public class PublisherAdminServlet extends AdminServlet { NameValuePairs params = new NameValuePairs(); // implName is always required so always send it. - params.add(Constants.PR_RULE_IMPL_NAME, ""); + params.put(Constants.PR_RULE_IMPL_NAME, ""); if (configParams != null) { for (int i = 0; i < configParams.size(); i++) { String kv = (String) configParams.elementAt(i); int index = kv.indexOf('='); - params.add(kv.substring(0, index), + params.put(kv.substring(0, index), kv.substring(index + 1)); } } @@ -2102,7 +2102,7 @@ public class PublisherAdminServlet extends AdminServlet { Vector configParams = ruleInst.getInstanceParams(); NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_RULE_IMPL_NAME, + params.put(Constants.PR_RULE_IMPL_NAME, getRulePluginName(ruleInst)); // implName is always required so always send it. if (configParams != null) { @@ -2110,7 +2110,7 @@ public class PublisherAdminServlet extends AdminServlet { String kv = (String) configParams.elementAt(i); int index = kv.indexOf('='); - params.add(kv.substring(0, index), + params.put(kv.substring(0, index), kv.substring(index + 1)); } } @@ -2167,13 +2167,13 @@ public class PublisherAdminServlet extends AdminServlet { NameValuePairs saveParams = new NameValuePairs(); // implName is always required so always include it it. - saveParams.add("pluginName", implname); + saveParams.put("pluginName", implname); if (oldConfigParms != null) { for (int i = 0; i < oldConfigParms.size(); i++) { String kv = oldConfigParms.elementAt(i); int index = kv.indexOf('='); - saveParams.add(kv.substring(0, index), + saveParams.put(kv.substring(0, index), kv.substring(index + 1)); } } @@ -2523,7 +2523,7 @@ public class PublisherAdminServlet extends AdminServlet { NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_PUBLISHER_IMPL_NAME, implname); + params.put(Constants.PR_PUBLISHER_IMPL_NAME, implname); sendResponse(SUCCESS, null, params, resp); return; } @@ -2550,7 +2550,7 @@ public class PublisherAdminServlet extends AdminServlet { desc = lp.getDescription(); } catch (Exception exp) { } - params.add(name, value.getClassPath() + "," + desc); + params.put(name, value.getClassPath() + "," + desc); } sendResponse(SUCCESS, null, params, resp); return; @@ -2579,7 +2579,7 @@ public class PublisherAdminServlet extends AdminServlet { if (value == null) continue; - params.add(name, getPublisherPluginName(value) + ";visible"); + params.put(name, getPublisherPluginName(value) + ";visible"); } sendResponse(SUCCESS, null, params, resp); return; @@ -2725,16 +2725,16 @@ public class PublisherAdminServlet extends AdminServlet { NameValuePairs params = new NameValuePairs(); // implName is always required so always send it. - params.add(Constants.PR_PUBLISHER_IMPL_NAME, ""); + params.put(Constants.PR_PUBLISHER_IMPL_NAME, ""); if (configParams != null) { for (int i = 0; i < configParams.size(); i++) { String kv = (String) configParams.elementAt(i); int index = kv.indexOf('='); if (index == -1) { - params.add(kv, ""); + params.put(kv, ""); } else { - params.add(kv.substring(0, index), + params.put(kv.substring(0, index), kv.substring(index + 1)); } } @@ -2771,7 +2771,7 @@ public class PublisherAdminServlet extends AdminServlet { Vector configParams = publisherInst.getInstanceParams(); NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_PUBLISHER_IMPL_NAME, + params.put(Constants.PR_PUBLISHER_IMPL_NAME, getPublisherPluginName(publisherInst)); // implName is always required so always send it. if (configParams != null) { @@ -2779,7 +2779,7 @@ public class PublisherAdminServlet extends AdminServlet { String kv = (String) configParams.elementAt(i); int index = kv.indexOf('='); - params.add(kv.substring(0, index), + params.put(kv.substring(0, index), kv.substring(index + 1)); } } @@ -2848,7 +2848,7 @@ public class PublisherAdminServlet extends AdminServlet { String pubType = ""; // implName is always required so always include it it. - saveParams.add("pluginName", implname); + saveParams.put("pluginName", implname); if (oldConfigParms != null) { for (int i = 0; i < oldConfigParms.size(); i++) { String kv = (String) oldConfigParms.elementAt(i); @@ -2860,7 +2860,7 @@ public class PublisherAdminServlet extends AdminServlet { pubType = "crl"; } - saveParams.add(kv.substring(0, index), + saveParams.put(kv.substring(0, index), kv.substring(index + 1)); } } @@ -2876,11 +2876,11 @@ public class PublisherAdminServlet extends AdminServlet { // get objects added and deleted if (pubType.equals("cacert")) { - saveParams.add("caObjectClassAdded", instancesConfig.getString(id + ".caObjectClassAdded", "")); - saveParams.add("caObjectClassDeleted", instancesConfig.getString(id + ".caObjectClassDeleted", "")); + saveParams.put("caObjectClassAdded", instancesConfig.getString(id + ".caObjectClassAdded", "")); + saveParams.put("caObjectClassDeleted", instancesConfig.getString(id + ".caObjectClassDeleted", "")); } else if (pubType.equals("crl")) { - saveParams.add("crlObjectClassAdded", instancesConfig.getString(id + ".crlObjectClassAdded", "")); - saveParams.add("crlObjectClassDeleted", instancesConfig.getString(id + ".crlObjectClassDeleted", "")); + saveParams.put("crlObjectClassAdded", instancesConfig.getString(id + ".crlObjectClassAdded", "")); + saveParams.put("crlObjectClassDeleted", instancesConfig.getString(id + ".crlObjectClassDeleted", "")); } // create new substore. @@ -3054,9 +3054,9 @@ public class PublisherAdminServlet extends AdminServlet { } catch (Exception e) { } - oldOC = saveParams.getValue(objName); - oldAdded = saveParams.getValue(objName + "Added"); - oldDeleted = saveParams.getValue(objName + "Deleted"); + oldOC = saveParams.get(objName); + oldAdded = saveParams.get(objName + "Added"); + oldDeleted = saveParams.get(objName + "Deleted"); if ((oldOC == null) || (newOC == null)) return; @@ -3106,11 +3106,8 @@ public class PublisherAdminServlet extends AdminServlet { store.removeSubStore(id); IConfigStore rstore = store.makeSubStore(id); - Enumeration keys = saveParams.getNames(); - - while (keys.hasMoreElements()) { - String key = keys.nextElement(); - String value = saveParams.getValue(key); + for (String key : saveParams.keySet()) { + String value = saveParams.get(key); if (value != null) rstore.put(key, value); diff --git a/pki/base/common/src/com/netscape/cms/servlet/admin/RAAdminServlet.java b/pki/base/common/src/com/netscape/cms/servlet/admin/RAAdminServlet.java index 1adde0891..5bdb14177 100644 --- a/pki/base/common/src/com/netscape/cms/servlet/admin/RAAdminServlet.java +++ b/pki/base/common/src/com/netscape/cms/servlet/admin/RAAdminServlet.java @@ -199,10 +199,10 @@ public class RAAdminServlet extends AdminServlet { continue; if (name.equals(Constants.PR_ENABLE)) continue; - params.add(name, rc.getString(name, "")); + params.put(name, rc.getString(name, "")); } - params.add(Constants.PR_ENABLE, + params.put(Constants.PR_ENABLE, rc.getString(PROP_ENABLED, Constants.FALSE)); //System.out.println("Send: "+params.toString()); sendResponse(SUCCESS, null, params, resp); @@ -265,10 +265,10 @@ public class RAAdminServlet extends AdminServlet { continue; if (name.equals(Constants.PR_ENABLE)) continue; - params.add(name, riq.getString(name, "")); + params.put(name, riq.getString(name, "")); } - params.add(Constants.PR_ENABLE, + params.put(Constants.PR_ENABLE, riq.getString(PROP_ENABLED, Constants.FALSE)); //System.out.println("Send: "+params.toString()); sendResponse(SUCCESS, null, params, resp); @@ -423,7 +423,7 @@ public class RAAdminServlet extends AdminServlet { if (name.equals(Constants.OP_TYPE)) continue; - params.add(name, caConnectorConfig.getString(name, "")); + params.put(name, caConnectorConfig.getString(name, "")); } } sendResponse(SUCCESS, null, params, resp); diff --git a/pki/base/common/src/com/netscape/cms/servlet/admin/RegistryAdminServlet.java b/pki/base/common/src/com/netscape/cms/servlet/admin/RegistryAdminServlet.java index 0175d9aa4..aa5494f0d 100644 --- a/pki/base/common/src/com/netscape/cms/servlet/admin/RegistryAdminServlet.java +++ b/pki/base/common/src/com/netscape/cms/servlet/admin/RegistryAdminServlet.java @@ -251,7 +251,7 @@ public class RegistryAdminServlet extends AdminServlet { String id = impls.nextElement(); IPluginInfo info = mRegistry.getPluginInfo(scope, id); - nvp.add(id, info.getClassName() + "," + + nvp.put(id, info.getClassName() + "," + info.getDescription(getLocale(req)) + "," + info.getName(getLocale(req))); } @@ -289,7 +289,7 @@ public class RegistryAdminServlet extends AdminServlet { if (policyConstraintClass.isApplicable(policyDefaultClass)) { CMS.debug("RegistryAdminServlet: getSUpportedConstraint isApplicable " + constraintInfo.getClassName()); - nvp.add(constraintID, + nvp.put(constraintID, constraintInfo.getClassName() + "," + constraintInfo.getDescription(getLocale(req)) + "," @@ -352,7 +352,7 @@ public class RegistryAdminServlet extends AdminServlet { + getNonNull(desc.getDefaultValue()); CMS.debug("RegistryAdminServlet: getProfileImpl " + value); - nvp.add(name, value); + nvp.put(name, value); } catch (Exception e) { CMS.debug("RegistryAdminServlet: getProfileImpl skipped descriptor for " + name); diff --git a/pki/base/common/src/com/netscape/cms/servlet/admin/UsrGrpAdminServlet.java b/pki/base/common/src/com/netscape/cms/servlet/admin/UsrGrpAdminServlet.java index 3783ead6f..e5a6dd3c4 100644 --- a/pki/base/common/src/com/netscape/cms/servlet/admin/UsrGrpAdminServlet.java +++ b/pki/base/common/src/com/netscape/cms/servlet/admin/UsrGrpAdminServlet.java @@ -327,7 +327,7 @@ public class UsrGrpAdminServlet extends AdminServlet { val = "noType"; NameValuePairs params = new NameValuePairs(); - params.add(Constants.PR_USER_TYPE, val); + params.put(Constants.PR_USER_TYPE, val); sendResponse(SUCCESS, null, params, resp); } @@ -372,7 +372,7 @@ public class UsrGrpAdminServlet extends AdminServlet { } i++; } - params.add("userInfo", sb.toString()); + params.put("userInfo", sb.toString()); sendResponse(SUCCESS, null, params, resp); } @@ -416,10 +416,10 @@ public class UsrGrpAdminServlet extends AdminServlet { } if (user != null) { - params.add(Constants.PR_USER_FULLNAME, user.getFullName()); - params.add(Constants.PR_USER_EMAIL, user.getEmail()); - params.add(Constants.PR_USER_PHONE, user.getPhone()); - params.add(Constants.PR_USER_STATE, user.getState()); + params.put(Constants.PR_USER_FULLNAME, user.getFullName()); + params.put(Constants.PR_USER_EMAIL, user.getEmail()); + params.put(Constants.PR_USER_PHONE, user.getPhone()); + params.put(Constants.PR_USER_STATE, user.getState()); // get list of groups, and get a list of those that this // uid belongs to @@ -447,7 +447,7 @@ public class UsrGrpAdminServlet extends AdminServlet { } } - params.add(Constants.PR_USER_GROUP, grpString.toString()); + params.put(Constants.PR_USER_GROUP, grpString.toString()); sendResponse(SUCCESS, null, params, resp); return; @@ -516,7 +516,7 @@ public class UsrGrpAdminServlet extends AdminServlet { String base64 = CMS.getEncodedCert(certs[i]); // pretty print certs - params.add(getCertificateString(certs[i]), + params.put(getCertificateString(certs[i]), print.toString(clientLocale) + "\n" + base64); } sendResponse(SUCCESS, null, params, resp); @@ -567,9 +567,9 @@ public class UsrGrpAdminServlet extends AdminServlet { String desc = group.getDescription(); if (desc != null) { - params.add(group.getGroupID(), desc); + params.put(group.getGroupID(), desc); } else { - params.add(group.getGroupID(), ""); + params.put(group.getGroupID(), ""); } } @@ -612,8 +612,8 @@ public class UsrGrpAdminServlet extends AdminServlet { if (e.hasMoreElements()) { IGroup group = e.nextElement(); - params.add(Constants.PR_GROUP_GROUP, group.getGroupID()); - params.add(Constants.PR_GROUP_DESC, + params.put(Constants.PR_GROUP_GROUP, group.getGroupID()); + params.put(Constants.PR_GROUP_DESC, group.getDescription()); Enumeration members = group.getMemberNames(); @@ -631,7 +631,7 @@ public class UsrGrpAdminServlet extends AdminServlet { } } - params.add(Constants.PR_GROUP_USER, membersString.toString()); + params.put(Constants.PR_GROUP_USER, membersString.toString()); sendResponse(SUCCESS, null, params, resp); return; diff --git a/pki/base/common/src/com/netscape/cmscore/security/JssSubsystem.java b/pki/base/common/src/com/netscape/cmscore/security/JssSubsystem.java index ac44672d2..42768060c 100644 --- a/pki/base/common/src/com/netscape/cmscore/security/JssSubsystem.java +++ b/pki/base/common/src/com/netscape/cmscore/security/JssSubsystem.java @@ -90,7 +90,6 @@ import com.netscape.certsrv.base.EBaseException; import com.netscape.certsrv.base.IConfigStore; import com.netscape.certsrv.base.ISubsystem; import com.netscape.certsrv.common.Constants; -import com.netscape.certsrv.common.NameValuePair; import com.netscape.certsrv.common.NameValuePairs; import com.netscape.certsrv.logging.ILogger; import com.netscape.certsrv.security.ICryptoSubsystem; @@ -1043,11 +1042,11 @@ public final class JssSubsystem implements ICryptoSubsystem { X509CertImpl impl = new X509CertImpl(b); NameValuePairs results = new NameValuePairs(); - results.add(Constants.PR_CERT_SUBJECT_NAME, impl.getSubjectDN().getName()); - results.add(Constants.PR_ISSUER_NAME, impl.getIssuerDN().getName()); - results.add(Constants.PR_SERIAL_NUMBER, impl.getSerialNumber().toString()); - results.add(Constants.PR_BEFORE_VALIDDATE, impl.getNotBefore().toString()); - results.add(Constants.PR_AFTER_VALIDDATE, impl.getNotAfter().toString()); + results.put(Constants.PR_CERT_SUBJECT_NAME, impl.getSubjectDN().getName()); + results.put(Constants.PR_ISSUER_NAME, impl.getIssuerDN().getName()); + results.put(Constants.PR_SERIAL_NUMBER, impl.getSerialNumber().toString()); + results.put(Constants.PR_BEFORE_VALIDDATE, impl.getNotBefore().toString()); + results.put(Constants.PR_AFTER_VALIDDATE, impl.getNotAfter().toString()); // fingerprint is using MD5 hash @@ -1202,7 +1201,7 @@ public final class JssSubsystem implements ICryptoSubsystem { } String serialno = impl.getSerialNumber().toString(); String issuer = impl.getIssuerDN().toString(); - nvps.add(nickname + "," + serialno, issuer); + nvps.put(nickname + "," + serialno, issuer); Debug.trace("getRootCerts: nickname=" + nickname + ", serialno=" + serialno + ", issuer=" + issuer); continue; @@ -1265,7 +1264,7 @@ public final class JssSubsystem implements ICryptoSubsystem { } String serialno = impl.getSerialNumber().toString(); String issuer = impl.getIssuerDN().toString(); - nvps.add(nickname + "," + serialno, issuer); + nvps.put(nickname + "," + serialno, issuer); Debug.trace("getUserCerts: nickname=" + nickname + ", serialno=" + serialno + ", issuer=" + issuer); } catch (ObjectNotFoundException e) { @@ -1329,18 +1328,16 @@ public final class JssSubsystem implements ICryptoSubsystem { } Date date = impl.getNotAfter(); String dateStr = mFormatter.format(date); - NameValuePair pair = pairs.getPair(nickname); + String vvalue = pairs.get(nickname); /* always user cert here*/ String certValue = dateStr + "," + "u"; - if (pair == null) - pairs.add(nickname, certValue); + if (vvalue == null) + pairs.put(nickname, certValue); else { - String vvalue = pair.getValue(); - if (vvalue.endsWith(",u")) { - pair.setValue(vvalue + ";" + certValue); + pairs.put(nickname, vvalue + ";" + certValue); } } @@ -1441,15 +1438,13 @@ public final class JssSubsystem implements ICryptoSubsystem { impl = new X509CertImpl(icert.getEncoded()); Date date = impl.getNotAfter(); String dateStr = mFormatter.format(date); - NameValuePair pair = pairs.getPair(nickname); + String vvalue = pairs.get(nickname); String certValue = dateStr + "," + trust; - if (pair == null) - pairs.add(nickname, certValue); + if (vvalue == null) + pairs.put(nickname, certValue); else { - String vvalue = pair.getValue(); - - pair.setValue(vvalue + ";" + certValue); + pairs.put(nickname, vvalue + ";" + certValue); } } catch (CertificateException e) { log(ILogger.LL_FAILURE, diff --git a/pki/base/console/src/CMakeLists.txt b/pki/base/console/src/CMakeLists.txt index b164a33a0..175a4a1a7 100644 --- a/pki/base/console/src/CMakeLists.txt +++ b/pki/base/console/src/CMakeLists.txt @@ -85,7 +85,6 @@ set(pki-console_java_SRCS com/netscape/certsrv/common/ScopeDef.java com/netscape/certsrv/common/PrefixDef.java com/netscape/certsrv/common/ConfigConstants.java - com/netscape/certsrv/common/NameValuePair.java com/netscape/certsrv/common/OpDef.java com/netscape/certsrv/common/Constants.java com/netscape/admin/certsrv/CMSCCMUILoader.java diff --git a/pki/base/console/src/com/netscape/admin/certsrv/CMSCAUILoader.java b/pki/base/console/src/com/netscape/admin/certsrv/CMSCAUILoader.java index bad1868b5..f967d28e0 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/CMSCAUILoader.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/CMSCAUILoader.java @@ -233,9 +233,7 @@ public class CMSCAUILoader implements ISubSystemUILoader { } if (nvps != null && nvps.size() > 0) { - Enumeration names = nvps.getNames(); - while (names.hasMoreElements()) { - String name = (String)names.nextElement(); + for (String name : nvps.keySet()) { if (name.indexOf('.') == -1) { node = new CMSResourceObject(); node.setName(name); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/CMSKernelUILoader.java b/pki/base/console/src/com/netscape/admin/certsrv/CMSKernelUILoader.java index 2c4de161f..cb232c527 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/CMSKernelUILoader.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/CMSKernelUILoader.java @@ -306,8 +306,7 @@ public class CMSKernelUILoader implements ISubSystemUILoader { } //update the index - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = (String)e.nextElement(); + for (String entry : response.keySet()) { CMSResourceObject node = new CMSResourceObject(entry); node.setCustomPanel(new LogInstancePanel(entry, model)); node.setIcon( CMSAdminUtil.getImage(CMSAdminResources.IMAGE_LOGOBJECT)); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/CMSServerInfo.java b/pki/base/console/src/com/netscape/admin/certsrv/CMSServerInfo.java index d4646c2a0..d2d64b3fc 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/CMSServerInfo.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/CMSServerInfo.java @@ -18,14 +18,9 @@ package com.netscape.admin.certsrv; import java.util.*; -import java.awt.*; -import java.awt.event.*; import javax.swing.*; -import javax.swing.event.*; -import javax.swing.border.*; -import javax.swing.text.*; + import com.netscape.management.client.util.*; -import com.netscape.management.client.console.*; import com.netscape.certsrv.common.*; import com.netscape.admin.certsrv.connection.*; @@ -102,7 +97,7 @@ public class CMSServerInfo implements IConnectionListener { // Need to do authentication here NameValuePairs config = new NameValuePairs(); - config.add(Constants.PR_PING,""); + config.put(Constants.PR_PING, ""); NameValuePairs response; @@ -111,7 +106,7 @@ public class CMSServerInfo implements IConnectionListener { Constants.RS_ID_CONFIG, config); - if (!response.getValue(Constants.PR_PING).equalsIgnoreCase(Constants.TRUE)) { + if (!response.get(Constants.PR_PING).equalsIgnoreCase(Constants.TRUE)) { Debug.println("Ping failed -> Server off"); throw new EAdminException("PING_FAILED",false); } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/CMSUIFramework.java b/pki/base/console/src/com/netscape/admin/certsrv/CMSUIFramework.java index 6949e73f6..13028b003 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/CMSUIFramework.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/CMSUIFramework.java @@ -23,7 +23,6 @@ import com.netscape.management.client.util.*; import com.netscape.management.client.console.*; import com.netscape.certsrv.common.*; import com.netscape.admin.certsrv.connection.*; -import java.util.*; /** * Netscape Certificate Server 4.0 UI Framework @@ -88,7 +87,7 @@ public class CMSUIFramework { ScopeDef.SC_PLATFORM, new NameValuePairs()); if (response == null) throw new EAdminException("PROTOCOL_ERROR",false); - if (response.getValue(Constants.PR_NT).equals(Constants.TRUE)) + if (response.get(Constants.PR_NT).equals(Constants.TRUE)) return true; return false; } @@ -207,9 +206,9 @@ class HTTPSSubSystemLocator implements ISubSystemLocator { Debug.println("getInstalledSubSystem() - "+input.toString()); SubSystemInfo[] subsystems = new SubSystemInfo[input.size()]; int i =0; - for (Enumeration e = input.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - String value = input.getValue(entry); + for (String entry : input.keySet()) { + entry = entry.trim(); + String value = input.get(entry); subsystems[i] = new SubSystemInfo(); subsystems[i].mType = value; subsystems[i].mNickName = entry; diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/ACIDialog.java b/pki/base/console/src/com/netscape/admin/certsrv/config/ACIDialog.java index 7c2948795..e6085f820 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/ACIDialog.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/ACIDialog.java @@ -18,16 +18,12 @@ package com.netscape.admin.certsrv.config; import com.netscape.admin.certsrv.*; -import com.netscape.admin.certsrv.config.*; import com.netscape.admin.certsrv.connection.*; import javax.swing.*; -import javax.swing.event.*; -import javax.swing.table.*; -import javax.swing.text.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -250,13 +246,11 @@ public class ACIDialog extends JDialog } private boolean validateSyntax(String str, NameValuePairs nvps) { - Enumeration names = nvps.getNames(); - while (names.hasMoreElements()) { - String name = (String)names.nextElement(); + for (String name : nvps.keySet()) { if (str.startsWith(name)) { int len = name.length(); String leftover = str.substring(len).trim(); - String operators = nvps.getValue(name); + String operators = nvps.get(name); StringTokenizer st = new StringTokenizer(operators, ","); while (st.hasMoreTokens()) { String token = st.nextToken(); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/ACLEditDialog.java b/pki/base/console/src/com/netscape/admin/certsrv/config/ACLEditDialog.java index 9aa392eaf..b3bcb72e1 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/ACLEditDialog.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/ACLEditDialog.java @@ -18,16 +18,12 @@ package com.netscape.admin.certsrv.config; import com.netscape.admin.certsrv.*; -import com.netscape.admin.certsrv.config.*; import com.netscape.admin.certsrv.connection.*; import javax.swing.*; -import javax.swing.event.*; -import javax.swing.table.*; -import javax.swing.text.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -145,9 +141,9 @@ public class ACLEditDialog extends JDialog else str = str+(String)mDataModel.elementAt(i); } - pairs.add(Constants.PR_ACI, str); - pairs.add(Constants.PR_ACL_DESC, desc); - pairs.add(Constants.PR_ACL_RIGHTS, rights); + pairs.put(Constants.PR_ACI, str); + pairs.put(Constants.PR_ACL_DESC, desc); + pairs.put(Constants.PR_ACL_RIGHTS, rights); } try { @@ -204,8 +200,8 @@ public class ACLEditDialog extends JDialog } public void showDialog(NameValuePairs data) { - String aci = data.getValue(Constants.PR_ACI); - mOperations = data.getValue(Constants.PR_ACL_OPS); + String aci = data.get(Constants.PR_ACI); + mOperations = data.get(Constants.PR_ACL_OPS); if ((aci != null) && (!aci.trim().equals(""))) { StringTokenizer tokenizer = new StringTokenizer(aci, ";"); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/ACLImplTab.java b/pki/base/console/src/com/netscape/admin/certsrv/config/ACLImplTab.java index b0dd8bb9e..4693224b1 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/ACLImplTab.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/ACLImplTab.java @@ -204,9 +204,8 @@ public class ACLImplTab extends CMSBaseUGTab { int i=0; String[] vals = new String[response.size()]; - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - vals[i++] = entry; + for (String entry : response.keySet()) { + vals[i++] = entry.trim(); } CMSAdminUtil.bubbleSort(vals); @@ -217,7 +216,7 @@ public class ACLImplTab extends CMSBaseUGTab { v.addElement(new JLabel(name, CMSAdminUtil.getImage(CMSAdminResources.IMAGE_ACLPLUGIN), JLabel.LEFT)); - v.addElement(response.getValue(name)); + v.addElement(response.get(name)); mDataModel.addRow(v); } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/ACLPanel.java b/pki/base/console/src/com/netscape/admin/certsrv/config/ACLPanel.java index 3b3d8c529..2593e0c0b 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/ACLPanel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/ACLPanel.java @@ -202,9 +202,8 @@ public class ACLPanel extends CMSBaseUGTab { int i=0; String[] vals = new String[response.size()]; - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - vals[i++] = entry; + for (String entry : response.keySet()) { + vals[i++] = entry.trim(); } CMSAdminUtil.bubbleSort(vals); @@ -215,7 +214,7 @@ public class ACLPanel extends CMSBaseUGTab { v.addElement(new JLabel(name, CMSAdminUtil.getImage(CMSAdminResources.IMAGE_ACL), JLabel.LEFT)); - v.addElement(response.getValue(name)); + v.addElement(response.get(name)); mDataModel.addRow(v); } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/CACertsTab.java b/pki/base/console/src/com/netscape/admin/certsrv/config/CACertsTab.java index e57c56271..18590747a 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/CACertsTab.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/CACertsTab.java @@ -21,11 +21,10 @@ import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import com.netscape.admin.certsrv.ug.*; import javax.swing.*; -import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.management.client.console.*; import com.netscape.certsrv.common.*; @@ -123,17 +122,16 @@ public class CACertsTab extends CMSBaseUGTab { try { NameValuePairs nvps = new NameValuePairs(); - nvps.add(Constants.PR_NICK_NAME, nickname); - nvps.add(Constants.PR_SERIAL_NUMBER, serialno); - nvps.add(Constants.PR_ISSUER_NAME, issuername); + nvps.put(Constants.PR_NICK_NAME, nickname); + nvps.put(Constants.PR_SERIAL_NUMBER, serialno); + nvps.put(Constants.PR_ISSUER_NAME, issuername); NameValuePairs results = mConnection.process( DestDef.DEST_SERVER_ADMIN, ScopeDef.SC_CERT_PRETTY_PRINT, Constants.RS_ID_CONFIG, nvps); if (nvps.size() <= 0) return; - NameValuePair nvp = results.elementAt(0); - String name = nvp.getName(); - String print = nvp.getValue(); + String name = results.keySet().iterator().next(); // first element + String print = results.get(name); CertViewDialog certdialog = new CertViewDialog(mModel.getFrame()); certdialog.showDialog(nickname, print); } catch (EAdminException ex) { @@ -150,17 +148,16 @@ public class CACertsTab extends CMSBaseUGTab { try { NameValuePairs nvps = new NameValuePairs(); - nvps.add(Constants.PR_NICK_NAME, nickname); - nvps.add(Constants.PR_SERIAL_NUMBER, serialno); - nvps.add(Constants.PR_ISSUER_NAME, issuername); + nvps.put(Constants.PR_NICK_NAME, nickname); + nvps.put(Constants.PR_SERIAL_NUMBER, serialno); + nvps.put(Constants.PR_ISSUER_NAME, issuername); NameValuePairs results = mConnection.process( DestDef.DEST_SERVER_ADMIN, ScopeDef.SC_ROOTCERT_TRUSTBIT, Constants.RS_ID_CONFIG, nvps); if (nvps.size() <= 0) return; - NameValuePair nvp = results.elementAt(0); - String name = nvp.getName(); - String trust = nvp.getValue(); + String name = results.keySet().iterator().next(); // first element + String trust = results.get(name); int i; String[] params = new String[2]; if (trust.equals("U")) { @@ -174,7 +171,7 @@ public class CACertsTab extends CMSBaseUGTab { } if (i == JOptionPane.YES_OPTION) { - nvps.add("trustbit", params[1]); + nvps.put("trustbit", params[1]); mConnection.modify(DestDef.DEST_SERVER_ADMIN, ScopeDef.SC_ROOTCERT_TRUSTBIT, Constants.RS_ID_CONFIG, nvps); @@ -333,9 +330,8 @@ public class CACertsTab extends CMSBaseUGTab { if (response != null) { String[] vals = new String[response.size()]; int i=0; - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - vals[i++] = entry; + for (String entry : response.keySet()) { + vals[i++] = entry.trim(); } int sindex = 0; @@ -344,7 +340,7 @@ public class CACertsTab extends CMSBaseUGTab { for (i=0; i0) { - nvp.add("PASSWORD_CACHE_ADD",value+";"+password); + nvp.put("PASSWORD_CACHE_ADD", value + ";" + password); } } - nvp.add(paramName,value); + nvp.put(paramName, value); } - nvp.add(PolicyRuleDataModel.RULE_NAME,mRuleName); - nvp.add(mImplName_token,mImplName.getText()); + nvp.put(PolicyRuleDataModel.RULE_NAME, mRuleName); + nvp.put(mImplName_token, mImplName.getText()); mData = nvp; try { @@ -502,7 +495,7 @@ public class CMSBaseConfigDialog extends JDialog } else { if (mId != null && mId.length() > 0) { - nvp.add(Constants.PR_ID, mId); + nvp.put(Constants.PR_ID, mId); } mAdminConnection.modify(mDest, mInstanceScope, mRuleName, nvp); } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/CMSBaseLDAPPanel.java b/pki/base/console/src/com/netscape/admin/certsrv/config/CMSBaseLDAPPanel.java index cb8826d2a..dbba4a763 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/CMSBaseLDAPPanel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/CMSBaseLDAPPanel.java @@ -234,24 +234,24 @@ public abstract class CMSBaseLDAPPanel extends CMSBaseTab implements ItemListene mModel.progressStart(); NameValuePairs nvps = new NameValuePairs(); if (mPublishing) - nvps.add(Constants.PR_PUBLISHING_ENABLE,""); + nvps.put(Constants.PR_PUBLISHING_ENABLE, ""); if (mLDAPPublishing) - nvps.add(Constants.PR_ENABLE,""); - nvps.add(Constants.PR_HOST_NAME, ""); - nvps.add(Constants.PR_LDAP_PORT, ""); - nvps.add(Constants.PR_SECURE_PORT_ENABLED, ""); + nvps.put(Constants.PR_ENABLE, ""); + nvps.put(Constants.PR_HOST_NAME, ""); + nvps.put(Constants.PR_LDAP_PORT, ""); + nvps.put(Constants.PR_SECURE_PORT_ENABLED, ""); //nvps.add(Constants.PR_BASE_DN, ""); - nvps.add(Constants.PR_BIND_DN, ""); - nvps.add(Constants.PR_LDAP_VERSION, ""); - nvps.add(Constants.PR_AUTH_TYPE, ""); - nvps.add(Constants.PR_CERT_NAMES, ""); - nvps.add(Constants.PR_LDAP_CLIENT_CERT, ""); - - nvps.add(Constants.PR_PUBLISHING_QUEUE_ENABLE, ""); - nvps.add(Constants.PR_PUBLISHING_QUEUE_THREADS, ""); - nvps.add(Constants.PR_PUBLISHING_QUEUE_PAGE_SIZE, ""); - nvps.add(Constants.PR_PUBLISHING_QUEUE_PRIORITY, ""); - nvps.add(Constants.PR_PUBLISHING_QUEUE_STATUS, ""); + nvps.put(Constants.PR_BIND_DN, ""); + nvps.put(Constants.PR_LDAP_VERSION, ""); + nvps.put(Constants.PR_AUTH_TYPE, ""); + nvps.put(Constants.PR_CERT_NAMES, ""); + nvps.put(Constants.PR_LDAP_CLIENT_CERT, ""); + + nvps.put(Constants.PR_PUBLISHING_QUEUE_ENABLE, ""); + nvps.put(Constants.PR_PUBLISHING_QUEUE_THREADS, ""); + nvps.put(Constants.PR_PUBLISHING_QUEUE_PAGE_SIZE, ""); + nvps.put(Constants.PR_PUBLISHING_QUEUE_PRIORITY, ""); + nvps.put(Constants.PR_PUBLISHING_QUEUE_STATUS, ""); try { NameValuePairs val = mAdmin.read(mServletName, @@ -280,27 +280,26 @@ public abstract class CMSBaseLDAPPanel extends CMSBaseTab implements ItemListene int serverCertIndex = -1; String version = ""; - for (int i=0; i= 0) mAuthBox.setSelectedIndex(index); } else if (name.equals(Constants.PR_CERT_NAMES)) { mCertBox.removeAllItems(); - String certNames = nvp.getValue(); + String certNames = value; StringTokenizer tokenizer = new StringTokenizer(certNames, DELIMITER); for (int index=0; tokenizer.hasMoreTokens(); index++) { @@ -336,9 +335,9 @@ public abstract class CMSBaseLDAPPanel extends CMSBaseTab implements ItemListene mCertBox.addItem(str); } } else if (name.equals(Constants.PR_LDAP_CLIENT_CERT)) { - clientCert = nvp.getValue(); + clientCert = value; } else if (name.equals(Constants.PR_LDAP_VERSION)) { - version = nvp.getValue(); + version = value; } } @@ -512,26 +511,26 @@ public abstract class CMSBaseLDAPPanel extends CMSBaseTab implements ItemListene if (mPublishing) { if (mEnablePublishing.isSelected()) - nvps.add(Constants.PR_PUBLISHING_ENABLE,Constants.TRUE); + nvps.put(Constants.PR_PUBLISHING_ENABLE, Constants.TRUE); else - nvps.add(Constants.PR_PUBLISHING_ENABLE,Constants.FALSE); + nvps.put(Constants.PR_PUBLISHING_ENABLE, Constants.FALSE); } if (mLDAPPublishing) { if (mEnable.isSelected()) - nvps.add(Constants.PR_ENABLE,Constants.TRUE); + nvps.put(Constants.PR_ENABLE, Constants.TRUE); else - nvps.add(Constants.PR_ENABLE,Constants.FALSE); + nvps.put(Constants.PR_ENABLE, Constants.FALSE); } if (mEnableQueue.isSelected()) - nvps.add(Constants.PR_PUBLISHING_QUEUE_ENABLE, Constants.TRUE); + nvps.put(Constants.PR_PUBLISHING_QUEUE_ENABLE, Constants.TRUE); else - nvps.add(Constants.PR_PUBLISHING_QUEUE_ENABLE, Constants.FALSE); - nvps.add(Constants.PR_PUBLISHING_QUEUE_THREADS, mMaxNumberOfPublishingThreads); - nvps.add(Constants.PR_PUBLISHING_QUEUE_PAGE_SIZE, mPublishingQueuePageSize); - nvps.add(Constants.PR_PUBLISHING_QUEUE_PRIORITY, mPublishingQueuePriorityLevel); - nvps.add(Constants.PR_PUBLISHING_QUEUE_STATUS, mPublishingQueueStatus); + nvps.put(Constants.PR_PUBLISHING_QUEUE_ENABLE, Constants.FALSE); + nvps.put(Constants.PR_PUBLISHING_QUEUE_THREADS, mMaxNumberOfPublishingThreads); + nvps.put(Constants.PR_PUBLISHING_QUEUE_PAGE_SIZE, mPublishingQueuePageSize); + nvps.put(Constants.PR_PUBLISHING_QUEUE_PRIORITY, mPublishingQueuePriorityLevel); + nvps.put(Constants.PR_PUBLISHING_QUEUE_STATUS, mPublishingQueueStatus); if (mEnable.isSelected()) { String host = mHostNameText.getText().trim(); @@ -550,21 +549,21 @@ public abstract class CMSBaseLDAPPanel extends CMSBaseTab implements ItemListene showMessageDialog("NUMBERFORMAT"); return false; } - nvps.add(Constants.PR_HOST_NAME, host); - nvps.add(Constants.PR_LDAP_PORT, port); + nvps.put(Constants.PR_HOST_NAME, host); + nvps.put(Constants.PR_LDAP_PORT, port); if (mSecurePort.isSelected()) - nvps.add(Constants.PR_SECURE_PORT_ENABLED, Constants.TRUE); + nvps.put(Constants.PR_SECURE_PORT_ENABLED, Constants.TRUE); else - nvps.add(Constants.PR_SECURE_PORT_ENABLED, Constants.FALSE); + nvps.put(Constants.PR_SECURE_PORT_ENABLED, Constants.FALSE); if (mAuthBox.getSelectedIndex() == 0) { if (mPanelName.equals("CALDAPSETTING")) { - nvps.add(Constants.PR_BINDPWD_PROMPT, "CA LDAP Publishing"); + nvps.put(Constants.PR_BINDPWD_PROMPT, "CA LDAP Publishing"); } else { - nvps.add(Constants.PR_BINDPWD_PROMPT, "RA LDAP Publishing"); + nvps.put(Constants.PR_BINDPWD_PROMPT, "RA LDAP Publishing"); } - nvps.add(Constants.PR_BIND_DN, bindAs); + nvps.put(Constants.PR_BIND_DN, bindAs); passwd = mPasswordText.getText(); if (passwd.equals("")) { @@ -572,10 +571,10 @@ public abstract class CMSBaseLDAPPanel extends CMSBaseTab implements ItemListene return false; } - nvps.add(Constants.PR_DIRECTORY_MANAGER_PWD, passwd); + nvps.put(Constants.PR_DIRECTORY_MANAGER_PWD, passwd); } else { - nvps.add(Constants.PR_LDAP_CLIENT_CERT, - (String)(mCertBox.getSelectedItem())); + nvps.put(Constants.PR_LDAP_CLIENT_CERT, + (String) (mCertBox.getSelectedItem())); } int index = mAuthBox.getSelectedIndex(); @@ -585,11 +584,11 @@ public abstract class CMSBaseLDAPPanel extends CMSBaseTab implements ItemListene return false; } } - nvps.add(Constants.PR_AUTH_TYPE, AUTHTYPE[index]); - nvps.add(Constants.PR_LDAP_VERSION, - (String)mVersionBox.getSelectedItem()); + nvps.put(Constants.PR_AUTH_TYPE, AUTHTYPE[index]); + nvps.put(Constants.PR_LDAP_VERSION, + (String) mVersionBox.getSelectedItem()); - // test the connection before save + // test the connection before save /* LDAPConnection conn = null; if (mAuthBox.getSelectedIndex() == 1) { @@ -642,8 +641,8 @@ public abstract class CMSBaseLDAPPanel extends CMSBaseTab implements ItemListene Constants.RS_ID_CONFIG, nvps, false); // show test report String report = ""; - for (int i = 0; i 0) { - request.add(Constants.PR_ID, mId); + request.put(Constants.PR_ID, mId); } try { @@ -349,22 +348,20 @@ public abstract class CMSPluginInstanceTab extends CMSBaseUGTab { Debug.println(response.toString()); - Enumeration e = response.getNames(); - - /* format of each data element: plugin;visibility;enabled where plugin is the name of the plugin impl visibility is one of {visible,invisible} enabled is one of {enabled,disabled} */ - while (e.hasMoreElements()) { + + for (String entry : response.keySet()) { String plugin=""; String visibility=null; String enabled=null; - String entry = ((String)e.nextElement()).trim(); - String value = response.getValue(entry); + entry = entry.trim(); + String value = response.get(entry); StringTokenizer st = new StringTokenizer(value,";"); @@ -382,10 +379,10 @@ Debug.println("xxxxxxx plugin " + plugin); if (visibility != null && visibility.equals("visible")) { NameValuePairs data = new NameValuePairs(); - data.add(RULE_NAME,entry); - data.add(RULE_IMPL,plugin); + data.put(RULE_NAME, entry); + data.put(RULE_IMPL, plugin); if (enabled != null) { - data.add(RULE_STAT,enabled); + data.put(RULE_STAT, enabled); } mDataModel.processData(data); } @@ -411,7 +408,7 @@ Debug.println("xxxxxxx plugin " + plugin); try { mConnection.delete(mDestination, mScope, - data.getValue(RULE_NAME)); + data.get(RULE_NAME)); } catch (EAdminException e) { //display error dialog showErrorDialog(e.getMessage()); @@ -431,12 +428,12 @@ Debug.println("xxxxxxx plugin " + plugin); mDataModel.getObjectValueAt(mTable.getSelectedRow()); NameValuePairs request = new NameValuePairs(); if (mId != null && mId.length() > 0) { - request.add(mId, ""); + request.put(mId, ""); } NameValuePairs response = mConnection.read(mDestination, mScope, - data.getValue(RULE_NAME), + data.get(RULE_NAME), request); return response; } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/CMSRACLMPanel.java b/pki/base/console/src/com/netscape/admin/certsrv/config/CMSRACLMPanel.java index 3b2ee1ce4..734425b41 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/CMSRACLMPanel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/CMSRACLMPanel.java @@ -138,12 +138,12 @@ public class CMSRACLMPanel extends CMSBaseTab { public void refresh() { mModel.progressStart(); NameValuePairs nvps = new NameValuePairs(); - nvps.add(Constants.PR_RENEWAL_ENABLED, ""); - nvps.add(Constants.PR_RENEWAL_VALIDITY, ""); - nvps.add(Constants.PR_RENEWAL_EMAIL, ""); - nvps.add(Constants.PR_RENEWAL_EXPIREDNOTIFIEDENABLED, ""); - nvps.add(Constants.PR_RENEWAL_NUMNOTIFICATION, ""); - nvps.add(Constants.PR_RENEWAL_INTERVAL, ""); + nvps.put(Constants.PR_RENEWAL_ENABLED, ""); + nvps.put(Constants.PR_RENEWAL_VALIDITY, ""); + nvps.put(Constants.PR_RENEWAL_EMAIL, ""); + nvps.put(Constants.PR_RENEWAL_EXPIREDNOTIFIEDENABLED, ""); + nvps.put(Constants.PR_RENEWAL_NUMNOTIFICATION, ""); + nvps.put(Constants.PR_RENEWAL_INTERVAL, ""); try { NameValuePairs val = mAdmin.read(DestDef.DEST_RA_ADMIN, @@ -162,23 +162,22 @@ public class CMSRACLMPanel extends CMSBaseTab { protected void populate(NameValuePairs nvps) { boolean renewalEnabled = false; boolean notificationEnabled = false; - for (int i=0; i 0) { - nvps.add(Constants.PR_ID, mName); + nvps.put(Constants.PR_ID, mName); } else { - nvps.add(Constants.PR_ID, mNameText.getText().trim()); + nvps.put(Constants.PR_ID, mNameText.getText().trim()); } - nvps.add(Constants.PR_DESCRIPTION, mDescText.getText().trim()); + nvps.put(Constants.PR_DESCRIPTION, mDescText.getText().trim()); if (mEnable) { - nvps.add(Constants.PR_ENABLED, Constants.TRUE); + nvps.put(Constants.PR_ENABLED, Constants.TRUE); } else { - nvps.add(Constants.PR_ENABLED, Constants.FALSE); + nvps.put(Constants.PR_ENABLED, Constants.FALSE); } try { diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/ConnectorEditor.java b/pki/base/console/src/com/netscape/admin/certsrv/config/ConnectorEditor.java index b6b831d05..6f711eca9 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/ConnectorEditor.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/ConnectorEditor.java @@ -19,16 +19,13 @@ package com.netscape.admin.certsrv.config; import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.ug.*; -import com.netscape.admin.certsrv.config.*; import com.netscape.admin.certsrv.connection.*; import javax.swing.*; -import javax.swing.event.*; import javax.swing.table.*; -import javax.swing.text.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -111,18 +108,17 @@ public class ConnectorEditor extends JDialog implements ActionListener, MouseLis if (response != null) { String[] vals = new String[response.size()]; int i=0; - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - vals[i++] = entry; + for (String entry : response.keySet()) { + vals[i++] = entry.trim(); } int sindex = 0; String snickname = ""; - CMSAdminUtil.quickSort(vals, 0, response.size()-1); + CMSAdminUtil.quickSort(vals, 0, response.size() - 1); for (i=0; i data = new Hashtable(); + for (String entry : response.keySet()) { + entry = entry.trim(); + String value = response.get(entry); Debug.println("entry= "+entry); - Debug.println("value= "+value); + Debug.println("value= " + value); int x = value.indexOf(","); - Debug.println("x = "+x); + Debug.println("x = " + x); NameValuePairs obj = new NameValuePairs(); - obj.add(IMPL_NAME,entry); + obj.put(IMPL_NAME, entry); vals[i++]= entry ; if (x != -1) { - obj.add(IMPL_CLASS, value.substring(0,x)); - obj.add(IMPL_DESC, value.substring(x+1)); + obj.put(IMPL_CLASS, value.substring(0, x)); + obj.put(IMPL_DESC, value.substring(x + 1)); } else { - obj.add(IMPL_CLASS, value); + obj.put(IMPL_CLASS, value); } data.put(entry,obj); } @@ -309,7 +308,7 @@ public class JobsImplTab extends CMSBaseUGTab { try { mConnection.delete(mDestination, ScopeDef.SC_JOBS_IMPLS, - obj.getValue(IMPL_NAME)); + obj.get(IMPL_NAME)); } catch (EAdminException e) { //display error dialog showErrorDialog(e.getMessage()); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/JobsRuleDataModel.java b/pki/base/console/src/com/netscape/admin/certsrv/config/JobsRuleDataModel.java index e12114eb1..76fbd9a38 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/JobsRuleDataModel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/JobsRuleDataModel.java @@ -20,7 +20,6 @@ package com.netscape.admin.certsrv.config; import java.util.*; import javax.swing.*; import com.netscape.admin.certsrv.*; -import com.netscape.admin.certsrv.connection.*; import com.netscape.certsrv.common.*; import com.netscape.management.client.util.Debug; @@ -53,19 +52,19 @@ public class JobsRuleDataModel extends CMSRuleDataModel NameValuePairs obj = (NameValuePairs) data; //XXX NEED TO ADD STUFF - if (obj.getValue(RULE_STAT).equalsIgnoreCase("enabled")) { - v.addElement(new JLabel(obj.getValue(RULE_NAME), + if (obj.get(RULE_STAT).equalsIgnoreCase("enabled")) { + v.addElement(new JLabel(obj.get(RULE_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_RULE), JLabel.LEFT)); v.addElement(mResource.getString("JOBSRULE_LABEL_ENABLED_LABEL")); } else { - v.addElement(new JLabel(obj.getValue(RULE_NAME), + v.addElement(new JLabel(obj.get(RULE_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_RULE_DISABLE), JLabel.LEFT)); v.addElement(mResource.getString("JOBSRULE_LABEL_DISABLED_LABEL")); } addRow(v, data); - mRules.addElement(obj.getValue(RULE_NAME)); + mRules.addElement(obj.get(RULE_NAME)); } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/JobsSettingPanel.java b/pki/base/console/src/com/netscape/admin/certsrv/config/JobsSettingPanel.java index b33613014..b6a93d191 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/JobsSettingPanel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/JobsSettingPanel.java @@ -19,12 +19,10 @@ package com.netscape.admin.certsrv.config; import com.netscape.admin.certsrv.*; import com.netscape.certsrv.common.*; -import com.netscape.management.client.util.*; import com.netscape.admin.certsrv.connection.*; -import com.netscape.admin.certsrv.config.*; + import javax.swing.*; import java.awt.*; -import java.util.*; import java.awt.event.*; /** @@ -110,8 +108,8 @@ public class JobsSettingPanel extends CMSBaseTab implements ItemListener { public void refresh() { mModel.progressStart(); NameValuePairs nvps = new NameValuePairs(); - nvps.add(Constants.PR_ENABLE,""); - nvps.add(Constants.PR_JOBS_FREQUENCY, ""); + nvps.put(Constants.PR_ENABLE, ""); + nvps.put(Constants.PR_JOBS_FREQUENCY, ""); try { NameValuePairs val = mAdmin.read(mServletName, @@ -131,13 +129,12 @@ public class JobsSettingPanel extends CMSBaseTab implements ItemListener { String clientCert = ""; String version = ""; - for (int i=0; i data = new Hashtable(); + for (String entry : response.keySet()) { + entry = entry.trim(); + String value = response.get(entry); int x = value.indexOf(","); NameValuePairs obj = new NameValuePairs(); - obj.add(IMPL_NAME,entry); + obj.put(IMPL_NAME, entry); vals[i++]= entry ; - obj.add(IMPL_CLASS, value.substring(0,x)); - obj.add(IMPL_DESC, value.substring(x+1)); + obj.put(IMPL_CLASS, value.substring(0, x)); + obj.put(IMPL_DESC, value.substring(x + 1)); data.put(entry,obj); } @@ -301,7 +300,7 @@ public class LogImplTab extends CMSBaseUGTab { try { mConnection.delete(mDestination, ScopeDef.SC_LOG_IMPLS, - obj.getValue(IMPL_NAME)); + obj.get(IMPL_NAME)); } catch (EAdminException e) { //display error dialog showErrorDialog(e.getMessage()); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/LogRuleDataModel.java b/pki/base/console/src/com/netscape/admin/certsrv/config/LogRuleDataModel.java index 57b7b0471..1a64226f0 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/LogRuleDataModel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/LogRuleDataModel.java @@ -20,7 +20,6 @@ package com.netscape.admin.certsrv.config; import java.util.*; import javax.swing.*; import com.netscape.admin.certsrv.*; -import com.netscape.admin.certsrv.connection.*; import com.netscape.certsrv.common.*; import com.netscape.management.client.util.Debug; @@ -52,10 +51,10 @@ public class LogRuleDataModel extends CMSRuleDataModel Vector v = new Vector(); NameValuePairs obj = (NameValuePairs) data; - v.addElement(new JLabel(obj.getValue(RULE_NAME), + v.addElement(new JLabel(obj.get(RULE_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_RULE), JLabel.LEFT)); - v.addElement(obj.getValue(RULE_IMPL)); + v.addElement(obj.get(RULE_IMPL)); addRow(v, data); } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/MNSchemeWizardInfo.java b/pki/base/console/src/com/netscape/admin/certsrv/config/MNSchemeWizardInfo.java index fd2fe0843..8d90914b3 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/MNSchemeWizardInfo.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/MNSchemeWizardInfo.java @@ -17,12 +17,9 @@ // --- END COPYRIGHT BLOCK --- package com.netscape.admin.certsrv.config; -import java.awt.*; -import javax.swing.*; import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import com.netscape.admin.certsrv.wizard.*; -import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; /** @@ -95,16 +92,16 @@ public class MNSchemeWizardInfo extends WizardInfo { public void changeScheme() throws EAdminException { NameValuePairs param = new NameValuePairs(); - param.add(Constants.PR_RECOVERY_M, (String)get(Constants.PR_RECOVERY_M)); - param.add(Constants.PR_RECOVERY_N, (String)get(Constants.PR_RECOVERY_N)); - param.add(Constants.PR_RECOVERY_AGENT, (String)get(Constants.PR_RECOVERY_AGENT)); - param.add(Constants.PR_OLD_RECOVERY_AGENT, (String)get(Constants.PR_OLD_RECOVERY_AGENT)); + param.put(Constants.PR_RECOVERY_M, (String) get(Constants.PR_RECOVERY_M)); + param.put(Constants.PR_RECOVERY_N, (String) get(Constants.PR_RECOVERY_N)); + param.put(Constants.PR_RECOVERY_AGENT, (String) get(Constants.PR_RECOVERY_AGENT)); + param.put(Constants.PR_OLD_RECOVERY_AGENT, (String) get(Constants.PR_OLD_RECOVERY_AGENT)); mConnection.modify(DestDef.DEST_KRA_ADMIN, ScopeDef.SC_MNSCHEME, Constants.RS_ID_CONFIG, param); - //param.removeAllPairs(); + //param.clear(); } } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/MapperImplDataModel.java b/pki/base/console/src/com/netscape/admin/certsrv/config/MapperImplDataModel.java index 50728b13d..23d6ee0b4 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/MapperImplDataModel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/MapperImplDataModel.java @@ -20,7 +20,6 @@ package com.netscape.admin.certsrv.config; import java.util.*; import javax.swing.*; import com.netscape.admin.certsrv.*; -import com.netscape.admin.certsrv.connection.*; import com.netscape.certsrv.common.*; /** @@ -58,10 +57,10 @@ public class MapperImplDataModel extends CMSContentTableModel public void processData(Object data) { Vector v = new Vector(); NameValuePairs obj = (NameValuePairs) data; - v.addElement(new JLabel(obj.getValue(IMPL_NAME), + v.addElement(new JLabel(obj.get(IMPL_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_RULE_PLUGIN), JLabel.LEFT)); - v.addElement(obj.getValue(IMPL_CLASS)); + v.addElement(obj.get(IMPL_CLASS)); //v.addElement(obj.getValue(IMPL_DESC)); addRow(v, data); } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/MapperImplTab.java b/pki/base/console/src/com/netscape/admin/certsrv/config/MapperImplTab.java index 66d4ba4ce..2ad54da60 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/MapperImplTab.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/MapperImplTab.java @@ -21,11 +21,10 @@ import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import com.netscape.admin.certsrv.ug.*; import javax.swing.*; -import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -115,9 +114,9 @@ public class MapperImplTab extends CMSBaseUGTab { mDataModel.getObjectValueAt(mTable.getSelectedRow()); if (mViewer==null) mViewer = new ViewDialog(mModel.getFrame()); - mViewer.showDialog(obj.getValue(IMPL_NAME), - obj.getValue(IMPL_CLASS), - obj.getValue(IMPL_DESC)); + mViewer.showDialog(obj.get(IMPL_NAME), + obj.get(IMPL_CLASS), + obj.get(IMPL_DESC)); } if (e.getSource().equals(mHelp)) { helpCallback(); @@ -269,16 +268,16 @@ public class MapperImplTab extends CMSBaseUGTab { //parse the data int i=0; String[] vals = new String[response.size()]; - Hashtable data = new Hashtable(); - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - String value = response.getValue(entry); + Hashtable data = new Hashtable(); + for (String entry : response.keySet()) { + entry = entry.trim(); + String value = response.get(entry); int x = value.indexOf(","); NameValuePairs obj = new NameValuePairs(); - obj.add(IMPL_NAME,entry); + obj.put(IMPL_NAME, entry); vals[i++]= entry ; - obj.add(IMPL_CLASS, value.substring(0,x)); - obj.add(IMPL_DESC, value.substring(x+1)); + obj.put(IMPL_CLASS, value.substring(0, x)); + obj.put(IMPL_DESC, value.substring(x + 1)); data.put(entry,obj); } @@ -306,7 +305,7 @@ public class MapperImplTab extends CMSBaseUGTab { try { mConnection.delete(mDestination, ScopeDef.SC_MAPPER_IMPLS, - obj.getValue(IMPL_NAME)); + obj.get(IMPL_NAME)); } catch (EAdminException e) { //display error dialog showErrorDialog(e.getMessage()); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/MapperRuleDataModel.java b/pki/base/console/src/com/netscape/admin/certsrv/config/MapperRuleDataModel.java index 1235fcc14..a72b59f7c 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/MapperRuleDataModel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/MapperRuleDataModel.java @@ -20,7 +20,6 @@ package com.netscape.admin.certsrv.config; import java.util.*; import javax.swing.*; import com.netscape.admin.certsrv.*; -import com.netscape.admin.certsrv.connection.*; import com.netscape.certsrv.common.*; import com.netscape.management.client.util.Debug; @@ -52,10 +51,10 @@ public class MapperRuleDataModel extends CMSRuleDataModel Vector v = new Vector(); NameValuePairs obj = (NameValuePairs) data; - v.addElement(new JLabel(obj.getValue(RULE_NAME), + v.addElement(new JLabel(obj.get(RULE_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_RULE), JLabel.LEFT)); - v.addElement(obj.getValue(RULE_IMPL)); + v.addElement(obj.get(RULE_IMPL)); addRow(v, data); } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/OCSPStoresInstanceTab.java b/pki/base/console/src/com/netscape/admin/certsrv/config/OCSPStoresInstanceTab.java index f48339b25..ff234c7fd 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/OCSPStoresInstanceTab.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/OCSPStoresInstanceTab.java @@ -19,13 +19,10 @@ package com.netscape.admin.certsrv.config; import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; -import com.netscape.admin.certsrv.ug.*; + import javax.swing.*; -import javax.swing.event.*; import java.awt.event.*; -import java.awt.*; -import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -118,7 +115,7 @@ public class OCSPStoresInstanceTab extends CMSPluginInstanceTab { try { mConnection.modify(DestDef.DEST_OCSP_ADMIN, ScopeDef.SC_OCSPSTORE_DEFAULT, - data.getValue(RULE_NAME), nvps); + data.get(RULE_NAME), nvps); } catch (EAdminException e) { //display error dialog showErrorDialog(e.getMessage()); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/OCSPStoresRuleDataModel.java b/pki/base/console/src/com/netscape/admin/certsrv/config/OCSPStoresRuleDataModel.java index 30211b689..5cd67e276 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/OCSPStoresRuleDataModel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/OCSPStoresRuleDataModel.java @@ -20,7 +20,6 @@ package com.netscape.admin.certsrv.config; import java.util.*; import javax.swing.*; import com.netscape.admin.certsrv.*; -import com.netscape.admin.certsrv.connection.*; import com.netscape.certsrv.common.*; import com.netscape.management.client.util.Debug; @@ -52,19 +51,19 @@ public class OCSPStoresRuleDataModel extends CMSRuleDataModel Vector v = new Vector(); NameValuePairs obj = (NameValuePairs) data; - if (obj.getValue(RULE_STAT).equalsIgnoreCase("enabled")) { - v.addElement(new JLabel(obj.getValue(RULE_NAME), + if (obj.get(RULE_STAT).equalsIgnoreCase("enabled")) { + v.addElement(new JLabel(obj.get(RULE_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_RULE), JLabel.LEFT)); v.addElement(mResource.getString("OCSPSTORESRULE_LABEL_ENABLED_LABEL")); } else { - v.addElement(new JLabel(obj.getValue(RULE_NAME), + v.addElement(new JLabel(obj.get(RULE_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_RULE_DISABLE), JLabel.LEFT)); v.addElement(mResource.getString("OCSPSTORESRULE_LABEL_DISABLED_LABEL")); } addRow(v, data); - mRules.addElement(obj.getValue(RULE_NAME)); + mRules.addElement(obj.get(RULE_NAME)); } } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/PanelMapperConfigDialog.java b/pki/base/console/src/com/netscape/admin/certsrv/config/PanelMapperConfigDialog.java index f3db6374a..5de06c7f9 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/PanelMapperConfigDialog.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/PanelMapperConfigDialog.java @@ -20,13 +20,12 @@ package com.netscape.admin.certsrv.config; import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import javax.swing.*; -import javax.swing.event.*; import javax.swing.table.*; import javax.swing.text.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -207,8 +206,8 @@ public class PanelMapperConfigDialog extends JDialog Debug.println("MapperList: "+response.toString()); mSelection.removeAllItems(); - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - mSelection.addItem(((String)e.nextElement()).trim()); + for (String name : response.keySet()) { + mSelection.addItem(name.trim()); } mSelection.setSelectedItem(mapperName); @@ -230,11 +229,11 @@ public class PanelMapperConfigDialog extends JDialog mDataModel.removeAllRows(); - for (Enumeration e = mData.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); + for (String entry : mData.keySet()) { + entry = entry.trim(); if (!entry.equals(Constants.PR_MAPPER)) { - String value = mData.getValue(entry); - Vector v = new Vector(); + String value = mData.get(entry); + Vector v = new Vector(); v.addElement(entry); v.addElement(value); mDataModel.addRow(v); @@ -369,15 +368,15 @@ public class PanelMapperConfigDialog extends JDialog //save the configuration settings for the mapper private void saveConfiguration() throws EAdminException { NameValuePairs nvp = getData(); - nvp.add(Constants.PR_MAPPER, (String)mSelection.getSelectedItem()); + nvp.put(Constants.PR_MAPPER, (String) mSelection.getSelectedItem()); mConn.modify(mDest, mScope, Constants.RS_ID_CONFIG, nvp); } private NameValuePairs getData() { NameValuePairs response = new NameValuePairs(); for (int i=0; i< mDataModel.getRowCount(); i++) { - response.add((String)mDataModel.getValueAt(i,0), - (String)mDataModel.getValueAt(i,1)); + response.put((String) mDataModel.getValueAt(i, 0), + (String) mDataModel.getValueAt(i, 1)); } return response; } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/PluginSelectionDialog.java b/pki/base/console/src/com/netscape/admin/certsrv/config/PluginSelectionDialog.java index c88251f0b..437881186 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/PluginSelectionDialog.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/PluginSelectionDialog.java @@ -20,11 +20,10 @@ package com.netscape.admin.certsrv.config; import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import javax.swing.*; -import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -163,7 +162,7 @@ public class PluginSelectionDialog extends JDialog } Debug.println(response.toString()); String id = ((JLabel)mDataModel.elementAt(mList.getSelectedIndex())).getText(); - response.add(Constants.PR_POLICY_IMPL_NAME,id); + response.put(Constants.PR_POLICY_IMPL_NAME, id); CMSBaseConfigDialog dialog = null; if (mExtraDestination == null) { @@ -338,9 +337,9 @@ public class PluginSelectionDialog extends JDialog //parse the data String[] vals = new String[response.size()]; int i=0; - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - vals[i++] = ((String)e.nextElement()).trim(); - Debug.println("PluginSelectionDialog::update() - adding '"+vals[i-1]+"'"); + for (String name : response.keySet()) { + vals[i++] = name.trim(); + Debug.println("PluginSelectionDialog::update() - adding '" + vals[i - 1] + "'"); } CMSAdminUtil.bubbleSort(vals); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/PolicyImplDataModel.java b/pki/base/console/src/com/netscape/admin/certsrv/config/PolicyImplDataModel.java index 8ac219345..59ea36ec8 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/PolicyImplDataModel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/PolicyImplDataModel.java @@ -20,7 +20,6 @@ package com.netscape.admin.certsrv.config; import java.util.*; import javax.swing.*; import com.netscape.admin.certsrv.*; -import com.netscape.admin.certsrv.connection.*; import com.netscape.certsrv.common.*; /** @@ -58,10 +57,10 @@ public class PolicyImplDataModel extends CMSContentTableModel public void processData(Object data) { Vector v = new Vector(); NameValuePairs obj = (NameValuePairs) data; - v.addElement(new JLabel(obj.getValue(IMPL_NAME), + v.addElement(new JLabel(obj.get(IMPL_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_RULE_PLUGIN), JLabel.LEFT)); - v.addElement(obj.getValue(IMPL_CLASS)); + v.addElement(obj.get(IMPL_CLASS)); //v.addElement(obj.getValue(IMPL_DESC)); addRow(v, data); } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/PolicyImplTab.java b/pki/base/console/src/com/netscape/admin/certsrv/config/PolicyImplTab.java index 8e89dcf45..f4fab7260 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/PolicyImplTab.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/PolicyImplTab.java @@ -21,11 +21,10 @@ import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import com.netscape.admin.certsrv.ug.*; import javax.swing.*; -import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -117,9 +116,9 @@ public class PolicyImplTab extends CMSBaseUGTab { mDataModel.getObjectValueAt(mTable.getSelectedRow()); if (mViewer==null) mViewer = new ViewDialog(mModel.getFrame()); - mViewer.showDialog(obj.getValue(IMPL_NAME), - obj.getValue(IMPL_CLASS), - obj.getValue(IMPL_DESC)); + mViewer.showDialog(obj.get(IMPL_NAME), + obj.get(IMPL_CLASS), + obj.get(IMPL_DESC)); } if (e.getSource().equals(mHelp)) { helpCallback(); @@ -271,16 +270,16 @@ public class PolicyImplTab extends CMSBaseUGTab { //parse the data int i=0; String[] vals = new String[response.size()]; - Hashtable data = new Hashtable(); - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - String value = response.getValue(entry); + Hashtable data = new Hashtable(); + for (String entry : response.keySet()) { + entry = entry.trim(); + String value = response.get(entry); int x = value.indexOf(","); NameValuePairs obj = new NameValuePairs(); - obj.add(IMPL_NAME,entry); + obj.put(IMPL_NAME, entry); vals[i++]= entry ; - obj.add(IMPL_CLASS, value.substring(0,x)); - obj.add(IMPL_DESC, value.substring(x+1)); + obj.put(IMPL_CLASS, value.substring(0, x)); + obj.put(IMPL_DESC, value.substring(x + 1)); data.put(entry,obj); } @@ -308,7 +307,7 @@ public class PolicyImplTab extends CMSBaseUGTab { try { mConnection.delete(mDestination, ScopeDef.SC_POLICY_IMPLS, - obj.getValue(IMPL_NAME)); + obj.get(IMPL_NAME)); } catch (EAdminException e) { //display error dialog showErrorDialog(e.getMessage()); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/PolicyRuleDataModel.java b/pki/base/console/src/com/netscape/admin/certsrv/config/PolicyRuleDataModel.java index e379b53e9..9bbeda334 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/PolicyRuleDataModel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/PolicyRuleDataModel.java @@ -20,7 +20,6 @@ package com.netscape.admin.certsrv.config; import java.util.*; import javax.swing.*; import com.netscape.admin.certsrv.*; -import com.netscape.admin.certsrv.connection.*; import com.netscape.certsrv.common.*; import com.netscape.management.client.util.Debug; @@ -53,19 +52,19 @@ public class PolicyRuleDataModel extends CMSRuleDataModel NameValuePairs obj = (NameValuePairs) data; //XXX NEED TO ADD STUFF - if (obj.getValue(RULE_STAT).equalsIgnoreCase("enabled")) { - v.addElement(new JLabel(obj.getValue(RULE_NAME), + if (obj.get(RULE_STAT).equalsIgnoreCase("enabled")) { + v.addElement(new JLabel(obj.get(RULE_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_RULE), JLabel.LEFT)); v.addElement(mResource.getString("POLICYRULE_LABEL_ENABLED_LABEL")); } else { - v.addElement(new JLabel(obj.getValue(RULE_NAME), + v.addElement(new JLabel(obj.get(RULE_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_RULE_DISABLE), JLabel.LEFT)); v.addElement(mResource.getString("POLICYRULE_LABEL_DISABLED_LABEL")); } addRow(v, data); - mRules.addElement(obj.getValue(RULE_NAME)); + mRules.addElement(obj.get(RULE_NAME)); } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/PolicyRuleOrderDialog.java b/pki/base/console/src/com/netscape/admin/certsrv/config/PolicyRuleOrderDialog.java index ec7394d4e..1c99d999f 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/PolicyRuleOrderDialog.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/PolicyRuleOrderDialog.java @@ -20,11 +20,10 @@ package com.netscape.admin.certsrv.config; import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import javax.swing.*; -import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -285,7 +284,7 @@ public class PolicyRuleOrderDialog extends JDialog } NameValuePairs nvp = new NameValuePairs(); - nvp.add(Constants.PR_POLICY_ORDER, buf.toString()); + nvp.put(Constants.PR_POLICY_ORDER, buf.toString()); Debug.println("ORDER: "+buf.toString()); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileConfigDialog.java b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileConfigDialog.java index f83a7457c..a8d312d2a 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileConfigDialog.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileConfigDialog.java @@ -20,13 +20,10 @@ package com.netscape.admin.certsrv.config; import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import javax.swing.*; -import javax.swing.event.*; -import javax.swing.table.*; -import javax.swing.text.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -367,11 +364,11 @@ public class ProfileConfigDialog extends CMSBaseConfigDialog if (mModel != null) { mModel.progressStart(); } String instanceName = mPluginName.getText(); - nvp.add("impl", mImplName.getText()); - nvp.add("name", mNameField.getText()); - nvp.add("visible", (String)(mVisibleField.getSelectedItem())); - nvp.add("auth", mAuthField.getText()); - nvp.add("desc", mDescField.getText()); + nvp.put("impl", mImplName.getText()); + nvp.put("name", mNameField.getText()); + nvp.put("visible", (String) (mVisibleField.getSelectedItem())); + nvp.put("auth", mAuthField.getText()); + nvp.put("desc", mDescField.getText()); // nvp.add("config", mConfigField.getText()); // mAdminConnection.add(DestDef.DEST_CA_PROFILE_ADMIN, mAdminConnection.add(mDest, diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileEditDialog.java b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileEditDialog.java index 0a3ceebe2..d10e19215 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileEditDialog.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileEditDialog.java @@ -21,12 +21,10 @@ import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import javax.swing.*; import javax.swing.event.*; -import javax.swing.table.*; -import javax.swing.text.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -684,11 +682,11 @@ public class ProfileEditDialog extends CMSBaseConfigDialog mModel.progressStart(); String instanceName = mPluginName.getText(); - nvp.add("impl", mImplName.getText()); - nvp.add("name", mNameField.getText()); - nvp.add("desc", mDescField.getText()); - nvp.add("visible", (String)(mVisibleField.getSelectedItem())); - nvp.add("auth", mAuthField.getText()); + nvp.put("impl", mImplName.getText()); + nvp.put("name", mNameField.getText()); + nvp.put("desc", mDescField.getText()); + nvp.put("visible", (String) (mVisibleField.getSelectedItem())); + nvp.put("auth", mAuthField.getText()); // nvp.add("config", mConfigField.getText()); /* @@ -719,7 +717,7 @@ public class ProfileEditDialog extends CMSBaseConfigDialog private void deletePolicy(String profileId, String policyId) throws EAdminException{ NameValuePairs nvps = new NameValuePairs(); - nvps.add("POLICYID", policyId); + nvps.put("POLICYID", policyId); //mAdminConnection.delete(DestDef.DEST_CA_PROFILE_ADMIN, mAdminConnection.delete(mDest, ScopeDef.SC_PROFILE_POLICIES, profileId, nvps); @@ -728,7 +726,7 @@ public class ProfileEditDialog extends CMSBaseConfigDialog private void deleteInput(String profileId, String inputId) throws EAdminException{ NameValuePairs nvps = new NameValuePairs(); - nvps.add("INPUTID", inputId); + nvps.put("INPUTID", inputId); //mAdminConnection.delete(DestDef.DEST_CA_PROFILE_ADMIN, mAdminConnection.delete(mDest, ScopeDef.SC_PROFILE_INPUT, profileId, nvps); @@ -737,7 +735,7 @@ public class ProfileEditDialog extends CMSBaseConfigDialog private void deleteOutput(String profileId, String outputId) throws EAdminException{ NameValuePairs nvps = new NameValuePairs(); - nvps.add("OUTPUTID", outputId); + nvps.put("OUTPUTID", outputId); //mAdminConnection.delete(DestDef.DEST_CA_PROFILE_ADMIN, mAdminConnection.delete(mDest, ScopeDef.SC_PROFILE_OUTPUT, profileId, nvps); @@ -769,17 +767,17 @@ public class ProfileEditDialog extends CMSBaseConfigDialog } mModel.progressStop(); - String enable = response.getValue("enable"); + String enable = response.get("enable"); if (response != null) { mPluginName.setEnabled(false); mPluginName.setBackground(getBackground()); mPluginName.setText(name); - mNameField.setText(response.getValue("name")); - mDescField.setText(response.getValue("desc")); - mAuthField.setText(response.getValue("auth")); - mVisibleField.setSelectedItem(response.getValue("visible")); - mImplName.setText(response.getValue("plugin")); + mNameField.setText(response.get("name")); + mDescField.setText(response.get("desc")); + mAuthField.setText(response.get("auth")); + mVisibleField.setSelectedItem(response.get("visible")); + mImplName.setText(response.get("plugin")); // mConfigField.setText(response.getValue("config")); } if (enable != null && enable.equals("true")) { @@ -870,29 +868,27 @@ public class ProfileEditDialog extends CMSBaseConfigDialog } private void populatePolicies(NameValuePairs response, JTable table) { - Vector colNames = new Vector(); + Vector colNames = new Vector(); colNames.addElement("Set Id"); colNames.addElement("Id"); colNames.addElement("Defaults"); colNames.addElement("Constraints"); - Vector d = new Vector(); + Vector> d = new Vector>(); - Enumeration e = response.getNames(); - if (e != null) { - for (; e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - String value = response.getValue(entry); + for (String entry : response.keySet()) { + entry = entry.trim(); + String value = response.get(entry); Debug.println("populatePolicies entry= "+entry); Debug.println("populatePolicies value= "+value); StringTokenizer st = new StringTokenizer(value, ";"); - String def = (String)st.nextToken(); - String con = (String)st.nextToken(); - Vector row = new Vector(); + String def = st.nextToken(); + String con = st.nextToken(); + Vector row = new Vector(); StringTokenizer st1 = new StringTokenizer(entry, ":"); - String setId = (String)st1.nextToken(); - String id = (String)st1.nextToken(); + String setId = st1.nextToken(); + String id = st1.nextToken(); if (mDefSetId == null) { mDefSetId = setId; @@ -902,7 +898,6 @@ public class ProfileEditDialog extends CMSBaseConfigDialog row.addElement(def); row.addElement(con); d.addElement(row); - } } ProfileEditDataModel model = new ProfileEditDataModel(); model.setInfo(d, colNames); @@ -910,21 +905,21 @@ public class ProfileEditDialog extends CMSBaseConfigDialog } private void populateNonPolicy(NameValuePairs response, JTable table) { - Vector colNames = new Vector(); + Vector colNames = new Vector(); colNames.addElement("Id"); if (table == mInputTable) colNames.addElement("Inputs"); else if (table == mOutputTable) colNames.addElement("Outputs"); - Vector d = new Vector(); + Vector> d = new Vector>(); - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - String value = response.getValue(entry); - Debug.println("populateNonPolicy entry= "+entry); - Debug.println("populateNonPolicy value= "+value); + for (String entry : response.keySet()) { + entry = entry.trim(); + String value = response.get(entry); + Debug.println("populateNonPolicy entry= " + entry); + Debug.println("populateNonPolicy value= " + value); - Vector row = new Vector(); + Vector row = new Vector(); row.addElement(entry); row.addElement(value); d.addElement(row); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileImplDataModel.java b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileImplDataModel.java index 98c188380..eee2531cf 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileImplDataModel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileImplDataModel.java @@ -20,7 +20,6 @@ package com.netscape.admin.certsrv.config; import java.util.*; import javax.swing.*; import com.netscape.admin.certsrv.*; -import com.netscape.admin.certsrv.connection.*; import com.netscape.certsrv.common.*; /** @@ -59,11 +58,11 @@ public class ProfileImplDataModel extends CMSContentTableModel public void processData(Object data) { Vector v = new Vector(); NameValuePairs obj = (NameValuePairs) data; - v.addElement(new JLabel(obj.getValue(IMPL_NAME), + v.addElement(new JLabel(obj.get(IMPL_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_RULE_PLUGIN), JLabel.LEFT)); - v.addElement(obj.getValue(IMPL_TYPE)); - v.addElement(obj.getValue(IMPL_CLASS)); + v.addElement(obj.get(IMPL_TYPE)); + v.addElement(obj.get(IMPL_CLASS)); //v.addElement(obj.getValue(IMPL_DESC)); addRow(v, data); } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileImplTab.java b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileImplTab.java index e9047701c..c7d28dd07 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileImplTab.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileImplTab.java @@ -21,11 +21,10 @@ import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import com.netscape.admin.certsrv.ug.*; import javax.swing.*; -import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -111,9 +110,9 @@ public class ProfileImplTab extends CMSBaseUGTab { mDataModel.getObjectValueAt(mTable.getSelectedRow()); if (mViewer==null) mViewer = new ViewDialog(mModel.getFrame()); - mViewer.showDialog(obj.getValue(IMPL_NAME), - obj.getValue(IMPL_CLASS), - obj.getValue(IMPL_DESC)); + mViewer.showDialog(obj.get(IMPL_NAME), + obj.get(IMPL_CLASS), + obj.get(IMPL_DESC)); } if (e.getSource().equals(mHelp)) { helpCallback(); @@ -264,9 +263,9 @@ public class ProfileImplTab extends CMSBaseUGTab { mModel.progressStop(); return; } - for (Enumeration e = response1.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - response.add(entry+";profile", response1.getValue(entry)); + for (String entry : response1.keySet()) { + entry = entry.trim(); + response.put(entry + ";profile", response1.get(entry)); } try { response2 = mConnection.search(mDestination, @@ -278,9 +277,9 @@ public class ProfileImplTab extends CMSBaseUGTab { mModel.progressStop(); return; } - for (Enumeration e = response2.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - response.add(entry+";profileInput", response2.getValue(entry)); + for (String entry : response2.keySet()) { + entry = entry.trim(); + response.put(entry + ";profileInput", response2.get(entry)); } try { response3 = mConnection.search(mDestination, @@ -292,9 +291,9 @@ public class ProfileImplTab extends CMSBaseUGTab { mModel.progressStop(); return; } - for (Enumeration e = response3.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - response.add(entry+";profileOutput", response3.getValue(entry)); + for (String entry : response3.keySet()) { + entry = entry.trim(); + response.put(entry + ";profileOutput", response3.get(entry)); } try { response4 = mConnection.search(mDestination, @@ -306,9 +305,9 @@ public class ProfileImplTab extends CMSBaseUGTab { mModel.progressStop(); return; } - for (Enumeration e = response4.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - response.add(entry+";defaultPolicy", response4.getValue(entry)); + for (String entry : response4.keySet()) { + entry = entry.trim(); + response.put(entry + ";defaultPolicy", response4.get(entry)); } try { response5 = mConnection.search(mDestination, @@ -320,27 +319,27 @@ public class ProfileImplTab extends CMSBaseUGTab { mModel.progressStop(); return; } - for (Enumeration e = response5.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - response.add(entry+";constraintPolicy", response5.getValue(entry)); + for (String entry : response5.keySet()) { + entry = entry.trim(); + response.put(entry + ";constraintPolicy", response5.get(entry)); } //parse the data int i=0; String[] vals = new String[response.size()]; Hashtable data = new Hashtable(); - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - String value = response.getValue(entry); + for (String entry : response.keySet()) { + entry = entry.trim(); + String value = response.get(entry); int x = value.indexOf(","); int x_end = value.lastIndexOf(","); int y = entry.indexOf(";"); NameValuePairs obj = new NameValuePairs(); - obj.add(IMPL_NAME,entry.substring(0,y)); - obj.add(IMPL_TYPE,entry.substring(y+1)); + obj.put(IMPL_NAME, entry.substring(0, y)); + obj.put(IMPL_TYPE, entry.substring(y + 1)); vals[i++]= entry ; - obj.add(IMPL_CLASS, value.substring(0,x)); - obj.add(IMPL_DESC, value.substring(x+1,x_end)); + obj.put(IMPL_CLASS, value.substring(0, x)); + obj.put(IMPL_DESC, value.substring(x + 1, x_end)); data.put(entry,obj); } @@ -367,8 +366,8 @@ public class ProfileImplTab extends CMSBaseUGTab { //send comment to server for the removal of user try { mConnection.delete(mDestination, - obj.getValue(IMPL_TYPE), - obj.getValue(IMPL_NAME)); + obj.get(IMPL_TYPE), + obj.get(IMPL_NAME)); } catch (EAdminException e) { //display error dialog showErrorDialog(e.getMessage()); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileNonPolicyNewDialog.java b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileNonPolicyNewDialog.java index 05836bc87..e597b242b 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileNonPolicyNewDialog.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileNonPolicyNewDialog.java @@ -20,13 +20,10 @@ package com.netscape.admin.certsrv.config; import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import javax.swing.*; -import javax.swing.event.*; -import javax.swing.table.*; -import javax.swing.text.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -309,8 +306,8 @@ setLabelCellEditor(mTable, 1); } for (int i = 0; i < mTable.getRowCount(); i++) { - nvp.add((String)mTable.getValueAt(i,0), - (String)mTable.getValueAt(i,1)); + nvp.put((String) mTable.getValueAt(i, 0), + (String) mTable.getValueAt(i, 1)); } if (mIsNew) { @@ -384,11 +381,11 @@ setLabelCellEditor(mTable, 1); defcolNames.addElement("Value"); Vector defdata = new Vector(); - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - String value = response.getValue(entry); - Debug.println("entry= "+entry); - Debug.println("value= "+value); + for (String entry : response.keySet()) { + entry = entry.trim(); + String value = response.get(entry); + Debug.println("entry= " + entry); + Debug.println("value= " + value); int start_pos = value.indexOf(';'); int end_pos = value.lastIndexOf(';'); @@ -413,9 +410,9 @@ setLabelCellEditor(mTable, 1); mPluginName.setText(profileId); mPluginName.setBackground(getBackground()); mPluginName.setEnabled(false); - mNameField.setText(response.getValue("name")); + mNameField.setText(response.get("name")); if (mIsNew) - mDescField.setText(response.getValue("desc")); + mDescField.setText(response.get("desc")); } this.show(); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileNonPolicySelDialog.java b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileNonPolicySelDialog.java index 4487754f2..8d97ffd13 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileNonPolicySelDialog.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileNonPolicySelDialog.java @@ -20,11 +20,10 @@ package com.netscape.admin.certsrv.config; import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import javax.swing.*; -import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -348,9 +347,9 @@ public class ProfileNonPolicySelDialog extends JDialog //parse the data String[] vals = new String[response.size()]; int i=0; - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String name = ((String)e.nextElement()).trim(); - String val = response.getValue(name); + for (String name : response.keySet()) { + name = name.trim(); + String val = response.get(name); StringTokenizer st = new StringTokenizer(val, ","); String className = st.nextToken(); String desc = st.nextToken(); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfilePluginSelectionDialog.java b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfilePluginSelectionDialog.java index c3b033221..ec3bb4c7c 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfilePluginSelectionDialog.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfilePluginSelectionDialog.java @@ -20,11 +20,8 @@ package com.netscape.admin.certsrv.config; import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import javax.swing.*; -import javax.swing.event.*; import java.awt.event.*; -import java.awt.*; -import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -89,7 +86,7 @@ public class ProfilePluginSelectionDialog extends PluginSelectionDialog } Debug.println(response.toString()); String id =(String)(((ProfileListDataModel)mDataModel).getObjectValueAt(mList.getSelectedIndex())); - response.add(Constants.PR_POLICY_IMPL_NAME,id); + response.put(Constants.PR_POLICY_IMPL_NAME, id); CMSBaseConfigDialog dialog = null; if (mExtraDestination == null) { @@ -160,9 +157,8 @@ public class ProfilePluginSelectionDialog extends PluginSelectionDialog String[] classnames = new String[response.size()]; String[] ids = new String[response.size()]; int i=0; - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String id = (String)e.nextElement(); - String value = response.getValue(id); + for (String id : response.keySet()) { + String value = response.get(id); int pos = value.lastIndexOf(","); String className = value.substring(pos+1); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfilePolicyEditDialog.java b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfilePolicyEditDialog.java index 12781c6b5..34b95c0f1 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfilePolicyEditDialog.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfilePolicyEditDialog.java @@ -20,13 +20,11 @@ package com.netscape.admin.certsrv.config; import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import javax.swing.*; -import javax.swing.event.*; import javax.swing.table.*; -import javax.swing.text.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -462,7 +460,7 @@ public class ProfilePolicyEditDialog extends CMSBaseConfigDialog val = (String)(((JComboBox)comp).getSelectedItem()); } String name1 = ((JLabel)(mDefaultTable.getValueAt(i,0))).getText(); - nvp.add(name1, val); + nvp.put(name1, val); } @@ -485,7 +483,7 @@ public class ProfilePolicyEditDialog extends CMSBaseConfigDialog } String name1 = ((JLabel)(mConstraintTable.getValueAt(i,0))).getText(); - nvp.add(name1, val); + nvp.put(name1, val); } instanceName = mPluginName.getText(); //DestDef.DEST_CA_PROFILE_ADMIN, @@ -582,9 +580,9 @@ public class ProfilePolicyEditDialog extends CMSBaseConfigDialog defcolNames.addElement("Value"); Vector defdata = new Vector(); - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - String value = response.getValue(entry); + for (String entry : response.keySet()) { + entry = entry.trim(); + String value = response.get(entry); Debug.println("ProfilePolicyEditDialog entry= "+entry); Debug.println("ProfilePolicyEditDialog value= "+value); @@ -605,7 +603,7 @@ public class ProfilePolicyEditDialog extends CMSBaseConfigDialog mPluginName.setText(profileId); mPluginName.setBackground(getBackground()); mPluginName.setEnabled(false); - mNameField.setText(response.getValue("name")); + mNameField.setText(response.get("name")); mDescField.setText(setid); mDescField.setBackground(getBackground()); mDescField.setEnabled(false); @@ -633,9 +631,9 @@ public class ProfilePolicyEditDialog extends CMSBaseConfigDialog colNames.addElement("Value"); Vector d = new Vector(); - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - String value = response.getValue(entry); + for (String entry : response.keySet()) { + entry = entry.trim(); + String value = response.get(entry); Debug.println("entry= "+entry); Debug.println("value= "+value); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfilePolicyNewDialog.java b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfilePolicyNewDialog.java index 44e1aa512..941917808 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfilePolicyNewDialog.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfilePolicyNewDialog.java @@ -20,13 +20,11 @@ package com.netscape.admin.certsrv.config; import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import javax.swing.*; -import javax.swing.event.*; import javax.swing.table.*; -import javax.swing.text.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -483,7 +481,7 @@ public class ProfilePolicyNewDialog extends CMSBaseConfigDialog val = (String)(((JComboBox)comp).getSelectedItem()); } String name1 = ((JLabel)(mDefaultTable.getValueAt(i,0))).getText(); - nvp.add(name1, val); + nvp.put(name1, val); } @@ -505,7 +503,7 @@ public class ProfilePolicyNewDialog extends CMSBaseConfigDialog } String name1 = ((JLabel)(mConstraintTable.getValueAt(i,0))).getText(); - nvp.add(name1, val); + nvp.put(name1, val); } instanceName = mPluginName.getText(); //mAdminConnection.add(DestDef.DEST_CA_PROFILE_ADMIN, @@ -559,9 +557,9 @@ public class ProfilePolicyNewDialog extends CMSBaseConfigDialog defcolNames.addElement("Value"); Vector defdata = new Vector(); - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - String value = response.getValue(entry); + for (String entry : response.keySet()) { + entry = entry.trim(); + String value = response.get(entry); Debug.println("entry= "+entry); Debug.println("value= "+value); @@ -581,8 +579,8 @@ public class ProfilePolicyNewDialog extends CMSBaseConfigDialog mPluginName.setText(profileId); mPluginName.setBackground(getBackground()); mPluginName.setEnabled(false); - mNameField.setText(response.getValue("name")); - mDescField.setText(response.getValue("desc")); + mNameField.setText(response.get("name")); + mDescField.setText(response.get("desc")); } // retrieve policy information @@ -604,9 +602,9 @@ public class ProfilePolicyNewDialog extends CMSBaseConfigDialog colNames.addElement("Value"); Vector d = new Vector(); - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - String value = response.getValue(entry); + for (String entry : response.keySet()) { + entry = entry.trim(); + String value = response.get(entry); Debug.println("entry= "+entry); Debug.println("value= "+value); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfilePolicySelectionDialog.java b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfilePolicySelectionDialog.java index 69d502355..44fe9886b 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfilePolicySelectionDialog.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfilePolicySelectionDialog.java @@ -24,7 +24,7 @@ import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -460,9 +460,9 @@ public class ProfilePolicySelectionDialog extends JDialog //parse the data String[] vals = new String[response.size()]; int i=0; - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String name = ((String)e.nextElement()).trim(); - String val = response.getValue(name); + for (String name : response.keySet()) { + name = name.trim(); + String val = response.get(name); StringTokenizer st = new StringTokenizer(val, ","); String className = st.nextToken(); String desc = st.nextToken(); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileRegisterDialog.java b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileRegisterDialog.java index fb9ed20e7..c576adbe6 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileRegisterDialog.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileRegisterDialog.java @@ -20,14 +20,12 @@ package com.netscape.admin.certsrv.config; import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import javax.swing.*; -import javax.swing.*; import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; -import javax.swing.table.*; import com.netscape.certsrv.common.*; @@ -287,8 +285,8 @@ public class ProfileRegisterDialog extends JDialog //construct NVP NameValuePairs config = new NameValuePairs(); - config.add(Constants.PR_POLICY_CLASS, mClassField.getText()); - config.add(Constants.PR_POLICY_DESC, mDescField.getText()); + config.put(Constants.PR_POLICY_CLASS, mClassField.getText()); + config.put(Constants.PR_POLICY_DESC, mDescField.getText()); if (mType) { mScope=mTypeField.getText(); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileRuleDataModel.java b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileRuleDataModel.java index 73bf5ca91..7a3ba2827 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileRuleDataModel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/ProfileRuleDataModel.java @@ -20,9 +20,7 @@ package com.netscape.admin.certsrv.config; import java.util.*; import javax.swing.*; import com.netscape.admin.certsrv.*; -import com.netscape.admin.certsrv.connection.*; import com.netscape.certsrv.common.*; -import com.netscape.management.client.util.Debug; /** @@ -52,19 +50,19 @@ public class ProfileRuleDataModel extends CMSRuleDataModel NameValuePairs obj = (NameValuePairs) data; //XXX NEED TO ADD STUFF - if (obj.getValue(RULE_STAT).equalsIgnoreCase("enabled")) { - v.addElement(new JLabel(obj.getValue(RULE_NAME), + if (obj.get(RULE_STAT).equalsIgnoreCase("enabled")) { + v.addElement(new JLabel(obj.get(RULE_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_RULE), JLabel.LEFT)); v.addElement(mResource.getString("POLICYRULE_LABEL_ENABLED_LABEL")); } else { - v.addElement(new JLabel(obj.getValue(RULE_NAME), + v.addElement(new JLabel(obj.get(RULE_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_RULE_DISABLE), JLabel.LEFT)); v.addElement(mResource.getString("POLICYRULE_LABEL_DISABLED_LABEL")); } addRow(v, data); - mRules.addElement(obj.getValue(RULE_NAME)); + mRules.addElement(obj.get(RULE_NAME)); } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/PublisherImplDataModel.java b/pki/base/console/src/com/netscape/admin/certsrv/config/PublisherImplDataModel.java index 18ad05ba2..aefa4b1cc 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/PublisherImplDataModel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/PublisherImplDataModel.java @@ -20,7 +20,6 @@ package com.netscape.admin.certsrv.config; import java.util.*; import javax.swing.*; import com.netscape.admin.certsrv.*; -import com.netscape.admin.certsrv.connection.*; import com.netscape.certsrv.common.*; /** @@ -58,10 +57,10 @@ public class PublisherImplDataModel extends CMSContentTableModel public void processData(Object data) { Vector v = new Vector(); NameValuePairs obj = (NameValuePairs) data; - v.addElement(new JLabel(obj.getValue(IMPL_NAME), + v.addElement(new JLabel(obj.get(IMPL_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_RULE_PLUGIN), JLabel.LEFT)); - v.addElement(obj.getValue(IMPL_CLASS)); + v.addElement(obj.get(IMPL_CLASS)); //v.addElement(obj.getValue(IMPL_DESC)); addRow(v, data); } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/PublisherImplTab.java b/pki/base/console/src/com/netscape/admin/certsrv/config/PublisherImplTab.java index ebe4db5db..79fedfe5e 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/PublisherImplTab.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/PublisherImplTab.java @@ -21,11 +21,10 @@ import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import com.netscape.admin.certsrv.ug.*; import javax.swing.*; -import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -115,9 +114,9 @@ public class PublisherImplTab extends CMSBaseUGTab { mDataModel.getObjectValueAt(mTable.getSelectedRow()); if (mViewer==null) mViewer = new ViewDialog(mModel.getFrame()); - mViewer.showDialog(obj.getValue(IMPL_NAME), - obj.getValue(IMPL_CLASS), - obj.getValue(IMPL_DESC)); + mViewer.showDialog(obj.get(IMPL_NAME), + obj.get(IMPL_CLASS), + obj.get(IMPL_DESC)); } if (e.getSource().equals(mHelp)) { helpCallback(); @@ -271,15 +270,15 @@ public class PublisherImplTab extends CMSBaseUGTab { int i=0; String[] vals = new String[response.size()]; Hashtable data = new Hashtable(); - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - String value = response.getValue(entry); + for (String entry : response.keySet()) { + entry = entry.trim(); + String value = response.get(entry); int x = value.indexOf(","); NameValuePairs obj = new NameValuePairs(); - obj.add(IMPL_NAME,entry); + obj.put(IMPL_NAME, entry); vals[i++]= entry ; - obj.add(IMPL_CLASS, value.substring(0,x)); - obj.add(IMPL_DESC, value.substring(x+1)); + obj.put(IMPL_CLASS, value.substring(0, x)); + obj.put(IMPL_DESC, value.substring(x + 1)); data.put(entry,obj); } @@ -307,7 +306,7 @@ public class PublisherImplTab extends CMSBaseUGTab { try { mConnection.delete(mDestination, ScopeDef.SC_PUBLISHER_IMPLS, - obj.getValue(IMPL_NAME)); + obj.get(IMPL_NAME)); } catch (EAdminException e) { //display error dialog showErrorDialog(e.getMessage()); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/PublisherRuleDataModel.java b/pki/base/console/src/com/netscape/admin/certsrv/config/PublisherRuleDataModel.java index e8ff44e44..03faeac21 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/PublisherRuleDataModel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/PublisherRuleDataModel.java @@ -20,7 +20,6 @@ package com.netscape.admin.certsrv.config; import java.util.*; import javax.swing.*; import com.netscape.admin.certsrv.*; -import com.netscape.admin.certsrv.connection.*; import com.netscape.certsrv.common.*; import com.netscape.management.client.util.Debug; @@ -52,10 +51,10 @@ public class PublisherRuleDataModel extends CMSRuleDataModel Vector v = new Vector(); NameValuePairs obj = (NameValuePairs) data; - v.addElement(new JLabel(obj.getValue(RULE_NAME), + v.addElement(new JLabel(obj.get(RULE_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_RULE), JLabel.LEFT)); - v.addElement(obj.getValue(RULE_IMPL)); + v.addElement(obj.get(RULE_IMPL)); addRow(v, data); } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/RegisterDialog.java b/pki/base/console/src/com/netscape/admin/certsrv/config/RegisterDialog.java index e1ef3631a..fe48419c7 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/RegisterDialog.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/RegisterDialog.java @@ -24,9 +24,8 @@ import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; -import javax.swing.table.*; import com.netscape.certsrv.common.*; /** @@ -270,7 +269,7 @@ public class RegisterDialog extends JDialog //construct NVP NameValuePairs config = new NameValuePairs(); - config.add(Constants.PR_POLICY_CLASS, mClassField.getText()); + config.put(Constants.PR_POLICY_CLASS, mClassField.getText()); if (mType) { mScope=mTypeField.getText(); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/RuleImplDataModel.java b/pki/base/console/src/com/netscape/admin/certsrv/config/RuleImplDataModel.java index 26e065c04..86a4fa6d4 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/RuleImplDataModel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/RuleImplDataModel.java @@ -20,7 +20,6 @@ package com.netscape.admin.certsrv.config; import java.util.*; import javax.swing.*; import com.netscape.admin.certsrv.*; -import com.netscape.admin.certsrv.connection.*; import com.netscape.certsrv.common.*; /** @@ -58,10 +57,10 @@ public class RuleImplDataModel extends CMSContentTableModel public void processData(Object data) { Vector v = new Vector(); NameValuePairs obj = (NameValuePairs) data; - v.addElement(new JLabel(obj.getValue(IMPL_NAME), + v.addElement(new JLabel(obj.get(IMPL_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_RULE_PLUGIN), JLabel.LEFT)); - v.addElement(obj.getValue(IMPL_CLASS)); + v.addElement(obj.get(IMPL_CLASS)); //v.addElement(obj.getValue(IMPL_DESC)); addRow(v, data); } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/RuleImplTab.java b/pki/base/console/src/com/netscape/admin/certsrv/config/RuleImplTab.java index b0c262dea..098cbcbd1 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/RuleImplTab.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/RuleImplTab.java @@ -21,11 +21,10 @@ import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import com.netscape.admin.certsrv.ug.*; import javax.swing.*; -import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -115,9 +114,9 @@ public class RuleImplTab extends CMSBaseUGTab { mDataModel.getObjectValueAt(mTable.getSelectedRow()); if (mViewer==null) mViewer = new ViewDialog(mModel.getFrame()); - mViewer.showDialog(obj.getValue(IMPL_NAME), - obj.getValue(IMPL_CLASS), - obj.getValue(IMPL_DESC)); + mViewer.showDialog(obj.get(IMPL_NAME), + obj.get(IMPL_CLASS), + obj.get(IMPL_DESC)); } if (e.getSource().equals(mHelp)) { helpCallback(); @@ -270,15 +269,15 @@ public class RuleImplTab extends CMSBaseUGTab { int i=0; String[] vals = new String[response.size()]; Hashtable data = new Hashtable(); - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - String value = response.getValue(entry); + for (String entry : response.keySet()) { + entry = entry.trim(); + String value = response.get(entry); int x = value.indexOf(","); NameValuePairs obj = new NameValuePairs(); - obj.add(IMPL_NAME,entry); + obj.put(IMPL_NAME, entry); vals[i++]= entry ; - obj.add(IMPL_CLASS, value.substring(0,x)); - obj.add(IMPL_DESC, value.substring(x+1)); + obj.put(IMPL_CLASS, value.substring(0, x)); + obj.put(IMPL_DESC, value.substring(x + 1)); data.put(entry,obj); } @@ -306,7 +305,7 @@ public class RuleImplTab extends CMSBaseUGTab { try { mConnection.delete(mDestination, ScopeDef.SC_RULE_IMPLS, - obj.getValue(IMPL_NAME)); + obj.get(IMPL_NAME)); } catch (EAdminException e) { //display error dialog showErrorDialog(e.getMessage()); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/RuleRuleDataModel.java b/pki/base/console/src/com/netscape/admin/certsrv/config/RuleRuleDataModel.java index 0eb22433d..cee7727a1 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/RuleRuleDataModel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/RuleRuleDataModel.java @@ -20,7 +20,6 @@ package com.netscape.admin.certsrv.config; import java.util.*; import javax.swing.*; import com.netscape.admin.certsrv.*; -import com.netscape.admin.certsrv.connection.*; import com.netscape.certsrv.common.*; import com.netscape.management.client.util.Debug; @@ -52,19 +51,19 @@ public class RuleRuleDataModel extends CMSRuleDataModel Vector v = new Vector(); NameValuePairs obj = (NameValuePairs) data; - if (obj.getValue(RULE_STAT).equalsIgnoreCase("enabled")) { - v.addElement(new JLabel(obj.getValue(RULE_NAME), + if (obj.get(RULE_STAT).equalsIgnoreCase("enabled")) { + v.addElement(new JLabel(obj.get(RULE_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_RULE), JLabel.LEFT)); v.addElement(mResource.getString("RULERULE_LABEL_ENABLED_LABEL")); } else { - v.addElement(new JLabel(obj.getValue(RULE_NAME), + v.addElement(new JLabel(obj.get(RULE_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_RULE_DISABLE), JLabel.LEFT)); v.addElement(mResource.getString("RULERULE_LABEL_DISABLED_LABEL")); } addRow(v, data); - mRules.addElement(obj.getValue(RULE_NAME)); + mRules.addElement(obj.get(RULE_NAME)); } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/config/TKSKeysTab.java b/pki/base/console/src/com/netscape/admin/certsrv/config/TKSKeysTab.java index 458885e2a..ac8b5e610 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/config/TKSKeysTab.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/config/TKSKeysTab.java @@ -17,22 +17,17 @@ // --- END COPYRIGHT BLOCK --- package com.netscape.admin.certsrv.config; -import com.netscape.admin.certsrv.keycert.*; import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import com.netscape.admin.certsrv.ug.*; import javax.swing.*; -import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.console.*; import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; -import org.mozilla.jss.*; -import org.mozilla.jss.pkcs10.*; -import org.mozilla.jss.crypto.*; import org.mozilla.jss.CryptoManager; /** * User Certs Tab @@ -239,9 +234,8 @@ public class TKSKeysTab extends CMSBaseUGTab { mToken.removeAllItems(); String[] vals = new String[response.size()]; int i=0; - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - vals[i++] = entry; + for (String entry : response.keySet()) { + vals[i++] = entry.trim(); } int sindex = 0; @@ -249,7 +243,7 @@ public class TKSKeysTab extends CMSBaseUGTab { for (i=0; i= 0) { NameValuePairs nvp = (NameValuePairs)mDataModel.getObjectValueAt(index); - String type = nvp.getValue(IMPL_TYPE); + String type = nvp.get(IMPL_TYPE); if (type.equals(Constants.VIEW) || type.equals(Constants.VIEW)) mDelete.setEnabled(false); } @@ -264,9 +263,9 @@ public class AuthImplTab extends CMSBaseUGTab { int i=0; String[] vals = new String[response.size()]; Hashtable data = new Hashtable(); - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - String value = response.getValue(entry); + for (String entry : response.keySet()) { + entry = entry.trim(); + String value = response.get(entry); Debug.println("AuthImplTab.java update(): "+ "entry="+entry+ "value="+value @@ -274,34 +273,34 @@ public class AuthImplTab extends CMSBaseUGTab { int x = value.indexOf(","); int y = value.indexOf(TOKEN); NameValuePairs obj = new NameValuePairs(); - obj.add(IMPL_NAME,entry); + obj.put(IMPL_NAME, entry); Debug.println("x="+x); if(x != -1) { Debug.println("0 obj.add(IMPL_CLASS,"+value.substring(0,x)); - obj.add(IMPL_CLASS, value.substring(0,x)); + obj.put(IMPL_CLASS, value.substring(0, x)); if (y == -1) { Debug.println("1 obj.add(IMPL_DESC,"+value.substring(0,x)); - obj.add(IMPL_DESC, value.substring(x+1)); + obj.put(IMPL_DESC, value.substring(x + 1)); } else { Debug.println("1 obj.add(IMPL_DESC,"+value.substring(x+1,y)); - obj.add(IMPL_DESC, value.substring(x+1, y)); + obj.put(IMPL_DESC, value.substring(x + 1, y)); } } else { Debug.println(" 2 obj.add(IMPL_CLASS,"+value.substring(0,y)); - obj.add(IMPL_CLASS, value.substring(0,y)); + obj.put(IMPL_CLASS, value.substring(0, y)); Debug.println(" 2 obj.add(IMPL_DESC,\"\""); - obj.add(IMPL_DESC,""); + obj.put(IMPL_DESC, ""); } String type = ""; Debug.println("y="+y); if (y == -1) { type = Constants.EDIT; - obj.add(IMPL_TYPE, Constants.EDIT); + obj.put(IMPL_TYPE, Constants.EDIT); } else { type = value.substring(y+1); - obj.add(IMPL_TYPE, type); + obj.put(IMPL_TYPE, type); } if (type.equals(Constants.EDIT)) { data.put(entry,obj); @@ -339,7 +338,7 @@ public class AuthImplTab extends CMSBaseUGTab { try { mConnection.delete(mDestination, ScopeDef.SC_AUTH_IMPLS, - obj.getValue(IMPL_NAME)); + obj.get(IMPL_NAME)); } catch (EAdminException e) { //display error dialog showErrorDialog(e.getMessage()); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/ug/AuthInstanceTab.java b/pki/base/console/src/com/netscape/admin/certsrv/ug/AuthInstanceTab.java index a4db6dd0a..929f99aef 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/ug/AuthInstanceTab.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/ug/AuthInstanceTab.java @@ -21,12 +21,7 @@ import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import com.netscape.admin.certsrv.config.*; import javax.swing.*; -import javax.swing.event.*; -import java.awt.event.*; -import java.awt.*; -import java.util.*; -import com.netscape.management.client.*; -import com.netscape.management.client.util.*; + import com.netscape.certsrv.common.*; /** @@ -117,7 +112,7 @@ public class AuthInstanceTab extends CMSPluginInstanceTab { try { mConnection.delete(DestDef.DEST_AUTH_ADMIN, ScopeDef.SC_AUTH_MGR_INSTANCE, - data.getValue(RULE_NAME)); + data.get(RULE_NAME)); } catch (EAdminException e) { //display error dialog showErrorDialog(e.getMessage()); @@ -139,7 +134,7 @@ public class AuthInstanceTab extends CMSPluginInstanceTab { NameValuePairs response; response = mConnection.read(DestDef.DEST_AUTH_ADMIN, ScopeDef.SC_AUTH_MGR_INSTANCE, - data.getValue(RULE_NAME), + data.get(RULE_NAME), new NameValuePairs()); return response; } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/ug/AuthRuleDataModel.java b/pki/base/console/src/com/netscape/admin/certsrv/ug/AuthRuleDataModel.java index 41fabfcc4..35e08d0eb 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/ug/AuthRuleDataModel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/ug/AuthRuleDataModel.java @@ -21,7 +21,6 @@ import java.util.*; import javax.swing.*; import com.netscape.certsrv.common.*; import com.netscape.admin.certsrv.*; -import com.netscape.admin.certsrv.connection.*; import com.netscape.admin.certsrv.config.*; import com.netscape.management.client.util.Debug; @@ -54,10 +53,10 @@ public class AuthRuleDataModel extends CMSRuleDataModel NameValuePairs obj = (NameValuePairs) data; //XXX NEED TO ADD STUFF - v.addElement(new JLabel(obj.getValue(RULE_NAME), + v.addElement(new JLabel(obj.get(RULE_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_AUTH), JLabel.LEFT)); - v.addElement(obj.getValue(RULE_IMPL)); + v.addElement(obj.get(RULE_IMPL)); addRow(v, data); } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/ug/CertDataModel.java b/pki/base/console/src/com/netscape/admin/certsrv/ug/CertDataModel.java index b6eda1308..de6a588f6 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/ug/CertDataModel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/ug/CertDataModel.java @@ -18,7 +18,6 @@ package com.netscape.admin.certsrv.ug; import com.netscape.admin.certsrv.*; -import com.netscape.admin.certsrv.connection.*; import com.netscape.certsrv.common.*; import com.netscape.management.client.util.*; import java.util.*; @@ -60,7 +59,7 @@ public class CertDataModel extends CMSContentTableModel NameValuePairs obj = (NameValuePairs) data; //XXX NEED TO CHANGE if we are going to have multi-column table - v.addElement(new JLabel(obj.getValue(CERT_NAME), + v.addElement(new JLabel(obj.get(CERT_NAME), CMSAdminUtil.getImage(CMSAdminResources.IMAGE_CERTICON_SMALL), JLabel.LEFT)); addRow(v, data); @@ -77,7 +76,7 @@ public class CertDataModel extends CMSContentTableModel public String getDataBlob(int row) { try { NameValuePairs obj = (NameValuePairs)getObjectValueAt(row); - return obj.getValue(CERT_DATA); + return obj.get(CERT_DATA); } catch (Exception e) { Debug.println("CertDataModel: getDataBlob()- "+e.toString()); return ""; diff --git a/pki/base/console/src/com/netscape/admin/certsrv/ug/CertManagementDialog.java b/pki/base/console/src/com/netscape/admin/certsrv/ug/CertManagementDialog.java index c6a1a878c..82083649e 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/ug/CertManagementDialog.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/ug/CertManagementDialog.java @@ -20,11 +20,10 @@ package com.netscape.admin.certsrv.ug; import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import javax.swing.*; -import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -298,8 +297,8 @@ public class CertManagementDialog extends JDialog String[] vals = new String[response.size()]; int i=0; - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - vals[i++] = ((String)e.nextElement()).trim(); + for (String entry : response.keySet()) { + vals[i++] = entry.trim(); } CMSAdminUtil.bubbleSort(vals); @@ -309,7 +308,7 @@ public class CertManagementDialog extends JDialog mDataModel.addElement(new JLabel(str, CMSAdminUtil.getImage(CMSAdminResources.IMAGE_CERTICON_SMALL), JLabel.LEFT)); - mPPData.addElement(response.getValue(vals[y])); + mPPData.addElement(response.get(vals[y])); } return true; @@ -386,7 +385,7 @@ public class CertManagementDialog extends JDialog private void addCert(String B64E) { //send comment to server for the removal of user NameValuePairs config = new NameValuePairs(); - config.add(Constants.PR_USER_CERT, cleanupCertData(B64E)); + config.put(Constants.PR_USER_CERT, cleanupCertData(B64E)); try { mConnection.add(DestDef.DEST_USER_ADMIN, ScopeDef.SC_USER_CERTS, @@ -423,7 +422,7 @@ public class CertManagementDialog extends JDialog String dn = ((JLabel)mDataModel.elementAt(mList.getSelectedIndex())).getText(); dn = toServerFormat(dn); NameValuePairs config = new NameValuePairs(); - config.add(Constants.PR_USER_CERT,dn); + config.put(Constants.PR_USER_CERT, dn); //send comment to server for the removal of user try { diff --git a/pki/base/console/src/com/netscape/admin/certsrv/ug/GroupEditor.java b/pki/base/console/src/com/netscape/admin/certsrv/ug/GroupEditor.java index f25267be5..e9693b15d 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/ug/GroupEditor.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/ug/GroupEditor.java @@ -24,8 +24,7 @@ import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import javax.swing.table.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -203,8 +202,8 @@ public class GroupEditor extends JDialog response = mConnection.search(DestDef.DEST_USER_ADMIN, ScopeDef.SC_USERS, new NameValuePairs()); boolean hasNewUser = false; - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); + for (String entry : response.keySet()) { + entry = entry.trim(); if (currentUser.indexOf(entry)== -1) hasNewUser = true; } @@ -483,9 +482,9 @@ public class GroupEditor extends JDialog //construct NVP NameValuePairs config = new NameValuePairs(); - config.add(Constants.PR_GROUP_DESC, ""); - config.add(Constants.PR_GROUP_USER, ""); - config.add(Constants.PR_GROUP_GROUP, ""); + config.put(Constants.PR_GROUP_DESC, ""); + config.put(Constants.PR_GROUP_USER, ""); + config.put(Constants.PR_GROUP_GROUP, ""); NameValuePairs response; response = mConnection.read(DestDef.DEST_GROUP_ADMIN, @@ -496,12 +495,12 @@ public class GroupEditor extends JDialog Debug.println("Received Memebership: "+response.toString()); //setup the ui mGroupNameField.setText(mGroupName); - mGroupDescField.setText(response.getValue(Constants.PR_GROUP_DESC)); + mGroupDescField.setText(response.get(Constants.PR_GROUP_DESC)); //setup the member table //parse user entry - String user = response.getValue(Constants.PR_GROUP_USER).trim(); + String user = response.get(Constants.PR_GROUP_USER).trim(); StringTokenizer tokenizer = new StringTokenizer(user, ","); while (tokenizer.hasMoreTokens()) { String user_str = tokenizer.nextToken().trim(); @@ -515,7 +514,7 @@ public class GroupEditor extends JDialog //construct NVP NameValuePairs config = new NameValuePairs(); - config.add(Constants.PR_GROUP_DESC, mGroupName); + config.put(Constants.PR_GROUP_DESC, mGroupName); createUGString(config); //send request @@ -530,7 +529,7 @@ public class GroupEditor extends JDialog //construct NVP NameValuePairs config = new NameValuePairs(); - config.add(Constants.PR_GROUP_DESC, mGroupDescField.getText()); + config.put(Constants.PR_GROUP_DESC, mGroupDescField.getText()); createUGString(config); //send request @@ -567,7 +566,7 @@ public class GroupEditor extends JDialog } //set parameters - config.add(Constants.PR_GROUP_USER, userBuf.toString()); + config.put(Constants.PR_GROUP_USER, userBuf.toString()); } //refresh the table content diff --git a/pki/base/console/src/com/netscape/admin/certsrv/ug/GroupListDialog.java b/pki/base/console/src/com/netscape/admin/certsrv/ug/GroupListDialog.java index 706ed2ccd..d5931a6f8 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/ug/GroupListDialog.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/ug/GroupListDialog.java @@ -260,8 +260,8 @@ public class GroupListDialog extends JDialog } //parse the data - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); + for (String entry : response.keySet()) { + entry = entry.trim(); //check if not already in current list if (mCurrentGroups.indexOf(entry)== -1) mDataModel.processData(entry); diff --git a/pki/base/console/src/com/netscape/admin/certsrv/ug/GroupTab.java b/pki/base/console/src/com/netscape/admin/certsrv/ug/GroupTab.java index eca04bddc..e80cc7047 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/ug/GroupTab.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/ug/GroupTab.java @@ -20,11 +20,10 @@ package com.netscape.admin.certsrv.ug; import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import javax.swing.*; -import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -323,9 +322,8 @@ public class GroupTab extends CMSBaseUGTab { //parse the data Vector store = new Vector(); - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String entry = ((String)e.nextElement()).trim(); - store.addElement(entry); + for (String entry : response.keySet()) { + store.addElement(entry.trim()); } String[] vals = new String[store.size()]; @@ -334,7 +332,7 @@ public class GroupTab extends CMSBaseUGTab { CMSAdminUtil.bubbleSort(vals); for (int y=0; y< vals.length ; y++) { - String value = response.getValue(vals[y]); + String value = response.get(vals[y]); mDataModel.processData(vals[y],value); } diff --git a/pki/base/console/src/com/netscape/admin/certsrv/ug/MemberDataModel.java b/pki/base/console/src/com/netscape/admin/certsrv/ug/MemberDataModel.java index 6ebb84cb6..6b1fbfeec 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/ug/MemberDataModel.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/ug/MemberDataModel.java @@ -21,7 +21,6 @@ import java.util.*; import javax.swing.*; import com.netscape.certsrv.common.*; import com.netscape.admin.certsrv.*; -import com.netscape.admin.certsrv.connection.*; import com.netscape.management.client.util.*; /** @@ -66,13 +65,13 @@ public class MemberDataModel extends CMSContentTableModel NameValuePairs rec = (NameValuePairs)data; Icon icon; - icon = (rec.getValue(MEMBER_TYPE).equals(MEMBER_GROUP))? + icon = (rec.get(MEMBER_TYPE).equals(MEMBER_GROUP))? CMSAdminUtil.getImage(CMSAdminResources.IMAGE_USERGROUP): CMSAdminUtil.getImage(CMSAdminResources.IMAGE_USER); - String entry = rec.getValue(MEMBER_NAME); + String entry = rec.get(MEMBER_NAME); String name = entry; - if(rec.getValue(MEMBER_TYPE).equals(MEMBER_GROUP)) { + if(rec.get(MEMBER_TYPE).equals(MEMBER_GROUP)) { if (entry.startsWith(PrefixDef.PX_SYS)) name = entry.substring(PrefixDef.PX_SYS.length()); else @@ -101,10 +100,10 @@ public class MemberDataModel extends CMSContentTableModel { Debug.println("MemberDataModel: removeRow() - start"); NameValuePairs data = (NameValuePairs)getObjectValueAt(index); - if (data.getValue(MEMBER_TYPE).equals(MEMBER_GROUP)) - mGroups.removeElement(data.getValue(MEMBER_NAME)); + if (data.get(MEMBER_TYPE).equals(MEMBER_GROUP)) + mGroups.removeElement(data.get(MEMBER_NAME)); else - mUsers.removeElement(data.getValue(MEMBER_NAME)); + mUsers.removeElement(data.get(MEMBER_NAME)); super.removeRow(index); Debug.println("MemberDataModel: removeRow() - end"); } @@ -118,10 +117,10 @@ public class MemberDataModel extends CMSContentTableModel super.addRow(values); mObjectContainer.addElement(obj); NameValuePairs rec = (NameValuePairs)obj; - if (rec.getValue(MEMBER_TYPE).equals(MEMBER_GROUP)) - mGroups.addElement(rec.getValue(MEMBER_NAME)); + if (rec.get(MEMBER_TYPE).equals(MEMBER_GROUP)) + mGroups.addElement(rec.get(MEMBER_NAME)); else - mUsers.addElement(rec.getValue(MEMBER_NAME)); + mUsers.addElement(rec.get(MEMBER_NAME)); } /** diff --git a/pki/base/console/src/com/netscape/admin/certsrv/ug/UserEditor.java b/pki/base/console/src/com/netscape/admin/certsrv/ug/UserEditor.java index 561b8bb19..75908fe6a 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/ug/UserEditor.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/ug/UserEditor.java @@ -24,9 +24,8 @@ import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; -import javax.swing.table.*; import com.netscape.certsrv.common.*; /** @@ -520,11 +519,11 @@ public class UserEditor extends JDialog //construct NVP NameValuePairs config = new NameValuePairs(); - config.add(Constants.PR_USER_FULLNAME, ""); - config.add(Constants.PR_USER_EMAIL, ""); - config.add(Constants.PR_USER_PHONE, ""); - config.add(Constants.PR_USER_STATE, ""); - config.add(Constants.PR_USER_GROUP, ""); + config.put(Constants.PR_USER_FULLNAME, ""); + config.put(Constants.PR_USER_EMAIL, ""); + config.put(Constants.PR_USER_PHONE, ""); + config.put(Constants.PR_USER_STATE, ""); + config.put(Constants.PR_USER_GROUP, ""); NameValuePairs response; response = mConnection.read(DestDef.DEST_USER_ADMIN, @@ -534,13 +533,13 @@ public class UserEditor extends JDialog //setup the ui mUserNameField.setText(mUserName); - mFullNameField.setText(response.getValue(Constants.PR_USER_FULLNAME)); - mEMailField.setText(response.getValue(Constants.PR_USER_EMAIL)); - mPhoneField.setText(response.getValue(Constants.PR_USER_PHONE)); - mStateField.setText(response.getValue(Constants.PR_USER_STATE)); + mFullNameField.setText(response.get(Constants.PR_USER_FULLNAME)); + mEMailField.setText(response.get(Constants.PR_USER_EMAIL)); + mPhoneField.setText(response.get(Constants.PR_USER_PHONE)); + mStateField.setText(response.get(Constants.PR_USER_STATE)); //parse group entry - String userStr = response.getValue(Constants.PR_USER_GROUP); + String userStr = response.get(Constants.PR_USER_GROUP); if ( (userStr != null) && (!userStr.trim().equals("")) ) { StringTokenizer tokenizer = new StringTokenizer(userStr, ","); while (tokenizer.hasMoreTokens()) { @@ -556,8 +555,7 @@ public class UserEditor extends JDialog ScopeDef.SC_GROUPS, new NameValuePairs()); if (mGroupBox.getItemCount() > 0) mGroupBox.removeAllItems(); - for (Enumeration e = response.getNames(); e.hasMoreElements() ;) { - String groupname = (String)e.nextElement(); + for (String groupname : response.keySet()) { mGroupBox.addItem(groupname.trim()); } @@ -568,13 +566,13 @@ public class UserEditor extends JDialog //construct NVP NameValuePairs config = new NameValuePairs(); - config.add(Constants.PR_USER_FULLNAME, mFullNameField.getText().trim()); - config.add(Constants.PR_USER_PASSWORD,mPasswordField.getText().trim()); - config.add(Constants.PR_USER_EMAIL,mEMailField.getText().trim()); - config.add(Constants.PR_USER_PHONE,mPhoneField.getText().trim()); - config.add(Constants.PR_USER_STATE,mStateField.getText().trim()); - config.add(Constants.PR_USER_GROUP,(String)mGroupBox.getSelectedItem()); - config.add(Constants.PR_USER_TYPE, ""); + config.put(Constants.PR_USER_FULLNAME, mFullNameField.getText().trim()); + config.put(Constants.PR_USER_PASSWORD, mPasswordField.getText().trim()); + config.put(Constants.PR_USER_EMAIL, mEMailField.getText().trim()); + config.put(Constants.PR_USER_PHONE, mPhoneField.getText().trim()); + config.put(Constants.PR_USER_STATE, mStateField.getText().trim()); + config.put(Constants.PR_USER_GROUP, (String) mGroupBox.getSelectedItem()); + config.put(Constants.PR_USER_TYPE, ""); //config.add(Constants.PR_USER_CERT,""); //createCertEntry(config); @@ -590,12 +588,12 @@ public class UserEditor extends JDialog //construct NVP NameValuePairs config = new NameValuePairs(); - config.add(Constants.PR_USER_FULLNAME, mFullNameField.getText().trim()); - config.add(Constants.PR_USER_PASSWORD,mPasswordField.getText().trim()); - config.add(Constants.PR_USER_EMAIL,mEMailField.getText().trim()); - config.add(Constants.PR_USER_PHONE,mPhoneField.getText().trim()); - config.add(Constants.PR_USER_STATE,mStateField.getText().trim()); - config.add(Constants.PR_USER_TYPE, ""); + config.put(Constants.PR_USER_FULLNAME, mFullNameField.getText().trim()); + config.put(Constants.PR_USER_PASSWORD, mPasswordField.getText().trim()); + config.put(Constants.PR_USER_EMAIL, mEMailField.getText().trim()); + config.put(Constants.PR_USER_PHONE, mPhoneField.getText().trim()); + config.put(Constants.PR_USER_STATE, mStateField.getText().trim()); + config.put(Constants.PR_USER_TYPE, ""); //createCertEntry(config); //send request diff --git a/pki/base/console/src/com/netscape/admin/certsrv/ug/UserListDialog.java b/pki/base/console/src/com/netscape/admin/certsrv/ug/UserListDialog.java index b54ac7d8d..6f0c6215b 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/ug/UserListDialog.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/ug/UserListDialog.java @@ -20,13 +20,11 @@ package com.netscape.admin.certsrv.ug; import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import javax.swing.*; -import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; -import javax.swing.table.*; import com.netscape.certsrv.common.*; /** @@ -317,7 +315,7 @@ public class UserListDialog extends JDialog store.copyInto(vals); */ - String responseValue = response.getValue("userInfo"); + String responseValue = response.get("userInfo"); StringTokenizer tokenizer = new StringTokenizer(responseValue, ";"); StringTokenizer subTokenizer = null; diff --git a/pki/base/console/src/com/netscape/admin/certsrv/ug/UserTab.java b/pki/base/console/src/com/netscape/admin/certsrv/ug/UserTab.java index ccf7ac5ae..78b7e5252 100644 --- a/pki/base/console/src/com/netscape/admin/certsrv/ug/UserTab.java +++ b/pki/base/console/src/com/netscape/admin/certsrv/ug/UserTab.java @@ -20,11 +20,10 @@ package com.netscape.admin.certsrv.ug; import com.netscape.admin.certsrv.*; import com.netscape.admin.certsrv.connection.*; import javax.swing.*; -import javax.swing.event.*; import java.awt.event.*; import java.awt.*; import java.util.*; -import com.netscape.management.client.*; + import com.netscape.management.client.util.*; import com.netscape.certsrv.common.*; @@ -289,7 +288,7 @@ public class UserTab extends CMSBaseUGTab { Debug.println(response.toString()); - String responseValue = response.getValue("userInfo"); + String responseValue = response.get("userInfo"); StringTokenizer tokenizer = new StringTokenizer(responseValue, ";"); StringTokenizer subTokenizer = null; diff --git a/pki/base/console/src/com/netscape/certsrv/common/NameValuePair.java b/pki/base/console/src/com/netscape/certsrv/common/NameValuePair.java deleted file mode 100644 index b2ad3ac29..000000000 --- a/pki/base/console/src/com/netscape/certsrv/common/NameValuePair.java +++ /dev/null @@ -1,56 +0,0 @@ -// --- BEGIN COPYRIGHT BLOCK --- -// This program is free software; you can redistribute it and/or modify -// it under the terms of the GNU General Public License as published by -// the Free Software Foundation; version 2 of the License. -// -// This program is distributed in the hope that it will be useful, -// but WITHOUT ANY WARRANTY; without even the implied warranty of -// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -// GNU General Public License for more details. -// -// You should have received a copy of the GNU General Public License along -// with this program; if not, write to the Free Software Foundation, Inc., -// 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. -// -// (C) 2007 Red Hat, Inc. -// All rights reserved. -// --- END COPYRIGHT BLOCK --- -package com.netscape.certsrv.common; - - -import java.io.*; -import java.util.*; -import java.net.*; -import java.util.*; -import java.text.*; -import java.math.*; - - -/** - * A class represents a name value pair. - * - * @author thomask - * @version $Revision$, $Date$ - */ -public class NameValuePair { - - private String mName = null; - private String mValue = null; - - public NameValuePair(String name, String value) { - mName = name; - mValue = value; - } - - public String getName() { - return mName; - } - - public String getValue() { - return mValue; - } - - public void setValue(String value) { - mValue = value; - } -} diff --git a/pki/base/console/src/com/netscape/certsrv/common/NameValuePairs.java b/pki/base/console/src/com/netscape/certsrv/common/NameValuePairs.java index 2dd913ea7..5e7d79cde 100644 --- a/pki/base/console/src/com/netscape/certsrv/common/NameValuePairs.java +++ b/pki/base/console/src/com/netscape/certsrv/common/NameValuePairs.java @@ -17,111 +17,23 @@ // --- END COPYRIGHT BLOCK --- package com.netscape.certsrv.common; - -import java.io.*; -import java.util.*; -import java.net.*; -import java.util.*; -import java.text.*; -import java.math.*; - +import java.util.LinkedHashMap; +import java.util.StringTokenizer; /** - * A class represents an ordered list of name + * A class represents an ordered list of name * value pairs. * - * @author thomask * @version $Revision$, $Date$ */ -public class NameValuePairs { - - private Vector mPairs = new Vector(); - - // an index to speed up searching - // The key is the name. The element is the NameValuePair. - private Hashtable index = new Hashtable(); +public class NameValuePairs extends LinkedHashMap { /** * Constructs name value pairs. - */ - public NameValuePairs() { - } - - /** - * Adds a name value pair into this set. - * if the name already exist, the value will - * be replaced. - */ - public void add(String name, String value) { - NameValuePair pair = getPair(name); - - if (pair == null) { - pair = new NameValuePair(name, value); - mPairs.addElement(pair); - index.put(name, pair); - } else { - pair.setValue(value); - } - } - - /** - * Retrieves name value pair from this set. - */ - public NameValuePair getPair(String name) { - return (NameValuePair) index.get(name); - } - - /** - * Returns number of pairs in this set. - */ - public int size() { - return mPairs.size(); - } - - /** - * Retrieves name value pairs in specific position. - */ - public NameValuePair elementAt(int pos) { - return (NameValuePair) mPairs.elementAt(pos); - } - - /** - * Removes all name value pairs in this set. - */ - public void removeAllPairs() { - mPairs.removeAllElements(); - index.clear(); - } - - /** - * Retrieves value of the name value pairs that matches - * the given name. */ - public String getValue(String name) { - NameValuePair p = getPair(name); - - if (p != null) { - return p.getValue(); - } - return null; + public NameValuePairs() { } - /** - * Retrieves a list of names. - */ - public Enumeration getNames() { - Vector v = new Vector(); - int size = mPairs.size(); - - for (int i = 0; i < size; i++) { - NameValuePair p = (NameValuePair) mPairs.elementAt(i); - - v.addElement(p.getName()); - } - //System.out.println("getNames: "+v.size()); - return v.elements(); - } - /** * Show the content of this name value container as * string representation. @@ -131,15 +43,23 @@ public class NameValuePairs { public String toString() { StringBuffer buf = new StringBuffer(); - for (int i = 0; i < mPairs.size(); i++) { - NameValuePair p = (NameValuePair) mPairs.elementAt(i); + for (String name : keySet()) { + String value = get(name); - buf.append(p.getName() + "=" + p.getValue()); + buf.append(name + "=" + value); buf.append("\n"); } + return buf.toString(); } + /** + * Parses a string into name value pairs. + * + * @param s string + * @param nvp name value pairs + * @return true if successful + */ public static boolean parseInto(String s, NameValuePairs nvp) { StringTokenizer st = new StringTokenizer(s, "&"); @@ -153,12 +73,8 @@ public class NameValuePairs { String n = t.substring(0, i); String v = t.substring(i + 1); - nvp.add(n, v); - } + nvp.put(n, v); + } return true; } - - public Enumeration elements() { - return mPairs.elements(); - } -} +} -- cgit