From f66cf4ad99219716e221f2a6717b910e531bc7d3 Mon Sep 17 00:00:00 2001 From: Mikaël Capelle Date: Mon, 15 Jun 2020 22:47:21 +0200 Subject: Update for new organizer callbacks. --- src/organizercore.cpp | 50 ++++++++++++++++++++++++++++++++++++++++---------- 1 file changed, 40 insertions(+), 10 deletions(-) (limited to 'src/organizercore.cpp') diff --git a/src/organizercore.cpp b/src/organizercore.cpp index b167b8e3..45207910 100644 --- a/src/organizercore.cpp +++ b/src/organizercore.cpp @@ -151,6 +151,10 @@ OrganizerCore::OrganizerCore(Settings &settings) // make directory refresher run in a separate thread m_RefresherThread.start(); m_DirectoryRefresher->moveToThread(&m_RefresherThread); + + connect(&settings.plugins(), &PluginSettings::pluginSettingChanged, [this](auto const& ...args) { + m_PluginSettingChanged(args...); + }); } OrganizerCore::~OrganizerCore() @@ -165,8 +169,7 @@ OrganizerCore::~OrganizerCore() saveCurrentProfile(); // profile has to be cleaned up before the modinfo-buffer is cleared - delete m_CurrentProfile; - m_CurrentProfile = nullptr; + m_CurrentProfile.reset(); ModInfo::clear(); m_ModList.setProfile(nullptr); @@ -297,6 +300,10 @@ void OrganizerCore::disconnectPlugins() m_AboutToRun.disconnect_all_slots(); m_FinishedRun.disconnect_all_slots(); m_ModInstalled.disconnect_all_slots(); + m_UserInterfaceInitialized.disconnect_all_slots(); + m_ProfileChanged.disconnect_all_slots(); + m_PluginSettingChanged.disconnect_all_slots(); + m_ModList.disconnectSlots(); m_PluginList.disconnectSlots(); m_Updater.setPluginContainer(nullptr); @@ -364,6 +371,11 @@ void OrganizerCore::downloadRequestedNXM(const QString &url) } } +void OrganizerCore::userInterfaceInitialized(QMainWindow* mainWindow) +{ + m_UserInterfaceInitialized(mainWindow); +} + void OrganizerCore::externalMessage(const QString &message) { if (MOShortcut moshortcut{ message } ) { @@ -533,11 +545,12 @@ void OrganizerCore::setCurrentProfile(const QString &profileName) profileBaseDir.entryList(QDir::AllDirs | QDir::NoDotAndDotDot).at(0)); } - Profile *newProfile = new Profile(QDir(profileDir), managedGame()); + // Keep the old profile to emit signal-changed: + auto oldProfile = std::move(m_CurrentProfile); + + m_CurrentProfile = std::make_unique(QDir(profileDir), managedGame()); - delete m_CurrentProfile; - m_CurrentProfile = newProfile; - m_ModList.setProfile(newProfile); + m_ModList.setProfile(m_CurrentProfile.get()); if (m_CurrentProfile->invalidationActive(nullptr)) { m_CurrentProfile->activateInvalidation(); @@ -547,11 +560,13 @@ void OrganizerCore::setCurrentProfile(const QString &profileName) m_Settings.game().setSelectedProfileName(m_CurrentProfile->name()); - connect(m_CurrentProfile, SIGNAL(modStatusChanged(uint)), this, SLOT(modStatusChanged(uint))); - connect(m_CurrentProfile, SIGNAL(modStatusChanged(QList)), this, SLOT(modStatusChanged(QList))); + connect(m_CurrentProfile.get(), SIGNAL(modStatusChanged(uint)), this, SLOT(modStatusChanged(uint))); + connect(m_CurrentProfile.get(), SIGNAL(modStatusChanged(QList)), this, SLOT(modStatusChanged(QList))); refreshDirectoryStructure(); m_CurrentProfile->debugDump(); + + m_ProfileChanged(oldProfile.get(), m_CurrentProfile.get()); } MOBase::IModRepositoryBridge *OrganizerCore::createNexusBridge() const @@ -1100,6 +1115,21 @@ bool OrganizerCore::onModInstalled( return conn.connected(); } +bool OrganizerCore::onUserInterfaceInitialized(std::function const& func) +{ + return m_UserInterfaceInitialized.connect(func).connected(); +} + +bool OrganizerCore::onProfileChanged(std::function const& func) +{ + return m_ProfileChanged.connect(func).connected(); +} + +bool OrganizerCore::onPluginSettingChanged(std::function const& func) +{ + return m_PluginSettingChanged.connect(func).connected(); +} + void OrganizerCore::refreshModList(bool saveChanges) { // don't lose changes! @@ -1153,7 +1183,7 @@ void OrganizerCore::refreshBSAList() // found (which might // happen if ini files are missing) use hard-coded defaults (preferrably the // same the game would use) - m_DefaultArchives = archives->archives(m_CurrentProfile); + m_DefaultArchives = archives->archives(m_CurrentProfile.get()); if (m_DefaultArchives.length() == 0) { m_DefaultArchives = archives->vanillaArchives(); } @@ -1370,7 +1400,7 @@ void OrganizerCore::requestDownload(const QUrl &url, QNetworkReply *reply) ModListSortProxy *OrganizerCore::createModListProxyModel() { - ModListSortProxy *result = new ModListSortProxy(m_CurrentProfile, this); + ModListSortProxy *result = new ModListSortProxy(m_CurrentProfile.get(), this); result->setSourceModel(&m_ModList); return result; } -- cgit v1.3.1 From 8c410aff33395daf607ae8a1c62b8be70943ba4d Mon Sep 17 00:00:00 2001 From: Mikaël Capelle Date: Tue, 16 Jun 2020 21:30:24 +0200 Subject: Clean notification of userInterfaceInitialized between MainWindow and OrganizerCore. --- src/iuserinterface.h | 3 ++- src/mainwindow.cpp | 5 +++-- src/mainwindow.h | 7 ++++++- src/organizercore.cpp | 8 ++++---- src/organizercore.h | 2 +- src/processrunner.cpp | 4 ++-- 6 files changed, 18 insertions(+), 11 deletions(-) (limited to 'src/organizercore.cpp') diff --git a/src/iuserinterface.h b/src/iuserinterface.h index aa48194f..cce89070 100644 --- a/src/iuserinterface.h +++ b/src/iuserinterface.h @@ -7,6 +7,7 @@ #include #include #include +#include class IUserInterface @@ -30,7 +31,7 @@ public: virtual MOBase::DelayedFileWriterBase &archivesWriter() = 0; - virtual QWidget* qtWidget() = 0; + virtual QMainWindow* mainWindow() = 0; }; #endif // IUSERINTERFACE_H diff --git a/src/mainwindow.cpp b/src/mainwindow.cpp index b3e78121..91c906a1 100644 --- a/src/mainwindow.cpp +++ b/src/mainwindow.cpp @@ -502,6 +502,7 @@ MainWindow::MainWindow(Settings &settings m_Tutorial.expose("espList", m_OrganizerCore.pluginList()); m_OrganizerCore.setUserInterface(this); + connect(this, &MainWindow::userInterfaceInitialized, &m_OrganizerCore, &OrganizerCore::userInterfaceInitialized); for (const QString &fileName : m_PluginContainer.pluginFileNames()) { installTranslator(QFileInfo(fileName).baseName()); } @@ -1384,7 +1385,7 @@ void MainWindow::showEvent(QShowEvent *event) updateProblemsButton(); // Notify plugin that the UI is initialized: - m_OrganizerCore.userInterfaceInitialized(this); + emit userInterfaceInitialized(); } } @@ -2267,7 +2268,7 @@ void MainWindow::storeSettings() s.interface().setFilterOptions(FilterWidget::options()); } -QWidget* MainWindow::qtWidget() +QMainWindow* MainWindow::mainWindow() { return this; } diff --git a/src/mainwindow.h b/src/mainwindow.h index 94626ff3..60f22770 100644 --- a/src/mainwindow.h +++ b/src/mainwindow.h @@ -118,7 +118,7 @@ public: void processUpdates(); - QWidget* qtWidget() override; + QMainWindow* mainWindow() override; bool addProfile(); void updateBSAList(const QStringList &defaultArchives, const QStringList &activeArchives); @@ -175,6 +175,11 @@ signals: */ void styleChanged(const QString &styleFile); + /** + * @brief emitted when the user interface has been completely initialized + */ + void userInterfaceInitialized(); + void modListDataChanged(const QModelIndex &topLeft, const QModelIndex &bottomRight); diff --git a/src/organizercore.cpp b/src/organizercore.cpp index 45207910..062369d6 100644 --- a/src/organizercore.cpp +++ b/src/organizercore.cpp @@ -233,7 +233,7 @@ void OrganizerCore::setUserInterface(IUserInterface* ui) QWidget* w = nullptr; if (m_UserInterface) { - w = m_UserInterface->qtWidget(); + w = m_UserInterface->mainWindow(); } if (w) { @@ -371,9 +371,9 @@ void OrganizerCore::downloadRequestedNXM(const QString &url) } } -void OrganizerCore::userInterfaceInitialized(QMainWindow* mainWindow) +void OrganizerCore::userInterfaceInitialized() { - m_UserInterfaceInitialized(mainWindow); + m_UserInterfaceInitialized(m_UserInterface->mainWindow()); } void OrganizerCore::externalMessage(const QString &message) @@ -1834,7 +1834,7 @@ bool OrganizerCore::beforeRun( { QWidget* w = nullptr; if (m_UserInterface) { - w = m_UserInterface->qtWidget(); + w = m_UserInterface->mainWindow(); } QMessageBox::warning(w, tr("Error"), e.what()); return false; diff --git a/src/organizercore.h b/src/organizercore.h index ae4a34c4..2588d332 100644 --- a/src/organizercore.h +++ b/src/organizercore.h @@ -365,7 +365,7 @@ public slots: void requestDownload(const QUrl &url, QNetworkReply *reply); void downloadRequestedNXM(const QString &url); - void userInterfaceInitialized(QMainWindow *); + void userInterfaceInitialized(); bool nexusApi(bool retry = false); diff --git a/src/processrunner.cpp b/src/processrunner.cpp index 91443750..3ea40da2 100644 --- a/src/processrunner.cpp +++ b/src/processrunner.cpp @@ -520,7 +520,7 @@ ProcessRunner& ProcessRunner::setFromFile( QWidget* parent, const QFileInfo& targetInfo) { if (!parent && m_ui) { - parent = m_ui->qtWidget(); + parent = m_ui->mainWindow(); } // if the file is a .exe, start it directly; if it's anything else, ask the @@ -767,7 +767,7 @@ std::optional ProcessRunner::runBinary() } // parent widget used for any dialog popped up while checking for things - QWidget* parent = (m_ui ? m_ui->qtWidget() : nullptr); + QWidget* parent = (m_ui ? m_ui->mainWindow() : nullptr); const auto* game = m_core.managedGame(); auto& settings = m_core.settings(); -- cgit v1.3.1