NickvisionApps / Tagger

Tag your music
https://flathub.org/apps/details/org.nickvision.tagger
MIT License
211 stars 13 forks source link

Application is corrupting media [lyrics] tags after lookup #416

Open s38b35M5 opened 7 months ago

s38b35M5 commented 7 months ago

Description of the issue: Changes to media tags are corrupt or otherwise unreadable after applying changes from lookup to lyrics database. Application shows a file system error Library was changed on disk. Reload for each file loaded. Occasionally (but not regularly), Tagger reports the corruption and claims to repair it. However, The changes to media files are not read by my media manager Strawberry Version 1.0.21 until after opening the media and making an arbitrary change to force saving of tag/info data.

I can, however view the saved "corrupted" lyrics in picard

How can this issue be reproduced?

  1. Load one or more supported file types
  2. Select one or more of the media in the list pane and press CTRL-SHIFT-L to query remote lyrics database(s)
  3. Optional Confirm lyrics data is applied, by clicking the ""Lyrics option, then dismiss the lyrics window
  4. Click Apply and observe Library was changed on disk. Reload for each file loaded.
  5. Click Reload or x to reload or dismiss the error and exit the application
  6. Observe the saved lyrics are not saved in a way media players like Amarok/Clementine/Strawberry can access

Expected result: Saving tags does not result in multiple errors, and saves the tags/lyrics without apparent corruption

Platform: Debian 11 Linux

Tagger Debug Blob:

org.nickvision.tagger
2023.11.3
GTK 4.12.4
libadwaita 1.4.2
Flatpak
en-US
LANG=en_US.UTF-8
LC_CTYPE="en_US.UTF-8"
LC_NUMERIC="en_US.UTF-8"
LC_TIME="en_US.UTF-8"
LC_COLLATE="en_US.UTF-8"
LC_MONETARY="en_US.UTF-8"
LC_MESSAGES="en_US.UTF-8"
LC_PAPER="en_US.UTF-8"
LC_NAME="en_US.UTF-8"
LC_ADDRESS="en_US.UTF-8"
LC_TELEPHONE="en_US.UTF-8"
LC_MEASUREMENT="en_US.UTF-8"
LC_IDENTIFICATION="en_US.UTF-8"
LC_ALL=

Attachments: inxi.txt System info tagger.zip Before/After media file and screen recording archived in a 7z (inside zip to save space and get around upload filter)

nlogozzo commented 7 months ago

Changes to media tags are corrupt or otherwise unreadable after applying changes from lookup to lyrics database

Does this only happen with looking up Lyrics?

If you have a valid file and just change the title/artist and apply, does that remain ok? What if you manually edit the Lyrics instead of lookup?

s38b35M5 commented 7 months ago

Changes to media tags are corrupt or otherwise unreadable after applying changes from lookup to lyrics database

Does this only happen with looking up Lyrics?

If you have a valid file and just change the title/artist and apply, does that remain ok? What if you manually edit the Lyrics instead of lookup?

I performed some testing today, and can confirm that the following circumstances do NOT result in errors or corrupt files

Then I selected an artist with two dozen albums in it, and did a lyric lookup, and the problem with errors seems to have ceased. I will have more time later this week to mess with it, but so far, it seems this problem (which persisted across reboots and different network configs) seems have been transient.

If you have any suggestions to assist troubleshooting, I'm open to them.

Thank you for your time and support.

nlogozzo commented 6 months ago

It could have just been a fluke in the flatpak portal setup...

@s38b35M5 has this issue occurred again? or is it gone?

s38b35M5 commented 6 months ago

It could have just been a fluke in the flatpak portal setup...

@s38b35M5 has this issue occurred again? or is it gone?

I just did some extensive testing, and I can't replicate it again. Gremlins...

Thanks for the support.

nlogozzo commented 6 months ago

Like I said, could have been a flatpak backend issue that was fixed through an update 🤷

Let me know if it happens again :)

s38b35M5 commented 5 months ago

It is happening again, if you would like to take a closer look. May only be with OGG files. Not critical for me, so no rush.

nlogozzo commented 5 months ago

Could you send me the specific files that it's happening with?

Email at nlogozzo225@gmail.com

s38b35M5 commented 5 months ago

Could you send me the specific files that it's happening with?

Sure thing. I'll share something in the next day or two when I can. Love to help if I can.

nlogozzo commented 5 months ago

Yeah having the files is best so we can exam the internal strutture to see if there are any issues...

s38b35M5 commented 5 months ago

You've got mail 📬

nlogozzo commented 5 months ago

@s38b35M5 So I was kind of a dumb dumb and didn't save the email when I cleared my inbox 😅

Could you resend it please?

s38b35M5 commented 5 months ago

You should have that email now. Thanks for looking into it!