Steps to reproduce: 1. Make a receive for a product
2.Go to physical inventory and enter value in ‘stock on hand’ field
3.Add an issue, select a reason and fill ‘quantity’ field
4. Go to Stock on hand menu, click view button near just updated product, as you can see there is adjustment of 30 and stock on hand value is 16000 change.The
Expected behavior
The changes are shown correctly
Cause of the wrong calculations
This issue is indirectly connected to fact that currently system allows users to submit Physical Inventory with unaccounted quantities not equal to 0. After discussion we decided to add validation on Physical Inventory Submit to make sure that there’s no submitted items with unaccounted qty.
During testing, a regression has been found: not related to this pull request
Nikola Laskowska June 15, 2022 at 6:55 AM
code is ready for your review. Thanks.
Nikola Laskowska June 6, 2022 at 11:17 AM
On my local machine I have already added validation. Tomorrow I’m going to write some unit tests and then push for review.
Nikola Laskowska June 3, 2022 at 7:53 AM
This issue is indirectly connected to fact that currently system allows users to submit Physical Inventory with unaccounted quantities not equal to 0. After discussion we decided to add validation on Physical Inventory Submit to make sure that there’s no submitted items with unaccounted qty.
Done
Pinned fields
Click on the next to a field label to start pinning.
Steps to reproduce:
1. Make a receive for a product
2.Go to physical inventory and enter value in ‘stock on hand’ field
3.Add an issue, select a reason and fill ‘quantity’ field
4. Go to Stock on hand menu, click view button near just updated product, as you can see there is adjustment of 30 and stock on hand value is 16000 change.The
Expected behavior
The changes are shown correctly
Cause of the wrong calculations
This issue is indirectly connected to fact that currently system allows users to submit Physical Inventory with unaccounted quantities not equal to 0. After discussion we decided to add validation on Physical Inventory Submit to make sure that there’s no submitted items with unaccounted qty.
Implemented solution