[koha-commits] main Koha release repository branch 3.20.x updated. v3.20.04-8-gdcd5491

[Git repo owner]

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.20.x has been updated
       via  dcd54915d0ce56acd900de313715be416ddc9c7e (commit)
      from  7f1efe089871ab23f8fc4a738fb79601786b907d (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 dcd54915d0ce56acd900de313715be416ddc9c7e
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Mon Sep 28 10:58:16 2015 +0100

    Bug 14910: Redirect to the circulation module after a renew
    
    iIf a patron is renewed from the circulation module, the librarian
    should be redirected to the circulation module.
    This works correctly if the renew is done from the patron module
    (members).
    
    This is caused by a typo in the template: desintation vs destination.
    
    This patch also removes the cardnumber parameter to the setstatus.pl
    script, it is not needed given that borrowernumber is always passed.
    
    This has a good side-effect, it will fix bug 14691. The cardnumber does
    not exist anymore, so no need to escape it :)
    
    Test plan:
    0/ Do not apply this patch
    1/ Create a patron with a cardnumber with a quote (rm'me) and another
    one without a quote (rmme)
    2/ Go on the checkouts page (circ/circulation.pl)
    3/ Renew the 2 patrons
    => With rm'me you are redirected to the circ module - ok
    => With rmme you are redirected to the member module - nok
    4/ Go on the patron detail page (members/moremember.pl)
    5/ Renew the 2 patrons
    => you are redirected to the member module - ok
    6/ Delete the patrons
    => Nothing happend with rm'me, there is a JS error on the page - nok
    => rmme is deleted - ok
    
    7/ Apply the patch and recreate rmme
    8/ Repeat 2, 3, 4, 5
    => You are redirected to the correct module
    9/ Delete the patrons
    => They are successfully deleted
    
    Signed-off-by: Magnus Enger <magnus at libriotech.no>
    Followed the test plan, works as advertised. (I did have some problems
    initially, but that was caused by me not using the interface in
    English...)
    
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
    (cherry picked from commit 653d305452a27d19d929e2acff71643845686af3)
    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>

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

Summary of changes:
 .../prog/en/includes/members-toolbar.inc              |    4 ++--
 .../intranet-tmpl/prog/en/modules/circ/circulation.tt |    4 ++--
 members/setstatus.pl                                  |   17 ++++++++---------
 3 files changed, 12 insertions(+), 13 deletions(-)


hooks/post-receive
-- 
main Koha release repository