search

diasurgical / devilution

Diablo devolved - magic behind the 1996 computer game
Other
8.79k stars 919 forks source link

items: add BUGFIX for GetItemRecord when handling quest items #2253

Closed mewmew closed 3 years ago

mewmew commented 3 years ago

When spawing quest items (e.g. blood stone), no unique seed is set for the item. Therefor two quest items may share the same seed, this happens deterministically for the Valor quest, since three blood stones are spawned, each with item seed 0x00000000.

For this reason, if two or more such quest items with identical seed are looted within less than 6 seconds, the 2nd, 3rd, etc loot actions are ignored.

galaxyhaxz commented 3 years ago

This explains why when testing the Valor quest, picking up one stone and quickly teleporting to nab the other won't let you pick it up for a few seconds. Very effective anti-dupe code! /s

LGTM :P