1 files changed, 28 insertions, 24 deletions

diff --git a/framework/src/domain/mime/partmodel.cpp b/framework/src/domain/mime/partmodel.cpp
index bf11b52d..c4e002b7 100644
--- a/framework/src/domain/mime/partmodel.cpp
+++ b/framework/src/domain/mime/partmodel.cpp
@@ -139,6 +139,32 @@ static QString addCss(const QString &s)
     return header + s + QStringLiteral("</body></html>");
 }
 
+
+SignatureInfo *signatureInfo(MimeTreeParser::MessagePart *messagePart)
+{
+    auto signatureInfo = new SignatureInfo;
+    const auto signatureParts = messagePart->signatures();
+    if (signatureParts.size() > 1) {
+        qWarning() << "Can't deal with more than one signature";
+    }
+    for (const auto &p : signatureParts) {
+        signatureInfo->keyId = p->partMetaData()->keyId;
+        signatureInfo->keyMissing = p->partMetaData()->sigSummary & GpgME::Signature::KeyMissing;
+        signatureInfo->keyExpired = p->partMetaData()->sigSummary & GpgME::Signature::KeyExpired;
+        signatureInfo->keyRevoked = p->partMetaData()->sigSummary & GpgME::Signature::KeyRevoked;
+        signatureInfo->sigExpired = p->partMetaData()->sigSummary & GpgME::Signature::SigExpired;
+        signatureInfo->crlMissing = p->partMetaData()->sigSummary & GpgME::Signature::CrlMissing;
+        signatureInfo->crlTooOld = p->partMetaData()->sigSummary & GpgME::Signature::CrlTooOld;
+        signatureInfo->signer = p->partMetaData()->signer;
+        signatureInfo->signClass = p->partMetaData()->signClass;
+        signatureInfo->signerMailAddresses = p->partMetaData()->signerMailAddresses;
+        signatureInfo->signatureIsGood = p->partMetaData()->isGoodSignature;
+        signatureInfo->keyIsTrusted = p->partMetaData()->keyTrust & GpgME::Signature::Full ||
+                                        p->partMetaData()->keyTrust & GpgME::Signature::Ultimate;
+    }
+    return signatureInfo;
+}
+
 QVariant PartModel::data(const QModelIndex &index, int role) const
 {
     if (!index.isValid()) {
@@ -236,30 +262,8 @@ QVariant PartModel::data(const QModelIndex &index, int role) const
                 //No info
                 return "unknown";
             }
-            case SignatureDetails: {
-                auto signatureInfo = new SignatureInfo;
-
-                const auto signatureParts = messagePart->signatures();
-                if (signatureParts.size() > 1) {
-                    qWarning() << "Can't deal with more than one signature";
-                }
-                for (const auto &p : signatureParts) {
-                    signatureInfo->mKeyId = p->partMetaData()->keyId;
-                    signatureInfo->mKeyMissing = p->partMetaData()->sigSummary & GpgME::Signature::KeyMissing;
-                    signatureInfo->mKeyExpired = p->partMetaData()->sigSummary & GpgME::Signature::KeyExpired;
-                    signatureInfo->mKeyRevoked = p->partMetaData()->sigSummary & GpgME::Signature::KeyRevoked;
-                    signatureInfo->mSigExpired = p->partMetaData()->sigSummary & GpgME::Signature::SigExpired;
-                    signatureInfo->mCrlMissing = p->partMetaData()->sigSummary & GpgME::Signature::CrlMissing;
-                    signatureInfo->mCrlTooOld = p->partMetaData()->sigSummary & GpgME::Signature::CrlTooOld;
-                    signatureInfo->mSigner = p->partMetaData()->signer;
-                    signatureInfo->mSignClass = p->partMetaData()->signClass;
-                    signatureInfo->mSignerMailAddresses = p->partMetaData()->signerMailAddresses;
-                    signatureInfo->mSignatureIsGood = p->partMetaData()->isGoodSignature;
-                    signatureInfo->mKeyIsTrusted = p->partMetaData()->keyTrust & GpgME::Signature::Full ||
-                                                   p->partMetaData()->keyTrust & GpgME::Signature::Ultimate;
-                }
-                return QVariant::fromValue(signatureInfo);
-            }
+            case SignatureDetails:
+                return QVariant::fromValue(signatureInfo(messagePart));
             case ErrorType:
                 return messagePart->error();
             case ErrorString: {