[stable-3.9] Improve macOS Sparkle updater

src/gui/generalsettings.cpp

@@ -1,4 +1,4 @@
 /*
  * Copyright (C) by Daniel Molkentin <danimo@owncloud.com>
  *
  * This program is free software; you can redistribute it and/or modify
@@ -300,9 +300,15 @@
                                       ocupdater->downloadState() != OCUpdater::DownloadComplete);
     }
 #if defined(Q_OS_MAC) && defined(HAVE_SPARKLE)
-    else if (auto sparkleUpdater = qobject_cast<SparkleUpdater *>(updater)) {
+    else if (const auto sparkleUpdater = qobject_cast<SparkleUpdater *>(updater)) {
+        connect(sparkleUpdater, &SparkleUpdater::statusChanged, this, &GeneralSettings::slotUpdateInfo, Qt::UniqueConnection);
         _ui->updateStateLabel->setText(sparkleUpdater->statusString());
         _ui->restartButton->setVisible(false);
+
+        const auto updaterState = sparkleUpdater->state();
+        const auto enableUpdateButton = updaterState == SparkleUpdater::State::Idle ||
+                                        updaterState == SparkleUpdater::State::Unknown;
+        _ui->updateButton->setEnabled(enableUpdateButton);
     }
 #endif
 

src/gui/updater/sparkleupdater.h

@@ -1,4 +1,4 @@
 /*
  * Copyright (C) by Daniel Molkentin <danimo@owncloud.com>
  *
  * This program is free software; you can redistribute it and/or modify
@@ -24,22 +24,40 @@
 class SparkleUpdater : public Updater
 {
     Q_OBJECT
+    Q_PROPERTY(QString statusString READ statusString NOTIFY statusChanged)
+    Q_PROPERTY(State state READ state NOTIFY statusChanged)
+
 public:
-    SparkleUpdater(const QUrl &appCastUrl);
+    class SparkleInterface;
+    enum class State {
+        Unknown = 0,
+        Idle,
+        Working,
+        AwaitingUserInput
+    };
+
+    explicit SparkleUpdater(const QUrl &appCastUrl);
     ~SparkleUpdater() override;
 
-    void setUpdateUrl(const QUrl &url);
+    [[nodiscard]] static bool autoUpdaterAllowed();
 
-    // unused in this updater
+    [[nodiscard]] bool handleStartup() override { return false; }
+    [[nodiscard]] QString statusString() const;
+    [[nodiscard]] State state() const;
+
+public slots:
+    void setUpdateUrl(const QUrl &url);
     void checkForUpdate() override;
     void backgroundCheckForUpdate() override;
-    bool handleStartup() override { return false; }
 
-    QString statusString();
+signals:
+    void statusChanged();
 
 private:
-    class Private;
-    Private *d;
+    std::unique_ptr<SparkleInterface> _interface;
+    QString _statusString;
+    State _state = State::Unknown;
+    friend class SparkleInterface;
 };
 
 } // namespace OCC