Bug 5819

Summary: No toolbar in record view when quotes present in title
Product: Koha Reporter: Janusz Kaczmarek <januszop>
Component: CatalogingAssignee: Janusz Kaczmarek <januszop>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: normal    
Priority: PATCH-Sent (DO NOT USE) CC: chris, julian.maurice, katrin.fischer, koha.sekjal
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: A screenshot illustrating the problem
Proposed patch
Another patch

Description Janusz Kaczmarek 2011-03-01 22:20:47 UTC
Created attachment 3235 [details]
A screenshot illustrating the problem

If there are double quotes in title (but also if there were any in lccn, isbn, issn, author, dewey, subject) the nice toolbar with buttons (cattoolbar) in normal view and MARC view (in staff client) will not appear.  Instead there is a text list of options (cf. screenshot).

The reason is that the JavaScript function GetZ3950Terms called when constructing the buttons fails, which is because of additional quotes in strQuery.  The reason for this is that the C4::Search::z3950_search_args function does not escape quotes.

A simple workaround will be submitted shortly.

Regards,

Janusz
Comment 1 Janusz Kaczmarek 2011-03-01 22:26:45 UTC Comment hidden (obsolete)
Comment 2 Janusz Kaczmarek 2011-03-01 23:00:47 UTC
Created attachment 3237 [details] [review]
Another patch

Another possible fix inspired by Paul Poulain.

J.
Comment 3 Katrin Fischer 2011-03-14 06:22:21 UTC
Patch work for me. Signed off.
Comment 4 Chris Cormack 2011-03-14 08:02:27 UTC
Pushed please test and mark resolved