[koha-commits] main Koha release repository branch master updated. v17.11.00-1069-g7a10b66

Thu Apr 5 20:01:35 CEST 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, master has been updated
       via  7a10b66205dc93d9b9773703008ebcb412b711bc (commit)
       via  754994a90b0050539174e83c065080dd10a53bbd (commit)
       via  c2fcaac302fc544ba1c3b64eec9a649e72d3d58b (commit)
       via  61d51c9b11fb10af24e134201984ea759ac4fa52 (commit)
       via  4ac8e52f61223d44dfbb5a7ad8933c0ba674ca8a (commit)
       via  0bfad6106aec007ac4bcf17d00f72b5508c4895d (commit)
       via  199c2f33d93a33bb56d422a6abdeb334fd988faf (commit)
       via  6271f6c1374ac2dad573ae6784d1632d626b14be (commit)
       via  f5fd79222dedacd13d9a06aa7ab27f07b585f1a4 (commit)
       via  328d5e7007a8abedf3f8ba679eee910e8ae41d9a (commit)
       via  373b2f747bb5d67302ae1aa6567b8b593a39d9e9 (commit)
       via  27a21083d9878f029f01fcc7256fa116ee15dc28 (commit)
      from  aaea46594215c88b4459cd882f1d869faeb2e2d8 (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 7a10b66205dc93d9b9773703008ebcb412b711bc
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Thu Apr 5 14:13:37 2018 -0300

    Bug 20530: Remove trailing ':' in columns_settings.yml

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 754994a90b0050539174e83c065080dd10a53bbd
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Wed Mar 7 15:39:49 2018 +0000

    Bug 20350: Add column configuration to table of saved reports

    This patch makes changes to the saved reports template to enable a
    columns configuration button. Settings for the table have been added to
    columns_settings.yml.

    To test, apply the patch and go to Reports -> Use saved.

     - Confirm that there is a "Column visibility" button in the
       table's pagination toolbar.
     - Confirm that showing and hiding columns via the button is working
       correctly.
     - Confirm that the first, second, and last columns cannot be toggled.

    Go to Administration -> Columns settings.

    - Expand the "Reports" section.
    - Find "id=table_reports."
    - Make some selections to configure default settings of the reports
      table.
    - Return to reports and confirm that these defaults are applied.

    Signed-off-by: Jesse Maseto <jesse at bywatersolutions.com>

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit c2fcaac302fc544ba1c3b64eec9a649e72d3d58b
Author: Julian Maurice <julian.maurice at biblibre.com>
Date:   Wed Apr 4 14:08:37 2018 +0200

    Bug 20516: Show patron's library in pending discharges table

    Test plan:
    1. Enable syspref useDischarge
    2. Login to OPAC and ask for a discharge (tab on the left)
    3. Login to intranet, you should see a message about pending discharges,
       and a link. Click on the link.
    4. Confirm that there is a 'Library' column in the table that is
       displaying the patron's library

    Signed-off-by: Mark Tompsett <mtompset at hotmail.com>

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 61d51c9b11fb10af24e134201984ea759ac4fa52
Author: Josef Moravec <josef.moravec at gmail.com>
Date:   Tue Apr 3 08:24:56 2018 +0200

    Bug 20510: Remove unused sub TotalPaid from C4::Stats

    Test plan:
    0) Do not apply the patch
    1) Run git grep TotalPaid and confirm it is only defined and then used once in a test
    2) Apply the patch
    3) git grep TotalPaid should return nothing
    4) prove t/db_dependent/Stats.t should be green

    Signed-off-by: Mark Tompsett <mtompset at hotmail.com>

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 4ac8e52f61223d44dfbb5a7ad8933c0ba674ca8a
Author: Nick Clemens <nick at bywatersolutions.com>
Date:   Mon Jun 19 19:40:39 2017 +0000

    Bug 18825: Elasticsearch - Update heading main-heading and see fields

    Signed-off-by: Ere Maijala <ere.maijala at helsinki.fi>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 0bfad6106aec007ac4bcf17d00f72b5508c4895d
Author: Alex Arnaud <alex.arnaud at gmail.com>
Date:   Wed Mar 28 09:03:56 2018 +0000

    Bug 20261: Fix pagination of the results in authority search

    Test plan:

      - apply this patch,
      - do a search that return more than 20 results,
      - click on page 2,
      - check that you get results,
      - check other pages if possible

    Signed-off-by: Nicolas Legrand <nicolas.legrand at bulac.fr>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 199c2f33d93a33bb56d422a6abdeb334fd988faf
Author: Alex Arnaud <alex.arnaud at biblibre.com>
Date:   Mon Mar 12 16:12:20 2018 +0000

    Bug 17373: Add mappings for unimarc authorities - Match index

    Test plan:

    1. apply patches

    2. switch  SearchEngine syspref to ElasticSearch

    3. load new mapping
    (admin/searchengine/elasticsearch/mappings.pl?op=reset&i_know_what_i_am_doing=1)

    4. load some unimarc authorities like the sample attached to bug 17373

    5. reindex authorities
    (./misc/search_tools/rebuild_elastic_search.pl -d -a -v)

    6. search entire notice for shalev or שלו

    7. you should find stuff

    Signed-off-by: Nicolas Legrand <nicolas.legrand at bulac.fr>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 6271f6c1374ac2dad573ae6784d1632d626b14be
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Thu Oct 13 09:45:40 2016 +0200

    Bug 17373: Elasticsearch - Authority mappings for UNIMARC - patch1

    This patch starts the UNIMARC mapping for ES.
    It is needed to test.
    More patches will come.

    Signed-off-by: Nicolas Legrand <nicolas.legrand at bulac.fr>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit f5fd79222dedacd13d9a06aa7ab27f07b585f1a4
Author: Kyle M Hall <kyle at bywatetsolutions.com>
Date:   Thu Apr 5 06:12:44 2018 -0400

    Bug 20325: Remove warning, it is no longer true

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 328d5e7007a8abedf3f8ba679eee910e8ae41d9a
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Wed Apr 4 15:36:46 2018 -0300

    Bug 20325: Do not remove accountlines between tests

    This is part of the tests, to make sure they have not been deleted

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 373b2f747bb5d67302ae1aa6567b8b593a39d9e9
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Wed Apr 4 15:19:54 2018 -0300

    Bug 20325: Move tests to a subtest

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 27a21083d9878f029f01fcc7256fa116ee15dc28
Author: Kyle M Hall <kyle at bywatetsolutions.com>
Date:   Tue Mar 27 15:16:16 2018 -0400

    Bug 20325: C4::Accounts::purge_zero_balance_fees does not check account_offsets

    purge_zero_balance_fees is used in cleanup_database.pl to determine which fees can be cleaned up.

    It uses a simple SQL query to determine which rows in accountlines need to be removed:

    463     my $sth = $dbh->prepare(
    464         q{
    465             DELETE FROM accountlines
    466             WHERE date < date_sub(curdate(), INTERVAL ? DAY)
    467               AND ( amountoutstanding = 0 or amountoutstanding IS NULL );
    468         }

    The function comes with the following warning:

    451 B<Warning:> Because fines and payments are not linked in accountlines, it is
    452 possible for a fine to be deleted without the accompanying payment,
    453 or vise versa. This won't affect the account balance, but might be
    454 confusing to staff.

    This was a reasonable solution prior to the addition of account_offsets in 17.11. The problem now is that rows in accountlines which are linked as credits in accountlines will *always* have amountoutstanding marked as 0. These are linked to debits via account_offsets. purge_zero_balance_fees will delete credits and leave rows in account_offsets which link to deleted credits.

    Sites using the --fees option cleanup_database.pl which upgrade to 17.11 may have all of their credits removed without warning.

    Test Plan:
    1) Apply this patch
    2) prove t/db_dependent/Accounts.t

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

    Signed-off-by: Josef Moravec <josef.moravec at gmail.com>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

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

Summary of changes:
 C4/Accounts.pm                                     |   15 +-
 C4/Stats.pm                                        |   46 --
 admin/columns_settings.yml                         |   45 +-
 admin/searchengine/elasticsearch/mappings.yaml     |  625 +++++++++++++++++++-
 authorities/authorities-home.pl                    |   15 +-
 .../prog/en/modules/members/discharges.tt          |    2 +
 .../en/modules/reports/guided_reports_start.tt     |   13 +-
 misc/cronjobs/cleanup_database.pl                  |    3 -
 t/db_dependent/Accounts.t                          |   49 +-
 t/db_dependent/Stats.t                             |   11 +-
 10 files changed, 738 insertions(+), 86 deletions(-)

hooks/post-receive
-- 
main Koha release repository