diff options
author | Endi Sukma Dewata <edewata@redhat.com> | 2012-02-03 13:49:11 -0600 |
---|---|---|
committer | Endi Sukma Dewata <edewata@redhat.com> | 2012-02-06 12:11:23 -0600 |
commit | d1247e1a404426c9ca2ceeffc52f2b73a0fb6d06 (patch) | |
tree | 1aaf14e1dfe278db584d4175cc96e4b78e1be1b7 /pki/base/common/src/com/netscape/cms/servlet/admin | |
parent | 6c0e7f2156e0b218e4cc62029f0e3b0e89124e9d (diff) | |
download | pki-d1247e1a404426c9ca2ceeffc52f2b73a0fb6d06.tar.gz pki-d1247e1a404426c9ca2ceeffc52f2b73a0fb6d06.tar.xz pki-d1247e1a404426c9ca2ceeffc52f2b73a0fb6d06.zip |
Added generics (part 3).
This patch brings down the warnings from 3992 to 3500.
Ticket #2
Diffstat (limited to 'pki/base/common/src/com/netscape/cms/servlet/admin')
3 files changed, 104 insertions, 100 deletions
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 58cf327bb..799e1bb37 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 @@ -327,13 +327,13 @@ public class PublisherAdminServlet extends AdminServlet { private IExtendedPluginInfo getExtendedPluginInfo(IPublisherProcessor p) { - Enumeration mappers = p.getMapperInsts().keys(); - Enumeration publishers = p.getPublisherInsts().keys(); + Enumeration<String> mappers = p.getMapperInsts().keys(); + Enumeration<String> publishers = p.getPublisherInsts().keys(); StringBuffer map = new StringBuffer(); for (; mappers.hasMoreElements();) { - String name = (String) mappers.nextElement(); + String name = mappers.nextElement(); if (map.length() == 0) { map.append(name); @@ -372,7 +372,7 @@ public class PublisherAdminServlet extends AdminServlet { if (implType.equals(Constants.PR_EXT_PLUGIN_IMPLTYPE_PUBLISHRULE)) { IPublisherProcessor p_processor = mProcessor; - Plugin plugin = (Plugin) p_processor.getRulePlugins().get(implName); + RulePlugin plugin = p_processor.getRulePlugins().get(implName); // Should get the registered rules from processor // instead of plugin @@ -439,10 +439,11 @@ public class PublisherAdminServlet extends AdminServlet { IConfigStore ldapcfg = publishcfg.getSubStore(IPublisherProcessor.PROP_LDAP_PUBLISH_SUBSTORE); IConfigStore ldap = ldapcfg.getSubStore(IPublisherProcessor.PROP_LDAP); - Enumeration e = req.getParameterNames(); + @SuppressWarnings("unchecked") + Enumeration<String> e = req.getParameterNames(); while (e.hasMoreElements()) { - String name = (String) e.nextElement(); + String name = e.nextElement(); if (name.equals(Constants.OP_TYPE)) continue; @@ -502,7 +503,6 @@ public class PublisherAdminServlet extends AdminServlet { private void setLDAPDest(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException, EBaseException { - NameValuePairs params = new NameValuePairs(); //Save New Settings to the config file IConfigStore config = mAuth.getConfigStore(); @@ -521,11 +521,12 @@ public class PublisherAdminServlet extends AdminServlet { } //set reset of the parameters - Enumeration e = req.getParameterNames(); + @SuppressWarnings("unchecked") + Enumeration<String> e = req.getParameterNames(); String pwd = null; while (e.hasMoreElements()) { - String name = (String) e.nextElement(); + String name = e.nextElement(); if (name.equals(Constants.OP_TYPE)) continue; @@ -641,11 +642,12 @@ public class PublisherAdminServlet extends AdminServlet { } //set reset of the parameters - Enumeration e = req.getParameterNames(); + @SuppressWarnings("unchecked") + Enumeration<String> e = req.getParameterNames(); String pwd = null; while (e.hasMoreElements()) { - String name = (String) e.nextElement(); + String name = e.nextElement(); if (name.equals(Constants.OP_TYPE)) continue; @@ -1038,7 +1040,7 @@ public class PublisherAdminServlet extends AdminServlet { IConfigStore instancesConfig = destStore.getSubStore("impl"); // Does the class exist? - Class newImpl = null; + Class<?> newImpl = null; try { newImpl = Class.forName(classPath); @@ -1147,7 +1149,7 @@ public class PublisherAdminServlet extends AdminServlet { return; } - Vector configParams = mProcessor.getMapperDefaultParams(implname); + Vector<String> configParams = mProcessor.getMapperDefaultParams(implname); IConfigStore destStore = mConfig.getSubStore(mAuth.getId() + ".publish.mapper"); IConfigStore instancesConfig = destStore.getSubStore("instance"); @@ -1243,10 +1245,10 @@ public class PublisherAdminServlet extends AdminServlet { HttpServletResponse resp) throws ServletException, IOException, EBaseException { NameValuePairs params = new NameValuePairs(); - Enumeration e = mProcessor.getMapperPlugins().keys(); + Enumeration<String> e = mProcessor.getMapperPlugins().keys(); while (e.hasMoreElements()) { - String name = (String) e.nextElement(); + String name = e.nextElement(); MapperPlugin value = (MapperPlugin) mProcessor.getMapperPlugins().get(name); // get Description @@ -1284,10 +1286,10 @@ public class PublisherAdminServlet extends AdminServlet { IOException, EBaseException { NameValuePairs params = new NameValuePairs(); - Enumeration e = mProcessor.getMapperInsts().keys(); + Enumeration<String> e = mProcessor.getMapperInsts().keys(); for (; e.hasMoreElements();) { - String name = (String) e.nextElement(); + String name = e.nextElement(); ILdapMapper value = mProcessor.getMapperInstance(name); params.add(name, getMapperPluginName(value) + ";visible"); @@ -1373,7 +1375,7 @@ public class PublisherAdminServlet extends AdminServlet { // first check if any instances from this mapper // DON'T remove mapper if any instance - for (Enumeration e = mProcessor.getMapperInsts().keys(); e.hasMoreElements();) { + for (Enumeration<String> e = mProcessor.getMapperInsts().keys(); e.hasMoreElements();) { String name = (String) e.nextElement(); ILdapMapper mapper = mProcessor.getMapperInstance(name); @@ -1421,14 +1423,14 @@ public class PublisherAdminServlet extends AdminServlet { return; } - Vector configParams = mProcessor.getMapperDefaultParams(implname); + Vector<String> configParams = mProcessor.getMapperDefaultParams(implname); NameValuePairs params = new NameValuePairs(); // implName is always required so always send it. params.add(Constants.PR_MAPPER_IMPL_NAME, ""); if (configParams != null) { for (int i = 0; i < configParams.size(); i++) { - String kv = (String) configParams.elementAt(i); + String kv = configParams.elementAt(i); int index = kv.indexOf('='); params.add(kv.substring(0, index), @@ -1463,7 +1465,7 @@ public class PublisherAdminServlet extends AdminServlet { ILdapMapper mapperInst = (ILdapMapper) mProcessor.getMapperInstance(id); - Vector configParams = mapperInst.getInstanceParams(); + Vector<String> configParams = mapperInst.getInstanceParams(); NameValuePairs params = new NameValuePairs(); params.add(Constants.PR_MAPPER_IMPL_NAME, @@ -1471,7 +1473,7 @@ public class PublisherAdminServlet extends AdminServlet { // implName is always required so always send it. if (configParams != null) { for (int i = 0; i < configParams.size(); i++) { - String kv = (String) configParams.elementAt(i); + String kv = configParams.elementAt(i); int index = kv.indexOf('='); params.add(kv.substring(0, index), @@ -1528,14 +1530,14 @@ public class PublisherAdminServlet extends AdminServlet { ILdapMapper oldinst = (ILdapMapper) mProcessor.getMapperInstance(id); - Vector oldConfigParms = oldinst.getInstanceParams(); + Vector<String> oldConfigParms = oldinst.getInstanceParams(); NameValuePairs saveParams = new NameValuePairs(); // implName is always required so always include it it. saveParams.add("pluginName", implname); if (oldConfigParms != null) { for (int i = 0; i < oldConfigParms.size(); i++) { - String kv = (String) oldConfigParms.elementAt(i); + String kv = oldConfigParms.elementAt(i); int index = kv.indexOf('='); saveParams.add(kv.substring(0, index), @@ -1554,7 +1556,7 @@ public class PublisherAdminServlet extends AdminServlet { // create new substore. - Vector configParams = mProcessor.getMapperInstanceParams(id); + Vector<String> configParams = mProcessor.getMapperInstanceParams(id); instancesConfig.removeSubStore(id); @@ -1563,7 +1565,7 @@ public class PublisherAdminServlet extends AdminServlet { substore.put("pluginName", implname); if (configParams != null) { for (int i = 0; i < configParams.size(); i++) { - String kv = (String) configParams.elementAt(i); + String kv = configParams.elementAt(i); int index = kv.indexOf('='); String key = kv.substring(0, index); String val = req.getParameter(key); @@ -1683,7 +1685,7 @@ public class PublisherAdminServlet extends AdminServlet { IConfigStore instancesConfig = destStore.getSubStore("impl"); // Does the class exist? - Class newImpl = null; + Class<?> newImpl = null; try { newImpl = Class.forName(classPath); @@ -1781,7 +1783,7 @@ public class PublisherAdminServlet extends AdminServlet { return; } - Vector configParams = mProcessor.getRuleDefaultParams(implname); + Vector<String> configParams = mProcessor.getRuleDefaultParams(implname); IConfigStore destStore = mConfig.getSubStore(mAuth.getId() @@ -1792,7 +1794,7 @@ public class PublisherAdminServlet extends AdminServlet { if (configParams != null) { for (int i = 0; i < configParams.size(); i++) { - String kv = (String) configParams.elementAt(i); + String kv = configParams.elementAt(i); int index = kv.indexOf('='); String val = req.getParameter(kv.substring(0, index)); @@ -1882,10 +1884,10 @@ public class PublisherAdminServlet extends AdminServlet { HttpServletResponse resp) throws ServletException, IOException, EBaseException { NameValuePairs params = new NameValuePairs(); - Enumeration e = mProcessor.getRulePlugins().keys(); + Enumeration<String> e = mProcessor.getRulePlugins().keys(); while (e.hasMoreElements()) { - String name = (String) e.nextElement(); + String name = e.nextElement(); RulePlugin value = (RulePlugin) mProcessor.getRulePlugins().get(name); // get Description @@ -1909,11 +1911,10 @@ public class PublisherAdminServlet extends AdminServlet { HttpServletResponse resp) throws ServletException, IOException, EBaseException { NameValuePairs params = new NameValuePairs(); - String insts = null; - Enumeration e = mProcessor.getRuleInsts().keys(); + Enumeration<String> e = mProcessor.getRuleInsts().keys(); for (; e.hasMoreElements();) { - String name = (String) e.nextElement(); + String name = e.nextElement(); ILdapRule value = (ILdapRule) mProcessor.getRuleInsts().get((Object) name); String enabled = value.enabled() ? "enabled" : "disabled"; @@ -1959,9 +1960,8 @@ public class PublisherAdminServlet extends AdminServlet { // first check if any instances from this rule // DON'T remove rule if any instance - for (Enumeration e = mProcessor.getRuleInsts().elements(); e.hasMoreElements();) { - ILdapRule rule = (ILdapRule) - e.nextElement(); + for (Enumeration<ILdapRule> e = mProcessor.getRuleInsts().elements(); e.hasMoreElements();) { + ILdapRule rule = e.nextElement(); if (id.equals(getRulePluginName(rule))) { sendResponse(ERROR, CMS.getUserMessage(getLocale(req), "CMS_LDAP_SRVLT_IN_USE"), null, resp); @@ -2058,7 +2058,7 @@ public class PublisherAdminServlet extends AdminServlet { return; } - Vector configParams = mProcessor.getRuleDefaultParams(implname); + Vector<String> configParams = mProcessor.getRuleDefaultParams(implname); NameValuePairs params = new NameValuePairs(); // implName is always required so always send it. @@ -2099,7 +2099,7 @@ public class PublisherAdminServlet extends AdminServlet { ILdapRule ruleInst = (ILdapRule) mProcessor.getRuleInsts().get(id); - Vector configParams = ruleInst.getInstanceParams(); + Vector<String> configParams = ruleInst.getInstanceParams(); NameValuePairs params = new NameValuePairs(); params.add(Constants.PR_RULE_IMPL_NAME, @@ -2163,14 +2163,14 @@ public class PublisherAdminServlet extends AdminServlet { ILdapRule oldinst = (ILdapRule) mProcessor.getRuleInsts().get((Object) id); - Vector oldConfigParms = oldinst.getInstanceParams(); + Vector<String> oldConfigParms = oldinst.getInstanceParams(); NameValuePairs saveParams = new NameValuePairs(); // implName is always required so always include it it. saveParams.add("pluginName", implname); if (oldConfigParms != null) { for (int i = 0; i < oldConfigParms.size(); i++) { - String kv = (String) oldConfigParms.elementAt(i); + String kv = oldConfigParms.elementAt(i); int index = kv.indexOf('='); saveParams.add(kv.substring(0, index), @@ -2189,7 +2189,7 @@ public class PublisherAdminServlet extends AdminServlet { // create new substore. - Vector configParams = mProcessor.getRuleDefaultParams(implname); + Vector<String> configParams = mProcessor.getRuleDefaultParams(implname); instancesConfig.removeSubStore(id); @@ -2198,7 +2198,7 @@ public class PublisherAdminServlet extends AdminServlet { substore.put("pluginName", implname); if (configParams != null) { for (int i = 0; i < configParams.size(); i++) { - String kv = (String) configParams.elementAt(i); + String kv = configParams.elementAt(i); int index = kv.indexOf('='); String key = kv.substring(0, index); String val = req.getParameter(key); @@ -2322,7 +2322,7 @@ public class PublisherAdminServlet extends AdminServlet { IConfigStore instancesConfig = destStore.getSubStore("impl"); // Does the class exist? - Class newImpl = null; + Class<?> newImpl = null; try { newImpl = Class.forName(classPath); @@ -2422,7 +2422,7 @@ public class PublisherAdminServlet extends AdminServlet { return; } - Vector configParams = mProcessor.getPublisherDefaultParams(implname); + Vector<String> configParams = mProcessor.getPublisherDefaultParams(implname); IConfigStore destStore = mConfig.getSubStore(mAuth.getId() + ".publish.publisher"); @@ -2431,7 +2431,7 @@ public class PublisherAdminServlet extends AdminServlet { if (configParams != null) { for (int i = 0; i < configParams.size(); i++) { - String kv = (String) configParams.elementAt(i); + String kv = configParams.elementAt(i); int index = kv.indexOf('='); String val = null; @@ -2533,10 +2533,10 @@ public class PublisherAdminServlet extends AdminServlet { IOException, EBaseException { NameValuePairs params = new NameValuePairs(); - Enumeration e = mProcessor.getPublisherPlugins().keys(); + Enumeration<String> e = mProcessor.getPublisherPlugins().keys(); while (e.hasMoreElements()) { - String name = (String) e.nextElement(); + String name = e.nextElement(); PublisherPlugin value = (PublisherPlugin) mProcessor.getPublisherPlugins().get(name); // get Description @@ -2571,11 +2571,10 @@ public class PublisherAdminServlet extends AdminServlet { IOException, EBaseException { NameValuePairs params = new NameValuePairs(); - String insts = null; - Enumeration e = mProcessor.getPublisherInsts().keys(); + Enumeration<String> e = mProcessor.getPublisherInsts().keys(); for (; e.hasMoreElements();) { - String name = (String) e.nextElement(); + String name = e.nextElement(); ILdapPublisher value = mProcessor.getPublisherInstance(name); if (value == null) @@ -2613,8 +2612,8 @@ public class PublisherAdminServlet extends AdminServlet { // first check if any instances from this publisher // DON'T remove publisher if any instance - for (Enumeration e = mProcessor.getPublisherInsts().keys(); e.hasMoreElements();) { - String name = (String) e.nextElement(); + for (Enumeration<String> e = mProcessor.getPublisherInsts().keys(); e.hasMoreElements();) { + String name = e.nextElement(); ILdapPublisher publisher = mProcessor.getPublisherInstance(name); @@ -2722,7 +2721,7 @@ public class PublisherAdminServlet extends AdminServlet { return; } - Vector configParams = mProcessor.getPublisherDefaultParams(implname); + Vector<String> configParams = mProcessor.getPublisherDefaultParams(implname); NameValuePairs params = new NameValuePairs(); // implName is always required so always send it. @@ -2769,7 +2768,7 @@ public class PublisherAdminServlet extends AdminServlet { ILdapPublisher publisherInst = (ILdapPublisher) mProcessor.getPublisherInstance(id); - Vector configParams = publisherInst.getInstanceParams(); + Vector<String> configParams = publisherInst.getInstanceParams(); NameValuePairs params = new NameValuePairs(); params.add(Constants.PR_PUBLISHER_IMPL_NAME, @@ -2844,7 +2843,7 @@ public class PublisherAdminServlet extends AdminServlet { // save old instance substore params in case new one fails. ILdapPublisher oldinst = mProcessor.getPublisherInstance(id); - Vector oldConfigParms = oldinst.getInstanceParams(); + Vector<String> oldConfigParms = oldinst.getInstanceParams(); NameValuePairs saveParams = new NameValuePairs(); String pubType = ""; @@ -2886,7 +2885,7 @@ public class PublisherAdminServlet extends AdminServlet { // create new substore. - Vector configParams = mProcessor.getPublisherInstanceParams(id); + Vector<String> configParams = mProcessor.getPublisherInstanceParams(id); instancesConfig.removeSubStore(id); @@ -3107,10 +3106,10 @@ public class PublisherAdminServlet extends AdminServlet { store.removeSubStore(id); IConfigStore rstore = store.makeSubStore(id); - Enumeration keys = saveParams.getNames(); + Enumeration<String> keys = saveParams.getNames(); while (keys.hasMoreElements()) { - String key = (String) keys.nextElement(); + String key = keys.nextElement(); String value = saveParams.getValue(key); if (value != null) 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 cbabe1fd0..1adde0891 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 @@ -185,10 +185,11 @@ public class RAAdminServlet extends AdminServlet { HttpServletResponse resp, IConfigStore rc) throws ServletException, IOException, EBaseException { NameValuePairs params = new NameValuePairs(); - Enumeration e = req.getParameterNames(); + @SuppressWarnings("unchecked") + Enumeration<String> e = req.getParameterNames(); while (e.hasMoreElements()) { - String name = (String) e.nextElement(); + String name = e.nextElement(); if (name.equals(Constants.OP_TYPE)) continue; @@ -250,10 +251,11 @@ public class RAAdminServlet extends AdminServlet { IConfigStore riq = nc.getSubStore(IRegistrationAuthority.PROP_REQ_IN_Q_SUBSTORE); - Enumeration e = req.getParameterNames(); + @SuppressWarnings("unchecked") + Enumeration<String> e = req.getParameterNames(); while (e.hasMoreElements()) { - String name = (String) e.nextElement(); + String name = e.nextElement(); if (name.equals(Constants.OP_TYPE)) continue; @@ -285,10 +287,11 @@ public class RAAdminServlet extends AdminServlet { IConfigStore riq = nc.getSubStore(IRegistrationAuthority.PROP_REQ_IN_Q_SUBSTORE); //set rest of the parameters - Enumeration e = req.getParameterNames(); + @SuppressWarnings("unchecked") + Enumeration<String> e = req.getParameterNames(); while (e.hasMoreElements()) { - String name = (String) e.nextElement(); + String name = e.nextElement(); if (name.equals(Constants.OP_TYPE)) continue; @@ -322,10 +325,11 @@ public class RAAdminServlet extends AdminServlet { HttpServletResponse resp, IConfigStore rc, IRequestListener thisListener) throws ServletException, IOException, EBaseException { //set rest of the parameters - Enumeration e = req.getParameterNames(); + @SuppressWarnings("unchecked") + Enumeration<String> e = req.getParameterNames(); while (e.hasMoreElements()) { - String name = (String) e.nextElement(); + String name = e.nextElement(); if (name.equals(Constants.OP_TYPE)) continue; @@ -404,12 +408,13 @@ public class RAAdminServlet extends AdminServlet { } */ - Enumeration enum1 = req.getParameterNames(); + @SuppressWarnings("unchecked") + Enumeration<String> enum1 = req.getParameterNames(); NameValuePairs params = new NameValuePairs(); if (caConnectorConfig != null) { while (enum1.hasMoreElements()) { - String name = (String) enum1.nextElement(); + String name = enum1.nextElement(); if (name.equals(Constants.RS_ID)) continue; @@ -441,11 +446,12 @@ public class RAAdminServlet extends AdminServlet { caConnectorConfig = connectorConfig.getSubStore("KRA"); } - Enumeration enum1 = req.getParameterNames(); + @SuppressWarnings("unchecked") + Enumeration<String> enum1 = req.getParameterNames(); if (caConnectorConfig != null) { while (enum1.hasMoreElements()) { - String name = (String) enum1.nextElement(); + String name = enum1.nextElement(); if (name.equals(Constants.OP_TYPE)) continue; @@ -469,10 +475,11 @@ public class RAAdminServlet extends AdminServlet { private boolean isCAConnector(HttpServletRequest req) { - Enumeration enum1 = req.getParameterNames(); + @SuppressWarnings("unchecked") + Enumeration<String> enum1 = req.getParameterNames(); while (enum1.hasMoreElements()) { - String key = (String) enum1.nextElement(); + String key = enum1.nextElement(); if (key.equals("RS_ID")) { String val = req.getParameter(key); @@ -488,10 +495,11 @@ public class RAAdminServlet extends AdminServlet { private boolean isRAConnector(HttpServletRequest req) { - Enumeration enum1 = req.getParameterNames(); + @SuppressWarnings("unchecked") + Enumeration<String> enum1 = req.getParameterNames(); while (enum1.hasMoreElements()) { - String key = (String) enum1.nextElement(); + String key = enum1.nextElement(); if (key.equals("RS_ID")) { String val = req.getParameter(key); @@ -507,10 +515,11 @@ public class RAAdminServlet extends AdminServlet { private boolean isKRAConnector(HttpServletRequest req) { - Enumeration enum1 = req.getParameterNames(); + @SuppressWarnings("unchecked") + Enumeration<String> enum1 = req.getParameterNames(); while (enum1.hasMoreElements()) { - String key = (String) enum1.nextElement(); + String key = enum1.nextElement(); if (key.equals("RS_ID")) { String val = req.getParameter(key); 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 8481fffe1..6eaf004f9 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 @@ -90,8 +90,6 @@ public class UsrGrpAdminServlet extends AdminServlet { private IUGSubsystem mMgr = null; - private IAuthzSubsystem mAuthz = null; - private static String[] mMultiRoleGroupEnforceList = null; private final static String MULTI_ROLE_ENABLE = "multiroles.enable"; private final static String MULTI_ROLE_ENFORCE_GROUP_LIST = "multiroles.false.groupEnforceList"; @@ -345,7 +343,7 @@ public class UsrGrpAdminServlet extends AdminServlet { NameValuePairs params = new NameValuePairs(); - Enumeration e = null; + Enumeration<IUser> e = null; try { e = mMgr.listUsers("*"); @@ -359,7 +357,7 @@ public class UsrGrpAdminServlet extends AdminServlet { int i = 0; while (e.hasMoreElements()) { - IUser user = (IUser) e.nextElement(); + IUser user = e.nextElement(); if (i > 0) { sb.append(";"); @@ -424,7 +422,7 @@ public class UsrGrpAdminServlet extends AdminServlet { // get list of groups, and get a list of those that this // uid belongs to - Enumeration e = null; + Enumeration<IGroup> e = null; try { e = mMgr.findGroups("*"); @@ -438,7 +436,7 @@ public class UsrGrpAdminServlet extends AdminServlet { StringBuffer grpString = new StringBuffer(); while (e.hasMoreElements()) { - IGroup group = (IGroup) e.nextElement(); + IGroup group = e.nextElement(); if (group.isMember(id) == true) { if (grpString.length() != 0) { @@ -553,7 +551,7 @@ public class UsrGrpAdminServlet extends AdminServlet { IOException, EBaseException { NameValuePairs params = new NameValuePairs(); - Enumeration e = null; + Enumeration<IGroup> e = null; try { e = mMgr.listGroups("*"); @@ -564,7 +562,7 @@ public class UsrGrpAdminServlet extends AdminServlet { } while (e.hasMoreElements()) { - IGroup group = (IGroup) e.nextElement(); + IGroup group = e.nextElement(); String desc = group.getDescription(); if (desc != null) { @@ -600,7 +598,7 @@ public class UsrGrpAdminServlet extends AdminServlet { return; } - Enumeration e = null; + Enumeration<IGroup> e = null; try { e = mMgr.findGroups(id); @@ -611,13 +609,13 @@ public class UsrGrpAdminServlet extends AdminServlet { } if (e.hasMoreElements()) { - IGroup group = (IGroup) e.nextElement(); + IGroup group = e.nextElement(); params.add(Constants.PR_GROUP_GROUP, group.getGroupID()); params.add(Constants.PR_GROUP_DESC, group.getDescription()); - Enumeration members = group.getMemberNames(); + Enumeration<String> members = group.getMemberNames(); StringBuffer membersString = new StringBuffer(); if (members != null) { @@ -626,7 +624,7 @@ public class UsrGrpAdminServlet extends AdminServlet { membersString.append(", "); } - String mn = (String) members.nextElement(); + String mn = members.nextElement(); membersString.append(mn); } @@ -814,7 +812,7 @@ public class UsrGrpAdminServlet extends AdminServlet { Constants.PR_USER_GROUP); if (groupName != null) { - Enumeration e = null; + Enumeration<IGroup> e = null; try { e = mMgr.findGroups(groupName); @@ -836,7 +834,7 @@ public class UsrGrpAdminServlet extends AdminServlet { } if (e.hasMoreElements()) { - IGroup group = (IGroup) e.nextElement(); + IGroup group = e.nextElement(); group.addMemberName(id); try { @@ -902,8 +900,6 @@ public class UsrGrpAdminServlet extends AdminServlet { } return; } catch (LDAPException e) { - String errMsg = "addUser()" + e.toString(); - log(ILogger.LL_FAILURE, CMS.getLogMessage("ADMIN_SRVLT_ADD_USER_FAIL", e.toString())); // store a message in the signed audit log file @@ -1530,7 +1526,7 @@ public class UsrGrpAdminServlet extends AdminServlet { return; } // get list of groups, and see if uid belongs to any - Enumeration e = null; + Enumeration<IGroup> e = null; try { e = mMgr.findGroups("*"); @@ -2090,7 +2086,7 @@ public class UsrGrpAdminServlet extends AdminServlet { } private boolean isDuplicate(String groupName, String memberName) { - Enumeration groups = null; + Enumeration<IGroup> groups = null; // Let's not mess with users that are already a member of this group boolean isMember = false; @@ -2105,15 +2101,15 @@ public class UsrGrpAdminServlet extends AdminServlet { try { groups = mMgr.listGroups("*"); while (groups.hasMoreElements()) { - IGroup group = (IGroup) groups.nextElement(); + IGroup group = groups.nextElement(); String name = group.getName(); - Enumeration g = mMgr.findGroups(name); - IGroup g1 = (IGroup) g.nextElement(); + Enumeration<IGroup> g = mMgr.findGroups(name); + IGroup g1 = g.nextElement(); if (!name.equals(groupName)) { if (isGroupInMultiRoleEnforceList(name)) { - Enumeration members = g1.getMemberNames(); + Enumeration<String> members = g1.getMemberNames(); while (members.hasMoreElements()) { - String m1 = (String) members.nextElement(); + String m1 = members.nextElement(); if (m1.equals(memberName)) return true; } |