[koha-commits] main Koha release repository branch master updated. v3.14.00-507-gdaf2ebc

Git repo owner gitmaster at git.koha-community.org
Fri Feb 28 20:35:35 CET 2014


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "main Koha release repository".

The branch, master has been updated
       via  daf2ebc4f5aeaf3a73e11915a17a2a81eafc5cb9 (commit)
       via  5159a43d4341b03bd1b30926e33bf9c686e9fb46 (commit)
       via  dc86557c34284b1a14a3aa13c4474ba5302d912c (commit)
      from  4ce0848fe35814f2324a8e3d59b3c04f061a858b (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit daf2ebc4f5aeaf3a73e11915a17a2a81eafc5cb9
Author: Tomas Cohen Arazi <tomascohen at gmail.com>
Date:   Mon Oct 21 16:17:57 2013 -0700

    Bug 11096: support the retrieval of large MARCXML records
    
    This patch makes Koha <-> Zebra use MARCXML for the serialization when
    using DOM, and USMARC for GRS-1.
    
    * The following functions are modified to set the Zebra record syntax
    according to the current sysprefs and configuration:
    
    - C4::Context->Zconn
    - C4::Context-_new_Zconn
    
    * A new function 'new_record_from_zebra' is introduced, which checks the
    context we are in, and creates the MARC::Record object using the right
    constructor.
    
    The following packages get touched to make use of the new function:
    - C4::Search
    - C4::AuthoritiesMarc
    
    and the same happens to the UI scripts that make use of them (both in
    the OPAC and STAFF interfaces).
    
    * Calls to the unsafe ZOOM::Record->render()[1] method are removed.
    
    Due to this last change the code for building facets was rewritten. And
    for performance on the facets creation I pushed higher version
    dependencies for MARC::File::XML and MARC::Record (we rely on
    MARC::Field->as_string).
    
    * Calls to MARC::Record->new_from_xml and MARC::Record->new_from_usmarc
    are wrapped with eval for catching problems [2].
    
    * As of bug 3087, UNIMARC uses the 'unimarc' record syntax. this case is
      correctly handled.
    * As of bug 7818 misc/migration_tools/rebuild_zebra.pl behaves like:
    
    - bib_index_mode (defaults to 'grs1' if not specified)
    - auth_index_mode (defaults to 'dom')
    
    here we do exactly the same.
    
    To test:
     - prove t/db_dependent/Search.t should pass.
     - Searching should remain functional.
     - Indexing and searching for a big record should work (that's what the
       unit tests do).
     - Test an index scan search (on the staff interface):
        Search > More options > Check "Scan indexes".
     - Enable 'itemBarcodeFallbackSearch' and try to circulate any word, it
       shouldn't break.
     - Searching for a biblio in a new subscription shouldn't break.
     - Running bulkmarcimport.pl shouldn't break.
     - And so on... for the rest of the .pl files.
    
    [1] http://search.cpan.org/~mirk/Net-Z3950-ZOOM/lib/ZOOM.pod#render()
    [2] a record that cannot be parsed by MARC::Record is simply skipped (bug 10684)
    
    Sponsored-by: Universidad Nacional de Cordoba
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit 5159a43d4341b03bd1b30926e33bf9c686e9fb46
Author: Tomas Cohen Arazi <tomascohen at gmail.com>
Date:   Wed Jan 8 14:15:28 2014 -0300

    Bug 11096: Wrap each test set configuration in Search.t into a subtest
    
    This makes it easier to read. Also fixes the test count, and is rebased
    against 11499 which is needed to correctly test the patches for this bug
    (i.e. indexing mode needs to be correctly set in koha-conf.xml)
    
    To test:
    
    [1] Run prove -v t/db_dependent/Search.t
    
    Sponsored-by: Universidad Nacional de Cordoba
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit dc86557c34284b1a14a3aa13c4474ba5302d912c
Author: Galen Charlton <gmc at esilibrary.com>
Date:   Mon Dec 9 23:39:07 2013 +0000

    Bug 11096: add regression tests for indexing/retrieving large MARC records
    
    This adds direct tests of indexing and retrieving
    bib records that are larger than the ISO2709 limit
    of 99,999 octets.
    
    To test: verify that prove -v t/db_dependent/Search.t passes.
    
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

-----------------------------------------------------------------------

Summary of changes:
 C4/AuthoritiesMarc.pm                              |   40 +-
 C4/Context.pm                                      |   50 +-
 C4/Installer/PerlDependencies.pm                   |    4 +-
 C4/Matcher.pm                                      |    2 +-
 C4/Search.pm                                       |  121 +-
 C4/XISBN.pm                                        |    2 +-
 acqui/neworderbiblio.pl                            |    2 +-
 catalogue/search.pl                                |    5 +-
 .../value_builder/marc21_linking_section.pl        |    2 +-
 cataloguing/value_builder/unimarc_field_4XX.pl     |    2 +-
 circ/circulation.pl                                |    2 +-
 labels/label-item-search.pl                        |    2 +-
 misc/migration_tools/bulkmarcimport.pl             |    2 +-
 serials/subscription-bib-search.pl                 |    2 +-
 t/db_dependent/Search.t                            |  100 +-
 .../large_biblio_dom/exported_records.xml          | 1667 ++++++++++++++++++++
 .../large_biblio_grs1/exported_records.xml         | 1665 +++++++++++++++++++
 17 files changed, 3572 insertions(+), 98 deletions(-)
 create mode 100644 t/db_dependent/data/marc21/zebraexport/large_biblio_dom/exported_records.xml
 create mode 100644 t/db_dependent/data/marc21/zebraexport/large_biblio_grs1/exported_records.xml


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list