[koha-commits] main Koha release repository branch 17.11.x updated. v17.11.11-37-g878627b

Fri Nov 9 16:39:40 CET 2018

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, 17.11.x has been updated
       via  878627b61e44615eb666c38d851fdd6db2410373 (commit)
       via  bc54704cd65b06639a1fbfb222b873f2d2fdd35b (commit)
       via  b5b249290efebbbe97723b356da0d7a4c308f754 (commit)
       via  c9124a794c91e58c08b3ecf909b24d7664ca8936 (commit)
      from  c2ba4a05fde5a9db976a6e871adac78f0b57f5ca (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 878627b61e44615eb666c38d851fdd6db2410373
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Thu Oct 18 08:32:59 2018 -0300

    Bug 21599: Fix item type creation by defining default values

    Same as what we have in Koha::Patron->new, empty strings should not be
    inserted in integer or date column type

    DBD::mysql::st execute failed: Incorrect decimal value: '' for column 'defaultreplacecost' at row 1 [for Statement "INSERT INTO `itemtypes` ( `checkinmsg`, `checkinmsgtype`, `defaultreplacecost`, `description`, `hideinopac`, `imageurl`, `itemtype`, `notforloan`, `processfee`, `rentalcharge`, `searchcategory`, `sip_media_type`, `summary`) VALUES ( ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ? )" with ParamValues: 0="", 1="message", 2="", 3="xx", 4=0, 5='', 6="XX", 7=0, 8="", 9="", 10="", 11=undef, 12=""] at /usr/share/perl5/DBIx/Class/Storage/DBI.pm line 1832.

    Test plan:
    Create a new itemtype

    Signed-off-by: Claire Gravely <claire.gravely at bsz-bw.de>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 33c142ba3b7cfd37dd90569de2a5802849b09ef5)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit 6c2eeaa43443e0591eebbf00f48dacfff005b647)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit bc54704cd65b06639a1fbfb222b873f2d2fdd35b
Author: Andreas Roussos <arouss1980 at gmail.com>
Date:   Sun Oct 7 21:00:09 2018 +0300

    Bug 21513: Add a 'Cancel' button to the authority editor and remove duplicate 'Save' button

    The authority editor (authorities/authorities.pl) does not have
    a 'Cancel' button. Also, it has two 'Save' buttons.

    This patch fixes that by:
    a) adding a 'Cancel' button to the floating toolbar at the top
    b) removing the 'Save' button from the bottom of the page

    Test plan:
    1) Go to Home > Authorities and launch the authority editor by
       clicking on 'New authority'. Notice how the editor doesn't
       have a 'Cancel' button. Also notice the two 'Save' buttons.
    2) Apply the patch.
    3) Re-launch the authority editor, and hit cancel; you should
       be taken back to Home > Authorities.
    4) Edit an existing authority, and hit cancel; you should be
       taken back to the details page for the authority you chose.
    5) Edit an authority as new (duplicate), and hit cancel; you
       should be taken back to Home > Authorities (this behaviour
       is mimicking the bibliographic editor: when duplicating a
       biblio the 'Cancel' button will take you back to Home >
       Cataloging).

    Signed-off-by: Michal Denar <black23 at gmail.com>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 2cffe3a574e14d5f35ce7b1d9ffe28a153dcb37e)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit 81ceef6b8410988dbf14ae0db3dfac64ae084fc9)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit b5b249290efebbbe97723b356da0d7a4c308f754
Author: Martin Renvoize <martin.renvoize at ptfs-europe.com>
Date:   Mon Oct 8 15:25:50 2018 +0100

    Bug 21456: (QA follow-up) Catch missed case in authorities merge

    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 796e4e5d2be5ec87970f8b586f940e4abddb687d)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit b9f85edd14aa4fc89b1a20457066980b4e7d4b15)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit c9124a794c91e58c08b3ecf909b24d7664ca8936
Author: Andreas Roussos <arouss1980 at gmail.com>
Date:   Sun Sep 30 13:06:28 2018 +0300

    Bug 21456: The 'New authority' button lists authority types inconsistently

    The 'New authority' button does not always list the available
    authority types in A-Z order based on their description.

    This patch fixes that.

    Test plan:
    1) Go to Home > Authorities in the staff client; notice how the
       'New authority' button lists auth types sorted by description.
    2) Go to an authority's detail page; notice how the same 'New
       authority' button will list auth types sorted by their code.
    3) Apply the patch.
    4) In an authority's detail page, the 'New authority' button will
       now list the available authority types sorted by description.

    Signed-off-by: Michal Denar <black23 at gmail.com>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit bf7c8303ca7f9e4955f25fc4d1f30d65d8c52dc9)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit 1f8356d2df19a704a5a12597a9443eb459941aad)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

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

Summary of changes:
 Koha/ItemType.pm                                           |   12 ++++++++++++
 authorities/detail.pl                                      |    2 +-
 authorities/merge.pl                                       |    2 +-
 .../prog/en/modules/authorities/authorities.tt             |   11 +++++++----
 4 files changed, 21 insertions(+), 6 deletions(-)

hooks/post-receive
-- 
main Koha release repository