[GH-ISSUE #1491] FLAC Tags (Vorbis Tags / Comments) are not registered. #854

Closed
opened 2026-02-26 02:34:27 +03:00 by kerem · 1 comment
Owner

Originally created by @jojoxd on GitHub (Aug 31, 2022).
Original GitHub issue: https://github.com/koel/koel/issues/1491

Describe the bug
Just as the title says, Vorbis tags are not imported into the database.

To reproduce
Steps to reproduce the behavior:

  1. Use a flac file that is commented using vorbis comments
  2. run artisan koel:sync
  3. See that all flac albums are incorrectly tagged as "Unknown Artist"/"Unknown Album"

Expected behavior
I Expect flac files to be correctly imported using artisan koel:sync

Screenshots
image
image

Environment

  • Koel v6.0.5 (Using official Koel docker image)
  • OS: Fedora 35
  • Browser Vivaldi 5.3.2679.68 (Stable channel) stable (64-bit)
  • PHP version: official docker image PHP
  • Node version official docker image Node

Additional context
I've noticed this line, which is slightly different from how getID3's basic demo does it:
github.com/koel/koel@ddd2dff65d/app/Services/FileSynchronizer.php (L51)

I think this would work great if we could use getID3's CopyTagsToComments($data) function here, and only read from $data['comments'] after that in fromGetId3Info

Originally created by @jojoxd on GitHub (Aug 31, 2022). Original GitHub issue: https://github.com/koel/koel/issues/1491 **Describe the bug** Just as the title says, Vorbis tags are not imported into the database. **To reproduce** Steps to reproduce the behavior: 1. Use a flac file that is commented using vorbis comments 2. run `artisan koel:sync` 3. See that all flac albums are incorrectly tagged as "Unknown Artist"/"Unknown Album" **Expected behavior** I Expect flac files to be correctly imported using `artisan koel:sync` **Screenshots** ![image](https://user-images.githubusercontent.com/5726497/187717370-edba3183-903e-47a9-b470-892b2b32fb43.png) ![image](https://user-images.githubusercontent.com/5726497/187717660-97276974-5052-4ae5-a801-c3466599b740.png) **Environment** - Koel v6.0.5 (Using official Koel docker image) - OS: Fedora 35 - Browser Vivaldi 5.3.2679.68 (Stable channel) stable (64-bit) - PHP version: official docker image PHP - Node version official docker image Node **Additional context** I've noticed this line, which is slightly different from how getID3's [basic demo](https://github.com/JamesHeinrich/getID3/blob/master/demos/demo.basic.php) does it: https://github.com/koel/koel/blob/ddd2dff65d6b39328323677f95004b2616d6cb62/app/Services/FileSynchronizer.php#L51 I think this would work great if we could use getID3's `CopyTagsToComments($data)` function here, and only read from `$data['comments']` after that in [`fromGetId3Info`](https://github.com/koel/koel/blob/ddd2dff65d6b39328323677f95004b2616d6cb62/app/Values/SongScanInformation.php#L28)
kerem closed this issue 2026-02-26 02:34:27 +03:00
Author
Owner

@phanan commented on GitHub (Sep 23, 2022):

Fixed and released in v6.2.0.

<!-- gh-comment-id:1256033032 --> @phanan commented on GitHub (Sep 23, 2022): Fixed and released in v6.2.0.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/koel-koel#854
No description provided.