[koha-commits] main Koha release repository branch master updated. v3.16.00-494-g2bbb27c

Git repo owner gitmaster at git.koha-community.org
Tue Aug 26 20:42:27 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, master has been updated
       via  2bbb27c841bb4c3810d4db81a4366df29cdba833 (commit)
       via  0033d287bad3344fad0861134d8dd3e03e56ed24 (commit)
       via  2031c3e5ca9ede22c0f70082c2b90a2416b2b5c3 (commit)
      from  794fa6e80558fc4fd5a3d4fd1fdb41983d316346 (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 2bbb27c841bb4c3810d4db81a4366df29cdba833
Author: David Cook <dcook at prosentient.com.au>
Date:   Fri May 30 16:18:03 2014 +1000

    Bug 12325 - Marc and Card view in Z39.50 Results doesn't work properly in IE 8
    
    This patch removes an unnecessary and problematic "return true" which
    causes conflicts with Greybox when clicking on "MARC" or "Card" in
    the search results of a Z39.50 search. The result is that the user is
    taken to the showmarc.pl page and cannot return to the results, which
    makes Z39.50 searching pretty much unusable for IE 8 users.
    
    This patch also cleans up the "inline links" display for the separate
    Z39.50 search pages. The 'biblio' version now has inline links for LCCN,
    and the 'authority' version now prevents inline links for the MARC
    and Import cells.
    
    _TEST PLAN_
    
    1) Open Internet Explorer 8 (if you don't have IE 8, use whatever Internet
    Explorer version you do have. You can use F12 developer tools to emulate it.)
    2) Go to Cataloguing > New from Z39.50
    3) Do a search that will get results
    
    (N.B. if you're using IE9+, hit F12 in the pop-up and change to IE 8 mode)
    
    4) Click on "MARC" or "Card".
    5) Note that Greybox tries to open a pop-over, but instead follows the
    link to showmarc.pl. (If you're on IE 8, you can't use "Back" to go
    back to the search results. If you're using an emulated version, you
    can.)
    
    (If you're testing the inline links, click on the LCCN cell for a biblio
    search (not that no inline links appear), or MARC and Import cells when
    doing an authority search (not that inline links do appear). Look at
    bug 10096 for authority z39.50 server details.)
    
    6) Apply the patch
    
    7) Repeat steps 1-5
    8) Note that Greybox now appears correctly when clicking "MARC" and
    "Card"
    
    9) Note also that inline links now appear for LCCN cell for biblio
    searches, and inline links no longer appear for MARC and Import
    cells when doing authority searches.
    
    10) Profit
    
    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Tested for regressions in Firefox and Chromium, no problems found.
    Passes tests and QA script.
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 0033d287bad3344fad0861134d8dd3e03e56ed24
Author: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Date:   Sat May 3 00:21:48 2014 -0300

    Bug 12176 - Remove HTML from additem.pl
    
    This patch removes HTML code from additem.pl
    
    To test:
    1. Chech no regressions on Add/Edit/Save items
    2. Update translation files for a language,
    check new string "Tag editor" on staff PO file
    3. Check it pass xt/tt_valid.t
    
    Rebased
    
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 2031c3e5ca9ede22c0f70082c2b90a2416b2b5c3
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Wed Aug 20 13:26:49 2014 +0200

    Bug 12717: Library no longer receiving Overdue email for patrons without email address
    
    Bug 10832 changes the fallback behavior if a patron does not have email
    address: a print notice is generated into the message_queue table.
    But this can cause issue for some libraries. The script should sent an
    email and (generated csv, html, text file) with the list of all unsent
    notices.
    
    Test plan:
    1/ Add overdue to a patron without email address (or smsalertnumber)
    2/ Check email in the overdue rules configuration (or sms)
    3/ Launch the overdue_notices.pl cronjob
    4/ Verify the message_queue contain a print notice AND an email notice
    for the library
    
    Signed-off-by: Nick Clemens <nick at quecheelibrary.org>
    Signed-off-by: Katrin Fischer <katrin.fischer at bsz-bw.de>
    This works as described and restores the old behaviour for now, with
    the difference that you have a print notice generated and visible
    in the patron account notices tab that will say 'pending'.
    
    We will have to figure out how we can change the workflows nicely
    to have only one script deal with print notice in the future.
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

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

Summary of changes:
 cataloguing/additem.pl                             |   91 +++++++++++++-------
 .../prog/en/modules/cataloguing/additem.tt         |   27 +++++-
 .../en/modules/cataloguing/z3950_auth_search.tt    |    4 +-
 .../prog/en/modules/cataloguing/z3950_search.tt    |    4 +-
 misc/cronjobs/overdue_notices.pl                   |   17 ++++
 5 files changed, 108 insertions(+), 35 deletions(-)


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list