Skip to content

OBPIH- 6901 bugfix. Make QuantityVariance properly handle 0 quantity#5128

Merged
awalkowiak merged 1 commit intodevelopfrom
bug/OBPIH-6901-fix-null-discrepancy
Mar 14, 2025
Merged

OBPIH- 6901 bugfix. Make QuantityVariance properly handle 0 quantity#5128
awalkowiak merged 1 commit intodevelopfrom
bug/OBPIH-6901-fix-null-discrepancy

Conversation

@ewaterman
Copy link
Member

@ewaterman ewaterman commented Mar 13, 2025

✨ Description of Change

A concise summary of what is being changed. Please provide enough context for reviewers to be able to understand the change and why it is necessary. If the issue/ticket already provides enough information, you can put "See ticket" as the description.

Link to GitHub issue or Jira ticket: https://pihemr.atlassian.net/browse/OBPIH-6901

Description: Fix CycleCountItem.getQuantityVariance() so that it doesn't return null when either quantityCounted or quantityOnHand == 0


📷 Screenshots & Recordings (optional)

https://jam.dev/c/ed162de1-0420-4c82-839d-fcc395b7f185

8581a846-4b1e-49e0-90d6-37d28346d590.webm

@ewaterman ewaterman self-assigned this Mar 13, 2025
@github-actions github-actions bot added type: bug Addresses unintended behaviours of the app domain: backend Changes or discussions relating to the backend server labels Mar 13, 2025
@ewaterman ewaterman changed the title OBPIH- 6901 FIX. Make QuantityVariance properly handle 0 quantity OBPIH- 6901 bugfix. Make QuantityVariance properly handle 0 quantity Mar 13, 2025
@codecov
Copy link

codecov bot commented Mar 13, 2025

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 8.12%. Comparing base (a9ae2be) to head (864fac6).
Report is 169 commits behind head on develop.

Files with missing lines Patch % Lines
.../org/pih/warehouse/inventory/CycleCountItem.groovy 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             develop   #5128      +/-   ##
============================================
- Coverage       8.15%   8.12%   -0.04%     
+ Complexity       945     942       -3     
============================================
  Files            636     636              
  Lines          43123   43124       +1     
  Branches       10486   10486              
============================================
- Hits            3518    3505      -13     
- Misses         39058   39076      +18     
+ Partials         547     543       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ewaterman ewaterman requested review from alannadolny, awalkowiak and kchelstowski and removed request for awalkowiak March 13, 2025 23:19
@awalkowiak awalkowiak merged commit 35278b9 into develop Mar 14, 2025
9 checks passed
@awalkowiak awalkowiak deleted the bug/OBPIH-6901-fix-null-discrepancy branch March 14, 2025 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: backend Changes or discussions relating to the backend server type: bug Addresses unintended behaviours of the app

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants