Skip to content

Commit

Permalink
#5629 Ensure setEdition signals main UI
Browse files Browse the repository at this point in the history
  • Loading branch information
SymlessRemoved authored and Andrew Nelless committed Oct 4, 2016
1 parent 66ca696 commit af9b4ce
Show file tree
Hide file tree
Showing 8 changed files with 21 additions and 24 deletions.
5 changes: 0 additions & 5 deletions src/gui/src/ActivationDialog.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -161,10 +161,5 @@ void ActivationDialog::accept()

message.information (this, "Activated!",
tr("Thanks for activating %1!").arg (getEditionName (edition)));
MainWindow& mainWindow = dynamic_cast<MainWindow&>(*this->parent());
mainWindow.setEdition(edition);
mainWindow.updateLocalFingerprint();
mainWindow.saveSettings();

QDialog::accept();
}
5 changes: 4 additions & 1 deletion src/gui/src/AppConfig.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -238,7 +238,10 @@ void AppConfig::setAutoConfigPrompted(bool prompted)
m_AutoConfigPrompted = prompted;
}

void AppConfig::setEdition(int e) { m_Edition = e; }
void AppConfig::setEdition(int e) {
m_Edition = e;
emit editionSet (e);
}

int AppConfig::edition() const { return m_Edition; }

Expand Down
8 changes: 7 additions & 1 deletion src/gui/src/AppConfig.h
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@

#define APPCONFIG_H

#include <QObject>
#include <QString>
#include "ElevateMode.h"

Expand Down Expand Up @@ -47,8 +48,10 @@ enum ProcessMode {
Desktop
};

class AppConfig
class AppConfig: public QObject
{
Q_OBJECT

friend class SettingsDialog;
friend class MainWindow;
friend class SetupWizard;
Expand Down Expand Up @@ -144,6 +147,9 @@ class AppConfig
static const char m_SynergysName[];
static const char m_SynergycName[];
static const char m_SynergyLogDir[];

signals:
void editionSet(int);
};

#endif
17 changes: 8 additions & 9 deletions src/gui/src/MainWindow.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -138,15 +138,8 @@ MainWindow::MainWindow(QSettings& settings, AppConfig& appConfig) :
setEdition(m_AppConfig.edition());

m_pLabelPadlock->hide();

if (appConfig.getCryptoEnabled()) {
m_pSslCertificate = new SslCertificate(this);
m_pSslCertificate->generateCertificate();
}

updateLocalFingerprint();

connect (this, SIGNAL(windowShown()), this, SLOT(on_windowShown()), Qt::QueuedConnection);
connect (&m_AppConfig, SIGNAL(editionSet(int)), this, SLOT(setEdition(int)), Qt::QueuedConnection);
}

MainWindow::~MainWindow()
Expand Down Expand Up @@ -1034,11 +1027,17 @@ void MainWindow::serverDetected(const QString name)
void MainWindow::setEdition(int edition)
{
setWindowTitle(getEditionName(edition));
if (m_AppConfig.getCryptoEnabled()) {
m_pSslCertificate = new SslCertificate(this);
m_pSslCertificate->generateCertificate();
}
updateLocalFingerprint();
saveSettings();
}

void MainWindow::updateLocalFingerprint()
{
if (Fingerprint::local().fileExists()) {
if (m_AppConfig.getCryptoEnabled() && Fingerprint::local().fileExists()) {
m_pLabelFingerprint->setVisible(true);
m_pLabelLocalFingerprint->setVisible(true);
m_pLabelLocalFingerprint->setText(Fingerprint::local().readFirst());
Expand Down
2 changes: 1 addition & 1 deletion src/gui/src/MainWindow.h
Original file line number Diff line number Diff line change
Expand Up @@ -115,10 +115,10 @@ class MainWindow : public QMainWindow, public Ui::MainWindowBase
void autoAddScreen(const QString name);
void updateZeroconfService();
void serverDetected(const QString name);
void setEdition(int edition);
void updateLocalFingerprint();

public slots:
void setEdition(int edition);
void appendLogRaw(const QString& text);
void appendLogInfo(const QString& text);
void appendLogDebug(const QString& text);
Expand Down
1 change: 0 additions & 1 deletion src/gui/src/SettingsDialog.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -151,6 +151,5 @@ void SettingsDialog::on_m_pCheckBoxEnableCrypto_toggled(bool checked)
sslCertificate.generateCertificate();
MainWindow& mainWindow = dynamic_cast<MainWindow&> (*this->parent());
mainWindow.updateLocalFingerprint();
mainWindow.saveSettings();
}
}
5 changes: 1 addition & 4 deletions src/gui/src/SetupWizard.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -28,9 +28,7 @@

SetupWizard::SetupWizard(MainWindow& mainWindow, bool startMain) :
m_MainWindow(mainWindow),
m_StartMain(startMain),
m_Edition(Unregistered),
m_LoginAttemps(0)
m_StartMain(startMain)
{
setupUi(this);

Expand Down Expand Up @@ -141,7 +139,6 @@ void SetupWizard::reject()

if (m_StartMain)
{
m_MainWindow.setEdition(m_Edition);
m_MainWindow.open();
}

Expand Down
2 changes: 0 additions & 2 deletions src/gui/src/SetupWizard.h
Original file line number Diff line number Diff line change
Expand Up @@ -47,8 +47,6 @@ class SetupWizard : public QWizard, public Ui::SetupWizardBase
MainWindow& m_MainWindow;
bool m_StartMain;
SynergyLocale m_Locale;
int m_Edition;
int m_LoginAttemps;

private slots:
void on_m_pComboLanguage_currentIndexChanged(int index);
Expand Down

0 comments on commit af9b4ce

Please sign in to comment.