[Koha-bugs] [Bug 10998] New: code duplication introduced by Z39.50 authority search
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Fri Oct 4 17:32:41 CEST 2013
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10998
Bug ID: 10998
Summary: code duplication introduced by Z39.50 authority search
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: normal
Priority: P5 - low
Component: Architecture, internals, and plumbing
Assignee: gmcharlt at gmail.com
Reporter: gmcharlt at gmail.com
QA Contact: testopia at bugs.koha-community.org
Bug 10096, which introduces a feature to import authority records via Z39.50,
also introduces some code duplication.
To summarize the differences:
- z3950_search.pl and z3950_auth_search.pl differ only in the input search
attributes they accept
- C4::Breeding::ImportBreeding() and C4::Breeding::ImportAuth() do mostly the
same thing
- the actual search logic in C4::Breeding::Z3950SearchAuth() and
C4::Breeding::Z3950Search() is the same; the two routines differ only in the
specific Z39.50 attributes they handle and in some details of post-retrieval
record processing. Also, Z3950SearchAuth() does not reflect the work that was
done for bug 10462.
Consequently, there is plenty of opportunity for code consolidation, as well as
adding unit tests.
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list