From a5825d031521670d19c7fd4e181862a3fc7cc6ba Mon Sep 17 00:00:00 2001
From: Mart Somermaa <mrts@users.noreply.github.com>
Date: Fri, 30 Aug 2024 19:56:01 +0300
Subject: [PATCH] Remove code duplication by implementing
 displayPinBlockedError() via displayFatalError()

WE2-479

Signed-off-by: Mart Somermaa <mrts@users.noreply.github.com>
---
 src/ui/webeiddialog.cpp | 19 ++++---------------
 1 file changed, 4 insertions(+), 15 deletions(-)

diff --git a/src/ui/webeiddialog.cpp b/src/ui/webeiddialog.cpp
index 3c91341a..76f7ab75 100644
--- a/src/ui/webeiddialog.cpp
+++ b/src/ui/webeiddialog.cpp
@@ -20,8 +20,6 @@
  * SOFTWARE.
  */
 
-#include "utils/erasedata.hpp"
-
 #include "webeiddialog.hpp"
 #include "application.hpp"
 #include "punycode.hpp"
@@ -465,7 +463,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 +479,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 +679,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 +694,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)