My original intention was to rename a main-folder in a wallet. because that is not possible, I tried to create a new folder and drag the passwords in it. But on dragging a password pair from one into another folder in the tree view crashes kwalletmanager. so btw.: I have the wish to be able to rename folders in kwalletmanager. why? for example, because the thunderbird extension creates a folder named "unknown", but it is possible to change that, but then, the passwords are not moved :-( Reproducible: Always
please always provide backtrace when reporting crashes.
I am not shure, if this helps: Application: Dienstprogramm für digitale Brieftaschen (kwalletmanager), signal: Segmentation fault Using host libthread_db library "/lib64/libthread_db.so.1". [KCrash Handler] #5 0x000000000042fb6c in _start () I have installed debug-symbols für kwalletmanager
I'm afraid current kwallet API won't allow implementing your wish. This is because kwallet identifies folders by name and not by ID. And applications always specify the folder's name. In this case, the thunderbird extensions should use a more specific folder name. I suggest you file a bug to that extension's developer. Meanwhile, I confirm I get a crash when dragging folders. Working on that.
Git commit dd8865824ac708ec48baad1617663cb4c4998add by Valentin Rusu. Committed on 30/09/2013 at 21:43. Pushed by vrusu into branch 'master'. Fix crash on drag&drop inside the wallet editor FIXED-IN: 4.12 M +2 -1 src/manager/allyourbase.cpp M +1 -0 src/manager/kwalleteditor.cpp http://commits.kde.org/kwallet/dd8865824ac708ec48baad1617663cb4c4998add
Thanks for fixing the crash. My original problem is still not solved. Explanation: > In this case, the thunderbird extensions should use a more specific folder name. I suggest you file a bug to that extension's developer. The problem is, that the extension can be used for thunderbird AND/OR firefox. So the extension's developer can not know, which name to set (firefox or thunderbird), so he built a configuration option for that name. Thats fine in general. I could add a "renameFolder" Method to the kwallet API and kwalletmanager. Would it be accepted? I already had a look at the code, the method in the backend would look similar to bool Backend::renameFolder(const QString &from, const QString &to) { if (!_entries.contains(from)) { return false; } if (_entries.contains(to)) { return false; } EntryMap entryMap = _entries.take(from); _entries.insert(to, entryMap); QCryptographicHash fromMd5(QCryptographicHash::Md5); fromMd5.addData(from.toUtf8()); QCryptographicHash toMd5(QCryptographicHash::Md5); toMd5.addData(to.toUtf8()); QList<MD5Digest> hashList = _hashes.take(MD5Digest(fromMd5.result())); _hashes.insert(MD5Digest(toMd5.result()), hashList); return true; }
It would be nice to get feedback about my last comment.
(In reply to H.H. from comment #6) > It would be nice to get feedback about my last comment. You will have more luck if you upload your patch here: https://git.reviewboard.kde.org/
I haven't written a fully patch yet, and also the gui part is missing completely. If you tell me, that my work will be accepted (at least after a few review-board iterations), I will prepare a patch for reviewboard. But I have not enough time, to invest much time into this, and then get told, that such a change would not be accepted in general.
(In reply to H.H. from comment #8) > I haven't written a fully patch yet, and also the gui part is missing > completely. If you tell me, that my work will be accepted (at least after a > few review-board iterations), I will prepare a patch for reviewboard. But I > have not enough time, to invest much time into this, and then get told, that > such a change would not be accepted in general. The main problem is that kwallet is currently unmaintained. But if your patch doesn't break anything and you convince enough people that it's something useful, I don't see why it shouldn't be accepted.
(In reply to H.H. from comment #8) > I haven't written a fully patch yet, and also the gui part is missing > completely. If you tell me, that my work will be accepted (at least after a > few review-board iterations), I will prepare a patch for reviewboard. But I > have not enough time, to invest much time into this, and then get told, that > such a change would not be accepted in general. Was this patch eventually added and/or merged? I believe this ticket can be closed anyway as it is about the crash which is now fixed according to comments. This bug has been set to "NEEDSINFO/WAITINGFORINFO", if you are able to update the bug report please mark it as reported when you add the new information, thank you.
Ok, I'm closing. But renaming was not added, and seems still a good idea to add as a feature.