Stock on hand calculation error

Description

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

Environment

None

Attachments

5

Activity

Show:

Szymon Rujner June 22, 2022 at 12:13 PM
Edited

  • Checked on Firefox and Chrome via UAT instance;

  • Issue is no longer reproducing;

  • The changes are shown correctly;

  • 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.

Details

Assignee

Reporter

Original estimate

Time tracking

1d 5h logged1d 3h remaining

Components

Fix versions

Affects versions

Priority

Time Assistant

Created May 9, 2022 at 6:57 AM
Updated June 22, 2022 at 12:14 PM
Resolved June 22, 2022 at 12:14 PM