Bugzilla – Attachment 96524 Details for
Bug 24286
FindDuplicateAuthority does not escape forward slash in 'GENRE/FORM'
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 24286: Escape forward slash in GENRE/FORM (and any authtype)
Bug-24286-Escape-forward-slash-in-GENREFORM-and-an.patch (text/plain), 1.60 KB, created by
Nick Clemens
on 2019-12-20 13:38:28 UTC
(
hide
)
Description:
Bug 24286: Escape forward slash in GENRE/FORM (and any authtype)
Filename:
MIME Type:
Creator:
Nick Clemens
Created:
2019-12-20 13:38:28 UTC
Size:
1.60 KB
patch
obsolete
>From 17ed5865b4b7c58ef24edae0318afd78f8254c59 Mon Sep 17 00:00:00 2001 >From: Nick Clemens <nick@bywatersolutions.com> >Date: Fri, 20 Dec 2019 13:33:28 +0000 >Subject: [PATCH] Bug 24286: Escape forward slash in GENRE/FORM (and any > authtype) > >Note that bug 24267 will fix the issue during Z39 searching for new auth, but the duplicate check at saving will still fail > >To test: > 0 - Set SearchEngine to Elasticsearch, QueryRegexEscapeOptions to 'Don't Escape' > 1 - Browse to Authorities > 2 - Select New->New from Z3950 > 3 - Search for subject 'Short stories' > 4 - Auth search explodes > 5 - Apply patch > 6 - Repeat and confirm the search results return > 7 - Import and save the GENRE/FORM record for Short stories > 8 - Repeat the search and import the same record > 9 - Confirm that upon save you are notified of duplicate, don't save >10 - set QueryRegexEscapeOptions to 'Escape' and confirm Z39 searching works and importing/save notifies of duplicate >11 - set SearchEngine to Zebra and confirm Z39 searching works and importing/save notifies of duplicate >--- > C4/AuthoritiesMarc.pm | 1 + > 1 file changed, 1 insertion(+) > >diff --git a/C4/AuthoritiesMarc.pm b/C4/AuthoritiesMarc.pm >index c0c42df2bd..69320e864d 100644 >--- a/C4/AuthoritiesMarc.pm >+++ b/C4/AuthoritiesMarc.pm >@@ -766,6 +766,7 @@ sub FindDuplicateAuthority { > } else { > $op = 'AND'; > } >+ $authtypecode =~ s/\//\\\//; # GENRE/FORM contains forward slash which is a reserved character > my $query='at:'.$authtypecode.' '; > my $filtervalues=qr([\001-\040\Q!'"`#$%&*+,-./:;<=>?@(){[}_|~\E\]]); > if ($record->field($auth_tag_to_report)) { >-- >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 24286
:
96524
|
96765
|
97543
|
97582
|
97671