470486104/Server
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix issues with incorrectly configured airdrops (!280)

Browse Source 
- Don't try to add items to the airdrop if the filtered item list is empty
- If we somehow get into `findAndAddRandomPresetToLoot` with an empty list, don't try to output the id of undefined

Co-authored-by: DrakiaXYZ <565558+TheDgtl@users.noreply.github.com>
Reviewed-on: https://dev.sp-tarkov.com/SPT-AKI/Server/pulls/280
Co-authored-by: DrakiaXYZ <drakiaxyz@noreply.dev.sp-tarkov.com>
Co-committed-by: DrakiaXYZ <drakiaxyz@noreply.dev.sp-tarkov.com>
This commit is contained in:
DrakiaXYZ 2024-04-08 17:39:08 +00:00 committed by chomp
parent f73802ec17
commit 2404e7a5ab
1 changed files with 35 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

55
project/src/generators/LootGenerator.ts
View File

@ -99,12 +99,15 @@ export class LootGenerator
&& options.itemTypeWhitelist.includes(x[1]._parent) && options.itemTypeWhitelist.includes(x[1]._parent)
); );
const randomisedItemCount = this.randomUtil.getInt(options.itemCount.min, options.itemCount.max); if (items.length > 0)
for (let index = 0; index < randomisedItemCount; index++)
{ {
if (!this.findAndAddRandomItemToLoot(items, itemTypeCounts, options, result)) const randomisedItemCount = this.randomUtil.getInt(options.itemCount.min, options.itemCount.max);
for (let index = 0; index < randomisedItemCount; index++)
{ {
index--; if (!this.findAndAddRandomItemToLoot(items, itemTypeCounts, options, result))
{
index--;
}
} }
} }
@ -122,13 +125,21 @@ export class LootGenerator
this.itemHelper.isOfBaseclass(preset._encyclopedia, BaseClasses.WEAPON) this.itemHelper.isOfBaseclass(preset._encyclopedia, BaseClasses.WEAPON)
); );
for (let index = 0; index < randomisedWeaponPresetCount; index++) if (weaponDefaultPresets.length > 0)
{ {
if ( for (let index = 0; index < randomisedWeaponPresetCount; index++)
!this.findAndAddRandomPresetToLoot(weaponDefaultPresets, itemTypeCounts, itemBlacklistArray, result)
)
{ {
index--; if (
!this.findAndAddRandomPresetToLoot(
weaponDefaultPresets,
itemTypeCounts,
itemBlacklistArray,
result,
)
)
{
index--;
}
} }
} }
} }
@ -146,18 +157,22 @@ export class LootGenerator
const levelFilteredArmorPresets = armorDefaultPresets.filter((armor) => const levelFilteredArmorPresets = armorDefaultPresets.filter((armor) =>
this.armorIsDesiredProtectionLevel(armor, options) this.armorIsDesiredProtectionLevel(armor, options)
); );
for (let index = 0; index < randomisedArmorPresetCount; index++)
if (levelFilteredArmorPresets.length > 0)
{ {
if ( for (let index = 0; index < randomisedArmorPresetCount; index++)
!this.findAndAddRandomPresetToLoot(
levelFilteredArmorPresets,
itemTypeCounts,
itemBlacklistArray,
result,
)
)
{ {
index--; if (
!this.findAndAddRandomPresetToLoot(
levelFilteredArmorPresets,
itemTypeCounts,
itemBlacklistArray,
result,
)
)
{
index--;
}
} }
} }
} }
@ -307,7 +322,7 @@ export class LootGenerator
const randomPreset = this.randomUtil.getArrayValue(globalDefaultPresets); const randomPreset = this.randomUtil.getArrayValue(globalDefaultPresets);
if (!randomPreset?._encyclopedia) if (!randomPreset?._encyclopedia)
{ {
this.logger.debug(`Airdrop - preset with id: ${randomPreset._id} lacks encyclopedia property, skipping`); this.logger.debug(`Airdrop - preset with id: ${randomPreset?._id} lacks encyclopedia property, skipping`);
return false; return false;
} }