0001188: Inserting leading or trailing spaces in column ROUTER_EXPRES…

…SION on table ROUTER produce ER in outgoing batch
JumpMind · Apr 26, 2013 · 8c0b0b4 · 8c0b0b4
1 parent 9e689fb
commit 8c0b0b4
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/symmetric-core/src/main/java/org/jumpmind/symmetric/route/LookupTableDataRouter.java b/symmetric-core/src/main/java/org/jumpmind/symmetric/route/LookupTableDataRouter.java
@@ -108,15 +108,18 @@ protected Map<String, String> getParams(Router router, SimpleRouterContext routi
 
     public Map<String, String> parse(String routerExpression) throws SyntaxParsingException {
         boolean valid = true;
-        Map<String, String> params =  new HashMap<String, String>();
+        Map<String, String> params = new HashMap<String, String>();
         if (!StringUtils.isBlank(routerExpression)) {
             String[] expTokens = routerExpression.split("\r\n|\r|\n");
             if (expTokens != null) {
                 for (String t : expTokens) {
                     if (!StringUtils.isBlank(t)) {
                         String[] tokens = t.split("=");
-                        if (tokens.length == 2 && !params.containsKey(tokens[0])) {
-                            params.put(tokens[0], tokens[1]);
+                        /*
+                         * tokens must be trimmed, removing leading and trailing spaces
+                         */
+                        if (tokens.length == 2 && !params.containsKey(tokens[0].trim())) {
+                            params.put(tokens[0].trim(), tokens[1].trim());
                         } else {
                             valid = false;
                             break;
@@ -139,8 +142,7 @@ public Map<String, String> parse(String routerExpression) throws SyntaxParsingEx
             log.warn("The provided lookup table router expression is empty");
         }
         return params;
-    }
-
+    }    
     @SuppressWarnings("unchecked")
     protected Map<String, Set<String>> getLookupTable(final Map<String, String> params, Router router,
             SimpleRouterContext routingContext) {