Changeset 45 in flacrip for trunk/lib

Timestamp:

09/19/15 18:59:30 (9 years ago)

Author:

peter

Message:

if the barcode element is present but empty, return '[none]' as the value of the BARCODE tag

File:

• trunk/lib/MusicBrainz.pm (modified) (1 diff)

trunk/lib/MusicBrainz.pm

@@ -110 +110 @@
     $info{LANGUAGE}            = $xpath->findvalue('text-representation/language', $release)->value;
     $info{SCRIPT}              = $xpath->findvalue('text-representation/script', $release)->value;
-    $info{BARCODE}             = $xpath->findvalue('barcode', $release)->value;
+
+    # add the barcode tag if the element exists
+    # if the barcode tag is empty, it is a release that explicitly has no barcode
+    # use the dummy value "[none]" for those cases
+    if ($xpath->exists('barcode', $release)) {
+        $info{BARCODE} = $xpath->findvalue('barcode', $release)->value || '[none]';
+    }
 
     # select the proper medium (important for multidisc releases)