[mb-commits] r13664 - search_server/trunk/servlet/src/main/java/org/musicbrainz/search/servlet/mmd2

[email protected] Thu, 25 Apr 2013 20:53:32 +0000
Newsgroups gmane.comp.audio.musicbrainz.cvs
Message-ID <E1UVTAa-0006Ne-Em@wiley>
Author: ijabz
Date: 2013-04-25 20:53:17 +0000 (Thu, 25 Apr 2013)
New Revision: 13664

Modified:
   search_server/trunk/servlet/src/main/java/org/musicbrainz/search/servlet/mmd2/RecordingWriter.java
Log:
SEARCH-284:REGRESSION:"-" is returned instead of an empty list when there are no ISRCs for a recording

Modified: search_server/trunk/servlet/src/main/java/org/musicbrainz/search/servlet/mmd2/RecordingWriter.java
===================================================================
--- search_server/trunk/servlet/src/main/java/org/musicbrainz/search/servlet/mmd2/RecordingWriter.java	2013-04-25 19:59:05 UTC (rev 13663)
+++ search_server/trunk/servlet/src/main/java/org/musicbrainz/search/servlet/mmd2/RecordingWriter.java	2013-04-25 20:53:17 UTC (rev 13664)
@@ -138,11 +138,15 @@
             if(isrcs.length>0) {
                 IsrcList isrcList = of.createIsrcList();
                 for (int i = 0; i < isrcs.length; i++) {
-                    Isrc isrc = of.createIsrc();
-                    isrc.setId(isrcs[i]);
-                    isrcList.getIsrc().add(isrc);
+                    if (isNotNoValue(isrcs[i])) {
+                        Isrc isrc = of.createIsrc();
+                        isrc.setId(isrcs[i]);
+                        isrcList.getIsrc().add(isrc);
+                    }
                 }
-                recording.setIsrcList(isrcList);
+                if(isrcList.getIsrc().size()>0) {
+                    recording.setIsrcList(isrcList);
+                }
             }
 
             String[] puids              = doc.getValues(RecordingIndexField.PUID);