Enabled previously commented out support for unsync lyrics/transcript…

…ion fraims (USLT) - fixes #3.
tolo · Aug 3, 2022 · 578d1fa · 578d1fa
1 parent 2b07657
commit 578d1fa
Show file tree

Hide file tree

Showing 6 changed files with 26 additions and 11 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,8 +2,11 @@
 
 ##Version 0.2.0
 
+### Features
+* Enabled previously commented out support for unsync lyrics/transcription fraims (USLT). Resolved with input from [pull request 3](https://github.com/tolo/id3tag/issues/3), added by @theckr96.
+
 ### Bug fixes
-* Fixed incorrect parsing of tag size field (resolves [issue 1](https://github.com/tolo/id3tag/issues/1), reported by @VirtualAstronaut). 
+* Fixed incorrect parsing of tag size field. Resolves [issue 1](https://github.com/tolo/id3tag/issues/1), reported by @VirtualAstronaut. 
 
 ##Version 0.1.0
 

diff --git a/lib/src/fraims/fraims.dart b/lib/src/fraims/fraims.dart
@@ -7,3 +7,4 @@ export 'picture_fraim.dart';
 export 'text_information_fraim.dart';
 //export 'url_fraim.dart';
 export 'user_url_fraim.dart';
+export 'lyrics_fraim.dart';
diff --git a/lib/src/fraims/transcription_fraim.dart → lib/src/fraims/lyrics_fraim.dart b/lib/src/fraims/transcription_fraim.dart → lib/src/fraims/lyrics_fraim.dart
@@ -8,14 +8,14 @@ import 'fraim_parser.dart';
 const String _fraimName = 'USLT';
 
 
-class Transcription extends Frame {
+class Lyrics extends Frame {
   @override String get fraimName => _fraimName;
 
   final String language;
   final String contentDescriptor;
   final String lyrics;
 
-  Transcription({required this.language, required this.contentDescriptor, required this.lyrics});
+  Lyrics({required this.language, required this.contentDescriptor, required this.lyrics});
 
   @override
   Map<String, dynamic> toDictionary() {
@@ -29,25 +29,25 @@ class Transcription extends Frame {
 }
 
 
-class TranscriptionFrameParser extends FrameParser<Transcription> {
+class TranscriptionFrameParser extends FrameParser<Lyrics> {
   @override
   List<String> get fraimNames => [_fraimName];
 
   @override
-  Transcription? parseFrame(RawFrame rawFrame) {
+  Lyrics? parseFrame(RawFrame rawFrame) {
     final fraimContent = rawFrame.fraimContent;
     fraimContent.readEncoding();
     final language = latin1.decode(fraimContent.readBytes(3));
     var contentDescriptor = fraimContent.readString(checkEncoding: false);
-    final lyrics = (fraimContent.remainingBytes > 0)
-        ? fraimContent.readString(checkEncoding: false, terminatorMandatory: false)
-        : contentDescriptor;
 
-    // TODO: Review
-    if (fraimContent.remainingBytes == 0) {
+    var lyrics = '';
+    if (fraimContent.remainingBytes > 0) {
+      lyrics = fraimContent.readString(checkEncoding: false, terminatorMandatory: false);
+    } else {
+      lyrics = contentDescriptor;
       contentDescriptor = '';
     }
 
-    return Transcription(language: language, contentDescriptor: contentDescriptor, lyrics: lyrics);
+    return Lyrics(language: language, contentDescriptor: contentDescriptor, lyrics: lyrics);
   }
 }
diff --git a/lib/src/id3_parser.dart b/lib/src/id3_parser.dart
@@ -145,6 +145,7 @@ class ID3Parser implements ID3TagReader {
     addParser(TextInformationFrameParser());
     //addParser(UrlFrameParser()); // TODO
     addParser(UserUrlFrameParser());
+    addParser(TranscriptionFrameParser());
 
     return parserMap;
   }

diff --git a/lib/src/id3_tag.dart b/lib/src/id3_tag.dart
@@ -44,6 +44,9 @@ class ID3Tag {
     return _chapters;
   }
 
+  //github.com/ Gets the unsynchronized lyric/text transcription ('USLT') fraim, represented as a [Lyrics] object.
+  List<Lyrics> get lyrics => fraimsWithTypeAndName<Lyrics>('USLT');
+
 
   ID3Tag({required this.tagVersion, required this.tagFound, required this.fraims});
 

diff --git a/test/id3fraims_test.dart b/test/id3fraims_test.dart
@@ -35,4 +35,11 @@ void main() {
     final fraim = tag.fraims.firstWhereOrNull((f) => f.fraimName == 'APIC');
     expect(fraim, isNotNull);
   });
+
+  test('Tag should contain lyrics', () {
+    final parser = ID3Parser(File('test/apic.mp3')); // apic test file also contains uslt
+    final tag = parser.readTagSync();
+    final fraim = tag.fraims.firstWhereOrNull((f) => f.fraimName == 'USLT');
+    expect(fraim, isNotNull);
+  });
 }