[koha-commits] main Koha release repository branch master updated. v3.18.00-310-g155ce56

Tue Feb 10 18:02:09 CET 2015

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  155ce56ad811954802090faf2994278f7c9dcbf2 (commit)
       via  18c28fe36cace2edcedcecad44b0637cc8a90625 (commit)
       via  5bd9482550cee62fde36c5a74d73714ec000dc0b (commit)
       via  87bcc8b303e506514318814f63c430a3a54f1181 (commit)
       via  f19a1777a0847ce2dc10b7f259bf76fc2d382d19 (commit)
       via  dc5b94486304fc5353640c0341cdfa11d3367fb2 (commit)
      from  2df541712fd691ee817568ed9d14e94bba13424e (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 155ce56ad811954802090faf2994278f7c9dcbf2
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Fri Feb 6 09:44:33 2015 -0500

    Bug 13240 [QA Followup]

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 18c28fe36cace2edcedcecad44b0637cc8a90625
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Wed Nov 12 12:24:10 2014 +0100

    Bug 13240: Remove commented warns

    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 5bd9482550cee62fde36c5a74d73714ec000dc0b
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Wed Nov 12 12:23:38 2014 +0100

    Bug 13240: Remove some code obfuscation

    my $bar;
    my $foo = $bar->{borrowernumber} ||= {};
    $foo->{one} ||= 'something';
    $foo->{two}++;

    What does $bar contain?

    $VAR1 = {
              'borrowernumber' => {
                                    'two' => 1,
                                    'bar' => 'something'
                                  }
            };

    Not really obvious.

    Maybe something I did not see is hidden.

    Test plan:
    Verify the digest for DUE and PREDUE work as before.

    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 87bcc8b303e506514318814f63c430a3a54f1181
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Thu Feb 5 09:21:12 2015 -0500

    Bug 13670 - Patron holds table not loading when holds tab anchor is used

    If a link to a patron record ends with #reserves, the holds tab is selected
    on page load, but the holds table does not load.

    Test Plan:
    1) Build your holds queue
    2) From the holds queue report, open any patron link
    3) Note the patron's reserves do not load
    4) Apply this patch
    5) Clear your browser cache
    6) Repeat step 2
    7) Note the holds table now loads

    Signed-off-by: Sean McGarvey <seanm at pascolibraries.org>

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit f19a1777a0847ce2dc10b7f259bf76fc2d382d19
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Fri Nov 14 16:18:09 2014 +0100

    Bug 13319: Fix Koha::Acq::Order should set parent_ordernumber

    The tests highligh the problem: if the parent_ordernumber attribute is
    not set on inserting an order, the object returned by the method does
    not contain the value (undef).

    Test plan:
    Verify the tests are consistent and
        prove t/db_dependent/Acquisition/CancelReceipt.t
    should return green.

    You can also verify that receive partially an order and cancel the
    receipt work as expected.

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

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit dc5b94486304fc5353640c0341cdfa11d3367fb2
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Wed May 21 11:09:04 2014 -0700

    Bug 12323 [SIGNED OFF] - Acquisitions search results fixed in order of biblionumber

    The acquisitions search is exhibiting the same behavior as bug 11410.
    Results are always fixed in order of biblionumber, among other possible
    issues ( including the ampersand issue ).

    Test Plan:
    1) From an open basket, choose "Add to basket"
    2) Run a search for "From an existing record"
    3) Note the "View MARC" URLs are fixed in order of biblionumber
    4) Apply this patch
    5) Refresh the page
    6) Note the results are no longer fixed in order of biblionumber

    Signed-off-by: David Cook <dcook at prosentient.com.au>

    Works as described.

    I think the code could be a bit tidier, but I think it makes sense to
    use buildQuery here. It'll detect CCL, CQL, and PQF queries, as well
    as parsing a regular keyword search as one would expect when searching
    the catalogue.

    It also has the added bonus of performing relevance searching,
    so long as QueryAutoTruncation is off, and so long as library staff
    avoid using the "*" truncation wildcard (see bug 12430).

    While there are simpler ways to fix this acq bug, I think this was
    probably the best move, as it adds a bit to the consistency of what
    librarians can expect from their search results.

    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Same result with and without the patch but I agree with the changes.
    BuildQuery should be called before SimpleSearch if QP is not used.

    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

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

Summary of changes:
 Koha/Acquisition/Order.pm                   |    3 +-
 acqui/neworderbiblio.pl                     |   13 ++++++--
 koha-tmpl/intranet-tmpl/prog/en/js/holds.js |   13 +++++---
 misc/cronjobs/advance_notices.pl            |   16 +++-------
 t/db_dependent/Acquisition/CancelReceipt.t  |   44 +++++++++++++++++++++++++--
 5 files changed, 68 insertions(+), 21 deletions(-)

hooks/post-receive
-- 
main Koha release repository