src/Controllers/Common/ContentController.cs

@@ -967,25 +967,8 @@ public class ContentController : Controller {
         
         uint gameVersion = ClientVersion.GetVersion(apiKey);
         UserMissionStateResult result = new UserMissionStateResult { Missions = new List<Mission>() };
-
-        HashSet<int> upcomingMissionsSet = new(missionStore.GetUpcomingMissions(gameVersion));
-        var toDiscardIds = new HashSet<int>(
-            viking.MissionStates
-                  .Where(x => x.MissionStatus != MissionStatus.Upcoming)
-                  .Select(x => x.MissionId)
-        );
-
-        var toAddIds = new HashSet<int>(
-            viking.MissionStates
-                  .Where(x => x.MissionStatus == MissionStatus.Upcoming)
-                  .Select(x => x.MissionId)
-        );
-
-        upcomingMissionsSet.UnionWith(toAddIds);
-        upcomingMissionsSet.ExceptWith(toDiscardIds);
-
-        foreach (var missionId in upcomingMissionsSet)
-            result.Missions.Add(missionService.GetMissionWithProgress(missionId, viking.Id, gameVersion));
+        foreach (var mission in viking.MissionStates.Where(x => x.MissionStatus == MissionStatus.Upcoming))
+            result.Missions.Add(missionService.GetMissionWithProgress(mission.MissionId, viking.Id, gameVersion));
 
         result.UserID = viking.Uid;
         return Ok(result);
@@ -1033,27 +1016,16 @@ public class ContentController : Controller {
     [Produces("application/xml")]
     [Route("ContentWebService.asmx/AcceptMission")]
     [VikingSession]
-    public IActionResult AcceptMission(Viking viking, [FromForm] Guid userId, [FromForm] int missionId, [FromForm] string apiKey) {
+    public IActionResult AcceptMission(Viking viking, [FromForm] Guid userId, [FromForm] int missionId) {
         if (viking.Uid != userId)
             return Unauthorized("Can't accept not owned mission");
 
-        uint gameVersion = ClientVersion.GetVersion(apiKey);
-        int[] upcomingMissions = missionStore.GetUpcomingMissions(gameVersion);
-
         MissionState? missionState = viking.MissionStates.FirstOrDefault(x => x.MissionId == missionId);
-        if (!upcomingMissions.Contains(missionId) || (missionState is not null && missionState.MissionStatus != MissionStatus.Upcoming))
+        if (missionState is null || missionState.MissionStatus != MissionStatus.Upcoming)
             return Ok(false);
 
-        if (missionState is null) {
-            viking.MissionStates.Add(new MissionState {
-                MissionId = missionId,
-                MissionStatus = MissionStatus.Active,
-                UserAccepted = true
-            });
-        } else {
-            missionState.MissionStatus = MissionStatus.Active;
-            missionState.UserAccepted = true;
-        }
+        missionState.MissionStatus = MissionStatus.Active;
+        missionState.UserAccepted = true;
         ctx.SaveChanges();
         return Ok(true);
     }
@@ -1068,19 +1040,10 @@ public class ContentController : Controller {
 
         uint gameVersion = ClientVersion.GetVersion(apiKey);
         UserMissionStateResult result = new UserMissionStateResult { Missions = new List<Mission>()  };
+        foreach (var mission in viking.MissionStates) {
+            Mission updatedMission = missionService.GetMissionWithProgress(mission.MissionId, viking.Id, gameVersion);
 
-        var missionStatesById = viking.MissionStates.ToDictionary(ms => ms.MissionId);
-        HashSet<int> upcomingMissionIds = new(missionStore.GetUpcomingMissions(gameVersion));
-        var combinedMissionIds = new HashSet<int>(missionStatesById.Keys);
-        combinedMissionIds.UnionWith(upcomingMissionIds);
-
-        foreach (var missionId in combinedMissionIds) {
-            MissionState? missionState = missionStatesById.TryGetValue(missionId, out var ms) ? ms : null;
-            Mission updatedMission = missionService.GetMissionWithProgress(missionId, viking.Id, gameVersion);
-
-            // Only upcoming missions are missing in the database, so if db is null, mission must be upcoming
-            MissionStatus status = missionState != null ? missionState.MissionStatus : MissionStatus.Upcoming;
-            if (status == MissionStatus.Upcoming) {
+            if (mission.MissionStatus == MissionStatus.Upcoming) {
                 // NOTE: in old SoD job board mission must be send as non active and required accept
                 //       (to avoid show all job board in journal and quest arrow pointing to job board)
                 //       do this in this place (instead of update missions.xml) to avoid conflict with newer versions of SoD
@@ -1089,8 +1052,8 @@ public class ContentController : Controller {
                     prerequisite.Value = "true";
             }
 
-            if (missionState != null && missionState.UserAccepted != null)
-                updatedMission.Accepted = (bool)missionState.UserAccepted;
+            if (mission.UserAccepted != null)
+                updatedMission.Accepted = (bool)mission.UserAccepted;
             result.Missions.Add(updatedMission);
         }
 

src/Services/MissionService.cs

@@ -219,12 +219,8 @@ public class MissionService {
 
             Schema.Task? t = mission.Tasks.Find(x => x.TaskID == task.Id);
             if (t != null) {
-                if (task.Completed) {
-                    t.Completed = 1;
-                    t.Payload = task.Payload ?? "<?xml version=\"1.0\" encoding=\"utf-8\"?>\r\n<Data>\r\n  <S>true</S>\r\n</Data>";
-                } else {
-                    t.Payload = task.Payload;
-                }
+                if (task.Completed) t.Completed = 1;
+                t.Payload = task.Payload;
             }
         }
 
@@ -251,16 +247,18 @@ public class MissionService {
                 MissionStatus = MissionStatus.Active
             });
         }
+        
+        foreach (int m in missionStore.GetUpcomingMissions(gameVersion)) {
+            viking.MissionStates.Add(new MissionState {
+                MissionId = m,
+                MissionStatus = MissionStatus.Upcoming
+            });
+        }
     }
 
-    private void SetTaskProgressDB(int missionId, int taskId, int userId, bool completed, string? xmlPayload) {
+    private void SetTaskProgressDB(int missionId, int taskId, int userId, bool completed, string xmlPayload) {
         Model.TaskStatus? status = ctx.TaskStatuses.FirstOrDefault(task => task.Id == taskId && task.MissionId == missionId && task.VikingId == userId);
 
-        // Based on the observation that no payloads on completed missions are null
-        // This is the most common task payload, so if the mission is completed with this payload we can set the payload to null and let the server autofill
-        if (completed && xmlPayload == "<?xml version=\"1.0\" encoding=\"utf-8\"?>\r\n<Data>\r\n  <S>true</S>\r\n</Data>")
-            xmlPayload = null;
-
         if (status is null) {
             status = new Model.TaskStatus {
                 Id = taskId,