forked from OpenWF/SpaceNinjaServer
fix: junction completion on steel path doesn't save (#2937)
Aka., an alternative approach to fixing the problem in #2866. Junctions don't have RewardInfo and therefore weren't reaching the new call to addMissionComplete. Reviewed-on: OpenWF/SpaceNinjaServer#2937 Co-authored-by: Sainan <63328889+Sainan@users.noreply.github.com> Co-committed-by: Sainan <63328889+Sainan@users.noreply.github.com>
This commit is contained in:
parent
0d21c73ab7
commit
23abe5de02
@ -2180,7 +2180,7 @@ export const addMissionComplete = (inventory: TInventoryDatabaseDocument, { Tag,
|
|||||||
|
|
||||||
if (itemIndex !== -1) {
|
if (itemIndex !== -1) {
|
||||||
Missions[itemIndex].Completes += Completes;
|
Missions[itemIndex].Completes += Completes;
|
||||||
if (Tier) {
|
if (Completes && Tier) {
|
||||||
Missions[itemIndex].Tier = Tier;
|
Missions[itemIndex].Tier = Tier;
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
|
|||||||
@ -310,6 +310,9 @@ export const addMissionInventoryUpdates = async (
|
|||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
|
case "Missions":
|
||||||
|
addMissionComplete(inventory, value);
|
||||||
|
break;
|
||||||
case "LastRegionPlayed":
|
case "LastRegionPlayed":
|
||||||
if (!(config.unfaithfulBugFixes?.ignore1999LastRegionPlayed && value === "1999MapName")) {
|
if (!(config.unfaithfulBugFixes?.ignore1999LastRegionPlayed && value === "1999MapName")) {
|
||||||
inventory.LastRegionPlayed = value;
|
inventory.LastRegionPlayed = value;
|
||||||
@ -1242,9 +1245,6 @@ export const addMissionRewards = async (
|
|||||||
if (missions && missions.Tag in ExportRegions) {
|
if (missions && missions.Tag in ExportRegions) {
|
||||||
const node = ExportRegions[missions.Tag];
|
const node = ExportRegions[missions.Tag];
|
||||||
|
|
||||||
// cannot add this with normal updates because { Tier: 1 } would mark the SP node as completed even on a failure
|
|
||||||
addMissionComplete(inventory, missions);
|
|
||||||
|
|
||||||
//node based credit rewards for mission completion
|
//node based credit rewards for mission completion
|
||||||
if (isEligibleForCreditReward(rewardInfo, missions, node)) {
|
if (isEligibleForCreditReward(rewardInfo, missions, node)) {
|
||||||
const levelCreditReward = getLevelCreditRewards(node);
|
const levelCreditReward = getLevelCreditRewards(node);
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user