[koha-commits] main Koha release repository branch master updated. v16.05.00-26-g71f0370

[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, master has been updated
       via  71f0370ddde0d8b8e2a666a091081186fb6531e8 (commit)
       via  cbdd9194eb3359d517fbda9a50a02fbc375bf5dd (commit)
      from  567a1a9021aaea3bf42b19f493b79f682ed9042b (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 71f0370ddde0d8b8e2a666a091081186fb6531e8
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Fri May 27 12:42:17 2016 +0000

    Bug 16610 - Regression in SIP2 user password handling
    
    Previous to bug 14507, SIP2 only did internal authentication. A change
    to the way we check empty passwords has caused any empty password to
    send back a CQ of Y. Previous to that patch set, a CQ of Y would only be
    sent back of the patron password column was NULL. Now, an empty AD field
    *always* returns a CQ of Y.
    
    Test Plan:
    1) Send a patron information request with an empty AD field
       Note: You must send the AD field or you won't get back a CQ field
    2) Note you get back a CQ of Y
    3) Apply this patch
    4) Repeat step 1
    5) Note you now get back a CQ of N
    
    Signed-off-by: Trent Roby <troby at bclib.info>
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Brendan Gallagher <brendan at bywatersolutions.com>

commit cbdd9194eb3359d517fbda9a50a02fbc375bf5dd
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Wed Jun 1 15:34:56 2016 +0200

    Bug 16610: [QA Follow-up] Add a test
    
    To verify the original patch, this test shows that before applying
    it the Patron Info request did not return CQ==N for an empty
    password. Note that the Patron Status did btw.
    
    After applying the original patch, the test passes for patron
    info as well as status.
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Brendan Gallagher <brendan at bywatersolutions.com>

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

Summary of changes:
 C4/SIP/ILS/Patron.pm         |    8 ++++----
 t/db_dependent/SIP/Message.t |   26 ++++++++++++++++++++++++--
 2 files changed, 28 insertions(+), 6 deletions(-)


hooks/post-receive
-- 
main Koha release repository