| 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, veron |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | 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 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 |