Clears info response when clearing init response

[mkjsf-moved-to-github.git] / src / main / java / com / indexdata / mkjsf / pazpar2 / commands / SearchCommand.java

diff --git a/src/main/java/com/indexdata/mkjsf/pazpar2/commands/SearchCommand.java b/src/main/java/com/indexdata/mkjsf/pazpar2/commands/SearchCommand.java
index dcbb323..0c8d745 100644 (file)
--- a/src/main/java/com/indexdata/mkjsf/pazpar2/commands/SearchCommand.java
+++ b/src/main/java/com/indexdata/mkjsf/pazpar2/commands/SearchCommand.java
@@ -1,9 +1,7 @@
 package com.indexdata.mkjsf.pazpar2.commands;\r
 \r
-import java.util.ArrayList;\r
 import java.util.Arrays;\r
 import java.util.List;\r
-import java.util.StringTokenizer;\r
 \r
 import javax.enterprise.context.SessionScoped;\r
 import javax.inject.Named;\r
@@ -73,25 +71,19 @@ public class SearchCommand extends Pazpar2Command implements ServiceProxyCommand
    * Sets the <code>filter</code> parameter. See Pazpar2 documentation for details.\r
    */  \r
   public void setFilter(String compoundExpression) {\r
-    if (compoundExpression != null && compoundExpression.length()>0) {\r
-      String[] subExpressions = compoundExpression.split(",");\r
+    if (getParameter("filter") != null) removeParameterInState("filter");\r
+    if (compoundExpression != null && compoundExpression.length()>0) {      \r
+      // Split expression by commas that are not escaped with backslash\r
+      String[] subExpressions = compoundExpression.split("(?<!\\\\),");\r
       for (int i=0; i<subExpressions.length; i++) {\r
-        if (subExpressions[i].split("[=~]").length==1) {\r
-          removeFilters(subExpressions[i].split("[=~]")[0]);\r
-        } else if (subExpressions[i].split("[=~]").length==2) {\r
-          if (getParameter("filter") == null) {\r
-            setParameter(new FilterParameter(new Expression(subExpressions[i])));\r
-          } else {\r
-            if (getParameter("filter").hasExpressions(subExpressions[i].split("[=~]")[0])) {\r
-              getParameter("filter").removeExpressions(subExpressions[i].split("[=~]")[0]);\r
-            }\r
-            getParameter("filter").addExpression(new Expression(subExpressions[i]));\r
-          }\r
+        if (i==0) {\r
+          setParameterInState(new FilterParameter(new Expression(subExpressions[i])));\r
         } else {\r
-          logger.error("Could not parse filter expression [" + subExpressions[i] + "]");\r
+          addExpressionInState("filter",new Expression(subExpressions[i]));\r
         }\r
       }\r
     }\r
+    checkInState(this);\r
   }\r
   \r
   /**\r
@@ -227,26 +219,20 @@ public class SearchCommand extends Pazpar2Command implements ServiceProxyCommand
   /**\r
    * Sets the <code>limit</code> parameter. See Pazpar2 documentation for details.\r
    */  \r
-  public void setLimit (String compoundExpression) {   \r
-    if (compoundExpression != null && compoundExpression.length()>0) {\r
-      String[] subExpressions = compoundExpression.split(",");\r
+  public void setLimit (String compoundExpression) {\r
+    if (getParameter("limit") != null) removeParameterInState("limit");\r
+    if (compoundExpression != null && compoundExpression.length()>0) {      \r
+      // Split expression by commas that are not escaped with backslash\r
+      String[] subExpressions = compoundExpression.split("(?<!\\\\),");\r
       for (int i=0; i<subExpressions.length; i++) {\r
-        if (subExpressions[i].split("[=~]").length==1) {\r
-          removeLimits(subExpressions[i].split("[=~]")[0]);\r
-        } else if (subExpressions[i].split("[=~]").length==2) {\r
-          if (getParameter("limit") == null) {\r
-            setParameter(new LimitParameter(new Expression(subExpressions[i])));\r
-          } else {\r
-            if (getParameter("limit").hasExpressions(subExpressions[i].split("[=~]")[0])) {\r
-              getParameter("limit").removeExpressions(subExpressions[i].split("[=~]")[0]);\r
-            }\r
-            getParameter("limit").addExpression(new Expression(subExpressions[i]));\r
-          }\r
+        if (i==0) {\r
+          setParameterInState(new LimitParameter(new Expression(subExpressions[i])));\r
         } else {\r
-          logger.error("Could not parse limit expression [" + subExpressions[i] + "]");\r
+          addExpressionInState("limit",new Expression(subExpressions[i]));\r
         }\r
       }\r
     }\r
+    checkInState(this);\r
   }\r
   \r
   /**\r