Move Not Available string away from the early convertion code and into ui code

generators/poppler/pdfsettingswidget.cpp

@@ -19,6 +19,14 @@
 #include <QTreeWidget>
 #include <QTreeWidgetItem>
 
+static QString notAvailableIfEmpty(const QString &string)
+{
+    if (string.isEmpty()) {
+        return i18n("Not Available");
+    }
+    return string;
+}
+
 PDFSettingsWidget::PDFSettingsWidget(QWidget *parent)
     : QWidget(parent)
 {
@@ -76,7 +84,9 @@ bool PDFSettingsWidget::event(QEvent *e)
 
         for (auto cert : certs) {
             new QTreeWidgetItem(m_tree,
-                                {cert->subjectInfo(Okular::CertificateInfo::EntityInfoKey::CommonName), cert->subjectInfo(Okular::CertificateInfo::EntityInfoKey::EmailAddress), cert->validityEnd().toString(QStringLiteral("yyyy-MM-dd"))});
+                                {notAvailableIfEmpty(cert->subjectInfo(Okular::CertificateInfo::EntityInfoKey::CommonName)),
+                                 notAvailableIfEmpty(cert->subjectInfo(Okular::CertificateInfo::EntityInfoKey::EmailAddress)),
+                                 cert->validityEnd().toString(QStringLiteral("yyyy-MM-dd"))});
         }
         qDeleteAll(certs);
 

generators/poppler/pdfsignatureutils.cpp

@@ -36,14 +36,12 @@ QByteArray PopplerCertificateInfo::serialNumber() const
 
 QString PopplerCertificateInfo::issuerInfo(PopplerCertificateInfo::EntityInfoKey key) const
 {
-    QString str = m_info.issuerInfo(static_cast<Poppler::CertificateInfo::EntityInfoKey>(key));
-    return !str.isEmpty() ? str : i18n("Not Available");
+    return m_info.issuerInfo(static_cast<Poppler::CertificateInfo::EntityInfoKey>(key));
 }
 
 QString PopplerCertificateInfo::subjectInfo(PopplerCertificateInfo::EntityInfoKey key) const
 {
-    QString str = m_info.subjectInfo(static_cast<Poppler::CertificateInfo::EntityInfoKey>(key));
-    return !str.isEmpty() ? str : i18n("Not Available");
+    return m_info.subjectInfo(static_cast<Poppler::CertificateInfo::EntityInfoKey>(key));
 }
 
 QString PopplerCertificateInfo::nickName() const

part/certificateviewer.cpp

@@ -111,6 +111,14 @@ static QString splitDNAttributes(const QString &text)
     return splitDNAttributes(QStringList {text});
 }
 
+static QString notAvailableIfEmpty(const QString &string)
+{
+    if (string.isEmpty()) {
+        return i18n("Not Available");
+    }
+    return string;
+}
+
 CertificateViewer::CertificateViewer(const Okular::CertificateInfo &certInfo, QWidget *parent)
     : KPageDialog(parent)
     , m_certificateInfo(certInfo)
@@ -132,16 +140,16 @@ CertificateViewer::CertificateViewer(const Okular::CertificateInfo &certInfo, QW
     auto issuerBox = new QGroupBox(i18n("Issued By"), generalPage);
     auto issuerFormLayout = new QFormLayout(issuerBox);
     issuerFormLayout->setLabelAlignment(Qt::AlignLeft);
-    issuerFormLayout->addRow(i18n("Common Name(CN)"), new QLabel(m_certificateInfo.issuerInfo(Okular::CertificateInfo::CommonName)));
-    issuerFormLayout->addRow(i18n("EMail"), new QLabel(m_certificateInfo.issuerInfo(Okular::CertificateInfo::EmailAddress)));
-    issuerFormLayout->addRow(i18n("Organization(O)"), new QLabel(m_certificateInfo.issuerInfo(Okular::CertificateInfo::Organization)));
+    issuerFormLayout->addRow(i18n("Common Name(CN)"), new QLabel(notAvailableIfEmpty(m_certificateInfo.issuerInfo(Okular::CertificateInfo::CommonName))));
+    issuerFormLayout->addRow(i18n("EMail"), new QLabel(notAvailableIfEmpty(m_certificateInfo.issuerInfo(Okular::CertificateInfo::EmailAddress))));
+    issuerFormLayout->addRow(i18n("Organization(O)"), new QLabel(notAvailableIfEmpty(m_certificateInfo.issuerInfo(Okular::CertificateInfo::Organization))));
 
     auto subjectBox = new QGroupBox(i18n("Issued To"), generalPage);
     auto subjectFormLayout = new QFormLayout(subjectBox);
     subjectFormLayout->setLabelAlignment(Qt::AlignLeft);
-    subjectFormLayout->addRow(i18n("Common Name(CN)"), new QLabel(m_certificateInfo.subjectInfo(Okular::CertificateInfo::CommonName)));
-    subjectFormLayout->addRow(i18n("EMail"), new QLabel(m_certificateInfo.subjectInfo(Okular::CertificateInfo::EmailAddress)));
-    subjectFormLayout->addRow(i18n("Organization(O)"), new QLabel(m_certificateInfo.subjectInfo(Okular::CertificateInfo::Organization)));
+    subjectFormLayout->addRow(i18n("Common Name(CN)"), new QLabel(notAvailableIfEmpty(m_certificateInfo.subjectInfo(Okular::CertificateInfo::CommonName))));
+    subjectFormLayout->addRow(i18n("EMail"), new QLabel(notAvailableIfEmpty(m_certificateInfo.subjectInfo(Okular::CertificateInfo::EmailAddress))));
+    subjectFormLayout->addRow(i18n("Organization(O)"), new QLabel(notAvailableIfEmpty(m_certificateInfo.subjectInfo(Okular::CertificateInfo::Organization))));
 
     auto validityBox = new QGroupBox(i18n("Validity"), generalPage);
     auto validityFormLayout = new QFormLayout(validityBox);