GpgEX - Preselection to enc/sign/both does not match
ClosedPublic

Authored by sengelhardt on Apr 5 2016, 9:52 AM.

Details

Summary

see https://phabricator.kde.org/T2132

When performing tasks like "sign and encrypt files" from the contextmenu in Windows Explorer, the preselected entrys in the Kleo do not match with the previous selection.

Test Plan

ensured the program behaves how expected using
gpg-connect-agent -v ~/.gnupg/S.uiserver

and using the required commands to sign, encrypt or sign_encrypt a file.

Compiled Kleo for windows, patching existing kleo installation with newly created Kleopatra.exe

Diff Detail

Repository
R168 Kleopatra
Lint
Lint Skipped
Unit
Unit Tests Skipped
sengelhardt updated this revision to Diff 3135.Apr 5 2016, 9:52 AM
sengelhardt retitled this revision from to GpgEX - Preselection to enc/sign/both does not match.
sengelhardt updated this object.
sengelhardt edited the test plan for this revision. (Show Details)
sengelhardt added a reviewer: aheinecke.
sengelhardt set the repository for this revision to R168 Kleopatra.
sengelhardt added a subscriber: aheinecke.

I need to mention:

when using a "force command", like "sign" or "sign_encrypt" what will happen ist:

wizard->setSigningUserMutable(true);
wizard->setSigningPreset(false);
wizard->setSigningPreset(true);
(and the same for encrypt)

This seems inconvenient at first.
To ensure that signing and encrypting are working at the same time, the wizard currently needs this behaviour (or it will just not preselct anything)

aheinecke accepted this revision.Apr 5 2016, 10:18 AM
aheinecke edited edge metadata.

Looks good to me. Makes sense to interpret "Forced" as selected by default as we don't want to force the user to do anything.

Maybe in a second commit you could replace s/SignForced/SignSelected/ s/EncryptForced/EncryptSelected/ s/ArchiveForced/ArchiveSelected/ in kleo uiserver? So that it's more understandable in the code how the commands are interpreted.

This revision is now accepted and ready to land.Apr 5 2016, 10:18 AM
aacid closed this revision.Feb 26 2017, 10:19 PM