Bugzilla – Bug 1701
Don't add playlist file contents to playlist when adding whole directory
Last modified: 2008-09-15 14:36:01 UTC
When browsing the music folder, playlist files (e.g. .cue and .m3u) show up in the listing. If you click > or + against "All subfolders", the files in the directory get added, but also the files listed in the playlist files get added too. Default behaviour should be that playlist files do not get added to the playlist unless they are clicked on explicitly.
and here I was thinking that add all was for adding all...
Ah well, you know what thought did...
Robin - this should be fixed now.
Hmmm. In what way is it "fixed" ? I've just updated to r3557 and the behaviour doesn't seem to have changed. What behaviour should I now expect? R.
This seems to work from playerUI, but not webUI. adding a folder from playerUI, however, seems to come up in the current playlist with a lot of missing metadata for all but song1. adding from webUI shows all metadata, but is adding the entire playlist if one exists in the folder
Robin - this is too risky of a fix to try for 6.1 - as a work around, can you move your playlists out of the audiodir?
Robin - can you try this patch? --- Slim/Utils/Scan.pm (revision 3028) +++ Slim/Utils/Scan.pm (local) @@ -636,7 +636,7 @@ }); } - } else { + } elsif (Slim::Utils::Misc::inPlaylistFolder($playlisturl)) {
Dan, I've tried your patch and it seems to work. After applying the patch, I used "Browse Music Folder" to browse to an album listing and clicked "+" to add the contents to the current (empty) playlist. All the music (flac) files were added but the two .m3u files in the directory were ignored. R.
Robin - I've checked in a slight variation of the patch as subversion change 3969