[koha-commits] main Koha release repository branch master updated. v19.05.00-166-g4dc64b4

Mon Jun 24 16:11:24 CEST 2019

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  4dc64b4b124abf4bd2437004879e083972e20492 (commit)
       via  0df3fc3dae575ceb16b85de6c201f5e82d01059c (commit)
       via  286f151009f778d6d4d0890b1a8bead1a6e62bde (commit)
       via  ed7b3760965ec5a52783efe64bcba13b45b3725a (commit)
       via  66975364b672ea5f362d49f6e5fca8c9a0e8326c (commit)
       via  48e5713e6ff78d7aad7eff9e239ec8779d62655c (commit)
       via  f1bb6178e5bfe990da747c9d5b0e7a4f04148a55 (commit)
       via  c3ced5d89320f85774d945957b2b4e68d0336016 (commit)
       via  efc21e6f47801503e55e3ba709a48c3f426f90ac (commit)
       via  49275f27d43990746b4af5c6335d4fe7f38de7b4 (commit)
       via  7c5e7ca9c47ac6c8a8b7d89da46e7354d8fea69d (commit)
      from  d08e776cf1e25ffa407bb3a5d668c1b140be6ec2 (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 4dc64b4b124abf4bd2437004879e083972e20492
Author: Martin Renvoize <martin.renvoize at ptfs-europe.com>
Date:   Mon Jun 24 14:48:27 2019 +0100

    Bug 23034: (RM follow-up) Consistent coding style

    We had a mix of q||, q{} and '' in this one script. This followup just
    makes them all conform to our standard style of q{}.

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

commit 0df3fc3dae575ceb16b85de6c201f5e82d01059c
Author: Joonas Kylmälä <joonas.kylmala at helsinki.fi>
Date:   Mon Jun 3 14:58:34 2019 +0000

    Bug 23034: Remove uninitialized value warning in Mana KB settings

    This gets rid of the warning "Use of uninitialized value $auto_share
    in string ne at [...]" by initializing the variable with empty string.

    Test plan:
    1) Go to page cgi-bin/koha/admin/share_content.pl and unselect "Auto
       subscription sharing". Click "Save".
    2) Check plack-error.log and notice the warning "Use of uninitialized
       value $auto_share [...]"
    3) Apply the patch and repeat the previous steps but now this time the
       warning is gone in plack-error.log.
    4) Make sure the AutoShareWithMana syspref with and without the patch
       is in unselected state after clicking "Save".

    Sponsored-by: The National Library of Finland
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

commit 286f151009f778d6d4d0890b1a8bead1a6e62bde
Author: Katrin Fischer <katrin.fischer.83 at web.de>
Date:   Mon Jun 17 23:20:41 2019 +0200

    Bug 23098: Reword success message of KOC upload process

    When uploading KOC files the processing of transactions needs
    to happen in a separate second step. This is why the success
    message "Your file was processed" was found to be misleading.

    This patch changes it to the wording suggested by Benjamin
    Daeuber in the original bug report. Thx!

    To test:
    - Create a KOC file either manually or by downloading it
      from the plugin
    - Upload the KOC file into Koha using Circulation >
      Upload offline circulation file (.koc)
    - Verify the new message is clear and has no typos :)

    Signed-off-by: Mark Tompsett <mtompset at hotmail.com>
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

commit ed7b3760965ec5a52783efe64bcba13b45b3725a
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Tue Jun 11 15:31:45 2019 +0200

    Bug 22944: remove obsolete unit test

    Test 'AnonymiseIssueHistory should not return any error if success' is obsolete,
    previous statement will explode if something went wrong.

    Test plan :
    Run t/db_dependent/Koha/Patrons.t

    Signed-off-by: Arthur Bousquet <arthur.bousquet at inlibro.com>
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

commit 66975364b672ea5f362d49f6e5fca8c9a0e8326c
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Tue May 21 11:55:40 2019 +0200

    Bug 22944: avoid AnonymousPatron in search_patrons_to_anonymise

    In Koha::Patrons::search_patrons_to_anonymise() old issues already affected to the anonymous patron should be avoided. It is useless and can impact performance.

    It is already working if AnymousPatron is not defined or 0 because of :
      'old_issues.borrowernumber' => { 'not' => undef }

    In theory the anymous patron should have privacy=0 but in case its not it should be explicitly avoided.

    Test plan :
    Run t/db_dependent/Koha/Patrons.t

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

commit 48e5713e6ff78d7aad7eff9e239ec8779d62655c
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Tue May 21 11:17:11 2019 +0200

    Bug 22944: change unit tests

    Add a dedicated UT and correct a test return value

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

commit f1bb6178e5bfe990da747c9d5b0e7a4f04148a55
Author: Martin Renvoize <martin.renvoize at ptfs-europe.com>
Date:   Mon Jun 24 14:24:26 2019 +0100

    Bug 23115: Compiled CSS

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

commit c3ced5d89320f85774d945957b2b4e68d0336016
Author: Nick Clemens <nick at bywatersolutions.com>
Date:   Fri Jun 21 16:20:28 2019 +0000

    Bug 23115: (QA follow-up) Address QA tools complaints

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

commit efc21e6f47801503e55e3ba709a48c3f426f90ac
Author: Nick Clemens <nick at bywatersolutions.com>
Date:   Fri Jun 21 16:18:15 2019 +0000

    Bug 23115: (follow-up) Move switches to a TT block

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

commit 49275f27d43990746b4af5c6335d4fe7f38de7b4
Author: Martin Renvoize <martin.renvoize at ptfs-europe.com>
Date:   Thu Jun 13 11:42:02 2019 +0100

    Bug 23115: Tweak display of totals on 'Fines and charges' page

    This patch tweaks the display of totals on the OPAC 'Fines and charges'
    page to clarify between the totals before and after applying credits.

    Test Plan:
    1) Add some debts and credits to a patrons account
    2) Look at the 'Fines and charges' page before applying the patch
    3) Look at the 'Fines and charges' page after applying the patch
    4) Varify the new layout is an improvment or not

    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    Signed-off-by: Nadine Pierre <nadine.pierre at inLibro.com>
    Signed-off-by: Maryse Simard <maryse.simard at inlibro.com>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

commit 7c5e7ca9c47ac6c8a8b7d89da46e7354d8fea69d
Author: Martin Renvoize <martin.renvoize at ptfs-europe.com>
Date:   Thu Jun 13 10:02:35 2019 +0100

    Bug 23106: Tweak display of totals on payment page

    This patch tweaks the display of totals on the fines payments page to
    clarify between the totals before and after applying credits.

    Test Plan:
    1) Add some debts and credits to a patrons account
    2) Look at the pay page before applying the patch
    3) Look at the pay page after applying the patch
    4) Verify the new layout is an improvment or not

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

    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    Signed-off-by: Maryse Simard <maryse.simard at inlibro.com>
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

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

Summary of changes:
 Koha/Patrons.pm                                    |    2 +
 admin/share_content.pl                             |    6 +-
 .../intranet-tmpl/prog/en/modules/members/pay.tt   |   16 ++--
 .../prog/en/modules/offline_circ/enqueue_koc.tt    |    2 +-
 koha-tmpl/opac-tmpl/bootstrap/css/opac.css         |    2 +-
 koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss    |    8 +-
 .../bootstrap/en/includes/account-table.inc        |   79 +++++++++++++-------
 opac/opac-account.pl                               |   20 ++---
 t/db_dependent/Koha/Patrons.t                      |    9 ++-
 9 files changed, 91 insertions(+), 53 deletions(-)

hooks/post-receive
-- 
main Koha release repository