Bug 358469

Summary:	kwalletmanager5 resets Auto Allow setting
Product:	[Applications] kwalletmanager	Reporter:	Filippe <gasinvein>
Component:	general	Assignee:	Valentin Rusu <valir>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	openSUSE
OS:	Linux
Latest Commit:	http://commits.kde.org/kwallet/0a7279fc0ebab9c8dbc94c6035751d0500aa342d	Version Fixed In:
Sentry Crash Report:

Description Filippe 2016-01-24 10:43:34 UTC

If "Prompt on Open" in kwalletrc is set to true, opening (even without changing any settings) kwalletmanager5 causes deletion of [Auto Allow] section.

Reproducible: Always

Steps to Reproduce:
1. Check "Prompt when application acceses a wallet" setting.
2. Select "always allow" when prompted to give access to wallet for a program.
3. Open kwalletmanager.
4. Re-login and launch that program again.

Actual Results:  
Access request appeares again.

Comment 1 Burkhard Lück 2016-08-11 13:05:41 UTC

Git commit 0a7279fc0ebab9c8dbc94c6035751d0500aa342d by Burkhard Lück.
Committed on 11/08/2016 at 13:04.
Pushed by lueck into branch 'Applications/16.08'.

Fix wrong path to existing wallet files

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 in konfigurator.cpp used QStandardPaths::DataLocation) + "/kwallet/" therefore existing wallet files were not found
and the config entries "Auto Allow" and "Auto Deny" were always deleted
With the patch the config entries "Auto Allow" and "Auto Deny" are preserved in kwalletrc
REVIEW:128354
Related: bug 363532

M  +2    -2    src/konfigurator/konfigurator.cpp

http://commits.kde.org/kwallet/0a7279fc0ebab9c8dbc94c6035751d0500aa342d