[koha-commits] main Koha release repository branch 3.16.x updated. v3.16.00-27-g1857c44

Git repo owner gitmaster at git.koha-community.org
Thu Jun 5 17:06:22 CEST 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, 3.16.x has been updated
       via  1857c444cc6666331eae34e9a99fa1eaa3f1c760 (commit)
       via  4b9409863eab03072212308e386b3dd622eb9b46 (commit)
      from  fd0ca0edde6b7c465543d1eb2762fb439d9e2bfa (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 1857c444cc6666331eae34e9a99fa1eaa3f1c760
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Tue May 20 14:58:32 2014 -0400

    Bug 12295: fix Javascript error when merging authorities
    
    Attempting to merge authorities results in the following error:
    Uncaught SyntaxError: Unexpected token u authorities-home.pl:284
        showMergingInProgress authorities-home.pl:284
        (anonymous function) authorities-home.pl:297
        o jquery.js:2
        p.fireWith jquery.js:2
        e.extend.ready jquery.js:2
        c.addEventListener.B
    
    This was the result of the upgrade of jquery-cookie by the patch
    for bug 11369; newer versions of jquery-cookie changed the return of
    $.cookie('foo') from null to undefined when the cookie is not present.
    
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    No test plan, no errors.
    
    Test
    1. search some authorities
    2. click merge, on browser dev console pops reported error message
    'Unexpected token u authorities-home.pl...'
    No way to marge auths
    3. with pach applied, merging works again
    
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit 4b9409863eab03072212308e386b3dd622eb9b46
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Wed May 21 07:14:50 2014 -0400

    Bug 12025 - Inventory tool fails with "Could not reads headers", or sometimes a timeout
    
    For libraries with large numbers of records, using a file of barcodes in
    the inventory tool without selecting filters to limit the catalog
    selection to compare against can result in an error of "could not read
    headers" or a page timeout.
    
    When you upload a file of barcodes, and do not specify any filters on
    the rest of the form, it effectively runs GetItemsForInventory() such
    that *every* item in the database is returned.  On a large enough
    database, this takes so long that the script times out.
    
    The current documentation in the manual implies that if you upload a
    file of barcodes, the *only* thing it does is set the date-last-seen
    field.  This is not entirely true; it also tries to compare the list of
    scanned items to what is expected to be seen in the list of items.
    
    The work-around for libraries with a large number of item records is to,
    when uploading a file of barcodes, to also set the filters (e.g.,
    library, shelving location, call number range, etc.) to
    specify the range of items that the file of barcodes corresponds to.
    
    This patch adds a warning when a librarian attempts to upload a
    barcode list file for comparison without selecting any filters
    to limit the catalog selection.
    
    Test Plan:
    1) Apply this patch
    2) Browse to the inventory tool
    3) Click the "Choose File" button, select a file
    4) Click the "Submit" button
    5) You should see a warning message
    6) Click "Cancel"
    7) Select one of more filters
    8) Click the "Submit" button
    9) This time the script should proceed without triggering the warning.
    
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    Work as described following test plan.
    No errors
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

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

Summary of changes:
 .../prog/en/includes/authorities_js.inc            |    4 +--
 .../prog/en/modules/tools/inventory.tt             |   29 +++++++++++++++-----
 2 files changed, 24 insertions(+), 9 deletions(-)


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list