If two players click same chest, first one's inventory will be lost. #1

New Issue

whosit · 2025-09-06T06:35:48+02:00

whosit commented

2025-09-06 06:35:48 +02:00

Needs a check if there's something stored already.
Probably needs some logging.
Maybe needs some indication that chest is already in use.
Also, with this implementation, if server crashes, there's no in-game way to restore inventory...

Needs a check if there's something stored already. Probably needs some logging. Maybe needs some indication that chest is already in use. Also, with this implementation, if server crashes, there's no in-game way to restore inventory...

whosit commented

2025-09-06 06:39:50 +02:00

Also, takes into account only main inv, bags are still accessible?

Also, takes into account only `main` inv, bags are still accessible?

Boot self-assigned this 2025-09-06 07:07:52 +02:00

Boot referenced this issue

2025-09-06 07:08:50 +02:00

bags are still accessible? #2

Boot commented

2025-09-06 09:21:11 +02:00

Needs a check if there's something stored already.

Now a player can only use the crate once.

Probably needs some logging.

Logging has been added.

Maybe needs some indication that chest is already in use.

Now a player receives the message "You have already used an Arena Chest!".

Also, with this implementation, if server crashes, there's no in-game way to restore inventory...

Now we store data persistently with mod_storage. Backups are saved during shutdown, loaded at restart.

> Needs a check if there's something stored already. Now a player can only use the crate once. > Probably needs some logging. Logging has been added. > Maybe needs some indication that chest is already in use. Now a player receives the message "You have already used an Arena Chest!". > Also, with this implementation, if server crashes, there's no in-game way to restore inventory... Now we store data persistently with `mod_storage`. Backups are saved during shutdown, loaded at restart.

whosit commented

2025-09-06 09:33:21 +02:00

Now a player receives the message "You have already used an Arena Chest!".

It does, but I think initial issue is still the same - player can lose their inventory.
Try it yourself with 2 players and single chest?

> Now a player receives the message "You have already used an Arena Chest!". It does, but I think initial issue is still the same - player can lose their inventory. Try it yourself with 2 players and single chest?

whosit commented

2025-09-06 09:34:50 +02:00

Modstorage may be overkill for this - you already store inventory and owner name in meta - this should be enough to allow people to take their stuff back, no modstorage needed.

Boot added the

1a - bug

1c - in progress

labels 2025-09-06 11:19:17 +02:00

tour referenced this issue

2025-09-06 14:28:06 +02:00

Several design problems #6

Boot added the