[Bug 34768] New: Can't pay fines on OPAC if patron has a guarantee and they can see their fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34768 Bug ID: 34768 Summary: Can't pay fines on OPAC if patron has a guarantee and they can see their fines Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: major Priority: P5 - low Component: OPAC Assignee: oleonard@myacpl.org Reporter: nick@bywatersolutions.com QA Contact: testopia@bugs.koha-community.org This requires having an OPAC payment system setup, but it doesn't need to be functional for testing To recreate: 1 - Set system preference: AllowStaffToSetFinesVisibilityForGuarantor to allow 2 - Find a patron 3 - Add some fines to their account 4 - Install and enable an opac payment plugin (https://github.com/bywatersolutions/koha-plugin-pay-via-govolution for example) 5 - Sign in to OPAC as borrower 6 - Go to 'Charges' tab 7 - Select a fee 8 - Confirm 'Make payment' button enabled and 'amount to pay' is updated 9 - Add a guarantee to patron in staff interface 10 - Set 'Show charges to guarantors' to 'Yes' 11 - Add and pay a fine for the child 12 - View guarantor account on the OPAC 13 - Confirm you now see child's charges 14 - Confirm selecting your own fees does not enable the button or update amount to pay -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34768 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34768 --- Comment #1 from Nick Clemens <nick@bywatersolutions.com> --- Created attachment 155558 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155558&action=edit Bug 34768: Remove checkbox column from relatives fines on OPAC When a patron has a guarantee and can see their fines on the opac and an opac payment plugin is enabled there is an error on the 'Charges' tab. The table uses a 'COLSPAN' variable which is set to 5 or 6 depending on if OPAC payments are enabled. The guarantees table has one less column than the patron's however, so it is going out of range and causing a JS error. Additionally, we are adding a column to the child's table which is not used as their fines cannot be selected for payment at this time. This patch removes the checkbox column and sets the colspan directly to 4 I also fix an issue where the table is not being set as we need to reference the patron object in the guarantor info to obtain the id. To test: 1 - Set system preference: AllowStaffToSetFinesVisibilityForGuarantor to allow 2 - Find a patron 3 - Add some fines to their account 4 - Install and enable an opac payment plugin (https://github.com/bywatersolutions/koha-plugin-pay-via-govolution for example) 5 - Sign in to OPAC as borrower 6 - Go to 'Charges' tab 7 - Select a fee 8 - Confirm 'Make payment' button enabled and 'amount to pay' is updated 9 - Add a guarantee to patron in staff interface 10 - Set 'Show charges to guarantors' to 'Yes' 11 - Add and pay a fine for the child 12 - View guarantor account on the OPAC 13 - Confirm you now see child's charges 14 - Confirm selecting your own fees does not enable the button or update amount to pay 15 - Note JS error in console 16 - Apply patch 17 - Reload patrons page 18 - Confirm the table loads correctly (no JS error) 19 - Confirm selecting a fine updates the total and enables button 20 - Disable the payment plugin 21 - Confirm the display is still correct, no errors -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34768 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|oleonard@myacpl.org |nick@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34768 --- Comment #2 from Owen Leonard <oleonard@myacpl.org> --- (In reply to Nick Clemens from comment #1)
4 - Install and enable an opac payment plugin (https://github.com/bywatersolutions/koha-plugin-pay-via-govolution for example)
That's a 404 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34768 --- Comment #3 from Nick Clemens <nick@bywatersolutions.com> --- (In reply to Owen Leonard from comment #2)
(In reply to Nick Clemens from comment #1)
4 - Install and enable an opac payment plugin (https://github.com/bywatersolutions/koha-plugin-pay-via-govolution for example)
That's a 404
Try it now, I hadn't made the repo public -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34768 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Trivial patch Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34768 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #155558|0 |1 is obsolete| | --- Comment #4 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 155562 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155562&action=edit Bug 34768: Remove checkbox column from relatives fines on OPAC When a patron has a guarantee and can see their fines on the opac and an opac payment plugin is enabled there is an error on the 'Charges' tab. The table uses a 'COLSPAN' variable which is set to 5 or 6 depending on if OPAC payments are enabled. The guarantees table has one less column than the patron's however, so it is going out of range and causing a JS error. Additionally, we are adding a column to the child's table which is not used as their fines cannot be selected for payment at this time. This patch removes the checkbox column and sets the colspan directly to 4 I also fix an issue where the table is not being set as we need to reference the patron object in the guarantor info to obtain the id. To test: 1 - Set system preference: AllowStaffToSetFinesVisibilityForGuarantor to allow 2 - Find a patron 3 - Add some fines to their account 4 - Install and enable an opac payment plugin (https://github.com/bywatersolutions/koha-plugin-pay-via-govolution for example) 5 - Sign in to OPAC as borrower 6 - Go to 'Charges' tab 7 - Select a fee 8 - Confirm 'Make payment' button enabled and 'amount to pay' is updated 9 - Add a guarantee to patron in staff interface 10 - Set 'Show charges to guarantors' to 'Yes' 11 - Add and pay a fine for the child 12 - View guarantor account on the OPAC 13 - Confirm you now see child's charges 14 - Confirm selecting your own fees does not enable the button or update amount to pay 15 - Note JS error in console 16 - Apply patch 17 - Reload patrons page 18 - Confirm the table loads correctly (no JS error) 19 - Confirm selecting a fine updates the total and enables button 20 - Disable the payment plugin 21 - Confirm the display is still correct, no errors Signed-off-by: Owen Leonard <oleonard@myacpl.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34768 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34768 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #155562|0 |1 is obsolete| | --- Comment #5 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 155635 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155635&action=edit Bug 34768: Remove checkbox column from relatives fines on OPAC When a patron has a guarantee and can see their fines on the opac and an opac payment plugin is enabled there is an error on the 'Charges' tab. The table uses a 'COLSPAN' variable which is set to 5 or 6 depending on if OPAC payments are enabled. The guarantees table has one less column than the patron's however, so it is going out of range and causing a JS error. Additionally, we are adding a column to the child's table which is not used as their fines cannot be selected for payment at this time. This patch removes the checkbox column and sets the colspan directly to 4 I also fix an issue where the table is not being set as we need to reference the patron object in the guarantor info to obtain the id. To test: 1 - Set system preference: AllowStaffToSetFinesVisibilityForGuarantor to allow 2 - Find a patron 3 - Add some fines to their account 4 - Install and enable an opac payment plugin (https://github.com/bywatersolutions/koha-plugin-pay-via-govolution for example) 5 - Sign in to OPAC as borrower 6 - Go to 'Charges' tab 7 - Select a fee 8 - Confirm 'Make payment' button enabled and 'amount to pay' is updated 9 - Add a guarantee to patron in staff interface 10 - Set 'Show charges to guarantors' to 'Yes' 11 - Add and pay a fine for the child 12 - View guarantor account on the OPAC 13 - Confirm you now see child's charges 14 - Confirm selecting your own fees does not enable the button or update amount to pay 15 - Note JS error in console 16 - Apply patch 17 - Reload patrons page 18 - Confirm the table loads correctly (no JS error) 19 - Confirm selecting a fine updates the total and enables button 20 - Disable the payment plugin 21 - Confirm the display is still correct, no errors Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34768 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |m.de.rooy@rijksmuseum.nl |y.org | CC| |m.de.rooy@rijksmuseum.nl -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34768 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master Version(s)| |23.11.00 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34768 --- Comment #6 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Pushed to master for 23.11. Nice work everyone, thanks! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34768 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to master |Pushed to stable CC| |fridolin.somers@biblibre.co | |m Version(s)|23.11.00 |23.11.00,23.05.04 released in| | --- Comment #7 from Fridolin Somers <fridolin.somers@biblibre.com> --- Pushed to 23.05.x for 23.05.04 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34768 Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to stable |Pushed to oldstable Version(s)|23.11.00,23.05.04 |23.11.00,23.05.04,22.11.11 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34768 --- Comment #8 from Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> --- Nice work everyone! Pushed to oldstable for 22.11.x -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org