libcalamaresui: apply coding style to viewmanager

src/libcalamaresui/ViewManager.cpp

@@ -112,9 +112,7 @@ ViewManager::ViewManager( QObject* parent )
     m_next = makeButton( m_widget, "go-next" );
     m_quit = makeButton( m_widget, "dialog-cancel" );
 
-    CALAMARES_RETRANSLATE(
+    CALAMARES_RETRANSLATE( updateButtonLabels(); )
-        updateButtonLabels();
-    )
 
     QBoxLayout* bottomLayout = new QHBoxLayout;
     mainLayout->addLayout( bottomLayout );
@@ -128,16 +126,20 @@ ViewManager::ViewManager( QObject* parent )
     connect( m_back, &QPushButton::clicked, this, &ViewManager::back );
     m_back->setEnabled( false );
 
-    connect( m_quit, &QPushButton::clicked, this,
+    connect( m_quit, &QPushButton::clicked, this, [this]() {
-             [this]() { if ( this->confirmCancelInstallation() ) qApp->quit(); } );
+        if ( this->confirmCancelInstallation() )
-    connect( JobQueue::instance(), &JobQueue::failed,
+        {
-             this, &ViewManager::onInstallationFailed );
+            qApp->quit();
-    connect( JobQueue::instance(), &JobQueue::finished,
+        }
-             this, &ViewManager::next );
+    } );
+    connect( JobQueue::instance(), &JobQueue::failed, this, &ViewManager::onInstallationFailed );
+    connect( JobQueue::instance(), &JobQueue::finished, this, &ViewManager::next );
 
     if ( Calamares::Settings::instance()->disableCancel() )
+    {
         m_quit->setVisible( false );
     }
+}
 
 
 ViewManager::~ViewManager()
@@ -160,8 +162,10 @@ ViewManager::addViewStep( ViewStep* step )
     insertViewStep( m_steps.size(), step );
     // If this is the first inserted view step, update status of "Next" button
     if ( m_steps.count() == 1 )
+    {
         m_next->setEnabled( step->isNextEnabled() );
     }
+}
 
 
 void
@@ -170,19 +174,21 @@ ViewManager::insertViewStep( int before, ViewStep* step )
     m_steps.insert( before, step );
     QLayout* layout = step->widget()->layout();
     if ( layout )
+    {
         layout->setContentsMargins( 0, 0, 0, 0 );
+    }
     m_stack->insertWidget( before, step->widget() );
 
     connect( step, &ViewStep::enlarge, this, &ViewManager::enlarge );
-    connect( step, &ViewStep::nextStatusChanged,
+    connect( step, &ViewStep::nextStatusChanged, this, [this]( bool status ) {
-             this, [this]( bool status )
-    {
         ViewStep* vs = qobject_cast< ViewStep* >( sender() );
         if ( vs )
         {
             if ( vs == m_steps.at( m_currentStep ) )
+            {
                 m_next->setEnabled( status );
             }
+        }
     } );
 
     m_stack->setCurrentIndex( 0 );
@@ -199,15 +205,18 @@ ViewManager::onInstallationFailed( const QString& message, const QString& detail
     cDebug() << "- message:" << message;
     cDebug() << "- details:" << details;
 
-    QString heading = Calamares::Settings::instance()->isSetupMode()
+    QString heading
-        ? tr( "Setup Failed" )
+        = Calamares::Settings::instance()->isSetupMode() ? tr( "Setup Failed" ) : tr( "Installation Failed" );
-        : tr( "Installation Failed" );
     QString pasteMsg = tr( "Would you like to paste the install log to the web?" );
     QString text = "<p>" + message + "</p>";
     if ( !details.isEmpty() )
+    {
         text += "<p>" + details + "</p>";
+    }
     if ( shouldOfferWebPaste )
+    {
         text += "<p>" + pasteMsg + "</p>";
+    }
 
     QMessageBox* msgBox = new QMessageBox();
     msgBox->setIcon( QMessageBox::Critical );
@@ -230,9 +239,7 @@ ViewManager::onInstallationFailed( const QString& message, const QString& detail
     msgBox->show();
 
     cDebug() << "Calamares will quit when the dialog closes.";
-    connect( msgBox, &QMessageBox::buttonClicked,
+    connect( msgBox, &QMessageBox::buttonClicked, [msgBox]( QAbstractButton* button ) {
-             [msgBox] ( QAbstractButton* button )
-    {
         if ( button->text() != tr( "&Yes" ) )
         {
             QApplication::quit();
@@ -266,7 +273,8 @@ ViewManager::onInitFailed( const QStringList& modules)
     // Because this means the installer / setup program is broken by the distributor,
     // don't bother being precise about installer / setup wording.
     QString title( tr( "Calamares Initialization Failed" ) );
-    QString description( tr( "%1 can not be installed. Calamares was unable to load all of the configured modules. This is a problem with the way Calamares is being used by the distribution." ) );
+    QString description( tr( "%1 can not be installed. Calamares was unable to load all of the configured modules. "
+                             "This is a problem with the way Calamares is being used by the distribution." ) );
     QString detailString;
 
     if ( modules.count() > 0 )
@@ -275,7 +283,9 @@ ViewManager::onInitFailed( const QStringList& modules)
         QStringList details;
         details << QLatin1Literal( "<ul>" );
         for ( const auto& m : modules )
+        {
             details << QLatin1Literal( "<li>" ) << m << QLatin1Literal( "</li>" );
+        }
         details << QLatin1Literal( "</ul>" );
         detailString = details.join( QString() );
     }
@@ -311,10 +321,8 @@ ViewManager::currentStepIndex() const
 static inline bool
 stepIsExecute( const ViewStepList& steps, int index )
 {
-    return
+    return ( 0 <= index ) && ( index < steps.count() )
-        ( 0 <= index ) &&
+        && ( qobject_cast< ExecutionViewStep* >( steps.at( index ) ) != nullptr );
-        ( index < steps.count() ) &&
-        ( qobject_cast< ExecutionViewStep* >( steps.at( index ) ) != nullptr );
 }
 
 void
@@ -331,9 +339,8 @@ ViewManager::next()
         // Depending on Calamares::Settings, we show an "are you sure" prompt or not.
         if ( settings->showPromptBeforeExecution() && stepIsExecute( m_steps, m_currentStep + 1 ) )
         {
-            QString title = settings->isSetupMode()
+            QString title
-                ? tr( "Continue with setup?" )
+                = settings->isSetupMode() ? tr( "Continue with setup?" ) : tr( "Continue with installation?" );
-                : tr( "Continue with installation?" );
             QString question = settings->isSetupMode()
                 ? tr( "The %1 setup program is about to make changes to your "
                       "disk in order to set up %2.<br/><strong>You will not be able "
@@ -341,12 +348,10 @@ ViewManager::next()
                 : tr( "The %1 installer is about to make changes to your "
                       "disk in order to install %2.<br/><strong>You will not be able "
                       "to undo these changes.</strong>" );
-            QString confirm = settings->isSetupMode()
+            QString confirm = settings->isSetupMode() ? tr( "&Set up now" ) : tr( "&Install now" );
-                ? tr( "&Set up now" )
-                : tr( "&Install now" );
 
-            int reply =
+            int reply = QMessageBox::question(
-                QMessageBox::question( m_widget,
+                m_widget,
                 title,
                 question.arg( *Calamares::Branding::ShortProductName, *Calamares::Branding::ShortVersionedName ),
                 confirm,
@@ -355,8 +360,10 @@ ViewManager::next()
                 0,
                 1 );
             if ( reply == 1 )
+            {
                 return;
             }
+        }
 
         m_currentStep++;
 
@@ -397,9 +404,7 @@ ViewManager::updateButtonLabels()
 {
     const auto* const settings = Calamares::Settings::instance();
 
-    QString nextIsInstallationStep = settings->isSetupMode()
+    QString nextIsInstallationStep = settings->isSetupMode() ? tr( "&Set up" ) : tr( "&Install" );
-        ? tr( "&Set up" )
-        : tr( "&Install" );
     QString quitOnCompleteTooltip = settings->isSetupMode()
         ? tr( "Setup is complete. Close the setup program." )
         : tr( "The installation is complete. Close the installer." );
@@ -434,8 +439,11 @@ ViewManager::updateButtonLabels()
     else
     {
         if ( settings->disableCancel() )
+        {
             m_quit->setVisible( false );  // In case we went back from final
-        updateCancelEnabled( !settings->disableCancel() && !( stepIsExecute( m_steps, m_currentStep ) && settings->disableCancelDuringExec() ) );
+        }
+        updateCancelEnabled( !settings->disableCancel()
+                             && !( stepIsExecute( m_steps, m_currentStep ) && settings->disableCancelDuringExec() ) );
 
         m_quit->setText( tr( "&Cancel" ) );
         m_quit->setToolTip( cancelBeforeInstallationTooltip );
@@ -456,46 +464,53 @@ ViewManager::back()
         emit currentStepChanged();
     }
     else if ( !step->isAtBeginning() )
+    {
         step->back();
-    else return;
+    }
+    else
+    {
+        return;
+    }
 
     m_next->setEnabled( m_steps.at( m_currentStep )->isNextEnabled() );
     m_back->setEnabled( m_steps.at( m_currentStep )->isBackEnabled() );
 
     if ( m_currentStep == 0 && m_steps.first()->isAtBeginning() )
+    {
         m_back->setEnabled( false );
+    }
 
     updateButtonLabels();
 }
 
-bool ViewManager::confirmCancelInstallation()
+bool
+ViewManager::confirmCancelInstallation()
 {
     const auto* const settings = Calamares::Settings::instance();
 
     // When we're at the very end, then it's always OK to exit.
     if ( isAtVeryEnd() )
+    {
         return true;
+    }
 
     // Not at the very end, cancel/quit might be disabled
     if ( settings->disableCancel() )
+    {
         return false;
+    }
     if ( settings->disableCancelDuringExec() && stepIsExecute( m_steps, m_currentStep ) )
+    {
         return false;
+    }
 
     // Otherwise, confirm cancel/quit.
-    QString title = settings->isSetupMode()
+    QString title = settings->isSetupMode() ? tr( "Cancel setup?" ) : tr( "Cancel installation?" );
-        ? tr( "Cancel setup?" )
+    QString question = settings->isSetupMode() ? tr( "Do you really want to cancel the current setup process?\n"
-        : tr( "Cancel installation?" );
-    QString question = settings->isSetupMode()
-        ? tr( "Do you really want to cancel the current setup process?\n"
                                                      "The setup program will quit and all changes will be lost." )
                                                : tr( "Do you really want to cancel the current install process?\n"
                                                      "The installer will quit and all changes will be lost." );
-    QMessageBox mb( QMessageBox::Question,
+    QMessageBox mb( QMessageBox::Question, title, question, QMessageBox::Yes | QMessageBox::No, m_widget );
-                    title,
-                    question,
-                    QMessageBox::Yes | QMessageBox::No,
-                    m_widget );
     mb.setDefaultButton( QMessageBox::No );
     mb.button( QMessageBox::Yes )->setText( tr( "&Yes" ) );
     mb.button( QMessageBox::No )->setText( tr( "&No" ) );
@@ -510,4 +525,4 @@ ViewManager::updateCancelEnabled( bool enabled )
     emit cancelEnabled( enabled );
 }
 
-}  // namespace
+}  // namespace Calamares

src/libcalamaresui/ViewManager.h

@@ -133,7 +133,8 @@ private:
 
     bool isAtVeryEnd() const
     {
-        return ( m_currentStep >= m_steps.count() ) || ( m_currentStep == m_steps.count() - 1 && m_steps.last()->isAtEnd() );
+        return ( m_currentStep >= m_steps.count() )
+            || ( m_currentStep == m_steps.count() - 1 && m_steps.last()->isAtEnd() );
     }
 
     static ViewManager* s_instance;
@@ -148,6 +149,6 @@ private:
     QPushButton* m_quit;
 };
 
-}
+}  // namespace Calamares
 
 #endif  // VIEWMANAGER_H