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

added check to make sure same item isnt used, still has an edge case, if maxRequestedAmount is too high, and we cant offer enough items, dupes will appear, but this will fix normal use

Browse Source
This commit is contained in:
CWXDEV 2024-01-12 23:44:58 +00:00
parent 04564d7e76
commit 091e4fa41b
1 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
project/src/generators/RepeatableQuestGenerator.ts
View File

@ -559,9 +559,19 @@ export class RepeatableQuestGenerator
// Draw items to ask player to retrieve // Draw items to ask player to retrieve
let isAmmo = 0 let isAmmo = 0
const randomNumbersUsed = [];
for (let i = 0; i < distinctItemsToRetrieveCount; i++) for (let i = 0; i < distinctItemsToRetrieveCount; i++)
{ {
const itemSelected = itemSelection[this.randomUtil.randInt(itemSelection.length)]; let randomNumber = this.randomUtil.randInt(itemSelection.length);
while (randomNumbersUsed.includes(randomNumber) && randomNumbersUsed.length !== itemSelection.length)
{
randomNumber = this.randomUtil.randInt(itemSelection.length);
}
randomNumbersUsed.push(randomNumber);
const itemSelected = itemSelection[randomNumber];
const itemUnitPrice = this.itemHelper.getItemPrice(itemSelected[0]); const itemUnitPrice = this.itemHelper.getItemPrice(itemSelected[0]);
let minValue = completionConfig.minRequestedAmount; let minValue = completionConfig.minRequestedAmount;
let maxValue = completionConfig.maxRequestedAmount; let maxValue = completionConfig.maxRequestedAmount;