human-essentials icon indicating copy to clipboard operation
human-essentials copied to clipboard

Total on Hand is Unknown for inactive Items on Itemized Reports -- should be 0.

Open cielf opened this issue 10 months ago • 15 comments

Summary

In itemized reports, the total on hand for inactive items should be 0, rather than "Unknown"

Why

We only allow items to be deactivated if the total on hand is 0, so we do know.

Details

To see this, sign in as [email protected],

Reduce the quantities in inventory on Adult Briefs (Medium/Large) to 0 (I usually use a couple of distributions to do this.), then go in to Inventory | Items & Inventory and deactivate the item

click reports, then 'Itemized Donations'.

You'll see that the Total On Hand for Adult Briefs (Medium/Large) is Unknown. We know that it is 0. (screenshot is from a different example, but you get the idea)

Image

This is left from when we allowed deactivation of items without first 0ing the inventory.

The Total On Hand should be 0 for any inactive item.

Confirm and fix this behaviour on Donations - Itemized, Distributions - Itemized, and Purchases - Itemized.

Criteria for Completion

  • [ ] Total On Hand shows 0 for any inactive item
  • [ ] Tests to confirm

cielf avatar Feb 09 '25 16:02 cielf

Hi, I can take a shot at this one!

lydmcb avatar Mar 02 '25 23:03 lydmcb

Please do!

cielf avatar Mar 03 '25 17:03 cielf

This issue is marked as stale due to no activity within 30 days. If no further activity is detected within 7 days, it will be unassigned.

github-actions[bot] avatar Apr 03 '25 00:04 github-actions[bot]

Automatically unassigned after 7 days of inactivity.

github-actions[bot] avatar Apr 10 '25 00:04 github-actions[bot]

I would like to take this story.

pshong79 avatar Apr 16 '25 03:04 pshong79

Please do!

cielf avatar Apr 16 '25 11:04 cielf

This issue is marked as stale due to no activity within 30 days. If no further activity is detected within 7 days, it will be unassigned.

github-actions[bot] avatar May 17 '25 00:05 github-actions[bot]

Automatically unassigned after 7 days of inactivity.

github-actions[bot] avatar May 25 '25 00:05 github-actions[bot]

Back to Help Wanted. Hopefully 3rd times the charm!

cielf avatar May 25 '25 15:05 cielf

HI! I can do this?

GiovannyCordeiro avatar Jun 09 '25 12:06 GiovannyCordeiro

Sure!

cielf avatar Jun 09 '25 19:06 cielf

@cielf I couldn't find the "Purchases - Itemized" page, it was removed in some change or renamed?

GiovannyCordeiro avatar Jun 11 '25 14:06 GiovannyCordeiro

@GiovannyCordeiro My error -- Apparently we don't have one for the purchases, which is against our usual pattern.

cielf avatar Jun 11 '25 16:06 cielf

Ah, alright cool, it would be nice to have an issue to create it!

GiovannyCordeiro avatar Jun 11 '25 16:06 GiovannyCordeiro

We haven't had users agitating to get it, apparently.

cielf avatar Jun 12 '25 01:06 cielf

This issue is marked as stale due to no activity within 30 days. If no further activity is detected within 7 days, it will be unassigned.

github-actions[bot] avatar Jul 13 '25 00:07 github-actions[bot]

Automatically unassigned after 7 days of inactivity.

github-actions[bot] avatar Jul 20 '25 00:07 github-actions[bot]

If someone else wants to take this on (as @GiovannyCordeiro has had to go quiet for awhile), please follow on with PR #5235.

cielf avatar Jul 24 '25 23:07 cielf

@cielf I’d like to take on this issue.

hagiya0121 avatar Sep 05 '25 03:09 hagiya0121

Please do

cielf avatar Sep 05 '25 17:09 cielf

This issue is marked as stale due to no activity within 30 days. If no further activity is detected within 7 days, it will be unassigned.

github-actions[bot] avatar Oct 06 '25 00:10 github-actions[bot]

Automatically unassigned after 7 days of inactivity.

github-actions[bot] avatar Oct 13 '25 00:10 github-actions[bot]

I would like to take this story @awwaiid

DasTapan avatar Oct 13 '25 09:10 DasTapan

Hi @DasTapan, I’ve already created a PR for this issue, and it’s currently under review. 5004 Show 0 instead of “Unknown” for inactive items in itemized reports #5400

hagiya0121 avatar Oct 13 '25 22:10 hagiya0121

hi @hagiya0121 , I thought it was stale, my bad

DasTapan avatar Oct 14 '25 04:10 DasTapan