| Summary: | Circ always asking for confirmation if RentalFeesCheckoutConfirmation is used | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Katrin Fischer <katrin.fischer> |
| Component: | Circulation | Assignee: | Katrin Fischer <katrin.fischer> |
| Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
| Severity: | major | ||
| Priority: | P5 - low | CC: | chris, gmcharlt, kyle, martin.renvoize, mtj, paola.rossi, tomascohen |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | Trivial patch |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
| Attachments: |
Bug 13461 - Circ always asking for confirmation if RentalFeesCheckoutConfirmation is used
Bug 13461 - Circ always asking for confirmation if RentalFeesCheckoutConfirmation is used [PASSED QA] Bug 13461 - Circ always asking for confirmation if RentalFeesCheckoutConfirmation is used |
||
|
Description
Katrin Fischer
2014-12-14 21:25:49 UTC
Created attachment 34383 [details] [review] Bug 13461 - Circ always asking for confirmation if RentalFeesCheckoutConfirmation is used To test: - Check RentalFeesCheckoutConfirmation is activated - Try to check out an item without rental fine - Verify confirmation message without explanation is shown - Apply patch - Verify confirmation message is no longer shown - Configure itemtype to have rental fee - Veirfy now the confirmation message appears as it should Created attachment 34415 [details] [review] Bug 13461 - Circ always asking for confirmation if RentalFeesCheckoutConfirmation is used I've applied the patch against 3.19.00.000 Everything is OK as required. So I pass the patch to "Signed Off" status. Created attachment 34450 [details] [review] [PASSED QA] Bug 13461 - Circ always asking for confirmation if RentalFeesCheckoutConfirmation is used To test: - Check RentalFeesCheckoutConfirmation is activated - Try to check out an item without rental fine - Verify confirmation message without explanation is shown - Apply patch - Verify confirmation message is no longer shown - Configure itemtype to have rental fee - Veirfy now the confirmation message appears as it should Signed-off-by: Paola Rossi <paola.rossi@cineca.it> Works as expected, super trivial but super needed patch.. Passing QA Thanks Katrin :) Pätch pushed to master. Gracias Katrina! Pushed to 3.18.x, will be in 3.18.2 (In reply to Chris Cormack from comment #6) > Pushed to 3.18.x, will be in 3.18.2 skipping patch for 3.16.x |