X-Git-Url: http://jsfdemo.indexdata.com/?a=blobdiff_plain;f=src%2Fmain%2Fjava%2Fcom%2Findexdata%2Fmkjsf%2Fpazpar2%2FCommandThread.java;h=678e4a65011be986c9e5c5eafc4ed0c96d5254ac;hb=74ea81b22375676bf687d614161c19f268289e39;hp=ce3df290cbe15cce5a4909c8eb0b2887b8ca3a36;hpb=ccb28ae8d5d46d29c40bd8b1637522c212b80636;p=mkjsf-moved-to-github.git

diff --git a/src/main/java/com/indexdata/mkjsf/pazpar2/CommandThread.java b/src/main/java/com/indexdata/mkjsf/pazpar2/CommandThread.java
index ce3df29..678e4a6 100644
--- a/src/main/java/com/indexdata/mkjsf/pazpar2/CommandThread.java
+++ b/src/main/java/com/indexdata/mkjsf/pazpar2/CommandThread.java
@@ -1,75 +1,45 @@
 package com.indexdata.mkjsf.pazpar2;
 
-import java.io.ByteArrayOutputStream;
-import java.io.IOException;
-
 import org.apache.log4j.Logger;
 
-import com.indexdata.masterkey.pazpar2.client.exceptions.Pazpar2ErrorException;
 import com.indexdata.mkjsf.pazpar2.commands.Pazpar2Command;
-import com.indexdata.mkjsf.pazpar2.data.CommandError;
+import com.indexdata.mkjsf.pazpar2.data.Responses;
 
+/**
+ * Helper class for running multiple concurrent Pazpar2 commands. Basically 
+ * used for updating display data (show,stat,bytarget,termlist) together.
+ * 
+ * @author Niels Erik
+ *
+ */
 public class CommandThread extends Thread {
 
   private static Logger logger = Logger.getLogger(CommandThread.class);
   Pazpar2Command command;
   SearchClient client;
-  private ByteArrayOutputStream baos = new ByteArrayOutputStream();
-  private StringBuilder response = new StringBuilder("");  
+  Responses pzresp;
+  HttpResponseWrapper commandResponse = null;      
   
-  public CommandThread (Pazpar2Command command, SearchClient client) {
+  public CommandThread (Pazpar2Command command, SearchClient client, Responses pzresp) {
     this.command = command;
     this.client = client;
+    this.pzresp = pzresp;
   }
   
   /**
-   * Runs the specified command using the specified Pazpar2 client
-   * Sets the Pazpar2 response as an XML response string to be retrieved by
-   * getResponse().
-   * 
-   * In case of an exception, an error response is generated, the document
-   * element being the same as it would have been if successful (named after
-   * the command, that is).  
-   *  
+   * Executes the specified command using the specified Pazpar2 client
    */
-  public void run() {
-    
-    if (command.getName().equals("search")) {
-      client.setSearchCommand(command);
-    }
-    try {
-      long start = System.currentTimeMillis();
-      CommandResponse commandResponse = client.executeCommand(command, baos);
-      if (commandResponse.getStatusCode()==200) {
-        response.append(commandResponse.getResponseString());  
-      } else if (commandResponse.getStatusCode()==417) {        
-        logger.error("Pazpar2 status code 417: " + baos.toString("UTF-8"));
-        response.append(CommandError.insertPazpar2ErrorXml(command.getName(), "Expectation failed (417)", commandResponse.getResponseString()));        
-      } else {
-        String resp = baos.toString("UTF-8");
-        logger.error("Pazpar2 status code was " + commandResponse.getStatusCode() + ": " + resp);
-        throw new Pazpar2ErrorException(resp,commandResponse.getStatusCode(),resp,null);
-      }       
-      long end = System.currentTimeMillis();      
-      logger.debug("Executed " + command.getName() + " in " + (end-start) + " ms." );
-    } catch (IOException e) {
-      response.append(CommandError.createErrorXml(command.getName(), "io", e.getMessage())); 
-      logger.error(response.toString());
-    } catch (Pazpar2ErrorException e) {
-      response.append(CommandError.createErrorXml(command.getName(), "pazpar2error", e.getMessage())); 
-      logger.error(response.toString());
-    } catch (Exception e) {
-      response.append(CommandError.createErrorXml(command.getName(), "general", e.getMessage())); 
-      logger.error(response.toString());      
-    }
+  public void run() {    
+    logger.debug(command.getCommandName() + " executing asynchronously");
+    command.run(client,pzresp);
   }
   
   /**
    * 
    * @return Pazpar2 response as an XML string, possibly a generated error XML
    */
-  public String getResponse () {
-    return response.toString();
+  public HttpResponseWrapper getCommandResponse () {
+    return commandResponse;
   }
     
   public Pazpar2Command getCommand() {