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

Git repo owner gitmaster at git.koha-community.org
Mon May 19 16:44:01 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  b322f5f13388fd7d64e9d2c3f8bd8c2cc1f60eeb (commit)
      from  1b02c031a34e7e3e3caabbe0184fb5fe71447d95 (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 b322f5f13388fd7d64e9d2c3f8bd8c2cc1f60eeb
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Tue Feb 18 09:47:32 2014 -0500

    Bug 11783: ensure CD field in SIP patron information response is populated
    
    If a patron has a record-level hold that is unavailable, any patron
    information request will send back an empty CD field when this field
    should have an item barcode in it [RM note: this actually isn't
    universally true -- the SIP2 standard is silent as to what is supposed
    to go in the CD field. Some SIP2 devices do indeed want an item
    barcode, but others are known to just want a display of the title
    and author of the request in question.  Providing an option is the
    topic of a new enhancement request, however.]
    
    This is due to a minor error in ILS::Patron::_get_outstanding_holds
    where GetItemnumbersForBiblio is assumed to return an array but in
    reality returns an arrayref.
    
    Test Plan:
    1) Create a record level hold for a patron and record
    2) Using SIP2, make a patron information request
    3) Note the empty CD fields
    4) Apply this patch, restart SIP server
    5) Repeat step 2
    6) Note the CD field now has a barcode
    
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    I did not test this patch but the following code shows me it is correct:
      use C4::Items;
      use Data::Dumper;
      my $biblionumber = 5035;
      my $itemnumber = (GetItemnumbersForBiblio($biblionumber))[0];
      say Dumper $itemnumber;
      $itemnumber = (GetItemnumbersForBiblio($biblionumber))->[0];
      say Dumper $itemnumber;
    
    displays:
    
    $VAR1 = [
              '23168',
              '23169',
              '23170',
              '23171',
              '23172'
            ];
    
    $VAR1 = '23168';
    
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit 37a0e888199f1a1f8eefd38ad6ada786403fe034)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

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

Summary of changes:
 C4/SIP/ILS/Patron.pm |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list