X-Git-Url: http://git.indexdata.com/?a=blobdiff_plain;f=src%2Fmain%2Fjava%2Fcom%2Findexdata%2Fmkjsf%2Fpazpar2%2Fcommands%2FRecordCommand.java;h=8991dde1e6dae6f3b7b50119d08906e50de19ad2;hb=b31cb1a9589fc04252c5bbfa11a0f6093f2f8ec6;hp=c163abad7b4253a7bb5940a9be62a73064524cea;hpb=9cd922c29f423b69da6f52eabf0c7fdb0b0b6ca5;p=mkjsf-moved-to-github.git

diff --git a/src/main/java/com/indexdata/mkjsf/pazpar2/commands/RecordCommand.java b/src/main/java/com/indexdata/mkjsf/pazpar2/commands/RecordCommand.java
index c163aba..8991dde 100644
--- a/src/main/java/com/indexdata/mkjsf/pazpar2/commands/RecordCommand.java
+++ b/src/main/java/com/indexdata/mkjsf/pazpar2/commands/RecordCommand.java
@@ -11,15 +11,30 @@ import com.indexdata.mkjsf.pazpar2.data.RecordResponse;
 import com.indexdata.mkjsf.pazpar2.data.ResponseDataObject;
 import com.indexdata.mkjsf.pazpar2.data.ResponseParser;
 
+/**
+ * Represents a Pazpar2 <code>record</code> command, can be accessed by <code>pzreq.record</code>
+ * 
+ * @author Niels Erik
+ *
+ */
 public class RecordCommand extends Pazpar2Command implements ServiceProxyCommand {
 
   private static final long serialVersionUID = 2817539422114569506L;
   private static Logger logger = Logger.getLogger(RecordCommand.class);
+  private RecordCommandSp spCommand = null;
 
   public RecordCommand() {
     super("record");
   }
   
+  /**
+   * Special handling of record responses since they come in three distinctly different ways
+   * <ol>
+   *  <li>As a regular &lt;record&gt; document</li>
+   *  <li>In arbitrary XML format, in case of an offset request to get the native format</li>
+   *  <li>In binary (non XML) format</li>
+   * </ol> 
+   */
   @Override
   public ResponseDataObject run() {
     ResponseDataObject responseObject = null;
@@ -30,21 +45,21 @@ public class RecordCommand extends Pazpar2Command implements ServiceProxyCommand
         responseObject = ResponseParser.getParser().getDataObject((ClientCommandResponse)commandResponse);
         if (ResponseParser.docTypes.contains(responseObject.getType())) {
           logger.debug("Storing " + responseObject.getType() + " in pzresp. ");
-        } else {        
+        } else {
           logger.debug("Command was 'record' but response not '<record>' - assuming raw record response.");
-          ResponseDataObject recordResponse = new RecordResponse(); 
+          ResponseDataObject recordResponse = new RecordResponse();
           recordResponse.setType("record");
-          recordResponse.setXml(responseObject.getXml());          
-          recordResponse.setAttribute("activeclients", "0");             
+          recordResponse.setXml(responseObject.getXml());
+          recordResponse.setAttribute("activeclients", "0");
+          responseObject = recordResponse;
         }
       } else if (commandResponse.isBinary()) {
-        responseObject = new RecordResponse(); 
+        responseObject = new RecordResponse();
         responseObject.setType(getCommandName());
         logger.info("Binary response");
         responseObject.setAttribute("activeclients", "0");
         responseObject.setXml("<record>binary response</record>");
         responseObject.setBinary(commandResponse.getBytes());
-        
       } else {
         logger.error("Response was not found to be XML or binary. The response was not handled.");
       }
@@ -56,58 +71,102 @@ public class RecordCommand extends Pazpar2Command implements ServiceProxyCommand
     return responseObject;
   }
   
+  /**
+   * Sets the <code>id</code> parameter. See Pazpar2 documentation for details.
+   * 
+   * @param recId record ID
+   */
   public void setId(String recId) {
     setParameter(new CommandParameter("id","=",recId));
   }
-  
+
+  /** 
+   * Returns the <code>id</code> parameter value.
+   */
   public String getId () {
     return getParameterValue("id");
   }
-  
+
+  /**
+   * Sets the <code>offset</code> parameter. See Pazpar2 documentation for details.
+   */  
   public void setOffset (String offset) {
     setParameter(new CommandParameter("offset","=",offset));
   }
-  
+
+  /** 
+   * Returns the <code>offset</code> parameter value.
+   */
   public String getOffset () {
     return getParameterValue("offset");
   }
   
+  /**
+   * Sets the <code>checksum</code> parameter. See Pazpar2 documentation for details.
+   */  
   public void setChecksum (String checksum) {
     setParameter(new CommandParameter("checksum","=",checksum));
   }
   
+  /** 
+   * Returns the <code>checksum</code> parameter value.
+   */
   public String getChecksum () {
     return getParameterValue("checksum");
   }
   
+  /**
+   * Sets the <code>nativesyntax</code> parameter. See Pazpar2 documentation for details.
+   */    
   public void setNativesyntax (String nativesyntax) {
     setParameterInState(new CommandParameter("nativesyntax","=",nativesyntax));
   }
   
+  /** 
+   * Returns the <code>nativesyntax</code> parameter value.
+   */
   public String getNativesyntax () {
     return getParameterValue("nativesyntax");
   }
   
+  /**
+   * Sets the <code>syntax</code> parameter. See Pazpar2 documentation for details.
+   */  
   public void setSyntax (String syntax) {
     setParameterInState(new CommandParameter("syntax","=",syntax));    
   }
   
+  /** 
+   * Returns the <code>syntax</code> parameter value.
+   */
   public String getSyntax () {
     return getParameterValue("syntax");
   }
   
+  /**
+   * Sets the <code>esn</code> parameter. See Pazpar2 documentation for details.
+   */  
   public void setEsn (String esn) {
     setParameter(new CommandParameter("esn","=",esn));
   }
   
+  /** 
+   * Returns the <code>esn</code> parameter value.
+   */
   public String getEsn () {
     return getParameterValue("esn");
   }
   
+  /**
+   * Sets the <code>binary</code> parameter. See Pazpar2 documentation for details.
+   */  
   public void setBinary (String binary) {
     setParameter(new CommandParameter("binary","=",binary));
   }
   
+  /** 
+   * Returns the <code>binary</code> parameter value.
+   */
   public String getBinary () {
     return getParameterValue("binary");
   }
@@ -118,6 +177,7 @@ public class RecordCommand extends Pazpar2Command implements ServiceProxyCommand
     for (String parameterName : parameters.keySet()) {
       newCommand.setParameterInState(parameters.get(parameterName).copy());      
     }    
+    newCommand.spCommand = new RecordCommandSp(newCommand);
     return newCommand;
   }
   
@@ -127,7 +187,10 @@ public class RecordCommand extends Pazpar2Command implements ServiceProxyCommand
    * 
    */
   public RecordCommandSp getSp () {
-    return new RecordCommandSp(this);
+    if (spCommand==null) {
+      spCommand = new RecordCommandSp(this);
+    } 
+    return spCommand;
   }
 
   @Override