Remove code duplication by implementing displayPinBlockedError() via …

…displayFatalError() WE2-479 Signed-off-by: Mart Somermaa <[email protected]>
web-eid · Aug 30, 2024 · 355ac18 · 355ac18
1 parent 4f11b50
commit 355ac18
Showing 1 changed file with 4 additions and 13 deletions.
diff --git a/src/ui/webeiddialog.cpp b/src/ui/webeiddialog.cpp
@@ -465,7 +465,6 @@ void WebEidDialog::onVerifyPinFailed(const VerifyPinFailed::Status status, const
         break;
     case Status::PIN_BLOCKED:
         displayPinBlockedError();
-        resizeHeight();
         return;
     case Status::INVALID_PIN_LENGTH:
         message = [] { return tr("Invalid PIN length"); };
@@ -482,7 +481,7 @@ void WebEidDialog::onVerifyPinFailed(const VerifyPinFailed::Status status, const
     case Status::UNKNOWN_ERROR:
         message = [] { return tr("Technical error"); };
         displayFatalError(message);
-        break;
+        return;
     }
 
     ui->pinErrorLabel->setVisible(bool(message));
@@ -682,17 +681,7 @@ void WebEidDialog::setupOK(Func func, const char* text, bool enabled)
 
 void WebEidDialog::displayPinBlockedError()
 {
-    ui->pinTitleLabel->hide();
-    ui->pinInput->hide();
-    ui->pinTimeoutTimer->stop();
-    ui->pinTimeRemaining->hide();
-    ui->pinEntryTimeoutProgressBar->hide();
-    setTrText(ui->pinErrorLabel, QT_TR_NOOP("PIN is locked. Unblock and try again."));
-    ui->pinErrorLabel->show();
-    ui->okButton->hide();
-    ui->cancelButton->setEnabled(true);
-    ui->cancelButton->show();
-    ui->helpButton->show();
+    displayFatalError([] { return tr("PIN is locked. Unblock and try again."); });
 }
 
 void WebEidDialog::displayFatalError(std::function<QString()> message)
@@ -707,6 +696,8 @@ void WebEidDialog::displayFatalError(std::function<QString()> message)
     ui->okButton->hide();
     ui->cancelButton->setEnabled(true);
     ui->cancelButton->show();
+    ui->helpButton->show();
+    resizeHeight();
 }
 
 void WebEidDialog::showPinInputWarning(bool show)