OpenWF/SpaceNinjaServer
28
29
Fork 35
Code Issues 21 Pull Requests Actions Packages Projects Releases Activity

chore: remove undefined as a possible argument when committing inventory change #1047

Merged
Sainan merged 1 commits from no-undefined-changes into main 2025-02-28 03:05:32 -08:00
Conversation 0 Commits 1 Files Changed 1 +25 -36
1 changed files with 25 additions and 36 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

61
src/services/inventoryService.ts
View File

@ -892,12 +892,12 @@ const addDrone = (
//TODO: wrong id is not erroring //TODO: wrong id is not erroring
export const addGearExpByCategory = ( export const addGearExpByCategory = (
inventory: TInventoryDatabaseDocument, inventory: TInventoryDatabaseDocument,
gearArray: IEquipmentClient[] | undefined, gearArray: IEquipmentClient[],
categoryName: TEquipmentKey categoryName: TEquipmentKey
): void => { ): void => {
const category = inventory[categoryName]; const category = inventory[categoryName];
gearArray?.forEach(({ ItemId, XP }) => { gearArray.forEach(({ ItemId, XP }) => {
if (!XP) { if (!XP) {
return; return;
} }
@ -921,10 +921,10 @@ export const addGearExpByCategory = (
}); });
}; };
export const addMiscItems = (inventory: TInventoryDatabaseDocument, itemsArray: IMiscItem[] | undefined): void => { export const addMiscItems = (inventory: TInventoryDatabaseDocument, itemsArray: IMiscItem[]): void => {
const { MiscItems } = inventory; const { MiscItems } = inventory;
itemsArray?.forEach(({ ItemCount, ItemType }) => { itemsArray.forEach(({ ItemCount, ItemType }) => {
if (ItemCount == 0) { if (ItemCount == 0) {
return; return;
} }
@ -943,13 +943,10 @@ export const addMiscItems = (inventory: TInventoryDatabaseDocument, itemsArray:
}); });
}; };
export const addShipDecorations = ( export const addShipDecorations = (inventory: TInventoryDatabaseDocument, itemsArray: IConsumable[]): void => {
inventory: TInventoryDatabaseDocument,
itemsArray: IConsumable[] | undefined
): void => {
const { ShipDecorations } = inventory; const { ShipDecorations } = inventory;
itemsArray?.forEach(({ ItemCount, ItemType }) => { itemsArray.forEach(({ ItemCount, ItemType }) => {
const itemIndex = ShipDecorations.findIndex(miscItem => miscItem.ItemType === ItemType); const itemIndex = ShipDecorations.findIndex(miscItem => miscItem.ItemType === ItemType);
if (itemIndex !== -1) { if (itemIndex !== -1) {
@ -960,10 +957,10 @@ export const addShipDecorations = (
}); });
}; };
export const addConsumables = (inventory: TInventoryDatabaseDocument, itemsArray: IConsumable[] | undefined): void => { export const addConsumables = (inventory: TInventoryDatabaseDocument, itemsArray: IConsumable[]): void => {
const { Consumables } = inventory; const { Consumables } = inventory;
itemsArray?.forEach(({ ItemCount, ItemType }) => { itemsArray.forEach(({ ItemCount, ItemType }) => {
const itemIndex = Consumables.findIndex(i => i.ItemType === ItemType); const itemIndex = Consumables.findIndex(i => i.ItemType === ItemType);
if (itemIndex !== -1) { if (itemIndex !== -1) {
@ -974,13 +971,10 @@ export const addConsumables = (inventory: TInventoryDatabaseDocument, itemsArray
}); });
}; };
export const addCrewShipRawSalvage = ( export const addCrewShipRawSalvage = (inventory: TInventoryDatabaseDocument, itemsArray: ITypeCount[]): void => {
inventory: TInventoryDatabaseDocument,
itemsArray: ITypeCount[] | undefined
): void => {
const { CrewShipRawSalvage } = inventory; const { CrewShipRawSalvage } = inventory;
itemsArray?.forEach(({ ItemCount, ItemType }) => { itemsArray.forEach(({ ItemCount, ItemType }) => {
const itemIndex = CrewShipRawSalvage.findIndex(i => i.ItemType === ItemType); const itemIndex = CrewShipRawSalvage.findIndex(i => i.ItemType === ItemType);
if (itemIndex !== -1) { if (itemIndex !== -1) {
@ -991,10 +985,10 @@ export const addCrewShipRawSalvage = (
}); });
}; };
export const addCrewShipAmmo = (inventory: TInventoryDatabaseDocument, itemsArray: ITypeCount[] | undefined): void => { export const addCrewShipAmmo = (inventory: TInventoryDatabaseDocument, itemsArray: ITypeCount[]): void => {
const { CrewShipAmmo } = inventory; const { CrewShipAmmo } = inventory;
itemsArray?.forEach(({ ItemCount, ItemType }) => { itemsArray.forEach(({ ItemCount, ItemType }) => {
const itemIndex = CrewShipAmmo.findIndex(i => i.ItemType === ItemType); const itemIndex = CrewShipAmmo.findIndex(i => i.ItemType === ItemType);
if (itemIndex !== -1) { if (itemIndex !== -1) {
@ -1005,10 +999,10 @@ export const addCrewShipAmmo = (inventory: TInventoryDatabaseDocument, itemsArra
}); });
}; };
export const addRecipes = (inventory: TInventoryDatabaseDocument, itemsArray: ITypeCount[] | undefined): void => { export const addRecipes = (inventory: TInventoryDatabaseDocument, itemsArray: ITypeCount[]): void => {
const { Recipes } = inventory; const { Recipes } = inventory;
itemsArray?.forEach(({ ItemCount, ItemType }) => { itemsArray.forEach(({ ItemCount, ItemType }) => {
const itemIndex = Recipes.findIndex(i => i.ItemType === ItemType); const itemIndex = Recipes.findIndex(i => i.ItemType === ItemType);
if (itemIndex !== -1) { if (itemIndex !== -1) {
@ -1019,10 +1013,10 @@ export const addRecipes = (inventory: TInventoryDatabaseDocument, itemsArray: IT
}); });
}; };
export const addMods = (inventory: TInventoryDatabaseDocument, itemsArray: IRawUpgrade[] | undefined): void => { export const addMods = (inventory: TInventoryDatabaseDocument, itemsArray: IRawUpgrade[]): void => {
const { RawUpgrades } = inventory; const { RawUpgrades } = inventory;
itemsArray?.forEach(({ ItemType, ItemCount }) => { itemsArray.forEach(({ ItemType, ItemCount }) => {
if (ItemCount == 0) { if (ItemCount == 0) {
return; return;
} }
@ -1041,12 +1035,9 @@ export const addMods = (inventory: TInventoryDatabaseDocument, itemsArray: IRawU
}); });
}; };
export const addFusionTreasures = ( export const addFusionTreasures = (inventory: TInventoryDatabaseDocument, itemsArray: IFusionTreasure[]): void => {
inventory: TInventoryDatabaseDocument,
itemsArray: IFusionTreasure[] | undefined
): void => {
const { FusionTreasures } = inventory; const { FusionTreasures } = inventory;
itemsArray?.forEach(({ ItemType, ItemCount, Sockets }) => { itemsArray.forEach(({ ItemType, ItemCount, Sockets }) => {
const itemIndex = FusionTreasures.findIndex(i => i.ItemType == ItemType && (i.Sockets || 0) == (Sockets || 0)); const itemIndex = FusionTreasures.findIndex(i => i.ItemType == ItemType && (i.Sockets || 0) == (Sockets || 0));
if (itemIndex !== -1) { if (itemIndex !== -1) {
@ -1057,7 +1048,7 @@ export const addFusionTreasures = (
}); });
}; };
export const addFocusXpIncreases = (inventory: TInventoryDatabaseDocument, focusXpPlus: number[] | undefined): void => { export const addFocusXpIncreases = (inventory: TInventoryDatabaseDocument, focusXpPlus: number[]): void => {
enum FocusType { enum FocusType {
AP_UNIVERSAL, AP_UNIVERSAL,
AP_ATTACK, AP_ATTACK,
@ -1071,14 +1062,12 @@ export const addFocusXpIncreases = (inventory: TInventoryDatabaseDocument, focus
AP_ANY AP_ANY
} }
if (focusXpPlus) { inventory.FocusXP ??= { AP_ATTACK: 0, AP_DEFENSE: 0, AP_TACTIC: 0, AP_POWER: 0, AP_WARD: 0 };
inventory.FocusXP ??= { AP_ATTACK: 0, AP_DEFENSE: 0, AP_TACTIC: 0, AP_POWER: 0, AP_WARD: 0 }; inventory.FocusXP.AP_ATTACK += focusXpPlus[FocusType.AP_ATTACK];
inventory.FocusXP.AP_ATTACK += focusXpPlus[FocusType.AP_ATTACK]; inventory.FocusXP.AP_DEFENSE += focusXpPlus[FocusType.AP_DEFENSE];
inventory.FocusXP.AP_DEFENSE += focusXpPlus[FocusType.AP_DEFENSE]; inventory.FocusXP.AP_TACTIC += focusXpPlus[FocusType.AP_TACTIC];
inventory.FocusXP.AP_TACTIC += focusXpPlus[FocusType.AP_TACTIC]; inventory.FocusXP.AP_POWER += focusXpPlus[FocusType.AP_POWER];
inventory.FocusXP.AP_POWER += focusXpPlus[FocusType.AP_POWER]; inventory.FocusXP.AP_WARD += focusXpPlus[FocusType.AP_WARD];
inventory.FocusXP.AP_WARD += focusXpPlus[FocusType.AP_WARD];
}
}; };
export const updateChallengeProgress = async ( export const updateChallengeProgress = async (