NOISSUE model Task states as one enum instead of multiple flags

This adds Task::State::AbortedByUser as a possibility

1 files changed, 11 insertions, 3 deletions

diff --git a/api/logic/tasks/Task.h b/api/logic/tasks/Task.h
index 58dbc4ca..c66622aa 100644
--- a/api/logic/tasks/Task.h
+++ b/api/logic/tasks/Task.h
@@ -25,6 +25,16 @@ class MULTIMC_LOGIC_EXPORT Task : public QObject
 {
     Q_OBJECT
 public:
+    enum class State
+    {
+        Inactive,
+        Running,
+        Succeeded,
+        Failed,
+        AbortedByUser
+    };
+
+public:
     explicit Task(QObject *parent = 0);
     virtual ~Task() {};
 
@@ -88,9 +98,7 @@ public slots:
     void setProgress(qint64 current, qint64 total);
 
 private:
-    bool m_running = false;
-    bool m_finished = false;
-    bool m_succeeded = false;
+    State m_state = State::Inactive;
     QStringList m_Warnings;
     QString m_failReason = "";
     QString m_status;