[mb-commits] r13687 - in search_server/trunk/index/src: main/java/org/musicbrainz/search/index test/java/org/musicbrainz/search/index

Author: ijabz
Date: 2013-05-20 09:23:17 +0000 (Mon, 20 May 2013)
New Revision: 13687

Modified:
   search_server/trunk/index/src/main/java/org/musicbrainz/search/index/ReleaseIndex.java
   search_server/trunk/index/src/test/java/org/musicbrainz/search/index/ReleaseIndexTest.java
Log:
SEARCH-294:Search results no longer include medium-list count attribute

Modified: search_server/trunk/index/src/main/java/org/musicbrainz/search/index/ReleaseIndex.java
===================================================================
--- search_server/trunk/index/src/main/java/org/musicbrainz/search/index/ReleaseIndex.java	2013-05-16 09:29:38 UTC (rev 13686)
+++ search_server/trunk/index/src/main/java/org/musicbrainz/search/index/ReleaseIndex.java	2013-05-20 09:23:17 UTC (rev 13687)
@@ -561,11 +561,12 @@
                 DiscList discList = of.createDiscList();
                 discList.setCount(BigInteger.valueOf(numDiscsOnMedium));
                 medium.setDiscList(discList);
-
                 mediumList.getMedium().add(medium);
-                release.setMediumList(mediumList);
             }
+            mediumList.setCount(BigInteger.valueOf(mediumList.getMedium().size()));
+            release.setMediumList(mediumList);
 
+
             //Num of mediums on the release
             doc.addNumericField(ReleaseIndexField.NUM_MEDIUMS, mediumCount);
 

Modified: search_server/trunk/index/src/test/java/org/musicbrainz/search/index/ReleaseIndexTest.java
===================================================================
--- search_server/trunk/index/src/test/java/org/musicbrainz/search/index/ReleaseIndexTest.java	2013-05-16 09:29:38 UTC (rev 13686)
+++ search_server/trunk/index/src/test/java/org/musicbrainz/search/index/ReleaseIndexTest.java	2013-05-20 09:23:17 UTC (rev 13687)
@@ -888,6 +888,8 @@
             assertEquals("EP", release.getReleaseGroup().getPrimaryType());
             assertEquals("EP", release.getReleaseGroup().getType());
             assertNotNull(release.getMediumList());
+            assertEquals(1,release.getMediumList().getCount().intValue());
+            assertEquals(1,release.getMediumList().getTrackCount().intValue());
 
 
         }
@@ -915,6 +917,9 @@
             assertEquals("B00005NTQ7", release.getAsin());
             assertNull(release.getStatus());
             assertNotNull(release.getMediumList());
+            assertEquals(1,release.getMediumList().getCount().intValue());
+            assertEquals(10,release.getMediumList().getTrackCount().intValue());
+
             ReleaseEventList rel = release.getReleaseEventList();
             assertNotNull(rel);
             assertEquals(4,rel.getReleaseEvent().size());