feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

CPaster: Use new message manager API

Browse Source 
Only show the panel if an error occurs, or if the user opted to have the
paste link shown to him.

Task-number: QTCREATORBUG-24430
Change-Id: Ie474acbe2b4d3b6d3332111bdd7ae18b16bd8618
Reviewed-by: hjk <hjk@qt.io>
This commit is contained in:
Eike Ziller
2020-12-01 14:41:17 +01:00
parent c400923308
commit 43249f655b
3 changed files with 12 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/plugins/cpaster/cpasterplugin.cpp
View File

@@ -326,7 +326,10 @@ void CodePasterPluginPrivate::finishPost(const QString &link)
{
if (m_settings.copyToClipboard)
QApplication::clipboard()->setText(link);
MessageManager::write(link, m_settings.displayOutput ? MessageManager::ModeSwitch : MessageManager::Silent);
if (m_settings.displayOutput)
MessageManager::writeDisrupting(link);
else
MessageManager::writeSilently(link);
}
// Extract the characters that can be used for a file name from a title
@@ -369,11 +372,12 @@ void CodePasterPluginPrivate::finishFetch(const QString &titleDescription,
{
// Failure?
if (error) {
MessageManager::write(content);
MessageManager::writeDisrupting(content);
return;
}
if (content.isEmpty()) {
MessageManager::write(tr("Empty snippet received for \"%1\".").arg(titleDescription));
MessageManager::writeDisrupting(
tr("Empty snippet received for \"%1\".").arg(titleDescription));
return;
}
// If the mime type has a preferred suffix (cpp/h/patch...), use that for
@@ -393,7 +397,7 @@ void CodePasterPluginPrivate::finishFetch(const QString &titleDescription,
saver.setAutoRemove(false);
saver.write(byteContent);
if (!saver.finalize()) {
MessageManager::write(saver.errorString());
MessageManager::writeDisrupting(saver.errorString());
return;
}
const QString fileName = saver.fileName();

4
src/plugins/cpaster/dpastedotcomprotocol.cpp
View File

@@ -59,7 +59,7 @@ void DPasteDotComProtocol::fetchFinished(const QString &id, QNetworkReply * cons
const QString location = QString::fromUtf8(reply->rawHeader("Location"));
if (status == 301 || status == 308) {
const QString m = QString("HTTP redirect (%1) to \"%2\"").arg(status).arg(location);
Core::MessageManager::write(m, Core::MessageManager::ModeSwitch);
Core::MessageManager::writeSilently(m);
}
QNetworkReply * const newRep = httpGet(location);
connect(newRep, &QNetworkReply::finished, this, [this, id, newRep] {
@@ -150,7 +150,7 @@ bool DPasteDotComProtocol::checkConfiguration(QString *errorMessage)
void DPasteDotComProtocol::reportError(const QString &message)
{
const QString fullMessage = tr("%1: %2").arg(protocolName(), message);
Core::MessageManager::write(fullMessage, Core::MessageManager::ModeSwitch);
Core::MessageManager::writeDisrupting(fullMessage);
}
} // namespace CodePaster

4
src/plugins/cpaster/fileshareprotocol.cpp
View File

@@ -206,10 +206,10 @@ void FileShareProtocol::paste(
saver.setResult(&writer);
}
if (!saver.finalize()) {
Core::MessageManager::write(saver.errorString());
Core::MessageManager::writeDisrupting(saver.errorString());
return;
}
Core::MessageManager::write(tr("Pasted: %1").arg(saver.fileName()));
Core::MessageManager::writeSilently(tr("Pasted: %1").arg(saver.fileName()));
}
} // namespace CodePaster