fix(ManagedPagePage): don't reload pack info when not needed

Signed-off-by: flow <flowlnlnln@gmail.com>

1 files changed, 9 insertions, 0 deletions

diff --git a/launcher/ui/pages/instance/ManagedPackPage.cpp b/launcher/ui/pages/instance/ManagedPackPage.cpp
index b3816ce9..16b2cb57 100644
--- a/launcher/ui/pages/instance/ManagedPackPage.cpp
+++ b/launcher/ui/pages/instance/ManagedPackPage.cpp
@@ -66,6 +66,7 @@ ManagedPackPage::ManagedPackPage(BaseInstance* inst, InstanceWindow* instance_wi
     connect(ui->reloadButton, &QPushButton::clicked, this, [this](bool){
         ui->reloadButton->setVisible(false);
 
+        m_loaded = false;
         // Pretend we're opening the page again
         openedImpl();
     });
@@ -179,6 +180,10 @@ void ModrinthManagedPackPage::parseManagedPack()
 {
     qDebug() << "Parsing Modrinth pack";
 
+    // No need for the extra work because we already have everything we need.
+    if (m_loaded)
+        return;
+
     if (m_fetch_job && m_fetch_job->isRunning())
         m_fetch_job->abort();
 
@@ -318,6 +323,10 @@ void FlameManagedPackPage::parseManagedPack()
         return;
     }
 
+    // No need for the extra work because we already have everything we need.
+    if (m_loaded)
+        return;
+
     if (m_fetch_job && m_fetch_job->isRunning())
         m_fetch_job->abort();