From 22ff90343259098541c3f589dffb3979d02416e4 Mon Sep 17 00:00:00 2001
From: Aron Roberts <aron@socrates.berkeley.edu>
Date: Thu, 9 May 2013 19:36:50 -0700
Subject: [PATCH] CSPACE-6000: Substitute constants for bare strings in string
 replacement. Add clarity to log statements.

---
 .../nuxeo/client/java/RepositoryJavaClientImpl.java         | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/services/common/src/main/java/org/collectionspace/services/nuxeo/client/java/RepositoryJavaClientImpl.java b/services/common/src/main/java/org/collectionspace/services/nuxeo/client/java/RepositoryJavaClientImpl.java
index 9c88719e5..ca7ae8200 100644
--- a/services/common/src/main/java/org/collectionspace/services/nuxeo/client/java/RepositoryJavaClientImpl.java
+++ b/services/common/src/main/java/org/collectionspace/services/nuxeo/client/java/RepositoryJavaClientImpl.java
@@ -113,6 +113,7 @@ public class RepositoryJavaClientImpl implements RepositoryClient<PoxPayloadIn,
     public static final String NUXEO_CORE_TYPE_WORKSPACEROOT = "WorkspaceRoot";
     // FIXME: Get this value from an existing constant, if available
     private static final String USER_SUPPLIED_WILDCARD = "*";
+    private static final String USER_SUPPLIED_WILDCARD_REGEX = "\\" + USER_SUPPLIED_WILDCARD;
     
     /**
      * Instantiates a new repository java client impl.
@@ -1793,12 +1794,13 @@ public class RepositoryJavaClientImpl implements RepositoryClient<PoxPayloadIn,
         // Partial term search expressions of 2 or fewer characters
         // currently aren't amenable to the use of wildcards
         if (len <= 2)  {
-            logger.warn("Partial term matching expression of 1 or 2 characters contains user-supplied wildcard:" + partialTerm);
+            logger.warn("Partial term match search expression of just 1-2 characters in length contains a user-supplied wildcard: " + partialTerm);
+            logger.warn("Will handle that character as a literal value, rather than as a wildcard ...");
             return partialTerm;
         }
         int lastCharPos = len - 1;
         return partialTerm.substring(0, 1) // first char
-                + partialTerm.substring(1, lastCharPos).replaceAll("\\*", "%") // middle of search expression, excluding first and last
+                + partialTerm.substring(1, lastCharPos).replaceAll(USER_SUPPLIED_WILDCARD_REGEX, JDBCTools.SQL_WILDCARD) // middle of search expression, excluding first and last
                 + partialTerm.substring(lastCharPos); // last char
 
     }
-- 
2.47.3