FS#7542 - This patch moves all the playlist creation from theroot dir to the PLAYLISTS directory.
This patch moves all the playlist creation etc… from the
root dir to the PLAYLISTS directory.
root.m3u8 and dynamic.m3u8 will be created in /PLAYLISTS.
The playlist catalog already uses /PLAYLISTS as default.
The OF creates this dir by default so it is always there
might as well use it to clean up the root.
GRaTT
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
Seeing as you can copy and paste files, shouldn’t the playlist be created in the folder that you created it in, rather than placing it in an arbitrary folder that may not even exist?
And not every player’s OF creates the folder. Rockbox needs to, on an application level, act functionally similar on all players.
The database uses /Playlists/ by default.
But all of my playlists regardless are created in the root.
My preference is to have all my playlists in one location
and since the database default is /playlists and the Sansa
OF creates this I felt this made sense.
GRaTT
The Rockbox database doesn’t use playlists at all. What are you talking about? Do you mean the “Playlist Catalog” feature?
Yes I mean the catalog feature.
GRaTT
It is a very good idea, as I would also like all my playlists in the one directory. If putting playlists in /PLAYLISTS/ is not within guidelines maybe it could at least be an option which is disabled by default? Also why is there no download available for download this patch?
Darky
here is the patch.
GRaTT
Why does it only enable for the Sansa?
I know for sure the sansa has the “Playlists” folder by default. The patch is small enough
that if you want to enable it for your player
just edit the #ifdef SANSA_E200.
GRaTT
it only has it if you go into the OF
Making it make the folder when built if it doesn’t already exist would be logical.
- check if folder ‘Playlists’ already exists
- merge with playlist_catalog
Thanks vmh.
@jdgordon: once rockbox has usb for PP targets then sansa owners will be free.
But I understand what you are saying. “If” this will get into svn then it
must address all targets. From previous forum posts and the first comment to this
patch I did not think this was wanted behavior. However I think this patch
makes the Playlist Catalog and playlists in general more user friendly,
and now thanks to vmh it does address all targets.
GRaTT
simplify and comment the added function playlist_check_dir()
Patch is out of sync with SVN
synchronized to r16201.
I don’t know if it still works correctly. I don’t have time to test it.
- synchronized to r17158
- simplified (using dir_exists()) and renamed playlist_check_dir to check_mkdir
Shouldn’t it obey the “playlist catalog directory” setting?
- use global_settings.playlist_catalog_dir (maybe it should be renamed to ‘playlist_dir’, but I don’t want to make changes in too many files)
- remove some unnecessary codes in playlist_catalog.c
Resync
Resync to rev. 19053
resync
Opps the above patch is incorrect this one should work