From 2b44b945cf50e3dbfeabe4e5aee10db4b13f5a64 Mon Sep 17 00:00:00 2001 From: Manuel Nickschas Date: Tue, 25 Mar 2014 00:11:39 +0100 Subject: [PATCH 1/1] Use QSslCertificate::isNull() instead of isValid() in IdentityEditWidget The intention is to check if the certificate could be loaded, and isNull() is actually the right thing to do. Saves us from dealing with the no-longer existing isValid() in Qt5, too. --- src/qtui/settingspages/identityeditwidget.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/qtui/settingspages/identityeditwidget.cpp b/src/qtui/settingspages/identityeditwidget.cpp index be889c29..65858912 100644 --- a/src/qtui/settingspages/identityeditwidget.cpp +++ b/src/qtui/settingspages/identityeditwidget.cpp @@ -300,7 +300,7 @@ void IdentityEditWidget::sslDropEvent(QDropEvent *event, bool isCert) if (isCert) { QSslCertificate cert = certByFilename(filename); - if (cert.isValid()) + if (!cert.isNull()) showCertState(cert); } else { @@ -393,7 +393,7 @@ QSslCertificate IdentityEditWidget::certByFilename(const QString &filename) for (int i = 0; i < 2; i++) { cert = QSslCertificate(certRaw, (QSsl::EncodingFormat)i); - if (cert.isValid()) + if (!cert.isNull()) break; } return cert; @@ -402,7 +402,7 @@ QSslCertificate IdentityEditWidget::certByFilename(const QString &filename) void IdentityEditWidget::showCertState(const QSslCertificate &cert) { - if (!cert.isValid()) { + if (cert.isNull()) { ui.certOrgLabel->setText(tr("No Certificate loaded")); ui.certCNameLabel->setText(tr("No Certificate loaded")); ui.clearOrLoadCertButton->setText(tr("Load")); -- 2.20.1