Bugzilla – Bug 7791
Web UI blocks creating playlists with illegal characters, player UI does not
Last modified: 2011-03-16 04:34:08 UTC
The work to close Bug 7610 made it so that the web UI would block playlists with illegal characters as well as not allowing a playlist name to begin with a dot. That much does work as I was unable to save a playlist called .jeff in the web UI. Saving a playlist from the player still allows the creation of a playlist called ".jeff" and is renamed to "jeff".
several options here. Very simple to do a bumpRight if the filename is illegal, but it may be nicer to display a warning message: Illegal Filename .jeff Press LEFT to re-enter or, simply show what the renamed version will be on the confirmation screen: Right Again to Save Playlist as: Jeff can be assigned to me for any of the above fixes.
I'd like to see a warning message and then have to re-enter the name. That would work with the "illegal" characters. For playlists that begin with a dot the message should state that playlists can't begin with a dot and either force a new name to be entered or suggest a new name without the dot. Dots are allowed in filenames everywhere except the first character so I'd rather this be a special case, since it's only illegal in that particular place.
The player display is a bit limited for giving detailed instructions. It may not be feasible to do something like: "Illegal characters in filename, please press left to edit or right to save as <fixed name>" Note that the web ui simply blocks saving and gives the warning: "Note: Illegal filename. Please check for illegal characters." The name is expected to be corrected before saving again. Player UI limitations dictate simplicity, and matching the web ui would be consistent. cc'ing Brian as well for a UI decision.
Gook suggestions KDF. IMHO I'd like to see the warning screen too. Assigning to you for implementation. Please apply an appropriate milestone.
in 7.0/trunk at change 18659. marking as fixed. If text and ui are good this can be verified. Feel free to reopen if either need a change (though string could also be fixed as part of the translation pass) I suspect something like this will also come up for SqueezePlay. CLI may also want to implement an error feedback for this case.
This bug has recently been fixed in the latest release of SqueezeCenter 7.0.1 Please try that version, if you still see the error, then reopen this bug. To download this version, please navigate to: http://www.slimdevices.com/su_downloads.html
Reduce number of active targets for SC