Re: [Gap-dev-discuss] Cynthiune - shuffle playlist

[Sebastian Reitenbach]

On Sunday, May 13, 2012 19:01 CEST, Philippe Roussel <address@hidden> wrote:

> Le 13/05/2012 16:27, Sebastian Reitenbach a écrit :
> >
> > On Friday, May 11, 2012 21:53 CEST, Philippe Roussel <address@hidden> wrote:
> >
> >> Le 11/05/2012 18:21, Riccardo Mottola a écrit :
> >>> Hi,
> >>>
> >>> On 05/11/12 10:55, Philippe Roussel wrote:
> >>>> I can now save a playlist in .m3u format and open it back but I got an
> >>>> error when reading it : 'Format error: /path/to/play/list.m3u' appears
> >>>> as the last line of the playlist window. Philippe
> >>> I can save/load a playlist in the m3u format without problems. I have
> >>> commited everything, I have no outstanding changes left.
> >>>
> >>> Can you cat the .m3u playlist? it is a plain text ascii file format.
> >>> Perhaps there are string encoding problems for you? (Accents, etc...)
> >>
> >> File attached. Nothing funny here, just filenames with spaces (not mine,
> >> I swear !).
> >>
> >> By the way, using absolute paths could be nice. If I move this .m3u file
> >> its content is useless.
> >
> > In the General Preferences, there is already a  switch, where you can choose
> > to use absolute file names.
>
> Yes, I saw that yesterday, I spoke too fast...
>
> Can you save and load playlists on your setup ? With the small patch I
> sent it works quite well here.

Here too. However, I recognized in the Code of Playlist.m and 
PlaylistController.m
there are checks to not load a file twice tot the playlist, which is just not 
working.
It is comparing the new Song Object, with the Objects in the playlist. But 
since they
have different Object IDs, this fails. I think it should better compare the 
filename
attribute of a Song instance?

Will probably look at it tomorrow.

cheers,
Sebastian

>
> Thanks,
> Philippe