quest status name refactoring

scripts/CSharp/Common/Quest/PopUpPostIt.cs

@@ -1,22 +0,0 @@
-using Godot;
-using System;
-using Babushka.scripts.CSharp.Common.Quest;
-
-public partial class PopUpPostIt : Control
-{
-    private Label Text => GetNode<Label>("Text");
-
-    private AnimationPlayer Animation => GetNode<AnimationPlayer>("AnimationPlayer");
-    
-    QuestManager QM => QuestManager.Instance!;
-    public override void _EnterTree()
-    {
-        QM.QuestBecomesActive += NewQuestPostIt;
-    }
-    
-    private void NewQuestPostIt(QuestResource questResource)
-    {
-        Text.Text = questResource.title;
-        Animation.Play("NewPostit");
-    }
-}

scripts/CSharp/Common/Quest/PopUpPostIt.cs.uid

@@ -1 +0,0 @@
-uid://byar6yqrlph6k

scripts/CSharp/Common/Quest/QuestListItemUi.cs

@@ -24,7 +24,7 @@ public partial class QuestListItemUi : Control
             _ => ""
         };
 
-        UpdateCircled(questResource == QuestManager.Instance.GetFollowQuest());
+        UpdateCircled(questResource == QuestManager.Instance.GetActiveQuest());
         ShowName(name);
     }
     

scripts/CSharp/Common/Quest/QuestLog.cs

@@ -17,10 +17,10 @@ public partial class QuestLog : Control
 
     public QuestResource? currentDetailQuest
     {
-        get => QuestManager.Instance!.GetFollowQuest();
+        get => QuestManager.Instance!.GetActiveQuest();
         set
         {
-            QuestManager.Instance!.SetFollowQuest(value); // TODO: fix setup
+            QuestManager.Instance!.SetActiveQuest(value); // TODO: fix setup
             EmitSignalDetailQuestChanged(this);
         }
     }

scripts/CSharp/Common/Quest/QuestManager.cs

@@ -11,7 +11,7 @@ public partial class QuestManager : Node
     public static QuestManager? Instance { get; private set; }
 
     [Signal]
-    public delegate void QuestBecomesActiveEventHandler(QuestResource questResource);
+    public delegate void QuestBecomesAvailableEventHandler(QuestResource questResource);
 
     [Signal]
     public delegate void QuestsChangedEventHandler();
@@ -26,7 +26,7 @@ public partial class QuestManager : Node
 
     private Godot.Collections.Dictionary<QuestResource, QuestStatus> _questStatus = new();
 
-    private QuestResource? _followQuest;
+    private QuestResource? _activeQuest;
 
 
     public void ChangeQuestStatus(QuestResource questResource, QuestStatus.Status newStatus)
@@ -40,11 +40,11 @@ public partial class QuestManager : Node
         value.status = newStatus;
 
         EmitSignalQuestsChanged();
-        EmitSignalDialogicActiveQuest(_followQuest?.id ?? "none");
+        EmitSignalDialogicActiveQuest(_activeQuest?.id ?? "none");
 
-        if (newStatus == QuestStatus.Status.Active)
+        if (newStatus == QuestStatus.Status.Available)
         {
-            EmitSignalQuestBecomesActive(questResource);
+            EmitSignalQuestBecomesAvailable(questResource);
         }
     }
 
@@ -53,9 +53,9 @@ public partial class QuestManager : Node
         return _questStatus.Where(qs => qs.Value.status != QuestStatus.Status.Hidden);
     }
 
-    public IEnumerable<QuestPair> GetActiveQuests()
+    public IEnumerable<QuestPair> GetAvailableQuests()
     {
-        return _questStatus.Where(qs => qs.Value.status == QuestStatus.Status.Active);
+        return _questStatus.Where(qs => qs.Value.status == QuestStatus.Status.Available);
     }
     public QuestStatus GetQuestStatus(QuestResource questResource)
     {
@@ -67,15 +67,15 @@ public partial class QuestManager : Node
         return status;
     }
 
-    public QuestResource? GetFollowQuest()
+    public QuestResource? GetActiveQuest()
     {
-        return _followQuest;
+        return _activeQuest;
     }
 
-    public void SetFollowQuest(QuestResource? questResource)
+    public void SetActiveQuest(QuestResource? questResource)
     {
-        _followQuest = questResource;
+        _activeQuest = questResource;
         EmitSignalQuestsChanged();
-        EmitSignalDialogicActiveQuest(_followQuest?.id ?? "none");
+        EmitSignalDialogicActiveQuest(_activeQuest?.id ?? "none");
     }
 }

scripts/CSharp/Common/Quest/QuestMessagePopup.cs

@@ -30,7 +30,7 @@ public partial class QuestMessagePopup : Control
 
     private void NewActiveQuest()
     {
-        var shownQuest = QuestManager.Instance!.GetFollowQuest();
+        var shownQuest = QuestManager.Instance!.GetActiveQuest();
         if (_currentlyShown == shownQuest)
             return;
 

scripts/CSharp/Common/Quest/QuestResource.cs

@@ -6,7 +6,7 @@ public partial class QuestResource : Resource
 {
     [Export]
     public string id = "";
-    [Export]
+    [Export(PropertyHint.MultilineText)]
     public string title = "";
     [Export(PropertyHint.MultilineText)]
     public string description = "";

scripts/CSharp/Common/Quest/QuestStatus.cs

@@ -7,7 +7,7 @@ public partial class QuestStatus : GodotObject
     public enum Status
     {
         Hidden = 0,
-        Active = 1,
+        Available = 1,
         Done = 2,
         Canceled = 3,
     }

scripts/CSharp/Common/Quest/QuestTestingScript.cs

@@ -8,15 +8,15 @@ using Godot.Collections;
 public partial class QuestTestingScript : Node
 {
     [Export(PropertyHint.ArrayType)]
-    private Array<QuestResource>? _questsToActivate;
+    private Array<QuestResource>? _questsToMakeAvailable;
 
     public override void _EnterTree()
     {
-        Debug.Assert(_questsToActivate != null);
+        Debug.Assert(_questsToMakeAvailable != null);
         
-        foreach (var questResource in _questsToActivate)
+        foreach (var questResource in _questsToMakeAvailable)
         {
-            QuestManager.Instance!.ChangeQuestStatus(questResource, QuestStatus.Status.Active);
+            QuestManager.Instance!.ChangeQuestStatus(questResource, QuestStatus.Status.Available);
         }
     }
 }

scripts/CSharp/Common/Quest/QuestTrigger.cs

@@ -4,30 +4,28 @@ using Babushka.scripts.CSharp.Common.Quest;
 
 public partial class QuestTrigger : Node
 {
-    [Export]
-    public QuestResource? questResource;
+    [Export] public QuestResource? questResource;
 
-    [Export]
-    public QuestStatus.Status toStatus;
+    [Export] public QuestStatus.Status toStatus;
 
-    [Export]
-    private bool makeCurrent = false;
+    [Export] private bool makeActive = false;
 
     public void Trigger()
     {
         GD.Print("trigger");
-        
-        if(questResource== null)
+
+        if (questResource == null)
             throw new Exception("QuestResource is not set on QuestTrigger node.");
-        
-        if(QuestManager.Instance == null)
-            throw new Exception("QuestManager instance is not available. Make sure it is initialized before calling Trigger.");
-        
+
+        if (QuestManager.Instance == null)
+            throw new Exception(
+                "QuestManager instance is not available. Make sure it is initialized before calling Trigger.");
+
         QuestManager.Instance.ChangeQuestStatus(questResource, toStatus);
-        
-        if (makeCurrent)
+
+        if (makeActive)
         {
-            QuestManager.Instance.SetFollowQuest(questResource);            
+            QuestManager.Instance.SetActiveQuest(questResource);
         }
     }
-}
+}

scripts/CSharp/Common/QuestBehaviour/QuestFulfillmentBase.cs

@@ -11,28 +11,28 @@ using Babushka.scripts.CSharp.Common.Quest;
 /// </summary>
 public abstract partial class QuestFulfillmentBase : Node
 {
-    [Export] private QuestResource _onActiveQuest;
-    [Export] private QuestResource _toNextQuest;
+    [Export] private QuestResource _onAvailableQuest = null!;
+    [Export] private QuestResource _toNextQuest = null!;
     
-    [Export] private bool _whenFulfilledSetActiveQuestToDone = true;
+    [Export] private bool _whenFulfilledSetAvailableQuestToDone = true;
+    [Export] private bool _whenFulfilledSetNextQuestToAvailable = true;
     [Export] private bool _whenFulfilledSetNextQuestToActive = true;
-    [Export] private bool _whenFulfilledSetNextQuestToFollow = true;
 
     [Signal] private delegate void OnFulfilledEventHandler();
 
     protected void Fulfill()
     {
-        if (_whenFulfilledSetActiveQuestToDone)
+        if (_whenFulfilledSetAvailableQuestToDone)
         {
-            QuestManager.Instance!.ChangeQuestStatus(_onActiveQuest, QuestStatus.Status.Done);
+            QuestManager.Instance!.ChangeQuestStatus(_onAvailableQuest, QuestStatus.Status.Done);
+        }
+        if (_whenFulfilledSetNextQuestToAvailable)
+        {
+            QuestManager.Instance!.ChangeQuestStatus(_toNextQuest, QuestStatus.Status.Available);
         }
         if (_whenFulfilledSetNextQuestToActive)
         {
-            QuestManager.Instance!.ChangeQuestStatus(_toNextQuest, QuestStatus.Status.Active);
-        }
-        if (_whenFulfilledSetNextQuestToFollow)
-        {
-            QuestManager.Instance!.SetFollowQuest(_toNextQuest);
+            QuestManager.Instance!.SetActiveQuest(_toNextQuest);
         }
         
         EmitSignalOnFulfilled();
@@ -40,6 +40,6 @@ public abstract partial class QuestFulfillmentBase : Node
 
     protected bool IsQuestActive()
     {
-        return QuestManager.Instance!.GetActiveQuests().Any(q => q.Key == _onActiveQuest);
+        return QuestManager.Instance!.GetAvailableQuests().Any(q => q.Key == _onAvailableQuest);
     }
 }