Bug 8680 - Z39.50 searches for ISBN/ISSN problematic in the Acquistions Module
Summary: Z39.50 searches for ISBN/ISSN problematic in the Acquistions Module
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Acquisitions (show other bugs)
Version: 3.8
Hardware: All All
: P5 - low normal (vote)
Assignee: Bugs List
QA Contact: Mason James
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-08-23 18:36 UTC by Joseph Alway
Modified: 2014-05-26 21:04 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Z39.50 ISBN Search is problematic and there is a missing ISSN field. (3.79 KB, patch)
2012-09-20 18:34 UTC, Joseph Alway
Details | Diff | Splinter Review
[SIGNED-OFF] Bug 8680 Z39.50 ISBN Search is problematic and there is a missing ISSN field. (3.91 KB, patch)
2012-09-29 17:12 UTC, Marc Véron
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Joseph Alway 2012-08-23 18:36:13 UTC
"Copy cataloging via Z39.50 is often done by searching on ISBN or ISSN. Unfortunately, the code for doing this is... problematic, and may cause problems with some Z39.50 servers." (Description from Bug 8550).

This Bug is the Exact Same bug as 8550. Except it is in the Z39.50 Search Script located in the Acquisitions Module. /usr/share/koha/intranet/cgi-bin/acqui/z3950_search.pl

The z3950_search.pl script for the Acquisitions Module has a few extra lines of code to handle Ordering of the item.
Comment 1 Joseph Alway 2012-09-20 18:34:34 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2012-09-29 17:12:48 UTC
Created attachment 12609 [details] [review]
[SIGNED-OFF] Bug 8680 Z39.50 ISBN Search is problematic and there is a missing ISSN field.

The changes here are duplicate fixes.
The cataloguing directory had the same bugs as the acqui directory.
The bugs in the cataloguing files of the same name were fixed in Bug 8550.
The bugs in the acqui files had not been fixed.
This will fix Bug 8680 which is in the Acquisitions Module.

Changes to the z3950_search.pl file:
Rather than treating ISBN and ISSN as the same, this patch breaks
them out into separate parts of the query, as well as changing
the truncation attribute to enable right truncation (the correct
behavior for the ISBN field in MARC21, and valid behavior in
UNIMARC).

Changes to the z3950_search.tt file:
Sometime in 2009 or so, the ISSN field was removed from the Z39.50 search
page, and the ISBN field replaced with a combined ISBN/ISSN field. Since
this breaks ISBN search for most Z39.50 targets (due to the
differing treatment of ISBN vs. ISSN), this is a bug not a feature.

Signed-off-by: Marc Veron <veron@veron.ch>

Works as expected.
Comment 3 Mason James 2012-10-04 01:53:44 UTC
(In reply to comment #2)
> Created attachment 12609 [details] [review]
> [SIGNED-OFF] Bug 8680 Z39.50 ISBN Search is problematic and there is a
> missing ISSN field.
> > Signed-off-by: Marc Veron <veron@veron.ch>
> 
> Works as expected.

tested works as expected, passing QA...

$ koha-qa.pl -c 1

testing 1 commit(s) (applied to commit 6c18204)
 * 76fc1dc Bug 8680 Z39.50 ISBN Search is problematic and there is a missing ISSN field.
      acqui/z3950_search.pl
      koha-tmpl/intranet-tmpl/prog/en/modules/acqui/z3950_search.tt

* acqui/z3950_search.pl                                                    OK
* koha-tmpl/intranet-tmpl/prog/en/modules/acqui/z3950_search.tt            OK
Comment 4 Paul Poulain 2012-10-08 15:57:49 UTC
Patch pushed to master
Comment 5 Chris Cormack 2012-10-09 21:07:04 UTC
Pushed to 3.8.x, will be in 3.8.6