OpenWF/SpaceNinjaServer
12
6
Fork 14
Code Issues 41 Pull Requests 1 Actions Packages Projects Releases Activity

fix: don't abort quest update when quest completion rewards are missing. #937

Merged
OrdisPrime merged 1 commits from quest-progression into main 2025-02-09 14:37:26 -08:00
Conversation 0 Commits 1 Files Changed 2 +10 -7
2 changed files with 10 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/controllers/api/updateQuestController.ts
View File

@ -30,10 +30,11 @@ export const updateQuestController: RequestHandler = async (req, res) => {
const questCompletionItems = getQuestCompletionItems(questKeyName); const questCompletionItems = getQuestCompletionItems(questKeyName);
logger.debug(`quest completion items`, questCompletionItems); logger.debug(`quest completion items`, questCompletionItems);
const inventoryChanges = await addItems(inventory, questCompletionItems); if (questCompletionItems) {
const inventoryChanges = await addItems(inventory, questCompletionItems);
updateQuestResponse.InventoryChanges = inventoryChanges;
}
inventory.ActiveQuest = ""; inventory.ActiveQuest = "";
updateQuestResponse.InventoryChanges = inventoryChanges;
} }
//TODO: might need to parse the custom data and add the associated items to inventory //TODO: might need to parse the custom data and add the associated items to inventory

10
src/services/itemDataService.ts
View File

@ -189,11 +189,13 @@ export const getNode = (nodeName: string): IRegion => {
}; };
export const getQuestCompletionItems = (questKey: string) => { export const getQuestCompletionItems = (questKey: string) => {
const items = (questCompletionItems as unknown as Record<string, ITypeCount[] | undefined>)[questKey]; const items = (questCompletionItems as unknown as Record<string, ITypeCount[]> | undefined)?.[questKey];
if (!items) {
throw new Error(`Quest ${questKey} not found in questCompletionItems`);
}
if (!items) {
logger.error(
`Quest ${questKey} not found in questCompletionItems, quest completion items have not been given. This is a temporary solution`
);
}
return items; return items;
}; };