[koha-commits] main Koha release repository branch 3.8.x updated. v3.08.07-27-gd3c5e31

[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.8.x has been updated
       via  d3c5e317f14ff0167bb65916d1027886107420c9 (commit)
       via  abdbad9409859f9d0abdcf22757bcdfa43bfb564 (commit)
       via  06c7f8613bd14b09730fed3380575e1ee10b9b72 (commit)
      from  8717f98508b32189c46e0a3592166bcf61aafd8a (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 d3c5e317f14ff0167bb65916d1027886107420c9
Author: Paul Poulain <paul.poulain at biblibre.com>
Date:   Thu Nov 29 17:38:09 2012 +0100

    Bug 8981 follow-up: case
    
    While QAing your patch, I saw you were writting "MarcFlavour", it should have been "marcflavour"
    
    Will work with the UC, but let's be consistent.
    
    This patch fixes another MarcFlavour just a few lines before I spotted by chance
    
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit abdbad9409859f9d0abdcf22757bcdfa43bfb564
Author: David Cook <dcook at prosentient.com.au>
Date:   Tue Oct 30 09:06:45 2012 +1100

    Bug 8981 - Remove Nonpublic General Note from OPAC authorities display
    
    Currently, the MARC21 667 Nonpublic General Note field is appearing on the opac-authoritiesdetail.pl page.
    
    Since it is a nonpublic note, it makes sense to remove it from this view.
    
    This patch adds in an extra condition that checks authority records for MARC21 667 fields before displaying them in this opac screen.
    
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    
    Does what it says, with no side effects
    
    Signed-off-by: Paul Poulain <paul.poulain at biblibre.com>
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit 06c7f8613bd14b09730fed3380575e1ee10b9b72
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Mon Oct 1 15:16:29 2012 -0400

    Bug 5801 - C4::Circulation::_GetCircControlBranch wrongly used in opac-reserve.pl
    
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    All tests and perlcritic pass.
    
    Tests done:
    holds policy vs. circulation rules seemed to make the difference for me,
    I am not totally sure why that is, but see no reason to fail this patch as
    results are ok.
    
    System preferences
    - CircControlBranch = the library the item is from
    - ReservesControlBranch = patron's home library
    
    Circulation rules
    - Library A - Holds policy: Books can not be put on hold.
    - Library B - Holds policy: Books can be put on hold. (no special setting needed)
    
    Catalog data
    - A record with an item for library B, itemtype Books.
    
    Test without patch:
    
    1) Patron A (home library A) places a hold in OPAC on an item from library B.
    
    Should be: hold is NOT allowed.
    Before patch: hold IS allowed.
    After patch: hold is NOT allowed.
    OK
    
    2) ReserveControlbranch = item's home library
    Patron A (home library A) places a hold in OPAC on an item from library B.
    
    Should be: hold is allowed.
    Before patch: hold is allowed.
    After patch: hold is allowed.
    OK
    
    3) ReservesControlBranch = patron's home library
    Circulation rules: Remove hold policy for library B.
    Library A: no holds allowed for books using 'Holds allowed (count)' = 0
    Library B: 5 holds allowed for books using 'Holds allowed (count)' = 5
    Patron A (home library A) places a hold in OPAC on an item from library B.
    
    Should be: hold is NOT allowed
    Before patch: hold is NOT allowed
    After patch:  hold is NOT allowed
    OK
    
    4) ReserveControlbranch = item's home library
    
    Should be: hold is allowed
    Before patch: hold is allowed
    After patch:  hold is allowed
    OK
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

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

Summary of changes:
 opac/opac-authoritiesdetail.pl |    5 ++++-
 opac/opac-reserve.pl           |    2 +-
 2 files changed, 5 insertions(+), 2 deletions(-)


hooks/post-receive
-- 
main Koha release repository