Commit ea9549c3 authored by Felix Tiede's avatar Felix Tiede Committed by Ingo Klöcker
Browse files

Refactor access to identity's transport name.

See !16 (comment 427791)

There is only the transport name of an identity, which is of interest
throughout the code, so instead of handling an identity in the calling
functions, now there's just the name of an identity's transport returned
to be used, further reducing overhead.
parent 0509ff5e
......@@ -31,14 +31,14 @@ using namespace Kleo;
using namespace Kleo::Commands;
using namespace GpgME;
static const KIdentityManagement::Identity identityForAddress(const QString &senderAddress) {
static const QString identityTransportForAddress(const QString &senderAddress) {
static const KIdentityManagement::IdentityManager *idManager = new KIdentityManagement::IdentityManager(true);
const KIdentityManagement::Identity identity = idManager->identityForAddress(senderAddress);
if (identity.isNull())
return idManager->defaultIdentity();
return idManager->defaultIdentity().transport();
return identity;
return identity.transport();
ExportOpenPGPCertToProviderCommand::ExportOpenPGPCertToProviderCommand(QAbstractItemView *v, KeyListController *c)
......@@ -60,11 +60,10 @@ ExportOpenPGPCertToProviderCommand::~ExportOpenPGPCertToProviderCommand() {}
bool ExportOpenPGPCertToProviderCommand::preStartHook(QWidget *parent) const
QString sender = senderAddress();
const QString sender = senderAddress();
const QString transportName = identityTransportForAddress(sender);
KIdentityManagement::Identity identity = identityForAddress(sender);
if (identity.transport().isEmpty()) {
if (transportName.isEmpty()) {
"<para><email>%1</email> has no usable transport for mailing a key available, "
......@@ -91,11 +90,8 @@ bool ExportOpenPGPCertToProviderCommand::preStartHook(QWidget *parent) const
void ExportOpenPGPCertToProviderCommand::postSuccessHook(QWidget *parent)
QString sender = senderAddress();
KIdentityManagement::Identity identity = identityForAddress(sender);
MailTransport::Transport *transport = MailTransport::TransportManager::self()->transportByName(
if (!transport)
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment