[koha-commits] main Koha release repository branch 3.12.x updated. v3.12.00-beta3-19-g978d7a9

Git repo owner gitmaster at git.koha-community.org
Wed May 8 16:01:21 CEST 2013


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, 3.12.x has been updated
       via  978d7a97323447ab4802a42cebb838cfef1a2845 (commit)
       via  15aa1afa7559c61be7b6a9006e2d0464135ed5e4 (commit)
       via  059cc4cd3ac5e40326fb8653b581a35dca585ed8 (commit)
       via  254173bc21043b92cecd3037962da832175bcdcc (commit)
       via  36e06a151c718d7d905e4cb0b8d818c0b12f3b4d (commit)
       via  640ab59a304fc67bc241ded052af618af3923044 (commit)
       via  a1f5f41d526c27156523eee264254ef86a139fed (commit)
       via  e63175d6b9874f1cd9c5d24238595db5be53c2ef (commit)
       via  135347db66406f5cb989de392a0851d76fc6d266 (commit)
       via  fda21fbc28867da1043c2ef7aed1213e7cd4defd (commit)
       via  0742457840edf7784d5bccafaa6b6383125a53ed (commit)
      from  cb9af9d0e667dcdebfdc89e77bdb8fdc2598493d (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 978d7a97323447ab4802a42cebb838cfef1a2845
Author: Jared Camins-Esakov <jcamins at cpbibliography.com>
Date:   Wed May 8 10:03:26 2013 -0400

    Bug 9681: Increment version number
    
    Database update for bug 9681 has been given version 3.11.00.301
    (cherry picked from commit 5a9a82dcc3ebdb3b7f130edbfb8c6ef67f97eb68)
    
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

commit 15aa1afa7559c61be7b6a9006e2d0464135ed5e4
Author: Paul Poulain <paul.poulain at biblibre.com>
Date:   Mon May 6 15:06:47 2013 +0200

    Bug 9681: Add some indexes in database
    
    Some table columns should have an index to speed SQL queries :
    - statistics table has only one index (used to make heavy queries for reports)
    - itemtype should have an index in items and biblioitems (this is used in circulation to check the number of existing issues of a specific item type)
    - issue.branchcode and issue.issuingbranch
    
    Test plan :
    - Create a new database using kohastructure.pl
    => check there is no SQL error and that new indexes are present (mysql> show create table)
    Apply update database on a master version
    => check there is no SQL error and that new indexes are present (mysql> show create table)
    
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    
    Comment: Indexes created as described. No errors.
    Fixing small merge conflict.
    Test presence of new indexes on all affected tables.
    
    Signed-off-by: Paul Poulain <paul.poulain at biblibre.com>
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    (cherry picked from commit 68ff49bb7d982737c41a225cd697b8fb60d0b4ab)
    
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

commit 059cc4cd3ac5e40326fb8653b581a35dca585ed8
Author: Julian Maurice <julian.maurice at biblibre.com>
Date:   Mon May 6 06:43:50 2013 +0000

    Bug 10185: Add missing columns in update 3.09.00.025
    
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    SQL only changes fixing an important database update.
    
    IMPORTANT: Patches must be backported to all versions containing
    the database update 3.09.00.025.
    
    I used the test plan provided by Julian with some additions:
    1/ Do a fresh 3.8 install
    2/ Do some reserves
    3/ Check your old_reserves table is empty
    4/ Update your sources to 3.10
    5/ Run updatedatabase.pl
    6/ Your reserves table is now empty!
    
    I made sure I had item level and title level holds, also
    suspended holds.
    
    I repeated the test with entries in my old_reserves table
    (cancelled and filled holds) to make sure this is also still
    working correctly.
    
    Before the patch we lost all suspended holds and when the
    old_reserves table was empty also all holds in general.
    After the patch the reserve_id is added but the other data
    in both tables is not altered.
    
    NOTE: Adding the reserve_id first to old_reserves and then
    to reserves we are missing one id in between.
    old_reserves: 1,2,3
    reserves: 5,6
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    (cherry picked from commit 30d067400e6d15cd86d8b35daed4bb10438d0e2e)
    
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

commit 254173bc21043b92cecd3037962da832175bcdcc
Author: Julian Maurice <julian.maurice at biblibre.com>
Date:   Fri May 3 17:20:17 2013 +0200

    Bug 10185: Fix update 3.09.00.025
    
    It can remove all reserves from reserves table when there is no entries
    in old_reserves.
    This is due to @ai which is set to NULL in
      SET @ai = ( SELECT MAX( reserve_id ) FROM tmp_reserves )
    and reserve_id > NULL returns no results in
      INSERT INTO reserves SELECT * FROM tmp_reserves WHERE reserve_id > @ai
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Test plan and comments on second patch.
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    (cherry picked from commit 37c52367b08f30514483b02a387cb5a463ea15d0)
    
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

commit 36e06a151c718d7d905e4cb0b8d818c0b12f3b4d
Author: David Cook <dcook at prosentient.com.au>
Date:   Wed May 8 18:00:29 2013 +1000

    Bug 10156 - Fix usr/bin/perl path for get_report_social_data.pl
    
    Currently, the perl path for get_report_social_data.pl is "#!/bin/perl"
    when it should be "#!/usr/bin/perl".
    
    While I'm not entirely sure how to test this, rangi did mention:
    
    23:47 (it wouldnt work on my debian install)
    23:48  zsh: no such file or directory: /bin/perl
    
    It's a pretty trivial change, so I'm not sure it needs much of a test
    plan.
    
    Signed-off-by: Magnus Enger <magnus at enger.priv.no>
    Before the patch "grep -r '#!/bin/perl' in the misc dir shows one file
    with this line, and it is indeed get_report_social_data.pl. After the
    patch there are no occurrences left.
    Passed-QA-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    (cherry picked from commit 2e103a018cedc1c57d1c0db8acad6f93ab4ca428)
    
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

commit 640ab59a304fc67bc241ded052af618af3923044
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Mon Apr 22 16:19:01 2013 +0200

    Bug 8278: build the structure with to_json instead of manually.
    
    Test plan:
    Same as previously.
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    (cherry picked from commit bb1b15d93937677aa61dbee8fce8c19846acb4d8)
    
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

commit a1f5f41d526c27156523eee264254ef86a139fed
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Mon Apr 15 15:09:24 2013 -0400

    Bug 8278 [Follow-up] Replace YUI autocomplete in UNIMARC 210c plugin
    
    This follow-up corrects the heading of the plugin pop-up window to match
    the UNIMARC field it targets.
    
    To test, trigger the plugin and confirm that the pop-up window refers to
    210c.
    
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    
    Smple patch, fixes label. No errors.
    
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    (cherry picked from commit e75fd34e92c599da8daa7ab40ef0153acbf05bf6)
    
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

commit e63175d6b9874f1cd9c5d24238595db5be53c2ef
Author: Chris Cormack <chrisc at catalyst.net.nz>
Date:   Tue Apr 2 19:48:34 2013 +1300

    Bug 8278 : Follow up, fixing the permissions on cataloguing/ysearch.pl
    
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    
    Comment: Works. No errors.
    
    Without patch and normal staff user, logged out
    With patch, normal staff, valid output.
    
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    (cherry picked from commit 588cc04571bd4eab1878c4abcac883ebe3ad060e)
    
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

commit 135347db66406f5cb989de392a0851d76fc6d266
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Fri Mar 8 12:56:24 2013 -0500

    Bug 8278 - Replace YUI autocomplete in UNIMARC 210c plugin
    
    This plugin, for pseudo-authority-control of publisher information,
    used YUI autocomplete to pull existing publisher data from the
    biblioitems table. Despite its name it is not UNIMARC-specific.
    It assumes publisher name data is saved in biblioitems.
    
    This patch converts the autocomplete to jQueryUI. The search
    script has been amended to return valid JSON. The template has
    been modified to enable the jQueryUI autocomplete and to add
    structure more consistent with other popups.
    
    To test, link the publisher name field in your MARC structure to
    the unimarc_field_210c_bis.pl plugin. Open a MARC record for
    editing and click the "tag editor" link to launch the plugin.
    Type the first few letters of a publisher which exists in your
    database. You should get an autocomplete menu of publishers
    which match your search. Select one and click the "choose"
    button to fill the field in the MARC editor.
    
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    Comment: work as described. No errors.
    
    Test: link 260b to plugin. Edit record and search first letters
    of present editors. Menú appears and can select and fill value.
    
    Why plugin 210c_bis talk about field 225a?
    
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Plugin now works correctly for me, even when linked
    to 260$b in a MARC21 installation. All tests and QA script
    pass.
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    (cherry picked from commit 626c27baf993b8a19a7cdace902c03ae26f23da5)
    
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

commit fda21fbc28867da1043c2ef7aed1213e7cd4defd
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Mon May 6 12:29:01 2013 -0400

    Bug 10204 - Patron image no longer appears in the OPAC
    
    With the addition of opac-memberentry.pl to the OPAC we lost a way to
    display the image associated with a patron's account. This patch adds
    display of the patron image to opac-memberentry.pl now that
    opac-userdetails.pl and opac-userupdate.pl are deprecated.
    
    To test:
    
    1. Log into the OPAC as a patron who has an image associated with their
       account. View the "my personal details" tab and confirm that the
       patron image appears with and without OPACPatronDetails enabled.
    
    2. Log into the OPAC as a patron who has no image associated with their
       account. View the "my personal details" tab and confirm that the
       layout looks correct.
    
    3. Turn off OPACpatronimages and confirm that the "my personal details"
       page looks correct.
    
    Signed-off-by: Mirko Tietgen <mirko at abunchofthings.net>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Tested with OpacPatronDetails and OpacPatronImags turned on/off
    and it's working well.
    Template only changes.
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    (cherry picked from commit afccbfcce7192c6e1fcd73f137135cb5289c3d3d)
    
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

commit 0742457840edf7784d5bccafaa6b6383125a53ed
Author: Jared Camins-Esakov <jcamins at cpbibliography.com>
Date:   Thu May 2 11:53:36 2013 -0400

    Bug 9961: Add truncation support to QP driver
    
    The original implementation of QueryParser did not handle truncation
    based on the QueryAutoTruncate system preference. This patch adds support.
    
    To test:
    1) Apply patch.
    2) Turn on UseQueryParser.
    3) Set QueryAutoTruncate to "automatically."
    4) Search for "har". Note that it returns results with words
       like "Harry" (i.e. with right truncation).
    5) Search for "har*". Note that it still returns results with right
       truncation.
    6) Set QueryAutoTruncate to "only when * is added."
    7) Search for "har". Note that it returns only records that have the
       exact word "har" in them (most likely there will be none unless you
       have Hebrew items).
    8) Search for "har*". Note that once again it returns results for "Harry"
       (i.e. right truncated results).
    9) Sign off.
    
    This patch also reindents a hash in Koha/QueryParser/Driver/PQF.pm
    because it was hard to read before.
    
    Signed-off-by: Mirko Tietgen <mirko at abunchofthings.net>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    All tests and QA script pass.
    Thx for fixing this Jared!
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    (cherry picked from commit 8c6fe639b812665339a2cd84083f8c75d3957fdd)
    
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

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

Summary of changes:
 C4/Search.pm                                       |    5 +
 Koha/QueryParser/Driver/PQF.pm                     |   26 +++---
 Koha/QueryParser/Driver/PQF/query_plan/node.pm     |    4 +
 cataloguing/ysearch.pl                             |   10 ++-
 installer/data/mysql/kohastructure.sql             |   22 +++++-
 installer/data/mysql/updatedatabase.pl             |   59 ++++++++++++-
 .../value_builder/unimarc_field_210c_bis.tt        |   91 ++++++++-----------
 .../opac-tmpl/prog/en/modules/opac-memberentry.tt  |   37 +++++---
 kohaversion.pl                                     |    2 +-
 .../cronjobs/social_data/get_report_social_data.pl |    2 +-
 opac/opac-memberentry.pl                           |   13 +++-
 11 files changed, 178 insertions(+), 93 deletions(-)


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list