Author: ijabz
Date: 2009-10-31 21:03:55 +0000 (Sat, 31 Oct 2009)
New Revision: 12276
Modified:
search_server/branches/ngs/index/src/main/java/org/musicbrainz/search/index/ReleaseIndex.java
search_server/branches/ngs/index/src/test/java/org/musicbrainz/search/index/ReleaseIndexTest.java
Log:
Fix numDiscIds field
Modified: search_server/branches/ngs/index/src/main/java/org/musicbrainz/search/index/ReleaseIndex.java
===================================================================
--- search_server/branches/ngs/index/src/main/java/org/musicbrainz/search/index/ReleaseIndex.java 2009-10-31 13:24:23 UTC (rev 12275)
+++ search_server/branches/ngs/index/src/main/java/org/musicbrainz/search/index/ReleaseIndex.java 2009-10-31 21:03:55 UTC (rev 12276)
@@ -240,7 +240,7 @@
addNumericFieldToDocument(doc,ReleaseIndexField.NUM_TRACKS,trackCount);
//Num Discs over the whole release
- addNumericFieldToDocument(doc,ReleaseIndexField.NUM_DISCIDS,trackCount);
+ addNumericFieldToDocument(doc,ReleaseIndexField.NUM_DISCIDS,discCount);
}
Modified: search_server/branches/ngs/index/src/test/java/org/musicbrainz/search/index/ReleaseIndexTest.java
===================================================================
--- search_server/branches/ngs/index/src/test/java/org/musicbrainz/search/index/ReleaseIndexTest.java 2009-10-31 13:24:23 UTC (rev 12275)
+++ search_server/branches/ngs/index/src/test/java/org/musicbrainz/search/index/ReleaseIndexTest.java 2009-10-31 21:03:55 UTC (rev 12276)
@@ -88,7 +88,9 @@
stmt.addBatch("INSERT INTO medium(id, tracklist, release, position, format, name, editpending) VALUES (1, 1, 491240, 1, 7, null, 1)");
stmt.addBatch("INSERT INTO medium_format(id, name,year )VALUES (1, 'CD', 1982)");
stmt.addBatch("INSERT INTO medium_format(id, name,year )VALUES (7, 'Vinyl', 1895)");
+ stmt.addBatch("INSERT INTO medium_cdtoc( id, medium, cdtoc, editpending) VALUES (1, 1, 1, 0)");
+
stmt.executeBatch();
stmt.close();
conn.close();
@@ -475,6 +477,27 @@
ir.close();
}
+ /**
+ * @throws Exception
+ */
+ public void testIndexReleaseNumDiscs() throws Exception {
+
+ addReleaseOne();
+ RAMDirectory ramDir = new RAMDirectory();
+ createIndex(ramDir);
+
+ IndexReader ir = IndexReader.open(ramDir, true);
+ assertEquals(1, ir.numDocs());
+ {
+ Document doc = ir.document(0);
+ assertEquals(1, doc.getFields(ReleaseIndexField.NUM_DISCIDS_MEDIUM.getName()).length);
+ assertEquals(1, NumericUtils.prefixCodedToInt(doc.getField(ReleaseIndexField.NUM_DISCIDS_MEDIUM.getName()).stringValue()));
+ assertEquals(1, doc.getFields(ReleaseIndexField.NUM_DISCIDS.getName()).length);
+ assertEquals(1, NumericUtils.prefixCodedToInt(doc.getField(ReleaseIndexField.NUM_DISCIDS.getName()).stringValue()));
+
+ }
+ ir.close();
+ }
/**
* @throws Exception
*/
_______________________________________________
MusicBrainz-commits mailing list
MusicBrainz-commits@...
http://lists.musicbrainz.org/mailman/listinfo/musicbrainz-commits
