Changeset 20294 for greenstone3/trunk/src/java/org/greenstone/gsdl3/service/GoogleSearch.java.tmp

Timestamp:

2009-08-17T10:24:54+12:00 (15 years ago)

Author:

kjdon

Message:

changed System.err to logger.error/info

File:

• greenstone3/trunk/src/java/org/greenstone/gsdl3/service/GoogleSearch.java.tmp (modified) (10 diffs)

greenstone3/trunk/src/java/org/greenstone/gsdl3/service/GoogleSearch.java.tmp

@@ -32 +32 @@
 import java.lang.Object;
 
+import org.apache.log4j.*;
 
 /**
@@ -42 +43 @@
     extends AbstractSearch {
     
+     static Logger logger = Logger.getLogger(org.greenstone.gsdl3.service.GoogleSearch.class.getName());
+     
     //Parameters connect to Proxy Server
     private boolean using_proxy = false;
@@ -62 +65 @@
         return false;
     }
-    System.err.println("Configuring GoogleSearch");
+    logger.info("Configuring GoogleSearch");
     Element server_elem = (Element)GSXML.getChildByTagName(info, "googleServer");
     if (server_elem == null) {
-        System.err.println("GoogleSearch.configure error: no googleServer element found");
+        logger.error("no googleServer element found");
         return false;
     }
@@ -71 +74 @@
     client_key = server_elem.getAttribute("key");
     if (client_key.equals("")) {
-        System.err.println("GoogleSearch.configure error: no client_key for the googleServer element");
+        logger.error("no client_key for the googleServer element");
         return false;
     }
@@ -86 +89 @@
         proxy_port = Integer.parseInt(System.getProperty("http.proxyPort").trim());
         } catch (Exception e) {
-        System.err.println("GoogleSearch.configure error: couldn't get proxy port, defaulting to 80");
+        logger.error("couldn't get proxy port, defaulting to 80");
         proxy_port = 80;
         }
@@ -127 +130 @@
     Element param_list = (Element) GSXML.getChildByTagName(request, GSXML.PARAM_ELEM+GSXML.LIST_MODIFIER);
     if (param_list == null) {
-        System.err.println("GoogleSearch.processTextQuery Error:: TextQuery request had no paramList.");
+        logger.error("TextQuery request had no paramList.");
         return result;  // Return the empty result
     }
@@ -137 +140 @@
     String query = (String) params.get(QUERY_PARAM);
     if (query == null || query.equals("")) {
-        System.err.println("GoogleSearch.processTextQuery Error: TextQuery request had no query string.");
+        logger.error("TextQuery request had no query string.");
         return result;  // Return the empty result
     }
@@ -150 +153 @@
         hits_per_page = Integer.parseInt(((String)params.get(HITS_PER_PAGE_PARAM)).trim());
     } catch (Exception e) {
-        System.err.println("GoogleSearch.processTextQuery error: couldn't get hits per page param, defaulting to 10");
+        logger.error("couldn't get hits per page param, defaulting to 10");
         hits_per_page = 10;
     }
@@ -158 +161 @@
         start_page = Integer.parseInt(((String) params.get(START_PAGE_PARAM)).trim());
     } catch (Exception e) {
-        System.err.println("GoogleSearch.processTextQuery error: couldn't get start page param, defaulting to 1");
+        logger.error("couldn't get start page param, defaulting to 1");
         start_page = 1;
     }
@@ -182 +185 @@
         google_result = search.doSearch();
         } catch (GoogleSearchFault ex) {
-        System.err.println("GoogleSearch.processTextQuery error: the call to the Google Web APIs failed:" + ex.toString());
+        logger.error("the call to the Google Web APIs failed:" + ex.toString());
         // add the error to the result
         return result;