Skip to content

OBPIH-7525 create 0 to 0 baseline transaction for products that were …#5545

Merged
awalkowiak merged 1 commit intodevelopfrom
bug/OBPIH-7525-cc-0-to-0-baseline
Oct 13, 2025
Merged

OBPIH-7525 create 0 to 0 baseline transaction for products that were …#5545
awalkowiak merged 1 commit intodevelopfrom
bug/OBPIH-7525-cc-0-to-0-baseline

Conversation

@ewaterman
Copy link
Member

…zeroed mid cycle count

✨ Description of Change

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

Description: Creates a 0 quantity baseline transaction in the default bin and lot for products who had their quantity set to zero while in the middle of a cycle count.

@ewaterman ewaterman self-assigned this Oct 10, 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 Oct 10, 2025
// longer holds an association to that product (cycle count item holds the relationship to product). We still
// want to create a baseline for those products (to record that a count happened), hence the need to loop on
// the original list of counted products.
for (Product product : countedProducts) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At what point don't cycleCount.products contain the "zeroed out" products? I'm asking because looking at the code and the changes, the countedProducts should be exactly the cycleCount.products.

I will leave the above if someone was also wondering.
I've answered myself - it's updateCycleCountItems method in the CycleCountProductAvailabilityService

@awalkowiak awalkowiak merged commit 28f3dab into develop Oct 13, 2025
3 of 5 checks passed
@awalkowiak awalkowiak deleted the bug/OBPIH-7525-cc-0-to-0-baseline branch October 13, 2025 09:55
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.

3 participants