Bug 355 - Provide "save-over" capability for playlists
: Provide "save-over" capability for playlists
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 5.x or older
: PC Windows XP
: P3 enhancement (vote)
: ---
Assigned To: KDF
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2004-06-08 08:44 UTC by Don Lagosz-Sinclair
Modified: 2008-10-13 22:35 UTC (History)
0 users

See Also:
Category: ---


Attachments
example patch for Fishbone skin (2.82 KB, patch)
2004-07-23 10:22 UTC, KDF
Details | Diff
full patch to EN, Default and Fishbone with strings. (6.22 KB, patch)
2004-07-23 12:29 UTC, KDF
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Don Lagosz-Sinclair 2004-06-08 08:44:56 UTC
When the user is saving a playlist, allow the option of saving over an existing 
playlist (e.g. via a message box, "Grunge.m3u already exists.  Replace it?").  
Also, use the most recently selected playlist as the default name, if possible.

To the best of my knowledge, this affects all platforms and operating systems.
Comment 1 KDF 2004-07-23 10:22:53 UTC
Created attachment 76 [details]
example patch for Fishbone skin

Here is one way to deal with this.  If the file exists, the server responds
with the same warning, but then also reveals a checkbox that the user can check
to confirm the overwrite.  This doesn't store the filename, just allows a user
to force an overwrite.
Comment 2 Blackketter Dean 2004-07-23 12:07:25 UTC
Looks reasonable, but I didn't see the necessary new text strings...
Comment 3 KDF 2004-07-23 12:09:27 UTC
strings.txt - line 4709

+ CONFIRM
+ 	EN	Confirm Overwrite

that's the basic idea.  just need to figure out how to carry over the filename.
 it keeps being reset to Untitled (just as it always does)
Comment 4 KDF 2004-07-23 12:29:26 UTC
Created attachment 77 [details]
full patch to EN, Default and Fishbone with strings.

This makes the change to EN Default and Fishbone, plus cleans up pages.pm to
avoid a perl warning when the checkbox isn't checked. It also keeps the
playlist name after its been typed the first time.
Comment 5 KDF 2004-07-23 19:08:59 UTC
committed July 23, 2004
Comment 6 Chris Owens 2006-06-16 14:42:20 UTC
There are 536 bugs in the database with targets of '---' that were fixed prior to new year 2006.  I am setting them to targets of 6.2.1 to keep them from showing up in my queries.