[koha-commits] main Koha release repository branch 3.14.x updated. v3.14.09-53-g6df3d7f

Git repo owner gitmaster at git.koha-community.org
Wed Aug 6 11:26:41 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.14.x has been updated
       via  6df3d7f44b3d323dfa9d61ed28299aefbbafb30d (commit)
      from  1e986908899e4beedbd5bf3a3b35edfb6e84599d (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 6df3d7f44b3d323dfa9d61ed28299aefbbafb30d
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Fri Nov 15 08:29:23 2013 -0800

    Bug 11258: fix another case where holds queue made transfer requests that contradict the library holds policy
    
    This patch fixes a problem where the holds queue generator
    was making requests where the pickup library is the
    same as the item's library but not the patron's branch,
    even if there is a "Default holds policy by item type" rule that states
    this item can only fill holds for patrons of the same library as the
    item.
    
    Test Plan:
    1) Create a test record with 2 items with different itemtypes
    2) Set the Default holds policy by item type for the first
       item to "From any library"
    3) Set the Default holds policy by item type for the second
       item to "From home library"
    4) Place a record level hold for a patron from another library,
       but for pickup at the same library as the item is from
    5) Rebuild the holds queue
    6) View the holds queue, note the item is listed, though this
       patron cannot place a hold on this item
    7) Apply this patch
    8) Repeat step 5, note the hold is no longer in the queue
    
    Signed-off-by: Liz Rea <liz at catalyst.net.nz>
    automated tests pass, functional tests pass. Bug replicated, eradicated by patch.
    
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    I finally managed to reproduce this, patch works as described.
    Passes tests and QA script, provided tests fail without patch, but
    succeed with the patch.
    
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit 79660377470e8d6ba9057eb59645886410852c92)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

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

Summary of changes:
 C4/HoldsQueue.pm            |    4 ++--
 t/db_dependent/HoldsQueue.t |    7 ++++---
 2 files changed, 6 insertions(+), 5 deletions(-)


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list