From 322e58391765339cba68eccf83ef53b5c2645b8e Mon Sep 17 00:00:00 2001 From: Serhii Hadzhilov <71632867+SerhiiGadzhilov@users.noreply.github.com> Date: Wed, 11 May 2022 12:22:17 +0300 Subject: [PATCH] SYNERGY-1-1494 Replace notifications with logs (#7166) * SYNERGY-1-1494 Replace notifications with logs * Update ChangeLog --- ChangeLog | 1 + src/lib/client/ServerProxy.cpp | 8 ++++---- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/ChangeLog b/ChangeLog index d2c6670fc..0e65d8ce5 100644 --- a/ChangeLog +++ b/ChangeLog @@ -3,6 +3,7 @@ v1.14.4-snapshot Enhancements: - #7143 Add ability to stop synergy on the login screen - #7157 | #7160 Synergy business accepts only business licenses +- #7166 Replace annoying language notifications with warnings in logs Bug fixes: - #7144 Fix double lines when pasting text from Linux to Windows diff --git a/src/lib/client/ServerProxy.cpp b/src/lib/client/ServerProxy.cpp index 864ffd29e..1583aebbc 100644 --- a/src/lib/client/ServerProxy.cpp +++ b/src/lib/client/ServerProxy.cpp @@ -976,7 +976,7 @@ ServerProxy::setActiveServerLanguage(const String& language) if (!m_languageManager.isLanguageInstalled(m_serverLanguage)) { if(!m_isUserNotifiedAboutLanguageSyncError) { - AppUtil::instance().showNotification("Language error", "Current server language is not installed on client."); + LOG((CLOG_WARN "Current server language is not installed on client.")); m_isUserNotifiedAboutLanguageSyncError = true; } } @@ -994,8 +994,8 @@ ServerProxy::checkMissedLanguages() const { auto missedLanguages = m_languageManager.getMissedLanguages(); if (!missedLanguages.empty()) { - AppUtil::instance().showNotification("Language synchronization error", - "You need to install these languages on this computer and restart Synergy to enable support for multiple languages: " - + missedLanguages); + LOG((CLOG_WARN + "You need to install these languages on this computer and restart Synergy to enable support for multiple languages: %s", + missedLanguages.c_str())); } }