Nicolas Weeger - 2020-11-02

Thanks for the patch :)

Unless I'm mistaking, the price deposited to the account is the price the seller wanted. But the price the buyer actually paid may be less because of bargaining... So you'll probably deposit more than actually paid... That IMO is an issue.

Some technical points:
- the value the player says in gold should be multiplied by 10 to get in silver. Your code actually does that, in 2 parts (x2 for the named price, then x5 when sold), so it should probably just multiply the gold price by 10 and let it there
- lines 72 and 73 will definitely won't work / lead to issues. When "Remove()" is called the object is edstroyed so becomes invalid, then the "Name" is invalid too. You should probably just call "Remove()" without any loop, or repeat the previous loop if you want to be sure to remove everything
- line 46, you can probably just copy the "Slaying" of the event object itself, it'd be equivalent