[Koha-bugs] [Bug 7417] Include alternate forms from authorities in bibliographic searches
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Thu Aug 9 06:46:33 CEST 2012
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7417
Katrin Fischer <katrin.fischer at bsz-bw.de> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #11460|0 |1
is obsolete| |
--- Comment #37 from Katrin Fischer <katrin.fischer at bsz-bw.de> ---
Created attachment 11465
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11465&action=edit
[SIGNED-OFF] Bug 7417 follow-up: incorporate QA comments
This patch incorporates the following QA comments:
* Renames Koha::Authority to Koha::DataObject::Authority
* Uses Modern::Perl instead of 'use strict; use warnings;'
* perltidied all new files again
* wrote an intelligent constructor for Koha::DataObject::Authority which
takes a hash with the construction criteria
* added rudimentary error handling in C4::AuthoritiesMarc::GetAuthority
* added missing t/AuthoritiesMarc.t
This patch also improves test coverage, and eliminates the need for a
database to test the EmbedSeeFromHeadings RecordProcessor plugin.
In order to make the Koha codebase more consistent, this patch switches
the RecordProcessor and Koha::DataObject::Authority code to use the
Moose-like syntax supported by Class::Accessor.
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Follow up passes test plan.
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list