Skip to content

Fix divide by zero error in restock process#153

Open
BaptTF wants to merge 2 commits intomainfrom
fix/restockdividebyzero
Open

Fix divide by zero error in restock process#153
BaptTF wants to merge 2 commits intomainfrom
fix/restockdividebyzero

Conversation

@BaptTF
Copy link
Member

@BaptTF BaptTF commented Apr 30, 2025

Adjust the AmountPerBundle to 1 when it is zero and log a warning to prevent potential divide by zero errors during item updates.

@aripot007
Copy link
Contributor

Est-ce que ce serait pas mieux de rejeter la réappro et de renvoyer une erreur à l'utilisateur dans ce cas la ? Ça ne devrait pas arriver sauf en cas d'erreur de l'utilisateur, et si on change juste le AmountPerBundle sans prévenir l'utilisateur ça va très probablement faire des erreurs de stock

@aripot007 aripot007 added bug Something isn't working question Further information is requested labels Aug 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working question Further information is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants