Crafting grid deletes items stacked over quantity 99 #4655

Open
opened 2023-06-01 15:14:35 +00:00 by niceride · 7 comments

Steps to reproduce:

  1. Furnace a couple of mese blocks yields two stacks of 162 mesecons wires. Store these
  2. Add 99 mesecons wires to crafting grid in any one of the crafting grid squares (i).
  3. Drag one stack of 162x mesecons wires into any other of the crafting grid squares (ii).
  4. Drag (ii) and drop onto (i), your stack of 162x mesecons wires has deleted all over 99... so you have just lost 63 mesecons wires to this bug.
Steps to reproduce: 1. Furnace a couple of mese blocks yields two stacks of 162 mesecons wires. Store these 2. Add 99 mesecons wires to crafting grid in any one of the crafting grid squares (i). 3. Drag one stack of 162x mesecons wires into any other of the crafting grid squares (ii). 4. Drag (ii) and drop onto (i), your stack of 162x mesecons wires has deleted all over 99... so you have just lost 63 mesecons wires to this bug.

fix by increasing max stack to 200?

fix by increasing max stack to 200?
flux added the
1. kind/bug
3. source/mod upstream
labels 2023-06-01 17:17:11 +00:00
Member

this doesn't just happen in the crafting grid, it happens in any inventory i've tried (main player inventory, chest inventory...)

this doesn't just happen in the crafting grid, it happens in any inventory i've tried (main player inventory, chest inventory...)
flux added
3. source/unknown
and removed
3. source/mod upstream
labels 2023-06-01 17:19:14 +00:00
flux added
3. source/engine
and removed
3. source/unknown
labels 2023-06-01 17:21:32 +00:00
Member
upstream issue: https://github.com/minetest/minetest/issues/13552
Member

upstream is fixed, though we'll have to wait for 5.9 engine to get the fix. this isn't critical, so it can wait.

upstream is fixed, though we'll have to wait for 5.9 engine to get the fix. this isn't critical, so it can wait.
flux added the
4. step/ready to QA test
label 2023-12-17 21:27:56 +00:00

Maybe we can fix the source of the oversized stack as well? I.e. change the recipe to not produce 162 wires, but rather 99+63 wires, i.e. split into 2 stacks?

Maybe we can fix the source of the oversized stack as well? I.e. change the recipe to not produce 162 wires, but rather 99+63 wires, i.e. split into 2 stacks?
Member

Maybe we can fix the source of the oversized stack as well? I.e. change the recipe to not produce 162 wires, but rather 99+63 wires, i.e. split into 2 stacks?

hm. that's slightly tricky. it'd require duplicating a lot of the furnace code logic and some tricky checks. also, i'm not 100% sure that's the only source of such stacks.

> Maybe we can fix the source of the oversized stack as well? I.e. change the recipe to not produce 162 wires, but rather 99+63 wires, i.e. split into 2 stacks? hm. that's slightly tricky. it'd require duplicating a lot of the furnace code logic and some tricky checks. also, i'm not 100% sure that's the only source of such stacks.

I'd rather keep as is and wait for 5.9.0

I'd rather keep as is and wait for 5.9.0
Sign in to join this conversation.
No Milestone
No project
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: your-land/bugtracker#4655
No description provided.