Attachment 88805 Details for Bug 22715 – Bug 22715: Searching for patrons with "" in the circulation note hangs patron search

[patch] Bug 22715: Searching for patrons with "" in the circulation note hangs patron search

Bug-22715-Searching-for-patrons-with--in-the-circu.patch (text/plain), 2.14 KB, created by Katrin Fischer on 2019-04-25 18:47:55 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2019-04-25 18:47:55 UTC

Size: 2.14 KB

patch

obsolete

>From b0f027645acce77c963f0d0a7cede4f7e00d6376 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Tue, 16 Apr 2019 10:34:56 -0400
>Subject: [PATCH] Bug 22715: Searching for patrons with "" in the circulation
> note hangs patron search
>
>To replicate:
>Edit a patron to have a circ note, include quoted text in it, such as "this is quoted string"
>Go to the patron search, search for a set of results that would include the patron with the quotes in the note
>Enjoy looking at "Processing" for forever.
>
>Test Plan:
>1) Apply this patch
>2) Attempt to replicate using the steps above
>
>Signed-off-by: Liz Rea <wizzyrea@gmail.com>
>I took this a step further and used <script>"something in
>quotes"</script> for my test note and this worked just fine. YAY.
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> .../intranet-tmpl/prog/en/modules/members/tables/members_results.tt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/tables/members_results.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/tables/members_results.tt
>index 607146e269..15b7c677ed 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/tables/members_results.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/tables/members_results.tt
>@@ -31,7 +31,7 @@
> "dt_fines":
> "[% IF data.fines < 0 %][% data.fines | $Price %] [% ELSIF data.fines > 0 %] [% data.fines | $Price %] [% ELSE %] [% data.fines | $Price %] [% END %]",
> "dt_borrowernotes":
>- "[% data.borrowernotes.replace('\\\\' , '\\\\') |html_line_break |collapse %]",
>+ "[% data.borrowernotes | html_line_break | collapse | html | $To %]",
> "dt_action":
> "<a href='/cgi-bin/koha/members/memberentry.pl?op=modify&amp;destination=circ&amp;borrowernumber=[% data.borrowernumber | html %]' class='btn btn-default btn-xs'> Edit</a>",
> "borrowernumber":
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 22715: 88074 | 88076 | 88077 | 88116 | 88749 | 88786 | 88804 | 88805 | 88806