[koha-commits] main Koha release repository branch master updated. v3.16.00-622-g81b9d8b

Thu Sep 18 15:09:10 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, master has been updated
       via  81b9d8b8aac5c67d01dbac64a1dfa9de880e021d (commit)
       via  3c1c5c04ceafb411dde1426967cb969a3969f80a (commit)
       via  45f474abdf935c928b4cd20e9823c96ca82af398 (commit)
       via  3ce2c0235ca8a1dc0d891e17cac5faeb6bea7caa (commit)
       via  bf26d347c17907878f31e4d37d9204c96b44f14b (commit)
       via  2aded36d08fab552bab112fcec74247c68d45d22 (commit)
      from  a414f413c82c71ce6d14aa1f416ec6fdf0300f13 (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 81b9d8b8aac5c67d01dbac64a1dfa9de880e021d
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Thu Sep 18 09:42:02 2014 +0200

    Bug 12953: FIX NewOrder test

    The patch for bug 12827 has been written some days before bug 10402. One
    occurrence of NewOrder has not been modified.

    Test plan:
      prove t/db_dependent/Letters.t

    should return green.

    Signed-off-by: Paola Rossi <paola.rossi at cineca.it>

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 3c1c5c04ceafb411dde1426967cb969a3969f80a
Author: Katrin Fischer <Katrin.Fischer.83 at web.de>
Date:   Mon Aug 18 11:02:01 2014 +0200

    Bug 8868: Follow-up: Update boostrap documentation page

    Updates the bootstrap documentation page for CancelHold.

commit 45f474abdf935c928b4cd20e9823c96ca82af398
Author: Julian Maurice <julian.maurice at biblibre.com>
Date:   Wed Oct 3 13:23:04 2012 +0200

    Bug 8868: ILS-DI: CancelHold needs to take a reserve_id

    CancelHold takes two parameters: patron_id and item_id.
    If item_id is considered as an itemnumber, holds on title can't be
    canceled.
    If item_id is considered as a biblionumber, all holds on this
    biblionumber (for a borrower) will be canceled.

    So CancelHold have to consider item_id as a reserve_id.

    - Added subroutine C4::Reserves::GetReserve
    - C4::ILSDI::Services::GetRecords now returns the reserve_id
    - Fix the text in the ilsdi.pl?service=Describe&verb=CancelHold page
    - Unit tests for CancelReserved and GetReserve
    - Do not delete row in reserves table if insert in old_reserves fails

    Signed-off-by: Leila and Sonia <koha.aixmarseille at gmail.com>
    Signed-off-by: Benjamin Rokseth <bensinober at gmail.com>
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

    Signing off, while noting a style issue in the patch review

    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Passes tests and QA script.
    Placed and cancelled a hold using ILS-DI successfully.
    Adding a follow-up to also update the ils-di documentation
    page in the bootstrap theme.

    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

    EDIT: I removed the changes it did to the prog theme.

commit 3ce2c0235ca8a1dc0d891e17cac5faeb6bea7caa
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Wed Jun 11 12:25:19 2014 +0200

    Bug 12405 - Search links on callnumber fails on intranet results page

    On intranet results page, the callnumber of items is a search like :
    /cgi-bin/koha/catalogue/search.pl?idx=callnum&q=[% result.itemcallnumber |url %]

    The callnumber should be URI-escaped to avoid special URI characters like & , ? ...
    If the callnumber contains some CCL words or parenthesis, the search will fail, it should be wrapped with double-quotes.

    This patch adds this to catalogue/results.pl and catalogue/shelves.pl :
    - uri TT filter instead of url
    - adds double-quotes using there URI code %22 since its in a HTML attribute using double-quotes

    Test plan :
    - Edit an item callnumber with : & ABC 123
    - Index zebraqueue
    - Perform a search returning this item
    - Click on the callnumber
    => Without this patch you get no result, because URL parameters are : idx=callnum&q=& ABC 123
    => With this patch you get results
    - Set syspref QueryWeightFields off (because this is no bug if on)
    - Edit an item callnumber with : AB(C) AND OR
    - Index zebraqueue
    - Perform a search returning this item
    - Click on the callnumber
    => Without this patch you get no result, because the search contains CCL words "OR" and "AND"
    => With this patch you get results

    Signed-off-by: Liz Rea <liz at catalyst.net.nz>
    Tested all with and without queryweightfields -
    all searches from clicked call numbers for given callnumbers fail without the patch, all are successful with the patch.

    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Works as described - no problems found.
    Passes tests and QA script.

    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit bf26d347c17907878f31e4d37d9204c96b44f14b
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Fri Aug 15 09:07:07 2014 -0400

    Bug 12763 [QA Followup] - Remove tabs and trailing space

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 2aded36d08fab552bab112fcec74247c68d45d22
Author: Marc Véron <veron at veron.ch>
Date:   Fri Aug 1 20:30:04 2014 +0200

    Bug 12763: Add more information for users to OPAC Error 404 page

    This patch adds more information to the OPAC 404 error page to make it more user friendly.

    To test:

    - Apply patch
    - Is the information clearly understandable and does it contain all necessary information?
    - Native speakers: Is the text correct?
    - Non native speakers: Is the text easy to translate?
    - Is the HTML correct?

    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

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

Summary of changes:
 C4/ILSDI/Services.pm                               |   30 ++++--------
 C4/Reserves.pm                                     |   51 +++++++++++---------
 .../prog/en/modules/catalogue/results.tt           |    8 +--
 .../prog/en/modules/virtualshelves/shelves.tt      |    2 +-
 .../opac-tmpl/bootstrap/en/modules/errors/404.tt   |   22 ++++++---
 koha-tmpl/opac-tmpl/bootstrap/en/modules/ilsdi.tt  |    2 +-
 t/db_dependent/Letters.t                           |    2 +-
 t/db_dependent/Reserves.t                          |   23 ++++++++-
 8 files changed, 80 insertions(+), 60 deletions(-)

hooks/post-receive
-- 
main Koha release repository