kindred/create
1
1
Fork 0
Code Issues 37 Pull Requests Actions Packages Projects 9 Releases 2 Wiki Activity

Gui: ReportView - include notifier string

Browse Source 
=========================================

Console error, warnings, ... allow for a notifiername. This appears separately in the notification area. However, the report view ignores it.

This commit prepends the notifier string separated by a semicolon, if not empty.

It solves the problem that, if the message did not include the notifier, this was not indicated in the Report View.

If the message included the notifier, then it was duplicated in the notification area.
This commit is contained in:
Abdullah Tahiri
2023-11-11 12:41:21 +01:00
committed by WandererFan
parent a9a25eae69
commit 4e7aa94b32
1 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/Gui/ReportView.cpp
View File

@@ -469,8 +469,6 @@ void ReportOutput::restoreFont()
void ReportOutput::SendLog(const std::string& notifiername, const std::string& msg, Base::LogStyle level,
Base::IntendedRecipient recipient, Base::ContentType content)
{
(void) notifiername;
// Do not log translated messages, or messages intended only to the user to the Report View
if( recipient == Base::IntendedRecipient::User ||
content == Base::ContentType::Translated)
@@ -497,7 +495,15 @@ void ReportOutput::SendLog(const std::string& notifiername, const std::string& m
break;
}
QString qMsg = QString::fromUtf8(msg.c_str());
QString qMsg;
if(!notifiername.empty()) {
qMsg = QStringLiteral("%1: %2").arg(QString::fromUtf8(notifiername.c_str()),
QString::fromUtf8(msg.c_str()));
}
else {
qMsg = QString::fromUtf8(msg.c_str());
}
// This truncates log messages that are too long
if (style == ReportHighlighter::LogText) {