libcalamaresui: don't rely on translations to interpret button

- Keep the debug-failure-call in the code, but commented out
 - Check ButtonRole, rather than the button's text, to see
   which one it is.
This commit is contained in:
Adriaan de Groot 2019-07-30 15:43:20 +02:00
parent 675a35bb2e
commit 2aae30f517

View file

@ -139,6 +139,8 @@ ViewManager::ViewManager( QObject* parent )
{ {
m_quit->setVisible( false ); m_quit->setVisible( false );
} }
// onInstallationFailed( "Tile of Failure", "Body of Failure"); // for testing paste functionality
} }
@ -199,7 +201,7 @@ ViewManager::insertViewStep( int before, ViewStep* step )
void void
ViewManager::onInstallationFailed( const QString& message, const QString& details ) ViewManager::onInstallationFailed( const QString& message, const QString& details )
{ {
bool shouldOfferWebPaste = true; // TODO: config var bool shouldOfferWebPaste = false; // TODO: config var
cError() << "Installation failed:"; cError() << "Installation failed:";
cDebug() << "- message:" << message; cDebug() << "- message:" << message;
@ -240,26 +242,21 @@ ViewManager::onInstallationFailed( const QString& message, const QString& detail
cDebug() << "Calamares will quit when the dialog closes."; cDebug() << "Calamares will quit when the dialog closes.";
connect( msgBox, &QMessageBox::buttonClicked, [this,msgBox]( QAbstractButton* button ) { connect( msgBox, &QMessageBox::buttonClicked, [this,msgBox]( QAbstractButton* button ) {
cDebug() << "Button role:" << msgBox->buttonRole( button ); if ( msgBox->buttonRole( button ) == QMessageBox::ButtonRole::YesRole )
if ( button->text() != tr( "&Yes" ) )
{ {
QApplication::quit(); // TODO: host and port should be configurable
return; QString pasteUrlMsg = CalamaresUtils::pastebin( msgBox, QStringLiteral( "termbin.com" ), 9999 );
QString pasteUrlTitle = tr( "Install Log Paste URL" );
if ( pasteUrlMsg.isEmpty() )
{
pasteUrlMsg = tr( "The upload was unsuccessful. No web-paste was done." );
}
QMessageBox::critical(nullptr,
pasteUrlTitle,
pasteUrlMsg);
} }
// TODO: host and port should be configurable
QString pasteUrlMsg = CalamaresUtils::pastebin( msgBox, QStringLiteral( "termbin.com" ), 9999 );
QString pasteUrlTitle = tr( "Install Log Paste URL" );
if ( pasteUrlMsg.isEmpty() )
{
pasteUrlMsg = tr( "The upload was unsuccessful. No web-paste was done." );
}
QMessageBox::critical(nullptr,
pasteUrlTitle,
pasteUrlMsg);
QApplication::quit(); QApplication::quit();
} ); } );
} }