Summary: | Odd error caused by URI Escape | ||
---|---|---|---|
Product: | Koha | Reporter: | Kyle M Hall (khall) <kyle> |
Component: | Patrons | Assignee: | Kyle M Hall (khall) <kyle> |
Status: | CLOSED INVALID | QA Contact: | Testopia <testopia> |
Severity: | major | ||
Priority: | P5 - low | CC: | gmcharlt, kyle.m.hall, mtompset |
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 12552 - Odd error caused by URI Escape |
Description
Kyle M Hall (khall)
2014-07-10 18:23:38 UTC
I cannot reproduce the issue, I suspect the issue may be URI::Escape version 3.30. I'm running version 3.31 and do not see the issue. I don't know the source of the error ( and thus cannot reproduce it on demand ) but it doesn't appear to be the forward slash after all. Created attachment 40245 [details] [review] Bug 12552 - Odd error caused by URI Escape uri_escape: Unmatched [ in regex; marked by <-- HERE in m/([ <-- HERE ])/ at (eval 133) line 1. The issue is the use of uri_escape at line 174 of members/pay.pl Test Plan: 1) Apply this patch 2) Attempt to pay a fine with the "Pay" button 3) No regressions should be noted Upping importance because this bug can prevent payment of fines. I should also note I've seen this bug multiple times but cannot reproduce on a test system. However, the patch does indeed fix the issue on production systems I've applied it to. Problem. I entered $_ as my note, and once saved, the note portion displayed as \$_. Could this be related to bug 8686 - version of URI::Escape? (In reply to Katrin Fischer from comment #7) > Could this be related to bug 8686 - version of URI::Escape? I was chatting with Kyle on IRC about this yesterday. I don't think it is. http://irc.koha-community.org/koha/2015-06-17#i_1689562 17:53 - 19:52 I believe the issue is no longer related to URI::Escape. |