Review Request 128354: Fix wrong path to existing wallet files

[Burkhard Lück]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128354/
-----------------------------------------------------------

(Updated Aug. 11, 2016, 1:05 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Utils and Valentin Rusu.


Changes
-------

Submitted with commit 0a7279fc0ebab9c8dbc94c6035751d0500aa342d by Burkhard Lück to branch Applications/16.08.


Bugs: 358469 and 363532
    http://bugs.kde.org/show_bug.cgi?id=358469
    http://bugs.kde.org/show_bug.cgi?id=363532


Repository: kwalletmanager


Description
-------

the wallet files *.kwl + *.salt are located in a subdir "/kwalletd/" in QStandardPaths::GenericDataLocation
see 
kwalletmanager.cpp:390:    const QString path = QStandardPaths::writableLocation(QStandardPaths::GenericDataLocation) + "/kwalletd/";
kwalletmanager.cpp:420:    const QString destinationDir = QStandardPaths::writableLocation(QStandardPaths::GenericDataLocation) + "/kwalletd/";

But the code uses QStandardPaths::DataLocation) + "/kwallet/" therefore existing wallet files are not found
and the config entries "Auto Allow" and "Auto Deny" are always deleted


Diffs
-----

  src/konfigurator/konfigurator.cpp e4fa19f 

Diff: https://git.reviewboard.kde.org/r/128354/diff/


Testing
-------

With the patch the config entries "Auto Allow" and "Auto Deny" are preserved in kwalletrc


Thanks,

Burkhard Lück

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20160811/1bd19179/attachment.html>