Platinum/the-blacklist
Platinum/the-blacklist
0
0
Fork 0
Code Issues Pull Requests Projects Releases 9 Wiki Activity

Invalid Price Data Added to Database #1

New Issue
Closed
opened 2023-07-12 00:17:43 -04:00 by DanW · 1 comment
DanW commented 2023-07-12 00:17:43 -04:00
Copy Link

I found that when loading this mod (version 1.1.0), a lot of records were added to the templates.prices dictionary that don't have valid data (null, undefined, or NaN). I attached an excerpt from one of my server logs that has the hashes of all affected items in base SPT-AKI 3.5.8 and the values this mod generated for each.

Could you please update your mod to only add records to the templates.prices dictionary if they contain valid data?

Currently, I'm testing for this by doing: if ((price == null) || isNaN(price))

Thanks!

I found that when loading this mod (version 1.1.0), a lot of records were added to the *templates.prices* dictionary that don't have valid data (null, undefined, or NaN). I attached an excerpt from one of my server logs that has the hashes of all affected items in base SPT-AKI 3.5.8 and the values this mod generated for each. Could you please update your mod to only add records to the *templates.prices* dictionary if they contain valid data? Currently, I'm testing for this by doing: `if ((price == null) || isNaN(price))` Thanks!
server-2023-07-11-23_excerpt.log
68 KiB
Platinum commented 2024-04-03 05:11:04 -04:00
Owner
Copy Link

Sorry, I just saw this. I did a big overhaul of the code for this mod. Since this is quite an old issue I'm going to close it for now, feel free to re-open if you still observe the same issue with the latest changes and SPT 3.8.0.

Sorry, I just saw this. I did a big overhaul of the code for this mod. Since this is quite an old issue I'm going to close it for now, feel free to re-open if you still observe the same issue with the latest changes and SPT 3.8.0.
👍 1
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Platinum/the-blacklist#1
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?