[koha-commits] main Koha release repository branch master updated. v3.18.00-12-gab2d2d2

Wed Dec 3 15:48:13 CET 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  ab2d2d2590fe4adf4adf6015f0ef441da8ad52a3 (commit)
       via  4179c63d7ea1a087b725b0a8da98537aa1d61949 (commit)
       via  66438d59e7268a06278ed29a5304a7a31cd0e67f (commit)
       via  fd38d8c565ce3efe21b2633f67f7349db28faddd (commit)
       via  46e3f8169c026ed9037f38669c5742e912326339 (commit)
       via  b449b08d91c536c3ec8a4b87e6c14d460f59878a (commit)
       via  d311722445f5d887e1054be2d1c99128f3b8207d (commit)
      from  a8c975d951694a80b73cd1f66e9a9095db2f484d (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 ab2d2d2590fe4adf4adf6015f0ef441da8ad52a3
Author: Liz Rea <liz at catalyst.net.nz>
Date:   Tue Nov 25 12:03:50 2014 +1300

    Bug 12123 - [Alternative patch] HTML notices can break the notice viewer

    Depending on the content of an html notice, it can cause the notice to
    either not be collapsible, be uncollapsible, or to be permanently
    collapsed.

    Test Plan:
    1) Set your CHECKOUT notice to the following ( with HTML Message checked ) :
    <p>The following items have been checked out:</p>

    ----
    <blockquote> <<biblio.title>> </blockquote>
    ----

    <p>Thank you for visiting the <<branches.branchname>> of HMCPL.</p>
    2) Check out some items to a patron
    3) View the patrn's notices
    4) Note the notice viewer is broken ( message is not collapsed, and
       con't be collapsed ).
    5) Apply this patch
    6) Reload the page
    7) Note the notice viewer is no longer broken

    Signed-off-by: Owen Leonard <oleonard at myacpl.org>

    This patch appears to fix the problem with the minimum required change.

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

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

commit 4179c63d7ea1a087b725b0a8da98537aa1d61949
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Tue Sep 23 12:39:55 2014 +0200

    Bug 12852: C4::Bookseller should not be imported anymore

    This script does not depend on C4::Bookseller anymore. The import can be
    removed.

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

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 66438d59e7268a06278ed29a5304a7a31cd0e67f
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Tue Sep 9 15:22:35 2014 +0200

    Bug 12852: Some unused stuffs more

    These parameters are never used.
    GetBookSeller takes a string (bookseller name) in parameter, not an id!

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

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit fd38d8c565ce3efe21b2633f67f7349db28faddd
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Fri Aug 29 15:16:19 2014 +0200

    Bug 12852: The "preview" param can be removed serials/claims.pl

    This case (preview=1) never appened. This patch remove all occurrences
    in the pl and the tt files.

    Test plan:
    Verify you don't manage to find a place where preview is set to 1 on the
    claim serials page.

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

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 46e3f8169c026ed9037f38669c5742e912326339
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Tue Sep 23 13:02:51 2014 +0200

    Bug 12980: GetHistory does useless processing

    GetHistory iterated on the orders to calculate the quantity and price.
    These values are never used by the called.
    It can be removed.

    Test plan:
    Verify there is no regression on acqui/histsearch.pl and
    catalogue/detail.pl
    Actually you just have to check that the total quantity and price are
    not displayed on these views.

    QA: note that 'count' and 'toggle' are never used in the template.

    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 b449b08d91c536c3ec8a4b87e6c14d460f59878a
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Tue Aug 26 10:50:22 2014 +0200

    Bug 12841: aqorders fields should not be mapped

    Bug 5336 introduced some code which should have been introduced by bug
    7294.
    Since the idea behind bug 7294 has been abandoned (map the aqorders
    fields), the code can be removed.

    Test plan:
    Verify that Koha does not allow you to map the aqorders fields with a
    MARC subfield.
    Verify there is no regression on adding/updating an order.

    Signed-off-by: Zeno Tajoli <z.tajoli at cineca.it>

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

commit d311722445f5d887e1054be2d1c99128f3b8207d
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Tue Aug 26 12:46:52 2014 +0200

    Bug 12839: aqbooksellers.gstreg is never used

    The aqbooksellers.gstreg is never used in the code.
    This patch does not remove the DB field but 3 useless occurrences in the
    neworderempty page.

    The both variable applygst and gstreg have never been took into account for prices calculation.

    Test plan:
    Verify there is no difference before and after the patch in the prices
    calculation.

    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>

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

Summary of changes:
 C4/Acquisition.pm                                  |   19 +----
 acqui/addorder.pl                                  |    7 --
 acqui/histsearch.pl                                |    7 +-
 acqui/neworderempty.pl                             |    1 -
 catalogue/detail.pl                                |    2 +-
 koha-tmpl/intranet-tmpl/prog/en/js/acq.js          |    1 -
 .../prog/en/modules/acqui/neworderempty.tt         |    1 -
 .../prog/en/modules/members/notices.tt             |    4 +-
 .../prog/en/modules/serials/claims.tt              |   79 --------------------
 serials/claims.pl                                  |   49 +++++-------
 t/db_dependent/Acquisition.t                       |    4 +-
 11 files changed, 28 insertions(+), 146 deletions(-)

hooks/post-receive
-- 
main Koha release repository