search

Lembas-Modding-Team / pvp-mode

A Minecraft mod offering various options to manage availability for and ability to PvP.
GNU Lesser General Public License v3.0
5 stars 3 forks source link

Soulbound crash with partial inventory loss/keeyInventory disabled #311

Closed DwarfyAssassin closed 5 years ago

DwarfyAssassin commented 5 years ago

If you have keepinv false and with partial inv drop and a soulbound item in your inv and you die you get a internal server error and all your items disapear even the unsoulbound items.

I have added the latest.log and the config file (in a zip file since you can't send .cfg files) latest.log .

pvp-mode.zip

EDIT: This bug was noticed while the development of 2.0.0-BETA, and it was fixed. For documentation reasons we'll assign it to the relevant milestone. ~CraftedMods

CraftedMods commented 5 years ago

I'll take a look at it later. Thanks for reporting it.

CraftedMods commented 5 years ago

Ah, I know what you mean. While porting 1.5.0-BETA to 2.0.0-BETA, I noticed exactly that bug with the soulbound command. I fixed it then. How servere is that for your server? Do you need a bugfix version, or can it wait until 2.0.0-BETA.1 (that version should be released within the next week)?

DwarfyAssassin commented 5 years ago

It's not a problem since I have keep inventory enabled

CraftedMods commented 5 years ago

The big is already fixed, but a changelog entry is missing. It has to be added to 2.0.0-BETA.1, then the issue can be closed.