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 2008/01/15 22:00:39 UTC

svn commit: r612215 - /lucene/solr/trunk/src/java/org/apache/solr/request/XMLWriter.java

Author: ryan
Date: Tue Jan 15 13:00:36 2008
New Revision: 612215

URL: http://svn.apache.org/viewvc?rev=612215&view=rev
Log:
SOLR-446 -- make sure TextResponseWriter limits fields to returnFields if specified

Modified:
    lucene/solr/trunk/src/java/org/apache/solr/request/XMLWriter.java

Modified: lucene/solr/trunk/src/java/org/apache/solr/request/XMLWriter.java
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/src/java/org/apache/solr/request/XMLWriter.java?rev=612215&r1=612214&r2=612215&view=diff
==============================================================================
--- lucene/solr/trunk/src/java/org/apache/solr/request/XMLWriter.java (original)
+++ lucene/solr/trunk/src/java/org/apache/solr/request/XMLWriter.java Tue Jan 15 13:00:36 2008
@@ -342,11 +342,14 @@
     startTag("doc", name, false);
     incLevel();
 
-    if (includeScore) {
-      writeVal("score", doc.getFirstValue("score"));
+    if (includeScore && returnFields != null ) {
+      returnFields.add( "score" );
     }
 
     for (String fname : doc.getFieldNames()) {
+      if (returnFields!=null && !returnFields.contains(fname)) {
+        continue;
+      }
       Object val = doc.getFieldValue(fname);
 
       if (val instanceof Collection) {