Changeset 28966 for main/trunk/greenstone3/src/java/org/greenstone/gsdl3/service/UserTracker.java

Timestamp:

2014-04-10T14:39:33+12:00 (10 years ago)

Author:

kjdon

Message:

Lots of changes. Mainly to do with removing this.doc from everywhere. Document is not thread safe. Now we tend to create a new Document everytime we are starting a new page/message etc. in service this.desc_doc is available as teh document to create service info stuff. But it should only be used for this and not for other messages. newDOM is now static for XMLConverter. method param changes for some GSXML methods.

File:

• main/trunk/greenstone3/src/java/org/greenstone/gsdl3/service/UserTracker.java (modified) (8 diffs)

main/trunk/greenstone3/src/java/org/greenstone/gsdl3/service/UserTracker.java

@@ -9 +9 @@
 import org.greenstone.gsdl3.util.DerbyWrapper;
 import org.greenstone.gsdl3.util.GSXML;
+import org.greenstone.gsdl3.util.XMLConverter;
 import org.greenstone.util.GlobalProperties;
+import org.w3c.dom.Document;
 import org.w3c.dom.Element;
 
@@ -38 +40 @@
         for (int i = 0; i < services.length; i++)
         {
-            Element service = this.doc.createElement(GSXML.SERVICE_ELEM);
+            Element service = this.desc_doc.createElement(GSXML.SERVICE_ELEM);
             service.setAttribute(GSXML.TYPE_ATT, GSXML.SERVICE_TYPE_RETRIEVE);
             service.setAttribute(GSXML.NAME_ATT, services[i]);
@@ -47 +49 @@
     }
 
-    protected Element getServiceDescription(String service_id, String lang, String subset)
+    protected Element getServiceDescription(Document doc, String service_id, String lang, String subset)
     {
         for (int i = 0; i < services.length; i++)
@@ -53 +55 @@
             if (service_id.equals(services[i]))
             {
-                Element service_elem = this.doc.createElement(GSXML.SERVICE_ELEM);
+                Element service_elem = doc.createElement(GSXML.SERVICE_ELEM);
                 service_elem.setAttribute(GSXML.TYPE_ATT, GSXML.SERVICE_TYPE_RETRIEVE);
                 service_elem.setAttribute(GSXML.NAME_ATT, services[i]);
@@ -65 +67 @@
     protected synchronized Element processRecordUserAction(Element request)
     {
-        Element result = GSXML.createBasicResponse(this.doc, RECORD_USER_ACTION);
+      Document result_doc = XMLConverter.newDOM();
+        Element result = GSXML.createBasicResponse(result_doc, RECORD_USER_ACTION);
 
         Element paramList = (Element) GSXML.getChildByTagName(request, GSXML.PARAM_ELEM + GSXML.LIST_MODIFIER);
         if (paramList == null)
         {
-            GSXML.addError(this.doc, result, "Request has no parameter list");
+            GSXML.addError(result, "Request has no parameter list");
             return result;
         }
@@ -90 +93 @@
     protected synchronized Element processGetActivityOnPage(Element request)
     {
-        Element result = GSXML.createBasicResponse(this.doc, GET_ACTIVITY_ON_PAGE);
+      Document result_doc = XMLConverter.newDOM();
+        Element result = GSXML.createBasicResponse(result_doc, GET_ACTIVITY_ON_PAGE);
         try
         {
@@ -97 +101 @@
             if (paramList == null)
             {
-                GSXML.addError(this.doc, result, "Request has no parameter list");
+                GSXML.addError(result, "Request has no parameter list");
                 return result;
             }
@@ -109 +113 @@
             ArrayList<HashMap<String, String>> userActions = database.getMostRecentUserActions(site, collection, oid);
 
-            Element userList = this.doc.createElement("userList");
+            Element userList = result_doc.createElement("userList");
             for (HashMap<String, String> userAction : userActions)
             {
-                Element user = this.doc.createElement("user");
+                Element user = result_doc.createElement("user");
                 user.setAttribute("username", userAction.get("username"));
                 user.setAttribute("action", userAction.get("action"));