Changeset 35 in flacrip for trunk

Timestamp:

03/25/15 18:59:54 (10 years ago)

Author:

peter

Message:

applymeta:

• if no track numbers are given, assume all tracks
• don't write an ID3v2 TPE2 frame (this may have been causing problems with ID3v1-only reading software)

File:

• trunk/applymeta (modified) (2 diffs)

trunk/applymeta

@@ -29 +29 @@
 
 my @track_numbers;
-foreach (split /,/, $TRACKS) {
-    if (/^(\d+)$/) {
-        push @track_numbers, $1;
-    } elsif (/^(\d+)-(\d+)$/) {
-        push @track_numbers, ($1 .. $2);
-    } elsif (/^-(\d+)$/) {
-        push @track_numbers, (1 .. $1);
-    } elsif (/^(\d+)-$/) {
-        push @track_numbers, ($1 .. $track_count);
+if ($TRACKS) {
+    foreach (split /,/, $TRACKS) {
+        if (/^(\d+)$/) {
+            push @track_numbers, $1;
+        } elsif (/^(\d+)-(\d+)$/) {
+            push @track_numbers, ($1 .. $2);
+        } elsif (/^-(\d+)$/) {
+            push @track_numbers, (1 .. $1);
+        } elsif (/^(\d+)-$/) {
+            push @track_numbers, ($1 .. $track_count);
+        }
     }
+} else {
+    @track_numbers = (1 .. $track_count);
 }
 
@@ -60 +64 @@
     $mp3->year_set($year) if $year;
     $mp3->track_set($i);
-    $mp3->select_id3v2_frame_by_descr('TPE2', $metadata->{ALBUMARTIST}) if $metadata->{ALBUMARTIST};
     $mp3->update_tags;
     printf "    Apply to %s\n", $audio_file;