From 499264a3ac955aafd1a9c3a0f280ec9a9bcdcfd0 Mon Sep 17 00:00:00 2001
From: "Niels Erik G. Nielsen" <nielserik@indexdata.com>
Date: Tue, 5 Nov 2013 19:57:14 -0500
Subject: [PATCH] Adds some support for compound expressions (filter,limit)

---
 .../mkjsf/pazpar2/commands/SearchCommand.java      |   56 ++++++++++++++------
 1 file changed, 39 insertions(+), 17 deletions(-)

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 b8003cc..dcbb323 100644
--- a/src/main/java/com/indexdata/mkjsf/pazpar2/commands/SearchCommand.java
+++ b/src/main/java/com/indexdata/mkjsf/pazpar2/commands/SearchCommand.java
@@ -1,7 +1,9 @@
 package com.indexdata.mkjsf.pazpar2.commands;
 
+import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;
+import java.util.StringTokenizer;
 
 import javax.enterprise.context.SessionScoped;
 import javax.inject.Named;
@@ -70,14 +72,24 @@ public class SearchCommand extends Pazpar2Command implements ServiceProxyCommand
   /**
    * Sets the <code>filter</code> parameter. See Pazpar2 documentation for details.
    */  
-  public void setFilter(String filterExpression) {
-    if (filterExpression != null && filterExpression.length()>0) {
-      if (filterExpression.split("[=~]").length==1) {
-        removeFilters(filterExpression.split("[=~]")[0]);
-      } else if (filterExpression.split("[=~]").length==2) {
-        setParameter(new FilterParameter(new Expression(filterExpression)));
-      } else {
-        logger.error("Could not parse filter expression [" + filterExpression + "]");
+  public void setFilter(String compoundExpression) {
+    if (compoundExpression != null && compoundExpression.length()>0) {
+      String[] subExpressions = compoundExpression.split(",");
+      for (int i=0; i<subExpressions.length; i++) {
+        if (subExpressions[i].split("[=~]").length==1) {
+          removeFilters(subExpressions[i].split("[=~]")[0]);
+        } else if (subExpressions[i].split("[=~]").length==2) {
+          if (getParameter("filter") == null) {
+            setParameter(new FilterParameter(new Expression(subExpressions[i])));
+          } else {
+            if (getParameter("filter").hasExpressions(subExpressions[i].split("[=~]")[0])) {
+              getParameter("filter").removeExpressions(subExpressions[i].split("[=~]")[0]);
+            }
+            getParameter("filter").addExpression(new Expression(subExpressions[i]));
+          }
+        } else {
+          logger.error("Could not parse filter expression [" + subExpressions[i] + "]");
+        }
       }
     }
   }
@@ -215,14 +227,24 @@ public class SearchCommand extends Pazpar2Command implements ServiceProxyCommand
   /**
    * Sets the <code>limit</code> parameter. See Pazpar2 documentation for details.
    */  
-  public void setLimit (String limitExpression) {   
-    if (limitExpression != null && limitExpression.length()>0) {
-      if (limitExpression.split("[=~]").length==1) {
-        removeLimits(limitExpression.split("[=~]")[0]);
-      } else if (limitExpression.split("[=~]").length==2) {
-        setParameter(new LimitParameter(new Expression(limitExpression)));
-      } else {
-        logger.error("Could not parse limit expression [" + limitExpression + "]");
+  public void setLimit (String compoundExpression) {   
+    if (compoundExpression != null && compoundExpression.length()>0) {
+      String[] subExpressions = compoundExpression.split(",");
+      for (int i=0; i<subExpressions.length; i++) {
+        if (subExpressions[i].split("[=~]").length==1) {
+          removeLimits(subExpressions[i].split("[=~]")[0]);
+        } else if (subExpressions[i].split("[=~]").length==2) {
+          if (getParameter("limit") == null) {
+            setParameter(new LimitParameter(new Expression(subExpressions[i])));
+          } else {
+            if (getParameter("limit").hasExpressions(subExpressions[i].split("[=~]")[0])) {
+              getParameter("limit").removeExpressions(subExpressions[i].split("[=~]")[0]);
+            }
+            getParameter("limit").addExpression(new Expression(subExpressions[i]));
+          }
+        } else {
+          logger.error("Could not parse limit expression [" + subExpressions[i] + "]");
+        }
       }
     }
   }
@@ -283,7 +305,7 @@ public class SearchCommand extends Pazpar2Command implements ServiceProxyCommand
   }
   
   /**
-   * Returns a list of limit expressions with fields that matches on of <code>expressionFields</code>
+   * Returns a list of limit expressions with fields that matches one of <code>expressionFields</code>
    * 
    * @param expressionFields limit expressions to look for
    */
-- 
1.7.10.4