[koha-commits] main Koha release repository branch 3.10.x updated. v3.10.02-25-g8c7c8ec

Fri Feb 1 09:52:27 CET 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.10.x has been updated
       via  8c7c8ec75a1b63baa85055efdbc90e0ad7472e77 (commit)
       via  fa106c3eb12566528657438f271e160995387d1b (commit)
       via  847df9ab3fca2ef30cf2069a82e719b889e84332 (commit)
       via  344d9ec63dd3401a8f59f4f233dfb68db6e054c2 (commit)
       via  d4c86ff52cead6806a1867ed5c7f4a803412aecf (commit)
       via  88fbb6c561cc27a2af3b64a870d93ff00af4c37d (commit)
       via  6844ccbfe1ce04ea6bae9f84b81b6bfadf2436f9 (commit)
       via  a6f2e86fa2b4725abb5c3506c5f57b374bfcc984 (commit)
      from  f9ee893e0edd61f2024804f51e9774f186bafb35 (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 8c7c8ec75a1b63baa85055efdbc90e0ad7472e77
Author: Magnus Enger <magnus at enger.priv.no>
Date:   Sun Jan 27 14:00:45 2013 +0100

    Bug 9497 - Make it easier to add new encodings for Z39.50 servers and add ISO 8859-1

    This patch does three things:
    - Makes admin/z3950servers.pl create one variable for encoding,
      not one variable for each encoding
    - Makes the template create encoding options from a list
    - Adds ISO 8859-1 to the list of available encodings
    See the bug for a lengthier description

    To test:
    - Apply the patch
    - Edit one Z39.50 server several times, choosing each available
      encoding in turn
    - Check that for each encoding, the correct encoding is shown both in
      the list of Z39.50 servers and in the dropdown on the edit screen

    Comment: Works as described, no errors.

    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    Signed-off-by: Mason James <mtj at kohaaloha.com>
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit fa106c3eb12566528657438f271e160995387d1b
Author: Jared Camins-Esakov <jcamins at cpbibliography.com>
Date:   Thu Jan 24 13:58:07 2013 -0500

    Bug 9486: t/Letters.t loads modules in wrong order

    Because the unit test t/Letters.t loads C4::Letters before C4::Context,
    and C4::Letters is not even vaguely thread-safe, the test tends to fail.
    Usually. Moving the dbh mocking to before the use_ok('C4::Letters') test
    fixes the problem.

    To test:
    1) Before applying patch, run `prove t/Letters.t` a few times. Note that
       it fails most of the time, if not all the time.
    2) Apply patch.
    3) Repeat step (1), noting that now it passes every time.

    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Works as advertised.
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit 847df9ab3fca2ef30cf2069a82e719b889e84332
Author: Tomas Cohen Arazi <tomascohen at gmail.com>
Date:   Tue Dec 4 15:32:28 2012 -0300

    Bug 9211 - days_between wrong behaviour

    As noted in comments #15, #16 and #17 of bug 8486, and its API,
    Koha::Calendar->days_between should always returns a positive number
    irrespective of the relative order of the parameters. This is still an
    open bug, which arised when rewriting the Calendar.t file (Bug 9209).

    Regards
    To+

    Sponsored-by: Universidad Nacional de Córdoba
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Paul Poulain <paul.poulain at biblibre.com>
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    Added Paul's sign-off line, as despite it showing up in Bugzilla, it
    wasn't on the patch.

    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit 344d9ec63dd3401a8f59f4f233dfb68db6e054c2
Author: Mathieu Saby <mathieu.saby at univ-rennes2.fr>
Date:   Wed Jan 16 12:19:23 2013 +0100

    Bug 9402: letter v between 2 option tags

    In HTML code generated for advanced search, a useless "v" letter
    is visible after "dissertation or thesis (revised)</option>".
    This patch suppress "v" letter is outside <option> markup
    (and cosmetic change : "Adult" -> "adult" in public types)

    To test : check HTML code generated by advanced search.

    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Fixes capitalization and a little problem in the HTML of an UNIMARC
    plugin.
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit d4c86ff52cead6806a1867ed5c7f4a803412aecf
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Thu Dec 13 14:26:54 2012 +0100

    Bug 9280: FIX price calculation adding an order from a staged file

    Test plan:
    Add an order from a staged file containing a price value (010$d for
    UNIMARC user).
    Check this patch with 2 different vendors (listprice=1 and listprice=0).
    Check the calculated price (depending discount and gstrate).

    Signed-off-by: mathieu saby <mathieu.saby at univ-rennes2.fr>
    Signed-off-by: Elliott Davis <elliott at bywatersolions.com>
    introduces no new bugs
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    I was unable to replicate the bug in MARC21, but the patch introduces
    no regressions that I could find, so I am pushing it anyway.
    Note: Acquisitions remains unusable under Plack. In order to test this
    patch I first applied the patch on bug 9432.

    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit 88fbb6c561cc27a2af3b64a870d93ff00af4c37d
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Thu Dec 13 14:17:07 2012 +0100

    Bug 9281: FIX add an order from a staged file

    In GetMember the hash parameter contained
    'C4::Members' => 'borrowernumber'
    XXX => undef

    Test plan:
    In the acquition module, try to add an order to a basket from a staged file.
    Without this patch, you get an error:
      DBD::mysql::st execute failed: called with 2 bind variables when 1 are needed at /home/koha/src/C4/Members.pm line 559

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    All tests pass, error only appearing in the log file.

    Test plan:
    1) Create a basket
    2) Add one or multiple order lines using a staged file and the 'batch' feature.
       This is done by using the "Save" button at the bottom of the page instead
       of the "Add order" link next to a single entry.
    3) Check your koha-error_log for the error above.
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit 6844ccbfe1ce04ea6bae9f84b81b6bfadf2436f9
Author: MJ Ray <mjr at phonecoop.coop>
Date:   Wed Jan 16 20:13:57 2013 +0000

    Bug 9407: patrons search should match substrings

        If a patron lives at 4345 Library Rd, a search on Street Address for
        Library should find it. However, it does not, but a search for 4345
        Library does.  This patch adds a "Search Type" drop-down, defaulting
        to the current behaviour.

        To test:
        1) Search for a patron based on a non-leading part of a field.
        2) Should return no results.
        3) Search again with "Search Type" of "Contains".
        4) Should return the patron.

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit a6f2e86fa2b4725abb5c3506c5f57b374bfcc984
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Tue Nov 20 15:57:13 2012 +0100

    Bug 9116: Remove some useless tmp variables

    Some variables are useless in C4::Items::PrepareItemrecordDisplay.

    This patch removes its.

    Test plan:
    Syspref AcqCreateItem = "receive an order" and try to receive an order.
    Check there is no regression.

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

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

Summary of changes:
 C4/Items.pm                                        |   38 ++++++-------------
 Koha/Calendar.pm                                   |    7 ++++
 acqui/addorderiso2709.pl                           |   10 ++----
 admin/z3950servers.pl                              |    4 +-
 .../prog/en/includes/patron-search.inc             |    6 +++
 .../prog/en/includes/subtypes_unimarc.inc          |    4 +-
 .../prog/en/modules/admin/z3950servers.tt          |   30 +++------------
 .../prog/en/includes/subtypes_unimarc.inc          |    4 +-
 members/member.pl                                  |    8 ++++-
 t/Letters.t                                        |    6 +--
 10 files changed, 49 insertions(+), 68 deletions(-)

hooks/post-receive
-- 
main Koha release repository