[koha-commits] main Koha release repository branch 3.8.x updated. v3.08.13-100-gc8fb62d

Sun Sep 15 06:59:31 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.8.x has been updated
       via  c8fb62d547c79565a611419a7723dc0f9d8e2288 (commit)
       via  b06b6f2438aca661929d4233f2b2ca24ba8589c6 (commit)
      from  e601e013db6954cb0d5f647337bb08b6e9b8317e (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 c8fb62d547c79565a611419a7723dc0f9d8e2288
Author: David Cook <dcook at prosentient.com.au>
Date:   Fri Jul 5 12:00:36 2013 +1000

    Bug 10541: enable cross-browser AJAX in additem.js

    Currently, the additem.js is using "indexOf" to search
    for a value in an array. While this works in Chrome, Firefox,
    and IE > 9, it fails miserably in IE 8 and 7 (which don't have
    the indexOf method). This means that users aren't able to add
    items using the acquisitions module!

    Instead of using "indexOf", we should be using the jQuery function
    $.inArray. It was added in jQuery v1.2 (3.8.0 uses v1.3.2 so even
    our oldest supported release can use this method). It's perfectly
    cross-browser compatible...works in Chrome, Firefox, and every
    version of IE that I've tried (i.e. 7, 8, 9).

    Test Plan:

    Before applying patch:

    0) Switch to Internet Explorer 7, or 8, or 9, or 10.

    If you're using IE 9 or 10, you'll need to change the Document Mode to
    IE7 standards or IE8 standards.

    You can do this by opening Internet Explorer 9 or 10, pressing F12 (or
    clicking on the gear in the top right corner and choosing
    F12 Developer Tools), and then clicking on "Document Mode" on the
    top toolbar. There, you can change to IE7 or IE8 standards.

    N.B. This is not always a perfect emulation in every case, but this
    time it does show you the bug.

    1) Set the system preference AcqCreateItem to "receiving an order"
    2) Go to Acquisitions
    3) Either:
        a) Receive a shipment for a basket with items
        b) Create a new basket, create an order, close the basket, and
           then do 3a)
    4) In the "Item" fieldset, fill out some fields such as barcode,
       Date acquiried, Public note, etc.
    5) Click "Add" at the bottom of the fieldset
    6) Note that while the item may have been added, the "Item" fieldset
    is not being shown again. You may also notice a Javascript error
    appearing in a pop-up window or you might see a yellow warning flag
    on the bottom status bar.

    APPLY THE PATCH

    7) Do a full refresh of the page (hold down shift and press the refresh
    button on the browser next to the address bar), and try adding items
    again.
    8) Note that you receive no warnings and that items are added correctly
    as they would be in Firefox or Chrome.

    OPTIONALLY

    9) To be sure that I haven't broken anything, go through the same steps
    in IE9 (with IE9 standards) or Chrome or Firefox. Everything should be
    working.

    Signed-off-by: Owen Leonard <oleonard at myacpl.org>

    Tested in IE10 in IE7 mode and IE9 mode. Also tested in Firefox.

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Passes koha-qa.pl, works as advertised.

    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit 53f82b1d7d9b7f06da1032faf0800d624eb4c1d3)
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    (cherry picked from commit afbe6a0f2c9b0d37327a57cf72eabfb7eebe156d)
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    (cherry picked from commit afbe6a0f2c9b0d37327a57cf72eabfb7eebe156d)
    Signed-off-by: Chris Hall <followingthepath at gmail.com>
    (cherry picked from commit bef4efbdc7e68b1c91b7234ba2d68ff5aeeb5131)

commit b06b6f2438aca661929d4233f2b2ca24ba8589c6
Author: Fridolyn SOMERS <fridolyn.somers at biblibre.com>
Date:   Tue Jul 9 17:38:04 2013 +0200

    Bug 9362: fix query that finds upcoming due loans

    C4:Circulation:GetUpcomingDueIssues is used in the advance_notices.pl
    script.  This patch corrects an error in its handling of the maxdays
    parameter that resulted in it picking up *all* upcoming due loans and
    recently overdue loans.

    Test plan :
    - Create an issue with a date due in the paste
    - Create an issue with a date due in two days
    - Launch advance notices with due date in max 2 days : perl misc/cronjobs/advance_notices.pl -c -n -v -m=2
    => You get a warn "found 0 issues"
    - Launch advance notices with due date in max 3 days : perl misc/cronjobs/advance_notices.pl -c -n -v -m=3
    => You get a warn "found 1 issues"

    Signed-off-by: Mathieu Saby <mathieu.saby at univ-rennes2.fr>
    I did the following test :

    - 1 book to check in 2 days
    - 2 books to check in in the past

    before applying the patch :

    $perl ../misc/cronjobs/advance_notices.pl -c -n -v -m=2
    getting upcoming due issues at ../misc/cronjobs/advance_notices.pl line 203.
    found 1 issues at ../misc/cronjobs/advance_notices.pl line 205.

    I changed the value of "-m" : 0, 1, 2, 3, 4
    => always 1 issue found (the book to check in in 2 days)

    after applying the patch :

    $perl ../misc/cronjobs/advance_notices.pl -c -n -v -m=2
    found 0 issues
    for m = 0, 1, 2 => 0 issues

    $perl ../misc/cronjobs/advance_notices.pl -c -n -v -m=3
    found 1 issues
    for m = 3,4,5 => 1 issues (the book to check in in 2 days)

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Passes koha-qa.pl, works as advertised.

    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit ff7f37202949bb6d60cdf30de4e237e6b1de93eb)
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    (cherry picked from commit f40924b317e37757f1e552ab466dc840304c6182)
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    (cherry picked from commit f40924b317e37757f1e552ab466dc840304c6182)
    Signed-off-by: Chris Hall <followingthepath at gmail.com>
    (cherry picked from commit fb11fb31a4357777328b80fd5ad6ce5694823d11)

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

Summary of changes:
 C4/Circulation.pm                             |    4 ++--
 koha-tmpl/intranet-tmpl/prog/en/js/additem.js |    4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

hooks/post-receive
-- 
main Koha release repository