NickvisionApps / Tagger

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

app crashes with too many lookups #233

Closed legacychimera247 closed 1 year ago

legacychimera247 commented 1 year ago

tried to lookup on musicbrainz for an entire folder (meaning as an artist folder) and the app always crash

if useful this is the terminal output

`UnhandledException - unhandled exception: System.Xml.XmlException: There are multiple root elements. Line 2, position 2. at AcoustID.Web.LookupService.GetAsync(String fingerprint, Int32 duration, String[] meta) at NickvisionTagger.Shared.Models.MusicFile.LoadTagFromMusicBrainzAsync(String acoustIdClientKey, String version, Boolean overwriteTagWithMusicBrainz, Boolean overwriteAlbumArtWithMusicBrainz) in /run/build/org.nickvision.tagger/NickvisionTagger.Shared/Models/MusicFile.cs:line 300 at NickvisionTagger.Shared.Controllers.MainWindowController.DownloadMusicBrainzMetadataAsync() in /run/build/org.nickvision.tagger/NickvisionTagger.Shared/Controllers/MainWindowController.cs:line 824 at NickvisionTagger.GNOME.Views.MainWindow.DownloadMusicBrainzMetadata(SimpleAction sender, EventArgs e) in /run/build/org.nickvision.tagger/NickvisionTagger.GNOME/Views/MainWindow.cs:line 734 at System.Threading.Tasks.Task.<>c.b128_0(Object state) at GLib.Internal.MainLoopSynchronizationContext.<>c__DisplayClass1_0.b0() at GLib.Internal.MainLoopSynchronizationContext.<>c__DisplayClass3_0.b__0() [luca247@fedora ~]$

`

nlogozzo commented 1 year ago

Please try out the fix in this release: https://github.com/NickvisionApps/Tagger/releases/tag/2023.8.2-rc1