[Fedora-directory-commits] console/src/com/netscape/management/client/security CertInstallCertPage.java, 1.1.1.1, 1.2 CertRequestWizard.java, 1.2, 1.3 ClientAuthPanel.java, 1.1.1.1, 1.2 EncryptionPanel.java, 1.1.1.1, 1.2

Nathan Kinder (nkinder) fedora-directory-commits at redhat.com
Wed Feb 8 22:13:26 UTC 2006


Author: nkinder

Update of /cvs/dirsec/console/src/com/netscape/management/client/security
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv22640/src/com/netscape/management/client/security

Modified Files:
	CertInstallCertPage.java CertRequestWizard.java 
	ClientAuthPanel.java EncryptionPanel.java 
Log Message:
Bug(s) fixed: 180544
Bug Description: Building Console fails with Java 1.5.  This is because "enum" has been added as
   a reserved keyword in Java 1.5, and we use it as a variable name.

   There is also a warning issued during build time about our usage of
   Method.invoke().  We are passing a "null" as the second parameter, but invoke()
   is overloaded and the compiler has to guess about which method to use.
Reviewed by: Rich
Files: see diffs
Branch: HEAD
Fix Description: Renamed our "enum" variable names.  Added a cast to our usage of Method.invoke()
   to avoid a compiler warning.
Platforms tested: RHEL4
Flag Day: no
Doc impact: no
QA impact: should be covered by regular nightly and manual testing
New Tests integrated into TET: none



Index: CertInstallCertPage.java
===================================================================
RCS file: /cvs/dirsec/console/src/com/netscape/management/client/security/CertInstallCertPage.java,v
retrieving revision 1.1.1.1
retrieving revision 1.2
diff -u -r1.1.1.1 -r1.2
--- CertInstallCertPage.java	18 Jul 2005 00:34:15 -0000	1.1.1.1
+++ CertInstallCertPage.java	8 Feb 2006 22:13:18 -0000	1.2
@@ -282,9 +282,9 @@
                                                    );
 
                         Debug.println("Restore session data");
-                        Enumeration enum = pref.getNames();
-                        while (enum.hasMoreElements()) {
-                            String name = enum.nextElement().toString();
+                        Enumeration names_enum = pref.getNames();
+                        while (names_enum.hasMoreElements()) {
+                            String name = names_enum.nextElement().toString();
                             ic.setProperty(name, pref.getString(name));
                         }
 


Index: CertRequestWizard.java
===================================================================
RCS file: /cvs/dirsec/console/src/com/netscape/management/client/security/CertRequestWizard.java,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -r1.2 -r1.3
--- CertRequestWizard.java	29 Nov 2005 18:37:06 -0000	1.2
+++ CertRequestWizard.java	8 Feb 2006 22:13:18 -0000	1.3
@@ -214,9 +214,9 @@
 									   "cn=CSRSession,"+consoleInfo.getCurrentDN()
 									   );
 
-				Enumeration enum = ic.getPropertyNames();
-				while (enum.hasMoreElements()) {
-				    String attr = enum.nextElement().toString();
+				Enumeration names_enum = ic.getPropertyNames();
+				while (names_enum.hasMoreElements()) {
+				    String attr = names_enum.nextElement().toString();
 				    pref.set(attr, ic.getProperty(attr));
 				}
 


Index: ClientAuthPanel.java
===================================================================
RCS file: /cvs/dirsec/console/src/com/netscape/management/client/security/ClientAuthPanel.java,v
retrieving revision 1.1.1.1
retrieving revision 1.2
diff -u -r1.1.1.1 -r1.2
--- ClientAuthPanel.java	18 Jul 2005 00:34:16 -0000	1.1.1.1
+++ ClientAuthPanel.java	8 Feb 2006 22:13:18 -0000	1.2
@@ -113,9 +113,9 @@
      * @param enable true to enable all buttons, false to disable and gray out all buttons
      */
     public void setEnabled(boolean enable) {
-	Enumeration enum = buttonGroup.getElements();
-	while (enum.hasMoreElements()) {
-	    ((JRadioButton)(enum.nextElement())).setEnabled(enable);
+	Enumeration button_enum = buttonGroup.getElements();
+	while (button_enum.hasMoreElements()) {
+	    ((JRadioButton)(button_enum.nextElement())).setEnabled(enable);
 	}
     }
 


Index: EncryptionPanel.java
===================================================================
RCS file: /cvs/dirsec/console/src/com/netscape/management/client/security/EncryptionPanel.java,v
retrieving revision 1.1.1.1
retrieving revision 1.2
diff -u -r1.1.1.1 -r1.2
--- EncryptionPanel.java	18 Jul 2005 00:34:17 -0000	1.1.1.1
+++ EncryptionPanel.java	8 Feb 2006 22:13:18 -0000	1.2
@@ -308,10 +308,10 @@
 			cert.setActionCommand("EVENT_HANDLE_DISABLED");
 			cert.removeAllItems();
 			Hashtable selectedDev = (Hashtable)(_secDevice.get((String)(device.getSelectedItem())));
-			Enumeration enum = selectedDev.keys();
+			Enumeration device_enum = selectedDev.keys();
 			DefaultComboBoxModel comboModel = (DefaultComboBoxModel) cert.getModel();
-			while (enum.hasMoreElements()) {
-				String key = (String)(enum.nextElement());
+			while (device_enum.hasMoreElements()) {
+				String key = (String)(device_enum.nextElement());
 				if (key.startsWith("CERT")) {
 					String certName = (String)selectedDev.get(key);
 					// Multiple certs with the same name can exit (bug 558903)




More information about the Fedora-directory-commits mailing list