Bugzilla – Bug 18066
Title information from M3U files are not saved
Last modified: 2014-02-08 22:26:19 UTC
There is a bug in the M3U parser Formats/Playlists/M3U.pm This parser tries to extract the duration, artist, album and title from comment lines and saves them to the database as metadata for the trackurl. But this doesn't work because the variables are defined within the entry loop and get lost when the next line with the trackurl is parsed. I fixed this by moving the definition line "my ($secs, $artist, $album, $title, $trackurl);" before the "while" statement. But this should probably be reviewed to make sure the variables are cleared before the next entry.
This is probably caused by a faulty fix for bug 17933 which ignores the possibility of multiple line entries.
Thanks - need to review this fix. There have been more reports of this in the forums. http://forums.slimdevices.com/showthread.php?t=100890
Could you please attach a sample playlist which is exposing this misbehavior to this bug report? Thanks!
Created attachment 7714 [details] Test playlist This playlist was working perfectly with squeezeboxserver-7.5.1, but it does not with the current version logitechmediaserver-7.8.0
Ok, thanks. I think I see a pattern: this is a playlist of remote streams only. Would you see the same problem with playlists of local tracks?
I only noticed it with this file. Perhaps the error does not show with other entries because they are already stored in the database and don't need to be read from the m3u file.
> Perhaps the error does not show with other entries because they are already > stored in the database and don't need to be read from the m3u file. Exactly. My assumption as well.
cedf697d25b76a9e4e43734ac2c580e8e02fd171 Ok, this time I hopefully fixed both, this issue and bug 17933.
*** Bug 17888 has been marked as a duplicate of this bug. ***