You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-commits@lucene.apache.org by dn...@apache.org on 2005/06/28 23:18:53 UTC

svn commit: r202282 - /lucene/java/trunk/src/java/org/apache/lucene/queryParser/MultiFieldQueryParser.java

Author: dnaber
Date: Tue Jun 28 14:18:51 2005
New Revision: 202282

URL: http://svn.apache.org/viewcvs?rev=202282&view=rev
Log:
small code simplification suggested by John Wang; added deprecated tag to javadoc

Modified:
    lucene/java/trunk/src/java/org/apache/lucene/queryParser/MultiFieldQueryParser.java

Modified: lucene/java/trunk/src/java/org/apache/lucene/queryParser/MultiFieldQueryParser.java
URL: http://svn.apache.org/viewcvs/lucene/java/trunk/src/java/org/apache/lucene/queryParser/MultiFieldQueryParser.java?rev=202282&r1=202281&r2=202282&view=diff
==============================================================================
--- lucene/java/trunk/src/java/org/apache/lucene/queryParser/MultiFieldQueryParser.java (original)
+++ lucene/java/trunk/src/java/org/apache/lucene/queryParser/MultiFieldQueryParser.java Tue Jun 28 14:18:51 2005
@@ -70,16 +70,12 @@
     return super.getFieldQuery(field, queryText);
   }
   
+  /**
+   * @deprecated use {@link #getFieldQuery(String, String)}
+   */
   protected Query getFieldQuery(String field, Analyzer analyzer, String queryText)
       throws ParseException {
-    if (field == null) {
-      Vector clauses = new Vector();
-      for (int i = 0; i < fields.length; i++)
-        clauses.add(new BooleanClause(super.getFieldQuery(fields[i], queryText),
-            BooleanClause.Occur.SHOULD));
-      return getBooleanQuery(clauses, true);
-    }
-    return super.getFieldQuery(field, queryText);
+    return getFieldQuery(field, queryText);
   }
   
   /**