You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-commits@lucene.apache.org by ry...@apache.org on 2009/02/24 03:11:13 UTC

svn commit: r747260 - /lucene/solr/trunk/src/solrj/org/apache/solr/client/solrj/util/ClientUtils.java

Author: ryan
Date: Tue Feb 24 02:11:13 2009
New Revision: 747260

URL: http://svn.apache.org/viewvc?rev=747260&view=rev
Log:
SOLR-1034 -- ClientUtils.escapeQuery should escape ;

Modified:
    lucene/solr/trunk/src/solrj/org/apache/solr/client/solrj/util/ClientUtils.java

Modified: lucene/solr/trunk/src/solrj/org/apache/solr/client/solrj/util/ClientUtils.java
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/src/solrj/org/apache/solr/client/solrj/util/ClientUtils.java?rev=747260&r1=747259&r2=747260&view=diff
==============================================================================
--- lucene/solr/trunk/src/solrj/org/apache/solr/client/solrj/util/ClientUtils.java (original)
+++ lucene/solr/trunk/src/solrj/org/apache/solr/client/solrj/util/ClientUtils.java Tue Feb 24 02:11:13 2009
@@ -180,9 +180,9 @@
     for (int i = 0; i < s.length(); i++) {
       char c = s.charAt(i);
       // These characters are part of the query syntax and must be escaped
-      if (c == '\\' || c == '+' || c == '-' || c == '!' || c == '(' || c == ')' || c == ':'
+      if (c == '\\' || c == '+' || c == '-' || c == '!'  || c == '(' || c == ')' || c == ':'
         || c == '^' || c == '[' || c == ']' || c == '\"' || c == '{' || c == '}' || c == '~'
-        || c == '*' || c == '?' || c == '|' || c == '&'
+        || c == '*' || c == '?' || c == '|' || c == '&'  || c == ';'
         || Character.isWhitespace(c)) {
         sb.append('\\');
       }