Bugzilla – Attachment 104925 Details for
Bug 25308
When cataloguing search fields are prefilled from record, content after & is cut off
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 25308: Escape querystring for Z39.50 popup from cataloging
Bug-25308-Escape-querystring-for-Z3950-popup-from-.patch (text/plain), 2.39 KB, created by
Marcel de Rooy
on 2020-05-15 09:45:34 UTC
(
hide
)
Description:
Bug 25308: Escape querystring for Z39.50 popup from cataloging
Filename:
MIME Type:
Creator:
Marcel de Rooy
Created:
2020-05-15 09:45:34 UTC
Size:
2.39 KB
patch
obsolete
>From 3e0f9d99642cd6c37bd0289404a6446fc9f0d2f9 Mon Sep 17 00:00:00 2001 >From: Phil Ringnalda <phil@chetcolibrary.org> >Date: Tue, 28 Apr 2020 17:16:51 -0700 >Subject: [PATCH] Bug 25308: Escape querystring for Z39.50 popup from > cataloging >Content-Type: text/plain; charset=utf-8 > >Test plan: > >1. For bonus fun, Administration -> MARC Frameworks -> New framework, >with the code A&B and description Ants & Bees >2. In the A&B Actions menu, choose MARC structure since Import won't >work >3. Accept the offer to copy in structure from default >4. Cataloging -> New record -> Ants & Bees >5. Type This & That in the 245$a (and for extra credit, in 020$a, and >create an author authority with an & in it so you can put it in the >100/110) >6. Click Z39.50/SRU search, make sure everything you typed is prefilled >including the parts after & >7. Search for anything that will retrieve a record (the ISBN This & >That probably won't), choose Import >8. Check that the record came back, and verify in the Settings menu that >it's still in the Ants & Bees framework. > >Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> > >Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> >--- > koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt >index bd7cc15165..66d253e363 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt >@@ -442,7 +442,7 @@ > */ > function GetZ3950Terms(){ > var frameworkcode = document.getElementById("frameworkcode").value; >- var strQuery = "&frameworkcode=" + frameworkcode; >+ var strQuery = "&frameworkcode=" + encodeURIComponent(frameworkcode); > var mandatories = new Array(); > var mandatories_label = new Array(); > [% FOREACH BIG_LOO IN BIG_LOOP %] >@@ -459,7 +459,7 @@ > for(var i=0,len=mandatories.length; i<len ; i++){ > var field_value = document.getElementById(mandatories[i]).value; > if( field_value ){ >- strQuery += "&"+mandatories_label[i]+"="+field_value; >+ strQuery += "&"+encodeURIComponent(mandatories_label[i])+"="+encodeURIComponent(field_value); > } > } > return strQuery; >-- >2.11.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 25308
:
103892
|
104240
| 104925