Summary: | The # in accountlines descriptions makes them un-writeoffable | ||
---|---|---|---|
Product: | Koha | Reporter: | Nicole C. Engard <nengard> |
Component: | Patrons | Assignee: | Marc Véron <veron> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | major | ||
Priority: | P5 - low | CC: | aloi54, brendan, cgdavis, f.demians, gmcharlt, julian.maurice, kyle.m.hall, veron |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 11217: The # in accountlines descriptions makes them un-writeoffable
Bug 11217: The # in accountlines descriptions makes them un-writeoffable Bug 11217: The # in accountlines descriptions makes them un-writeoffable |
Description
Nicole C. Engard
2013-11-07 17:30:27 UTC
Dear Colleagues, This bug is also giving my institution trouble. Anyone else? Thanks, Christopher Davis Confirmed. Created attachment 56078 [details] [review] Bug 11217: The # in accountlines descriptions makes them un-writeoffable This patch adds escaping to url params in mambers/pay.pl To test: - Apply patch - Go to Home > Patrons > Manual invoice and create an invoice with a description containing a # - Go to tab "Pay fines" - Write off the fine Expected result: Fine is written off and does no longer display in pay tab. Agree with Nicole. the # prevents the passage of the fine in write of. There is a line in the table but this does not affect the amount. (In reply to aloij from comment #4) > Agree with Nicole. the # prevents the passage of the fine in write of. There > is a line in the table but this does not affect the amount. The patch fixes the problem - and is ready to sign off :-) Created attachment 56304 [details] [review] Bug 11217: The # in accountlines descriptions makes them un-writeoffable This patch adds escaping to url params in mambers/pay.pl To test: - Apply patch - Go to Home > Patrons > Manual invoice and create an invoice with a description containing a # - Go to tab "Pay fines" - Write off the fine Expected result: Fine is written off and does no longer display in pay tab. Signed-off-by: Claire Gravely <claire_gravely@hotmail.com> Created attachment 56307 [details] [review] Bug 11217: The # in accountlines descriptions makes them un-writeoffable This patch adds escaping to url params in mambers/pay.pl To test: - Apply patch - Go to Home > Patrons > Manual invoice and create an invoice with a description containing a # - Go to tab "Pay fines" - Write off the fine Expected result: Fine is written off and does no longer display in pay tab. Signed-off-by: Claire Gravely <claire_gravely@hotmail.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Pushed to Master - Should be in the November 16.11 Release. Thanks! Pushed in 16.05. Will be in 16.05.05. Pushed in 3.22.x, will be in 3.22.12 |