-
Type: Sprint Defect
-
Status: Done (View Workflow)
-
Priority: Major
-
Resolution: Completed
-
Affects Version/s: CFA 2018R3
-
Fix Version/s: CFA 2018R3
-
Component/s: PaidInOut
-
Labels:
-
Sprint:CFAMX 2018R4 Sprint 1, CFAMX 2018R4 Sprint 2, CFAMX 2018R4 Sprint 3, CFAMX 2018R4 Sprint 4
Reproduction Steps:
- Create some records, save them,
- Run the Cash Day Summary Export
- Look at the Count in the Paid In/Out sections
- Edit the amount of one of the Paid In/Out Events
- Run the Cash Day Summary Export
- Look at the Count in the Paid In/Out sections
Expected Results:
- Edited records should not be counted twice in CDS export.
Actual Results:
Edited paid in/out records are increasing the "count" in CDS
Document walking through some scenarios, good and bad! https://docs.google.com/document/d/162p560oM88xwk-wI4ovsCPgihXfxwIJwE1eRQ0EnNms/edit?usp=sharing
For that Paid In\Out defect for Amount change, the place to fix is going to be in \Mobile\Areas\Financial\Mx.Financial.Services\CommandServices\TransactionPettyCashCommandService.cs around line 218 where I handle when just the amount changes. At a more detailed level, I think the fix is to make add "quantity" as an optional parameter to "AdjustCashier" defaulted to 1 so that this one place we call it can override that to 0. If doing that and using a 0 quantity here causes some other error, I guess we would have to reverse the original amount and then adjust by the new amount.
- Clones
-
CFAMX-118 TESTING - INF-10526 - Editing Amount of Paid In/Out records from new MxC Paid In/Out page does not remove them from Cash Day Summary count
- Done