Changeset 1607 for branches/3.0-Beta2/components/tools/WebAdmin/test/TestChangeGroups.java

Timestamp:

06/08/07 20:15:22 (18 months ago)

Author:

david

Message:

code cleanup ticket #383

Files:

• branches/3.0-Beta2/components/tools/WebAdmin/test/TestChangeGroups.java (modified) (5 diffs)

branches/3.0-Beta2/components/tools/WebAdmin/test/TestChangeGroups.java

@@ -1 +1 @@
 import java.util.ArrayList;
 import java.util.List;
+
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 
 import ome.api.IAdmin;
@@ -14 +17 @@
 
 public class TestChangeGroups {
+    
+    private static Log log = LogFactory.getLog(TestChangeGroups.class);
 
         /**
@@ -19 +24 @@
          */
         public static void main(String[] args) {
-                // TODO Auto-generated method stub
               
         Login l = new Login("root", "ome", "system", "User");
@@ -32 +36 @@
         Experimenter [] exps = as.containedExperimenters(group.getId());
        
-        System.out.println("group "+group.getName());
+        if (log.isDebugEnabled()) {
+            log.debug("group "+group.getName());
+        }
         for (int i=0; i<exps.length; i++) {
                 Experimenter tExp = exps[i];
-                System.out.println("ex "+tExp.getOmeName()+" id: "+tExp.getId());
+            if (log.isDebugEnabled()) {
+                log.debug("ex "+tExp.getOmeName()+" id: "+tExp.getId());
+            }
         }
-        
-        System.out.println("add new user to this group... ");
+        if (log.isDebugEnabled()) {
+            log.debug("add new user to this group... ");
+        }
         
         List<String> formExps = new ArrayList<String>();
@@ -49 +58 @@
                 List<String> exGrL = new ArrayList<String>();
                 for(int j=0; j<exGr.length; j++) {
-                        System.out.println("existing gr '"+exGr[j].getName()+"' id:"+exGr[j].getId());
+                if (log.isDebugEnabled()) {
+                    log.debug("existing gr '"+exGr[j].getName()+"' id:"+exGr[j].getId());
+                }
                         exGrL.add(exGr[j].getId().toString());
                 }
-                
-                System.out.println("if contains... " + formExps.get(i));
+            if (log.isDebugEnabled()) {
+                log.debug("if contains... " + formExps.get(i));
+            }
                 if(!exGrL.contains(group.getId().toString())) {
                         Experimenter tExp = as.getExperimenter(Long.parseLong(formExps.get(i)));
-                        System.out.println("exp can be add... '"+ tExp.getOmeName()+"' id:"+tExp.getId());
+                if (log.isDebugEnabled()) {
+                    log.debug("exp can be add... '"+ tExp.getOmeName()+"' id:"+tExp.getId());
+                }
                         
                         //as.addGroups(tExp, group);
                 } else {
-                        System.out.println("user exist on this group...");
+                if (log.isDebugEnabled()) {
+                    log.debug("user exist on this group...");
+                }
                 }