names of playlists won't be saved

Bug #426721 reported by nickless
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Exaile
Fix Released
Medium
Unassigned

Bug Description

Playlists are saved and ordered ascending, but if I rename them, upon restart all playlists will be called "Playlist 1-X" again.
Using Exaile 0.3.0.1, Linux Mint 7 (based on 32bit Ubuntu 9.04)

Revision history for this message
Johan.Mazel (johan-mazel) wrote :

I have the same bug on Exaile 0.3.0.0, Linux Ubuntu 9.04.
I don't know if this bug is supposed to be fixed by the 3.0.1, but it looks like it's not the case.

Revision history for this message
Steve Dodier-Lazaro (sidi) wrote : Re: [Bug 426721] Re: names of playlists won't be saved

Do you mean the name of the tabs opened, or the name of the playlists in the
side panel ?

Revision history for this message
Johan.Mazel (johan-mazel) wrote :

I mean the tab for sure.
And I don't understand for the side panel since I don't see any playlists
there.
Johan

2009/9/12 Steve Dodier <email address hidden>

> Do you mean the name of the tabs opened, or the name of the playlists in
> the
> side panel ?
>
> --
> names of playlists won't be saved
> https://bugs.launchpad.net/bugs/426721
> You received this bug notification because you are a direct subscriber
> of the bug.
>
> Status in Exaile: New
>
> Bug description:
> Playlists are saved and ordered ascending, but if I rename them, upon
> restart all playlists will be called "Playlist 1-X" again.
> Using Exaile 0.3.0.1, Linux Mint 7 (based on 32bit Ubuntu 9.04)
>

Revision history for this message
Steve Dodier-Lazaro (sidi) wrote :

Doing that requires a bit of code refactoring, which I don't think anyone will do in a minor release. Currently the tab name is only known to the Gtk Notebook, not to the backend playlists which are the things that are saved.

Aren, do you know if it's possible to do for playlists what you did for tracks in 0.3.1 (one track instance per loc), for instance by using a quick hash of the list's internal unique name ? Would it then be possible to add a display name property to the xl.playlists and use it in the tabs, reflects changes of that name to the playlist tab too, and setup an autosaving feature at the same time ? Then we could easily manage radio pls / custom pls / and ephemere pls only used in the notebook, and this would simplify the playlist saving and notebook tab code a lot, imo.

Feel free to set the milestone to whenever you want that to land.

Changed in exaile:
importance: Undecided → Low
milestone: none → 0.3.x
status: New → Confirmed
Revision history for this message
reacocard (reacocard) wrote :

Uniquifying playlists would be impossible since they don't have an inherent identifier that is guaranteed-unique like tracks do. I also fail to see how it would fix this bug.

The bug, so far as I can tell, comes from the backend storing all pl names as "Playlist %d" and then replacing the %d on restore. This needs to be fixed to always use the static pl name, and only ever generating playlist names on initial playlist creation.

Changed in exaile:
importance: Low → Medium
milestone: 0.3.x → 0.3.1
Revision history for this message
reacocard (reacocard) wrote :

fixed in r2516

Changed in exaile:
status: Confirmed → Fix Committed
reacocard (reacocard)
Changed in exaile:
milestone: 0.3.1 → 0.3.0.2
reacocard (reacocard)
Changed in exaile:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.