Bug 122797

Summary: Warn if files aren't writeable.
Product: [Applications] amarok Reporter: Malte Doersam <malte.d>
Component: generalAssignee: Amarok Developers <amarok-bugs-dist>
Status: RESOLVED FIXED    
Severity: wishlist CC: richlv
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Slackware   
OS: Linux   
Latest Commit: Version Fixed In:

Description Malte Doersam 2006-02-27 16:50:59 UTC 
Version:            (using KDE KDE 3.5.1)
Installed from:    Slackware Packages
OS:                Linux

I've ripped a new album, put it on my nfs server, tagged all the files via MusicBrainz: Edit Track Information -> Fill-in Tags Using MusicBrainz -> Select the right track -> next song -> repeat for each song. The time I tagged all, I pressed the 'save and close' button (I hope, that this is the english name, as this button is german here, called "speichern und schließen". )

As I forgot to change the ownerships of the files on my nfs server, I got several Fade-ins (popup-alikes) in the left bottom corner complaining that each of these files isn't writeable. 

Tagging the whole album took some time and all these taggings are lost. It's quite annoying.

I propose that the id3 edit window should have a small yellow exclamationmark to indicate that there will be problems with this file. And if writing fails, the new id3 tags should be somehow kept in memory until the application is closed or the tags are finally written to the files.
Comment 1 Alexandre Oliveira 2006-02-27 18:48:46 UTC 
The warning before is a good idea, but keeping the changes is something we don't intend to do, sorry. It would complicate things in an unnecessary way.
Also, it would be a dup of bug #102662.
Anyway, no bug here. It was not intended to work, so it's a wish!
Comment 2 Harald Sitter 2007-11-14 20:09:13 UTC 
*** Bug 126283 has been marked as a duplicate of this bug. ***
Comment 3 Dan Meltzer 2008-07-27 03:15:04 UTC 
SVN commit 838173 by dmeltzer:

Check if the file is writable before allowing it to be edited.
BUG: 122797


 M  +1 -0      ChangeLog  
 M  +2 -1      src/collection/sqlcollection/SqlMeta.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=838173