refactor: remove messages::showClientError, fix mainwindow calling showandactive for non error messages

This commit is contained in:
sithlord48
2026-02-24 07:31:55 -05:00
committed by Nick Bolton
parent 47ed13868f
commit c0692b6fc7
9 changed files with 60 additions and 113 deletions

View File

@ -1165,12 +1165,22 @@ void MainWindow::remoteHostChanged(const QString &newRemoteHost)
void MainWindow::showClientError(deskflow::client::ErrorType error, const QString &address) void MainWindow::showClientError(deskflow::client::ErrorType error, const QString &address)
{ {
if (!isVisible() || m_clientErrorVisible) if (!isVisible() || m_clientErrorVisible || error != deskflow::client::ErrorType::AlreadyConnected)
return; return;
m_clientErrorVisible = true; m_clientErrorVisible = true;
showAndActivate(); showAndActivate();
deskflow::gui::messages::showClientConnectError(this, error, address);
QMessageBox::warning(
this, tr("%1 Connection Error").arg(kAppName),
tr("<p>Failed to connect to the server '%1'.</p>"
"<p>A Client with your name is already connected to the server.</p>"
"Please ensure that you're using a unique name and that only a "
"single instance of the client process is running.</p>")
.arg(address)
);
m_clientErrorVisible = false; m_clientErrorVisible = false;
} }

View File

@ -183,43 +183,6 @@ void showFirstConnectedMessage(QWidget *parent, bool closeToTray, bool enableSer
QMessageBox::information(parent, title, message); QMessageBox::information(parent, title, message);
} }
void showClientConnectError(QWidget *parent, deskflow::client::ErrorType error, const QString &address)
{
using enum deskflow::client::ErrorType;
if (error == NoError || error == GenericError || error == HostnameError)
return;
auto message = QObject::tr("<p>Failed to connect to the server '%1'.</p>").arg(address);
if (error == AlreadyConnected) {
message.append(
QObject::tr( //
"<p>A Client with your name is already connected to the server.</p>"
"Please ensure that you're using a unique name and that only a "
"single instance of the client process is running.</p>"
)
);
} else {
qFatal("unknown client error");
}
auto title = QObject::tr("%1 Connection Error").arg(kAppName);
if (error != HostnameError) {
QMessageBox::warning(parent, title, message);
return;
}
auto dialog = QMessageBox(parent);
dialog.setWindowTitle(title);
dialog.setText(message);
dialog.setWindowModality(Qt::ApplicationModal);
dialog.setIcon(QMessageBox::Information);
dialog.setDefaultButton(QMessageBox::Ok);
dialog.exec();
}
bool showNewClientPrompt(QWidget *parent, const QString &clientName, bool serverRequiresPeerAuth) bool showNewClientPrompt(QWidget *parent, const QString &clientName, bool serverRequiresPeerAuth)
{ {
if (serverRequiresPeerAuth) { if (serverRequiresPeerAuth) {

View File

@ -25,8 +25,6 @@ void showFirstConnectedMessage(QWidget *parent, bool closeToTray, bool enableSer
void showCloseReminder(QWidget *parent); void showCloseReminder(QWidget *parent);
void showClientConnectError(QWidget *parent, deskflow::client::ErrorType error, const QString &address);
bool showNewClientPrompt(QWidget *parent, const QString &clientName, bool serverRequiresPeerAuth = false); bool showNewClientPrompt(QWidget *parent, const QString &clientName, bool serverRequiresPeerAuth = false);
bool showClearSettings(QWidget *parent); bool showClearSettings(QWidget *parent);

View File

@ -386,6 +386,10 @@ Do you want to connect to the server?
<extracomment>stop core shortcut</extracomment> <extracomment>stop core shortcut</extracomment>
<translation type="unfinished">Ctrl+T</translation> <translation type="unfinished">Ctrl+T</translation>
</message> </message>
<message>
<source>&lt;p&gt;Failed to connect to the server &apos;%1&apos;.&lt;/p&gt;&lt;p&gt;A Client with your name is already connected to the server.&lt;/p&gt;Please ensure that you&apos;re using a unique name and that only a single instance of the client process is running.&lt;/p&gt;</source>
<translation>&lt;p&gt;Error al conectar con el servidor &apos;%1&apos;.&lt;/p&gt;&lt;p&gt;A Client with your name is already connected to the server.&lt;/p&gt;Please ensure that you&apos;re using a unique name and that only a single instance of the client process is running.&lt;/p&gt;</translation>
</message>
<message> <message>
<source>Your current TLS key is smaller than the minimum allowed size, A new key 2048-bit key will be generated.</source> <source>Your current TLS key is smaller than the minimum allowed size, A new key 2048-bit key will be generated.</source>
<translation type="unfinished">Su clave TLS actual es más pequeña que el tamaño mínimo permitido. Se generará una nueva clave de 2048 bits.</translation> <translation type="unfinished">Su clave TLS actual es más pequeña que el tamaño mínimo permitido. Se generará una nueva clave de 2048 bits.</translation>
@ -430,6 +434,10 @@ Do you want to connect to the server?
<source>Disconnect</source> <source>Disconnect</source>
<translation type="unfinished">Desconectar</translation> <translation type="unfinished">Desconectar</translation>
</message> </message>
<message>
<source>%1 Connection Error</source>
<translation>%1 Error de conexión</translation>
</message>
<message> <message>
<source>No IP Detected</source> <source>No IP Detected</source>
<translation type="unfinished">No se detectó ninguna IP</translation> <translation type="unfinished">No se detectó ninguna IP</translation>
@ -610,18 +618,6 @@ Nombres válidos:
<source>%1 Connected</source> <source>%1 Connected</source>
<translation type="unfinished">%1 Conectado</translation> <translation type="unfinished">%1 Conectado</translation>
</message> </message>
<message>
<source>&lt;p&gt;Failed to connect to the server &apos;%1&apos;.&lt;/p&gt;</source>
<translation type="unfinished">&lt;p&gt;Error al conectar con el servidor &apos;%1&apos;.&lt;/p&gt;</translation>
</message>
<message>
<source>&lt;p&gt;A Client with your name is already connected to the server.&lt;/p&gt;Please ensure that you&apos;re using a unique name and that only a single instance of the client process is running.&lt;/p&gt;</source>
<translation type="unfinished">&lt;p&gt;Ya hay un cliente conectado al servidor con su nombre.&lt;/p&gt;Asegúrese de utilizar un nombre único y de que solo se esté ejecutando una única instancia del proceso del cliente.&lt;/p&gt;</translation>
</message>
<message>
<source>%1 Connection Error</source>
<translation type="unfinished">%1 Error de conexión</translation>
</message>
<message> <message>
<source>%1 - New Client</source> <source>%1 - New Client</source>
<translation type="unfinished">%1 - Nuevo cliente</translation> <translation type="unfinished">%1 - Nuevo cliente</translation>

View File

@ -374,6 +374,10 @@ Vuoi connetterti al server?
<extracomment>stop core shortcut</extracomment> <extracomment>stop core shortcut</extracomment>
<translation>Ctrl+F</translation> <translation>Ctrl+F</translation>
</message> </message>
<message>
<source>&lt;p&gt;Failed to connect to the server &apos;%1&apos;.&lt;/p&gt;&lt;p&gt;A Client with your name is already connected to the server.&lt;/p&gt;Please ensure that you&apos;re using a unique name and that only a single instance of the client process is running.&lt;/p&gt;</source>
<translation>&lt;p&gt;Impossibile connettersi al server &quot;%1&quot;.&lt;/p&gt;&lt;p&gt;Un client con il tuo nome è già connesso al server.&lt;/p&gt;Assicurati di utilizzare un nome univoco e che sia in esecuzione una sola istanza del processo client.&lt;/p&gt;</translation>
</message>
<message> <message>
<source>Your current TLS key is smaller than the minimum allowed size, A new key 2048-bit key will be generated.</source> <source>Your current TLS key is smaller than the minimum allowed size, A new key 2048-bit key will be generated.</source>
<translation>La tua chiave TLS attuale è più piccola della dimensione minima consentita. Verrà generata una nuova chiave a 2048 bit.</translation> <translation>La tua chiave TLS attuale è più piccola della dimensione minima consentita. Verrà generata una nuova chiave a 2048 bit.</translation>
@ -418,6 +422,10 @@ Vuoi connetterti al server?
<source>Disconnect</source> <source>Disconnect</source>
<translation>Disconnetti</translation> <translation>Disconnetti</translation>
</message> </message>
<message>
<source>%1 Connection Error</source>
<translation>Errore di connessione %1</translation>
</message>
<message> <message>
<source>No IP Detected</source> <source>No IP Detected</source>
<translation>Nessun IP rilevato</translation> <translation>Nessun IP rilevato</translation>
@ -610,18 +618,6 @@ Nomi validi:
<source>%1 Connected</source> <source>%1 Connected</source>
<translation>%1 Connesso</translation> <translation>%1 Connesso</translation>
</message> </message>
<message>
<source>&lt;p&gt;Failed to connect to the server &apos;%1&apos;.&lt;/p&gt;</source>
<translation>&lt;p&gt;Impossibile connettersi al server &quot;%1&quot;.&lt;/p&gt;</translation>
</message>
<message>
<source>&lt;p&gt;A Client with your name is already connected to the server.&lt;/p&gt;Please ensure that you&apos;re using a unique name and that only a single instance of the client process is running.&lt;/p&gt;</source>
<translation>&lt;p&gt;Un client con il tuo nome è già connesso al server.&lt;/p&gt;Assicurati di utilizzare un nome univoco e che sia in esecuzione una sola istanza del processo client.&lt;/p&gt;</translation>
</message>
<message>
<source>%1 Connection Error</source>
<translation>Errore di connessione %1</translation>
</message>
<message> <message>
<source>%1 - New Client</source> <source>%1 - New Client</source>
<translation>%1 - Nuovo Client</translation> <translation>%1 - Nuovo Client</translation>

View File

@ -390,6 +390,10 @@ Do you want to connect to the server?
<source>Disconnect</source> <source>Disconnect</source>
<translation></translation> <translation></translation>
</message> </message>
<message>
<source>&lt;p&gt;Failed to connect to the server &apos;%1&apos;.&lt;/p&gt;&lt;p&gt;A Client with your name is already connected to the server.&lt;/p&gt;Please ensure that you&apos;re using a unique name and that only a single instance of the client process is running.&lt;/p&gt;</source>
<translation>&lt;p&gt; &apos;%1&apos; &lt;/p&gt;&lt;p&gt;&lt;/p&gt;&lt;p&gt;&lt;/p&gt;</translation>
</message>
<message> <message>
<source>No IP Detected</source> <source>No IP Detected</source>
<translation>IPアドレスが見つかりません</translation> <translation>IPアドレスが見つかりません</translation>
@ -511,6 +515,10 @@ Valid names:
_ - _ -
1255</translation> 1255</translation>
</message> </message>
<message>
<source>%1 Connection Error</source>
<translation>%1 </translation>
</message>
<message> <message>
<source>Suggested IP: </source> <source>Suggested IP: </source>
<translation>IPアドレス: </translation> <translation>IPアドレス: </translation>
@ -612,18 +620,6 @@ Valid names:
<source>%1 Connected</source> <source>%1 Connected</source>
<translation>%1 </translation> <translation>%1 </translation>
</message> </message>
<message>
<source>&lt;p&gt;Failed to connect to the server &apos;%1&apos;.&lt;/p&gt;</source>
<translation>&lt;p&gt; &apos;%1&apos; &lt;/p&gt;</translation>
</message>
<message>
<source>&lt;p&gt;A Client with your name is already connected to the server.&lt;/p&gt;Please ensure that you&apos;re using a unique name and that only a single instance of the client process is running.&lt;/p&gt;</source>
<translation>&lt;p&gt;&lt;/p&gt;&lt;p&gt;&lt;/p&gt;</translation>
</message>
<message>
<source>%1 Connection Error</source>
<translation>%1 </translation>
</message>
<message> <message>
<source>%1 - New Client</source> <source>%1 - New Client</source>
<translation>%1 - </translation> <translation>%1 - </translation>

View File

@ -390,6 +390,10 @@ Do you want to connect to the server?
<source>Disconnect</source> <source>Disconnect</source>
<translation> </translation> <translation> </translation>
</message> </message>
<message>
<source>&lt;p&gt;Failed to connect to the server &apos;%1&apos;.&lt;/p&gt;&lt;p&gt;A Client with your name is already connected to the server.&lt;/p&gt;Please ensure that you&apos;re using a unique name and that only a single instance of the client process is running.&lt;/p&gt;</source>
<translation>&lt;p&gt; &apos;%1&apos; .&lt;/p&gt;&lt;p&gt; .&lt;/p&gt;&lt;p&gt; , .&lt;/p&gt;</translation>
</message>
<message> <message>
<source>No IP Detected</source> <source>No IP Detected</source>
<translation>IP를 </translation> <translation>IP를 </translation>
@ -511,6 +515,10 @@ Valid names:
_ - _ -
1~255</translation> 1~255</translation>
</message> </message>
<message>
<source>%1 Connection Error</source>
<translation>%1 </translation>
</message>
<message> <message>
<source>Suggested IP: </source> <source>Suggested IP: </source>
<translation type="unfinished"> IP: </translation> <translation type="unfinished"> IP: </translation>
@ -610,18 +618,6 @@ Valid names:
<source>%1 Connected</source> <source>%1 Connected</source>
<translation>%1 </translation> <translation>%1 </translation>
</message> </message>
<message>
<source>&lt;p&gt;Failed to connect to the server &apos;%1&apos;.&lt;/p&gt;</source>
<translation>&lt;p&gt; &apos;%1&apos; .&lt;/p&gt;</translation>
</message>
<message>
<source>&lt;p&gt;A Client with your name is already connected to the server.&lt;/p&gt;Please ensure that you&apos;re using a unique name and that only a single instance of the client process is running.&lt;/p&gt;</source>
<translation>&lt;p&gt; .&lt;/p&gt;&lt;p&gt; , .&lt;/p&gt;</translation>
</message>
<message>
<source>%1 Connection Error</source>
<translation>%1 </translation>
</message>
<message> <message>
<source>%1 - New Client</source> <source>%1 - New Client</source>
<translation>%1 - </translation> <translation>%1 - </translation>

View File

@ -390,6 +390,10 @@ Do you want to connect to the server?
<source>Disconnect</source> <source>Disconnect</source>
<translation>Отключиться</translation> <translation>Отключиться</translation>
</message> </message>
<message>
<source>&lt;p&gt;Failed to connect to the server &apos;%1&apos;.&lt;/p&gt;&lt;p&gt;A Client with your name is already connected to the server.&lt;/p&gt;Please ensure that you&apos;re using a unique name and that only a single instance of the client process is running.&lt;/p&gt;</source>
<translation>&lt;p&gt;Не удалось подключиться к серверу &apos;%1&apos;.&lt;/p&gt;&lt;p&gt;Клиент с таким именем уже подключен к серверу.&lt;/p&gt;Убедитесь, что вы используете уникальное имя и запущен только один процесс клиента.&lt;/p&gt;</translation>
</message>
<message> <message>
<source>No IP Detected</source> <source>No IP Detected</source>
<translation>IP-адрес не обнаружен</translation> <translation>IP-адрес не обнаружен</translation>
@ -511,6 +515,10 @@ Valid names:
Можно использовать _ или - Можно использовать _ или -
Длина от 1 до 255 символов</translation> Длина от 1 до 255 символов</translation>
</message> </message>
<message>
<source>%1 Connection Error</source>
<translation>Ошибка соединения %1</translation>
</message>
<message> <message>
<source>Suggested IP: </source> <source>Suggested IP: </source>
<translation>Рекомендуемый IP-адрес: </translation> <translation>Рекомендуемый IP-адрес: </translation>
@ -610,18 +618,6 @@ Valid names:
<source>%1 Connected</source> <source>%1 Connected</source>
<translation>%1 подключено</translation> <translation>%1 подключено</translation>
</message> </message>
<message>
<source>&lt;p&gt;Failed to connect to the server &apos;%1&apos;.&lt;/p&gt;</source>
<translation>&lt;p&gt;Не удалось подключиться к серверу &apos;%1&apos;.&lt;/p&gt;</translation>
</message>
<message>
<source>&lt;p&gt;A Client with your name is already connected to the server.&lt;/p&gt;Please ensure that you&apos;re using a unique name and that only a single instance of the client process is running.&lt;/p&gt;</source>
<translation>&lt;p&gt;Клиент с таким именем уже подключен к серверу.&lt;/p&gt;Убедитесь, что вы используете уникальное имя и запущен только один процесс клиента.&lt;/p&gt;</translation>
</message>
<message>
<source>%1 Connection Error</source>
<translation>Ошибка соединения %1</translation>
</message>
<message> <message>
<source>%1 - New Client</source> <source>%1 - New Client</source>
<translation>%1 - Новый клиент</translation> <translation>%1 - Новый клиент</translation>

View File

@ -390,6 +390,10 @@ Do you want to connect to the server?
<source>Disconnect</source> <source>Disconnect</source>
<translation></translation> <translation></translation>
</message> </message>
<message>
<source>&lt;p&gt;Failed to connect to the server &apos;%1&apos;.&lt;/p&gt;&lt;p&gt;A Client with your name is already connected to the server.&lt;/p&gt;Please ensure that you&apos;re using a unique name and that only a single instance of the client process is running.&lt;/p&gt;</source>
<translation>&lt;p&gt;%1&lt;/p&gt;&lt;p&gt;&lt;/p&gt;使&lt;/p&gt;</translation>
</message>
<message> <message>
<source>No IP Detected</source> <source>No IP Detected</source>
<translation> IP</translation> <translation> IP</translation>
@ -511,6 +515,10 @@ Valid names:
使 _ - 使 _ -
1 255 </translation> 1 255 </translation>
</message> </message>
<message>
<source>%1 Connection Error</source>
<translation>%1 </translation>
</message>
<message> <message>
<source>Suggested IP: </source> <source>Suggested IP: </source>
<translation type="unfinished"> IP </translation> <translation type="unfinished"> IP </translation>
@ -612,18 +620,6 @@ Valid names:
<source>%1 Connected</source> <source>%1 Connected</source>
<translation>%1 </translation> <translation>%1 </translation>
</message> </message>
<message>
<source>&lt;p&gt;Failed to connect to the server &apos;%1&apos;.&lt;/p&gt;</source>
<translation>&lt;p&gt;%1&lt;/p&gt;</translation>
</message>
<message>
<source>&lt;p&gt;A Client with your name is already connected to the server.&lt;/p&gt;Please ensure that you&apos;re using a unique name and that only a single instance of the client process is running.&lt;/p&gt;</source>
<translation type="unfinished">&lt;p&gt;&lt;/p&gt;使&lt;/p&gt;</translation>
</message>
<message>
<source>%1 Connection Error</source>
<translation>%1 </translation>
</message>
<message> <message>
<source>%1 - New Client</source> <source>%1 - New Client</source>
<translation>%1 - </translation> <translation>%1 - </translation>