Bug 168016 - "Key does not contain email address" warning when configuring an identity: cancel does not work as expected
Summary: "Key does not contain email address" warning when configuring an identity: ca...
Status: RESOLVED FIXED
Alias: None
Product: kmail
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR minor
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-01 18:02 UTC by Alex Merry
Modified: 2008-08-02 15:13 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Merry 2008-08-01 18:02:57 UTC
Version:           r840630 (using Devel)
Installed from:    Compiled sources
OS:                Linux

Steps:

1. Create a new identity (or modify an existing one) so that an OpenPGP signing key is specified, but where the email address for the identity does not match any email addresses attached to the signing key.
2. When clicking OK, a warning is displayed about this, with two options: continue or cancel.
3. Clicking cancel does not return you to the "Edit Identity" dialog, but proceeds as if you had clicked continue.

Instead, it should return you to the "Edit Identity" dialog so that you can correct it if necessary.
Comment 1 Thomas McGuire 2008-08-01 19:37:10 UTC
SVN commit 840749 by tmcguire:

Make the cancel button really work when the identity dialog complains about some error.
BUG: 168016


 M  +11 -5     identitydialog.cpp  
 M  +2 -1      identitydialog.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=840749
Comment 2 Thomas McGuire 2008-08-02 15:13:13 UTC
SVN commit 841027 by tmcguire:

Backport r840749 by tmcguire to the 4.1 branch:

Make the cancel button really work when the identity dialog complains about some error.
CCBUG: 168016



 M  +11 -5     identitydialog.cpp  
 M  +2 -1      identitydialog.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=841027