Version: (using KDE 3.1.9) Compiler: gcc version 3.3.1 20030626 (Debian prerelease) OS: Linux (i686) release 2.4.21-1-k7-smp It's quite self-explaining, but an undo feature for tagging (and multitagging) would be great. Specially when you realize that you've wiped out half of your ID3 tags by a typo.
Marking this one as "assigned" and changing it from wishlist to bug status. Hoping to get this one done for 3.2.
*** Bug 79549 has been marked as a duplicate of this bug. ***
CVS commit by mpyne: Implement bug 61340 (Add undo support for multi-tagging). What this does is introduce a class that manages changes to PlaylistItems. A number of PlaylistItems can be altered before the underlying files are changed. Once the files are changed, a record is kept of the changes, which can then be used to undo what happened. Also, this class takes care of items that are to be renamed. All code paths that are capable of altering a tag have been changed to use this class. Note that the underlying code could probably use some cleaning up. e.g. adding a copy ctor to Tag instead of making a duplicateTag() function. ;-) I've tested this on a temp directory of copied music, so I'm pretty confident this all works and is safe. However, you may want to test on a backup first to make sure it works on your system too. =D Now the question is whether I should backport this or not. :-/ CCMAIL:61340-done@bugs.kde.org M +1 -0 Makefile.am 1.69 M +9 -0 collectionlist.cpp 1.89 M +2 -0 collectionlist.h 1.59 M +9 -0 juk.cpp 1.225 M +1 -0 juk.h 1.96 M +10 -8 musicbrainzquery.cpp 1.23 M +21 -25 playlist.cpp 1.258 M +16 -0 playlistbox.cpp 1.127 M +3 -0 playlistbox.h 1.71 M +8 -8 playlistitem.cpp 1.95 M +2 -2 playlistitem.h 1.60 M +6 -4 tag.h 1.34 M +35 -84 tageditor.cpp 1.72 M +3 -18 treeviewitemplaylist.cpp 1.8
CVS commit by mpyne: Doh!! I forgot to add these rather important files to CVS for bug 61340. :-( CCMAIL:61340@bugs.kde.org A tagtransactionmanager.cpp 1.1 [GPL (v2+)] A tagtransactionmanager.h 1.1 [GPL (v2+)]