Updated step to fly

Directory.Build.targets

@@ -1,5 +1,5 @@
 <Project>
     <PropertyGroup>
-        <Version>2.19</Version>
+        <Version>2.17</Version>
     </PropertyGroup>
 </Project>

QuestPathGenerator/RoslynElements/QuestStepExtensions.cs

@@ -62,9 +62,6 @@ internal static class QuestStepExtensions
                             Assignment(nameof(QuestStep.IgnoreDistanceToObject),
                                     step.IgnoreDistanceToObject, emptyStep.IgnoreDistanceToObject)
                                 .AsSyntaxNodeOrToken(),
-                            Assignment(nameof(QuestStep.RestartNavigationIfCancelled),
-                                    step.RestartNavigationIfCancelled, emptyStep.RestartNavigationIfCancelled)
-                                .AsSyntaxNodeOrToken(),
                             Assignment(nameof(QuestStep.Comment), step.Comment, emptyStep.Comment)
                                 .AsSyntaxNodeOrToken(),
                             Assignment(nameof(QuestStep.Aetheryte), step.Aetheryte, emptyStep.Aetheryte)

QuestPaths/2.x - A Realm Reborn/MSQ-2/E6-2.55/427_The Parting Glass.json

@@ -53,7 +53,6 @@
             "Y": -0.63573146,
             "Z": -166.33862
           },
-          "StopDistance": 5,
           "TerritoryId": 141,
           "InteractionType": "Interact"
         }

QuestPaths/2.x - A Realm Reborn/MSQ-2/E6-2.55/428_Before the Dawn.json

@@ -35,7 +35,6 @@
           },
           "TerritoryId": 155,
           "InteractionType": "CompleteQuest",
-          "Mount": true,
           "DialogueChoices": [
             {
               "Type": "YesNo",

QuestPaths/2.x - A Realm Reborn/Raid Quests/1709_Legacy of Allag.json

@@ -337,6 +337,16 @@
           "InteractionType": "WalkTo",
           "Fly": true
         },
+        {
+          "Position": {
+            "X": 233.0817,
+            "Y": 8,
+            "Z": -21.83023
+          },
+          "TerritoryId": 146,
+          "InteractionType": "WalkTo",
+          "Fly": true
+        },
         {
           "DataId": 2000077,
           "Position": {
@@ -350,16 +360,18 @@
           "KillEnemyDataIds": [
             9489
           ],
-          "$": "0 0  0 0  0 0 -> 16 17  0 0  0 128",
-          "CompletionQuestVariablesFlags": [
-            null,
-            null,
-            null,
-            null,
-            null,
-            128
-          ],
-          "Fly": true
+          "$": "0 0  0 0  0 0 -> 16 17  0 0  0 128"
+        },
+        {
+          "Position": {
+            "X": 614.4023,
+            "Y": 301.81046,
+            "Z": -101.94888
+          },
+          "TerritoryId": 155,
+          "InteractionType": "WalkTo",
+          "Fly": true,
+          "AetheryteShortcut": "Coerthas Central Highlands - Camp Dragonhead"
         },
         {
           "DataId": 2000078,
@@ -374,17 +386,7 @@
           "KillEnemyDataIds": [
             9490
           ],
-          "Fly": true,
-          "AetheryteShortcut": "Coerthas Central Highlands - Camp Dragonhead",
-          "$": "16 17  0 0  0 128 -> 0 17  0 0  0 0",
-          "CompletionQuestVariablesFlags": [
-            null,
-            null,
-            null,
-            null,
-            null,
-            64
-          ]
+          "$": "16 17  0 0  0 128 -> 0 17  0 0  0 0"
         }
       ]
     },

QuestPaths/2.x - A Realm Reborn/Unlocks/Gold Saucer/435_World of Wonders.json

@@ -191,15 +191,6 @@
           "AethernetShard": "[Gold Saucer] Cactpot Board",
           "DelaySecondsAtStart": 3
         },
-        {
-          "Position": {
-            "X": 111.36922,
-            "Y": 13.000123,
-            "Z": -24.209782
-          },
-          "TerritoryId": 144,
-          "InteractionType": "WalkTo"
-        },
         {
           "DataId": 1011079,
           "Position": {
@@ -216,15 +207,6 @@
     {
       "Sequence": 6,
       "Steps": [
-        {
-          "Position": {
-            "X": 111.36922,
-            "Y": 13.000123,
-            "Z": -24.209782
-          },
-          "TerritoryId": 144,
-          "InteractionType": "WalkTo"
-        },
         {
           "TerritoryId": 144,
           "InteractionType": "None",

QuestPaths/4.x - Stormblood/Allied Societies/Ananta/Dailies/3052_When Feathers Fly.json

@@ -21,32 +21,6 @@
     {
       "Sequence": 1,
       "Steps": [
-        {
-          "DataId": 1025066,
-          "Position": {
-            "X": -155.90149,
-            "Y": 66.2623,
-            "Z": -366.93372
-          },
-          "TerritoryId": 612,
-          "InteractionType": "Emote",
-          "Emote": "rally",
-          "Fly": true,
-          "RequiredQuestVariables": [null, null, [16], null, null, null]
-        },
-        {
-          "DataId": 1025064,
-          "Position": {
-            "X": 185.62585,
-            "Y": 68.93902,
-            "Z": -372.15234
-          },
-          "TerritoryId": 612,
-          "InteractionType": "Emote",
-          "RequiredQuestVariables": [null, [1, 17], null, null, null, null],
-          "Emote": "soothe",
-          "Fly": true
-        },
         {
           "DataId": 1025063,
           "Position": {
@@ -56,7 +30,6 @@
           },
           "TerritoryId": 612,
           "InteractionType": "Emote",
-          "RequiredQuestVariables": [null, [2, 18], null, null, null, null],
           "Emote": "soothe",
           "Fly": true
         },
@@ -69,7 +42,6 @@
           },
           "TerritoryId": 612,
           "InteractionType": "Emote",
-          "RequiredQuestVariables": [null, null, [32], null, null, null],
           "Emote": "rally",
           "Fly": true
         }
@@ -86,7 +58,6 @@
           },
           "TerritoryId": 612,
           "InteractionType": "WalkTo",
-          "AetheryteShortcut": "Fringes - Peering Stones",
           "Fly": true
         },
         {

QuestPaths/4.x - Stormblood/Allied Societies/Ananta/Dailies/3052_When Feathers Fly.md

@@ -1,7 +0,0 @@
-```
-0 0 x  x  0 0 | 1025063 1025064 1025065 1025066 
-    1  16                  x               x
-    2  16          x                       x
-    1  32                  x       x
-    2  32          x               x
-```

Questionable/Controller/GameUi/InteractionUiController.cs

@@ -235,16 +235,6 @@ internal sealed class InteractionUiController : IDisposable
             _logger.LogInformation("Checking if current quest {Name} is on the list", currentQuest.Quest.Info.Name);
             if (CheckQuestSelection(addonSelectIconString, currentQuest.Quest, answers))
                 return;
-
-            var sequence = currentQuest.Quest.FindSequence(currentQuest.Sequence);
-            QuestStep? step = sequence?.FindStep(currentQuest.Step);
-            if (step is { InteractionType: EInteractionType.AcceptQuest, PickUpQuestId: not null } &&
-                _questRegistry.TryGetQuest(step.PickUpQuestId, out Quest? pickupQuest))
-            {
-                _logger.LogInformation("Checking if current picked-up {Name} is on the list", pickupQuest.Info.Name);
-                if (CheckQuestSelection(addonSelectIconString, pickupQuest, answers))
-                    return;
-            }
         }
 
         var nextQuest = _questController.NextQuest;

Questionable/Controller/Steps/Shared/MoveTo.cs

@@ -68,7 +68,7 @@ internal static class MoveTo
         public ITask Move(MoveParams moveParams)
         {
             return new MoveInternal(moveParams, movementController, gameFunctions,
-                loggerFactory.CreateLogger<MoveInternal>(), clientState, dataManager);
+                loggerFactory.CreateLogger<MoveInternal>(), condition, clientState, dataManager);
         }
 
         public ITask Land()
@@ -163,22 +163,24 @@ internal static class MoveTo
         private readonly string _cannotExecuteAtThisTime;
         private readonly MovementController _movementController;
         private readonly ILogger<MoveInternal> _logger;
+        private readonly ICondition _condition;
         private readonly IClientState _clientState;
 
         private readonly Action _startAction;
         private readonly Vector3 _destination;
         private readonly MoveParams _moveParams;
-        private bool _canRestart;
 
         public MoveInternal(MoveParams moveParams,
             MovementController movementController,
             GameFunctions gameFunctions,
             ILogger<MoveInternal> logger,
+            ICondition condition,
             IClientState clientState,
             IDataManager dataManager)
         {
             _movementController = movementController;
             _logger = logger;
+            _condition = condition;
             _clientState = clientState;
             _cannotExecuteAtThisTime = dataManager.GetString<LogMessage>(579, x => x.Text)!;
 
@@ -211,7 +213,6 @@ internal static class MoveTo
             }
 
             _moveParams = moveParams;
-            _canRestart = moveParams.RestartNavigation;
         }
 
         public bool Start()
@@ -230,20 +231,13 @@ internal static class MoveTo
             if (movementStartedAt == DateTime.MaxValue || movementStartedAt.AddSeconds(2) >= DateTime.Now)
                 return ETaskResult.StillRunning;
 
-            if (_canRestart &&
+            if (_moveParams.RestartNavigation &&
                 Vector3.Distance(_clientState.LocalPlayer!.Position, _destination) >
                 (_moveParams.StopDistance ?? QuestStep.DefaultStopDistance) + 5f)
             {
-                _canRestart = false;
-                if (_clientState.TerritoryType == _moveParams.TerritoryId)
-                {
-                    _logger.LogInformation("Looks like movement was interrupted, re-attempting to move");
-                    _startAction();
-                    return ETaskResult.StillRunning;
-                }
-                else
-                    _logger.LogInformation(
-                        "Looks like movement was interrupted, do nothing since we're in a different territory now");
+                _logger.LogInformation("Looks like movement was interrupted, re-attempting to move");
+                _startAction();
+                return ETaskResult.StillRunning;
             }
 
             return ETaskResult.TaskComplete;
@@ -253,7 +247,8 @@ internal static class MoveTo
 
         public bool OnErrorToast(SeString message)
         {
-            if (GameFunctions.GameStringEquals(_cannotExecuteAtThisTime, message.TextValue))
+            if (GameFunctions.GameStringEquals(_cannotExecuteAtThisTime, message.TextValue) &&
+                _condition[ConditionFlag.Diving])
                 return true;
 
             return false;

Questionable/External/QuestionableIpc.cs

@@ -1,31 +0,0 @@
-using System;
-using Dalamud.Plugin;
-using Dalamud.Plugin.Ipc;
-using Questionable.Controller;
-
-namespace Questionable.External;
-
-internal sealed class QuestionableIpc : IDisposable
-{
-    private const string IpcIsRunning = "Questionable.IsRunning";
-    private const string IpcGetCurrentQuestId = "Questionable.GetCurrentQuestId";
-
-    private readonly ICallGateProvider<bool> _isRunning;
-    private readonly ICallGateProvider<string?> _getCurrentQuestId;
-
-    public QuestionableIpc(QuestController questController, IDalamudPluginInterface pluginInterface)
-    {
-        _isRunning = pluginInterface.GetIpcProvider<bool>(IpcIsRunning);
-        _isRunning.RegisterFunc(() => questController.IsRunning);
-
-        _getCurrentQuestId = pluginInterface.GetIpcProvider<string?>(IpcGetCurrentQuestId);
-        _getCurrentQuestId.RegisterFunc(() => questController.CurrentQuest?.Quest.Id.ToString());
-    }
-
-
-    public void Dispose()
-    {
-        _getCurrentQuestId.UnregisterFunc();
-        _isRunning.UnregisterFunc();
-    }
-}

Questionable/QuestionablePlugin.cs

@@ -120,7 +120,6 @@ public sealed class QuestionablePlugin : IDalamudPlugin
         serviceCollection.AddSingleton<LifestreamIpc>();
         serviceCollection.AddSingleton<YesAlreadyIpc>();
         serviceCollection.AddSingleton<ArtisanIpc>();
-        serviceCollection.AddSingleton<QuestionableIpc>();
     }
 
     private static void AddTaskFactories(ServiceCollection serviceCollection)
@@ -236,7 +235,6 @@ public sealed class QuestionablePlugin : IDalamudPlugin
         serviceProvider.GetRequiredService<CreditsController>();
         serviceProvider.GetRequiredService<HelpUiController>();
         serviceProvider.GetRequiredService<LeveUiController>();
-        serviceProvider.GetRequiredService<QuestionableIpc>();
         serviceProvider.GetRequiredService<DalamudInitializer>();
     }
 

Questionable/Windows/QuestComponents/ActiveQuestComponent.cs

@@ -177,7 +177,7 @@ internal sealed partial class ActiveQuestComponent
             {
                 using var _ = ImRaii.PushColor(ImGuiCol.Text, ImGuiColors.DalamudYellow);
                 ImGui.TextUnformatted(
-                    $"Next Quest: {Shorten(nextQuest.Quest.Info.Name)} / {nextQuest.Sequence} / {nextQuest.Step}");
+                    $"Next Quest: {Shorten(currentQuest.Quest.Info.Name)} / {currentQuest.Sequence} / {currentQuest.Step}");
             }
         }
     }