[koha-commits] main Koha release repository branch 3.12.x updated. v3.12.06-10-ga89124f

Git repo owner gitmaster at git.koha-community.org
Wed Nov 6 15:16:47 CET 2013


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.12.x has been updated
       via  a89124f7b0ee3e53f1e1462c13bb492e23f4127c (commit)
       via  815a7cb6b94455d4e0a0ea88f7cd0bcb8621cd6e (commit)
       via  70a6226cf7de15a74cb6f82dc14bf4568255fb95 (commit)
       via  b0474ebcf0a83cbbe8484b95986b350162959188 (commit)
      from  75e51cdc63948da19779fd650d191d1a1785bfd7 (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 a89124f7b0ee3e53f1e1462c13bb492e23f4127c
Author: Colin Campbell <colin.campbell at ptfs-europe.com>
Date:   Wed Sep 4 11:18:11 2013 +0100

    Bug 10817: return hold items info in SIP Patron Info Response
    
    When requested by the summary flags the sipserver
    should return in the patron info response barcodes of
    the relevant titles.
    
    For available holds this is the barcode of the captured items.
    
    For unavailable holds ( i.e. current unsatisfied holds ),
    we need to send a barcode so that the unit can use this to
    request the title info. The barcode could be any one
    belonging to the title.
    
    This patch also corrects the erroneous return of empty items
    in the patron information response. If the unit supplies a
    range 1 - 100 unless the title has a hundred or more copies the
    unit expects all copies. The server was erroneously stuffing
    the returned arrays with null copies so that all summary requests
    returned 100 copies (mainly without barcodes)
    
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    
    Testing notes:
    Using the test script provided on the bug report, but changed it
    to match sip user and patron existing in my database.
    
    Before applying the patch the SIP responses show the behaviour
    pointed out above regarding the 100 items. After applying the
    patch and restarting the SIP server responses are much more clean
    not returning empty IDs.
    
    64 Patron information response
    AS = hold items
      hold items count is correct.
      AS contains barcodes of waiting holds.
      Before patch, all AS were empty.
    AT = overdue items
      overdue items count is correct.
      AT contains barcodes of overdue items.
    AU = charged items
      charged items count is correct.
      AU contains barcodes of charged items.
    AV = fine items
      Judging from behaviour seen and comment in
      Patrons.pm currently not implemented.
    BU = recall items
      Recalls are not implemented in Koha yet.
    CD = unavailable hold items
      unavailable items count is correct.
      CD contains barcode for item level holds and is empty
      for title level holds where  no item can be determined.
      Before patch, all CD were empty.
    
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit 63820d3973dae7514556172665a46c0584aaefc8)
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 815a7cb6b94455d4e0a0ea88f7cd0bcb8621cd6e
Author: Fridolyn SOMERS <fridolyn.somers at biblibre.com>
Date:   Wed Jul 10 11:06:39 2013 +0200

    Bug 10564 - use OPACURLOpenInNewWindow in UNIMARCslim2OPACResults.xsl
    
    When using UNIMARC XSLT for OPAC detail page, if record has an 856$u,
    its content is displayed as a link. The syspref OPACURLOpenInNewWindow
    adds the option of opening in a new window (attribute target="_blank").
    
    856$u is also used in UNIMARC results XSLT but the use of
    OPACURLOpenInNewWindow syspref is missing. It exists for MARC21.
    
    This patch adds the use of OPACURLOpenInNewWindow in UNIMARCslim2OPACResults.xsl.
    
    Test plan :
    - In a UNIMARC catalog, enable XSLT for OPAC results page
    - Create a record with a link in 856$u and set second indicator to 0
    - Set OPACURLOpenInNewWindow syspref on
    - At OPAC, perform a search that contains the record
    => You should see the link : "Click here to access online"
    - click on it
    => You shoud get to a new window (or tab depending on your browser configuration)
    - Set OPACURLOpenInNewWindow syspref off
    - re-perform the search
    - click on the link
    => You shoud stay in same window
    
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    
    Comment: Work as described, no koha-qa errors
    Tested following plan after applying patch
    
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    XSLT change only, works as described.
    To test: Catalog 856$u and $a, 2nd indicator = 0.
    
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit bc240a908d0fbc50b099df04e61da9fbb55fab52)
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 70a6226cf7de15a74cb6f82dc14bf4568255fb95
Author: Janusz Kaczmarek <januszop at gmail.com>
Date:   Fri May 24 14:38:10 2013 +0300

    Bug 10331 - Little errors in unimarc_field_4XX.pl (punctuation)
    
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    Comment: Work as described, no koha-qa errors
    
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Copying the test plan from the bug report:
    
    A.
    1) have $a active in one of the linking fields (like 461, 463, etc.).
    2) have a biblio record with the field 700 $a, $b, $f filled up
    3) go to cataloguing -> field 4XX of your choice, use the plugin to
      fetch data from biblio rec. mentioned in 2)
    4) see the punctuation in $a -- it will be wrong (no closing bracket)
    
    B.
    1) have $a active in one of the linking fields (like 461, 463, etc.).
    2) have a biblio record with the field 700 $a, $b, $c, $f filled up
    3) go to cataloguing -> field 4XX of your choice, use the plugin to
       fetch data from biblio rec. mentioned in 2)
    4) see the punctuation in $a -- it will be wrong (dash instead of
       semicolon between original $c and $f, no closing bracket)
    
    C.
    1) have $a active in one of the linking fields (like 461, 463, etc.).
    2) have a biblio record with the field 702 $a, $b, $c filled up
    3) go to cataloguing -> field 4XX of your choice, use the plugin to
       fetch data from biblio rec. mentioned in 2)
    4) see the punctuation in $a -- it will be wrong (unnecessary semicolon
       after original $c, n.b. lacking space).
    
    D.
    1) have $a active in one of the linking fields (like 461, 463, etc.).
    2) have a biblio record with the field 712 $a, $b filled up
    3) go to cataloguing -> field 4XX of your choice, use the plugin to
       fetch data from biblio rec. mentioned in b)
    4) see the punctuation in $a -- it will be wrong (unnecessary opening
       bracket at the end).
    
    Test plan, all tests and QA script pass.
    Thx Bernardo for his testing and explanations!
    
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit 86c08e63a25afa86c9051f070d819ddac675425d)
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit b0474ebcf0a83cbbe8484b95986b350162959188
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Fri Feb 1 10:40:50 2013 -0500

    Bug 9531: Make SIP2 message terminator configurable via SIPconfig.xml
    
    Add a terminator option to SIPConfig.xml, choices for 'terminator' are
    'CR' or 'CRLF'.  The default continues to be 'CRLF' if 'terminator' is
    undefined.
    
    Test Plan:
    1) Apply patch
    2) Start SIP server
    3) Run C4/SIP/t/04patron_status.t
    4) Stop SIP server
    5) Add terminator="CR" for account login 'term1'
    6) Run 04patron_status.t again, you should see no change
    
    Signed-off-by: Frédéric Demians <f.demians at tamil.fr>
    Signed-off-by: Adrien Saurat <adrien.saurat at biblibre.com>
    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit 90e2aafeb538f2d5e52fd1d32fce528f72076df8)
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

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

Summary of changes:
 C4/SIP/ILS/Patron.pm                               |   55 ++++++++++++++++----
 C4/SIP/Sip.pm                                      |   20 +++----
 C4/SIP/Sip/MsgType.pm                              |   34 ++++++------
 cataloguing/value_builder/unimarc_field_4XX.pl     |   16 ++++--
 etc/SIPconfig.xml                                  |    2 +-
 .../prog/en/xslt/UNIMARCslim2OPACResults.xsl       |    4 ++
 6 files changed, 85 insertions(+), 46 deletions(-)


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list