Bug 362018

Summary:	Export playlist feature in context menu does not add the playlist tracks to the exported file.
Product:	[Applications] amarok	Reporter:	Evert Vorster <evorster>
Component:	Playlists/Saved Playlists	Assignee:	Amarok Bugs <amarok-bugs-null>
Status:	CONFIRMED ---
Severity:	normal	CC:	bart.cerneels
Priority:	NOR
Version First Reported In:	2.8.90 (2.9 beta)
Target Milestone:	2.9
Platform:	Arch Linux
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Evert Vorster 2016-04-21 03:30:26 UTC

A little background information.... 
I was migrating from internal database to external one, and I have a sizable collection, with 80,000 songs. With this collection I also have quite a few playlists stored in the database. 

First thing I tried was to try and drag my from the database to files on disk. 
This appeared to work, and I could load the playlists into the main playlist. 
However, when I switched to the new database, I was no longer able to open the playlists that were saved to disk, as they appeared corrupted.

I then tried the export playlist function in the context menu of the database of saved playlists. However the exported file was not loadable by amarok. 

The next thing I tried was to switch back to the previous database, load a saved playlist, then save it to disk by clicking on the "save" button. This very helpfully saved the playlist back to the database. There is a little down arrow next to the save icon, which would normally mean that there are some options, like where to save the playlist. However, this little down arrow does nothing, and the playlist just gets saved to the database. 

Selecting all songs in the playlist and dragging the songs to a new playlist on disk worked. 
It was a bit tedious doing every saved playlist I have like this.

Reproducible: Always



Expected Results:  
Consistent and solid saved playlist handling, Icons to actually do what they suggest to be doing

Comment 1 Myriam Schweingruber 2016-04-21 10:04:38 UTC

Hm, let's try to grasp what exact bug you are reporting:

Save playlist button has options, and these options do work, one just has to stay clicked on the button to make them appear. Not reproducible here on current git master
You migrated your database, this means the database location has changed, and with that the links to files in the playlist, no wonder this becomes corrupt. Normally a full rescan of the collection should get those back for the ones saved in the database, but I fear not for playlists saved on disk, as they lost their reference, e.g. the old database.

You exported a playlist from database, into what type of file? There are 4 options, at least one of those should work. If you save them to the standard location for amarok playlists (on Kubuntu this is ~/.kde/share/apps/amarok/playlists/), quit Amarok and restart it and you will see all the newly saved items, but indeed those are not loadable, because they are empty files. 

Can I modify the description of the bug to this? Remember, one item per bug, else this is impossible to handle.

Comment 2 Evert Vorster 2016-04-21 11:37:13 UTC

My apologies for writing more than one bug in one bug report. 

My music collection is still mounted on the same place as always, a playlist on disk should always point to the music files on the disk. This is the point of exporting a playlist, right? In theory an exported playlist should be readable by another application that does not have access to the amarok database. 

Bug 1.
Save playlist button - clicking on down arrow does not produce menu of output options. 
Never in a million years would I have thought of holding down the button for more menu options. Completely counter-intuitive. 
Now that I am aware that there is a context menu, I am able to export to xspf, and this file contains all the tracks on the playlist. 

Bug 2. 
"Export As" option in the context menu (available with the standard right-click) on the saved playlists does not actually put any of the songs in the playlist in the exported file. This is clearly a bug, right?

I suppose the most important bit is that bug #2, so I am going to change the bug title. (Even though the save playlist feature still requires some polish :)

Comment 3 Myriam Schweingruber 2016-06-12 12:58:45 UTC

Thank you for your feedback.