[mb-commits] r13703 - search_server/trunk/index/src/main/java/org/musicbrainz/search/index

[email protected] Fri, 09 Aug 2013 11:22:02 +0000
Newsgroups gmane.comp.audio.musicbrainz.cvs
Message-ID <E1V7kle-0006Qj-KW@wiley>
Author: ijabz
Date: 2013-08-09 11:22:02 +0000 (Fri, 09 Aug 2013)
New Revision: 13703

Modified:
   search_server/trunk/index/src/main/java/org/musicbrainz/search/index/RecordingIndex.java
Log:
SEARCH-315:REGRESSION:Recording index is not outputting Release artist credit when different to recoridng artist credit (i.e. Various Artist)

Modified: search_server/trunk/index/src/main/java/org/musicbrainz/search/index/RecordingIndex.java
===================================================================
--- search_server/trunk/index/src/main/java/org/musicbrainz/search/index/RecordingIndex.java	2013-08-09 10:52:05 UTC (rev 13702)
+++ search_server/trunk/index/src/main/java/org/musicbrainz/search/index/RecordingIndex.java	2013-08-09 11:22:02 UTC (rev 13703)
@@ -765,6 +765,15 @@
                     MediumList ml = of.createMediumList();
                     release.setReleaseGroup(origRelease.getReleaseGroup());
                     release.setStatus(origRelease.getStatus());
+                    if(origRelease.getArtistCredit()!=null)
+                    {
+                        release.setArtistCredit(origRelease.getArtistCredit());
+                        doc.addField(RecordingIndexField.RELEASE_AC_VA, "1");
+                    }
+                    else {
+                        doc.addField(RecordingIndexField.RELEASE_AC_VA, Index.NO_VALUE);
+                    }
+
                     ml.setTrackCount(origRelease.getMediumList().getTrackCount());
                     release.setMediumList(ml);
                     release.setReleaseEventList(origRelease.getReleaseEventList());
@@ -850,12 +859,7 @@
                     doc.addNumericField(RecordingIndexField.NUM_TRACKS_RELEASE, release.getMediumList().getTrackCount().intValue());
 
 
-                    //Is Various Artist Release
-                    if (release.getArtistCredit() != null) {
-                        doc.addField(RecordingIndexField.RELEASE_AC_VA, "1");
-                    } else {
-                        doc.addField(RecordingIndexField.RELEASE_AC_VA, Index.NO_VALUE);
-                    }
+
                     trackNames.add(trackWrapper.getTrackName().toLowerCase(Locale.UK));
                     doc.addField(RecordingIndexField.POSITION, String.valueOf(trackWrapper.getMediumPosition()));
                     doc.addFieldOrNoValue(RecordingIndexField.FORMAT, trackWrapper.getMediumFormat());