Changeset 13124 for trunk/gsdl3/src/java/org/greenstone/gsdl3/service/GS2Construct.java

Timestamp:

2006-10-17T15:52:42+13:00 (18 years ago)

Author:

shaoqun

Message:

added the code to use log4j and replace system.out or system.err to logger.info or logger.error respectively. The commented out output has been replaced with logger.debug. The log file is in web/logs/greenstone.log

File:

• trunk/gsdl3/src/java/org/greenstone/gsdl3/service/GS2Construct.java (modified) (9 diffs)

trunk/gsdl3/src/java/org/greenstone/gsdl3/service/GS2Construct.java

@@ -36 +36 @@
 import java.util.Locale;
 
+import org.apache.log4j.*;
+
 /**
  * A Services class for building collections
@@ -46 +48 @@
     extends ServiceRack { 
     
+     static Category logger = Category.getInstance(org.greenstone.gsdl3.service.GS2Construct.class.getName()); 
+
     // services offered
     private static final String NEW_SERVICE = "NewCollection";
@@ -166 +170 @@
     GS2PerlListener listener = (GS2PerlListener)this.listeners.get(id);
     if (listener==null) {
-        System.err.println("somethings gone wrong, couldn't find the listener");
+        logger.error("somethings gone wrong, couldn't find the listener");
         status.setAttribute(GSXML.STATUS_ERROR_CODE_ATT, Integer.toString(GSStatus.ERROR));
         return response;
@@ -217 +221 @@
     if (get_status_only) {
         // at the moment, delete is synchronous. but it may take ages so should do the command in another thread maybe? in which case we will want to ask for status
-        System.err.println("had a status request for delete - this shouldn't happen!!");
+        logger.error("had a status request for delete - this shouldn't happen!!");
         //t = this.doc.createTextNode("");
         //status.appendChild(t);
@@ -267 +271 @@
     if (get_status_only) {
         // reload is synchronous - this makes no sense
-        System.err.println("had a status request for reload - this shouldn't happen!!");
+        logger.error("had a status request for reload - this shouldn't happen!!");
         //t = this.doc.createTextNode("");
         //status.appendChild(t);
@@ -300 +304 @@
         command.setAttribute(GSXML.TYPE_ATT, GSXML.SYSTEM_TYPE_ACTIVATE);
     } else {
-        System.err.println("invalid action name passed to systemRequest:"+action_name);
+        logger.error("invalid action name passed to systemRequest:"+action_name);
         return;
     }
@@ -330 +334 @@
     }
 
-    System.out.println("configuring GS2Construct");
+    logger.info("configuring GS2Construct");
 
     Element e = null;
@@ -427 +431 @@
         coll_name = (String)params.get(COL_PARAM);
     }
-    System.err.println("Coll name = "+coll_name);
+    logger.error("Coll name = "+coll_name);
     // makes a paramList of the relevant params
     Element other_params = extractOtherParams(params, type);
@@ -471 +475 @@
     File collectDir = new File(GSFile.collectDir(this.site_home));
     if (!collectDir.exists()) {
-        System.err.println("couldn't find collect dir: "+collectDir.toString());
+        logger.error("couldn't find collect dir: "+collectDir.toString());
         return null;
     }
-    System.out.println("GS2Construct: reading thru directory "+collectDir.getPath()+" to find collections.");
+    logger.info("GS2Construct: reading thru directory "+collectDir.getPath()+" to find collections.");
     File[] contents = collectDir.listFiles();
     int num_colls=0;