Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Items voidind on craft cancel if no more space left #3349

Closed
TiphaineLAURENT opened this issue Aug 1, 2022 · 4 comments
Closed

Items voidind on craft cancel if no more space left #3349

TiphaineLAURENT opened this issue Aug 1, 2022 · 4 comments
Labels
enhancement New feature or request

Comments

@TiphaineLAURENT
Copy link

Describe the bug

My use case is trying to craft an Infinite Storage Part.
I did have the resources before starting.

The craft get locked because there was not enough space to store the crafted part (I am using Mystical Agriculture so a few essences can easily get multiplied after crafted).
I cancel the craft and now that I want to restart it I am missing a lot of the needed resources. Best exemple are the quartz essences which I am now needing ~700 000.

I don't know if it is relevant but I am playing AllTheMods7

How can we reproduce this bug or crash?

  1. Start a big craft with a nearly full storage
  2. Once the disk are full but the craft not finished cancel it

What Minecraft version is this happening on?

Minecraft 1.18.2

What Forge version is this happening on?

40.1.60

What Refined Storage version is this happening on?

1.10.2

Relevant log output

-
@TiphaineLAURENT TiphaineLAURENT added the bug Something isn't working label Aug 1, 2022
@raoulvdberge
Copy link
Collaborator

Ok, but where should the items go if there is no space? Drop on the ground?

@TiphaineLAURENT
Copy link
Author

A simple fix would be to display an error on the craft status informing the user that there is no space left and he should add more but if he cancel it, items will be voided.

The complex option would be to precalculate the space needed before the craft and not start it if it is not possible. I am not really sure if it is possible in term of time. Maybe enable the check with a ingame toggle also informing the user about the risk of not checking ?

@Darkere
Copy link
Collaborator

Darkere commented Sep 30, 2022

I think it would be reasonable if we prevented users from cancelling a task if there is not enough space. Just need to figure out a way to make that transparent to users.

@raoulvdberge raoulvdberge added enhancement New feature or request and removed bug Something isn't working labels Nov 5, 2023
@raoulvdberge
Copy link
Collaborator

This has been added to the RS 2 tracker, and will be implemented for RS 2.x.

Closing this issue for RS 1.x on this repository.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants