| Summary: | Voided payments still appear as bankable in the cash register | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Benjamin Daeuber <bdaeuber> |
| Component: | Point of Sale | Assignee: | Laura Escamilla <Laura.escamilla> |
| Status: | Failed QA --- | QA Contact: | Testopia <testopia> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | clackman, david, Laura.escamilla, martin.renvoize |
| Version: | 25.05 | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | --- |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
| Attachments: |
Bug 40871: filter out VOID and VOID adjustments in outstanding_accountlines
Bug 40871: Exclude void payments from cashup/bankable transactions |
||
|
Description
Benjamin Daeuber
2025-09-24 14:16:29 UTC
Created attachment 192331 [details] [review] Bug 40871: filter out VOID and VOID adjustments in outstanding_accountlines Created attachment 192332 [details] [review] Bug 40871: Exclude void payments from cashup/bankable transactions To test: 1. Make sure registers are turned on and you know the current bankable amount. 2. Make a cash payment on a patron account. 3. Observe the bankable amount on that register has increased (as expected). 4. Void this payment on the patron account. 5. Note that the bankable amount has not decreased. 6. Apply the patch. 7. Repeat steps 2-4. Notice that this time the bankable amount remains the same / it does not show the amount that was voided. I attempted to test, but after applying the patches I get a 500: Internal Server Error when accessing the transaction history for the register. Developer tools says: Got 500 (Internal Server Error) DataTables warning: table id=table_cashups - Ajax error. Lots of these warnings in /var/log/koha/kohadev/plack-intranet-error.log: [WARN] Use of uninitialized value $CGI::Compile::ROOT::kohadevbox_koha_members_pay_2epl:: change_given in concatenation (.) or string at /kohadevbox/koha/members/pay.pl line 194. Note: I'm not sure what the result expected here is, for example, is this what the result should be after the patches? 1. Two $10 manual transactions added to a patron's account. 2. Two payments made. 3. Cash register bankable total = $20 (More > Tools > Cash management > Cash summary for Centerville) 4. One of the payments voided. 5. Cash register bankable total now = $10 Testing notes (using KTD): 1. Set UseCashRegisters to "Use". 2. Set up a cash register (Administration > Accounting > Cash registers > New cash register). 3. Add two $10 manual transactions to a patron's account. 4. Pay the two amounts. 5. Void one of the payments. 6. Cash register bankable total = $10 (More > Tools > Cash management > Cash summary for Centerville). |