Playlists containing '/' convert to '-'

markstuartwalker · Post by **markstuartwalker** » Fri Nov 18, 2011 9:15 am

Using MM version 4.0.0.1459

If I have a playlist called 'abc/def' it seems to be presented to the d_itunes4.dll as 'abc-def''. For Ludek or Jiri ... this is within the DEVICE_NotifyPlaylist() function.

The same problem occurs with either '/' or '\'. I think that there is something filtering to protecting against the playlist names being used as file paths.

I can't now check in MM3 but I am told that this was not the case in the MM3 version

nohitter151 · Post by **nohitter151** » Fri Nov 18, 2011 10:41 am

I'm guessing this:
http://www.ventismedia.com/mantis/view.php?id=8083

markstuartwalker · Post by **markstuartwalker** » Fri Nov 18, 2011 3:17 pm

Yes, I can foresee that on a file system based device where a playlist name maps to a file name then '/' and '\' are likely to cause problems. Not so on iDevices though because the file name is different to the playlist name.

Can it be fixed please?

markstuartwalker · Post by **markstuartwalker** » Mon Nov 21, 2011 3:55 am

This needs to be fixed as the playlist name provided to the driver doesn't match the one in MM. Playlists with these modified names cannot be synchronised in "Build Folders" mode.

markstuartwalker · Post by **markstuartwalker** » Wed Nov 23, 2011 2:39 am

Do I need to submit a bug report? This seems obvious and understood.

markstuartwalker · Post by **markstuartwalker** » Wed Nov 23, 2011 11:37 am

I've just discovered that this also exhibits when a '/' or '"' is within the filename of the track as well.

Not good. Needs fixing.

markstuartwalker · Post by **markstuartwalker** » Fri Nov 25, 2011 4:20 am

Ticket ZEP-491676 raised

Post by **jiri** » Tue Dec 27, 2011 9:52 am

It will be fixed in MM 4.0.3, a new flag 'dflNoPlstFilesystemReady' will return the old behavior for a device plug-in if needed.

Jiri

markstuartwalker · Post by **markstuartwalker** » Thu Jan 05, 2012 5:55 am

@jiri

What is the enumeration constant value for this that I need to put into the DeviceCommon.pas file?

Code: Select all

dflNoPlstFilesystemReady = 25;

Post by **jiri** » Thu Jan 05, 2012 4:22 pm

Yes, that's right.

Jiri

markstuartwalker · Post by **markstuartwalker** » Mon Jan 09, 2012 10:43 am

Ok, incorporated into the plugin build 2216

Playlists containing '/' convert to '-'

Playlists containing '/' convert to '-'

Re: Playlists containing '/' convert to '-'

Re: Playlists containing '/' convert to '-'

Re: Playlists containing '/' convert to '-'

Re: Playlists containing '/' convert to '-'

Re: Playlists containing '/' convert to '-'

Re: Playlists containing '/' convert to '-'

Re: Playlists containing '/' convert to '-'

Re: Playlists containing '/' convert to '-'

Re: Playlists containing '/' convert to '-'

Re: Playlists containing '/' convert to '-'