[Koha-bugs] [Bug 7417] Include alternate forms from authorities in bibliographic searches

[bugzilla-daemon at bugs.koha-community.org]

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.