[koha-commits] main Koha release repository branch 17.05.x updated. v17.05.10-23-gee58496

Tue Apr 24 17:24:13 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, 17.05.x has been updated
       via  ee5849653f5cedb0f041c86b0e569a599db45d80 (commit)
       via  d952f0915e933ca1f8651ceb9b5f90ea73694ed6 (commit)
       via  acef328d1840a7ea1f3b3b59b110039dbb85a869 (commit)
       via  dcb5961175dd6801a64083f11fb4c001a760f203 (commit)
       via  c678eccdb6844ed8d8e5a15ff263dfff2827f3fe (commit)
       via  55bb8b0c744ce120f9e59b7099759e843c3f77e1 (commit)
       via  ffb17dd0a694c3949315011bd34e8333eae6711b (commit)
       via  77c752d393c6f2db4eaa80dee6245525140b6f0f (commit)
       via  a92494aec3c0c50e88e4b7fde39dd1b2e6b36744 (commit)
       via  72b22d6dba025b819b2be7412a4c2fda9a67cc71 (commit)
       via  12dd65fda95fbf40f2d58c0a79197b1435b50430 (commit)
      from  38d62cec1d25efd6503305019e92a099bbb1d0f5 (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 ee5849653f5cedb0f041c86b0e569a599db45d80
Author: Victor Grousset <victor.grousset at biblibre.com>
Date:   Thu Mar 22 15:44:00 2018 +0100

    Bug 20461: Serials: fix ignored params in subscription creation form

    In subscription-add.pl, the two params aren't passed to
    NewSubscription() but they are to ModSubscription()

    == Test plan ==
    1. Enable syspref "makePreviousSerialAvailable"
    2. Create a minimal subscription with a value in the "Item type" and
         "item type for older issues" fields.
    3. Edit the subscription
    4. You should see that the two fields are empty. This is the bug.
    5. Apply this patch
    6. Do step 2 and 3
    7. You should see that the two fields have the right value
    8. Express the joy of a successful and easy sign off.
       (this is important, otherwise the sign off spell won't work!)

    Signed-off-by: delaye <stephane.delaye at biblibre.com>
    Signed-off-by: Séverine QUEUNE <severine.queune at bulac.fr>

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

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

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit b02528d8bb7e8e86d0157166e0d2868e64d19d51)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit d952f0915e933ca1f8651ceb9b5f90ea73694ed6
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Tue Jan 2 15:42:12 2018 -0300

    Bug 18593: Move suggestions when bibliographic records are merged

    When bibliographic records are merged, the suggestions.biblionumber
    field should be replaced with the new record.
    Ideally we should have kept an history of the merges, to know what was
    the original record. Now we cannot fix the broken links.

    Test plan:
    1) Create a suggestion in OPAC
    2) Accept the suggestion in staff
    3) Add order from this suggestion
    4) Go to cataloguing search and search for the record and another
    5) Merge the suggestion record with the catalog record - catalog record wins (should be the more common case when a patron suggests something that already exists)
    6) Verify that after merging, the new title displays in the acquisition record
    7) Verify that the suggestion info no longer displays in basket

    From Katrin's test plan:
    8) Verify that the new title doesn't display in suggestions. The old title will still show as suggested (not updated)
    => This is the title from suggestions.title, I think it makes sense to show this one.
    Maybe we should open a new bug report to improve the display of the suggestion, and link to the bibliographic record

    Signed-off-by: Lucie Gay <lucie.gay at ens-paris-saclay.fr>

    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>

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

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 3f118604af4ff965a1a320656e40762f9958e791)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit acef328d1840a7ea1f3b3b59b110039dbb85a869
Author: Julian Maurice <julian.maurice at biblibre.com>
Date:   Mon Mar 12 11:51:59 2018 +0100

    Bug 11936: (QA follow-up) Consistent log message for item insert

    Fix the same error in another place

    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>

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

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 174dc4cb2e1de70b310aea2b812ce959ad5ab174)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit dcb5961175dd6801a64083f11fb4c001a760f203
Author: Magnus Enger <magnus at libriotech.no>
Date:   Thu Feb 22 10:37:08 2018 +0100

    Bug 11936: Consistent log message for item insert

    If you use bulkmarcimport.pl to import records with items it looks
    like the successfull insert of the record is reported multiple time,
    but the second and subsequent "ok" is really related to importing
    the item(s).

    This patch changes the log message on successfully inserting an item
    to match the log message given when inserting an item fails.

    To test, the easy way:
    - Look at lines 530 and 536 of bulkmarcimport.pl, and note that the
      "op" in those two lines are different
    - Apply the patch
    - Look at lines 530 and 536 again, and note that the "op" is now
      identical, and that this makes sense, since they are both related
      to the same operation, specifically inserting an item

    To test, the hard way
    - Have some records with items
    - Import the records with bulkmarcimport.pl, and make sure to specify
      the -l option, to create a log of the actions taken
    - Look at the log and verify it looks something like this:
      id;operation;status
      1;insert;ok
      1;insert;ok
      2;insert;ok
      2;insert;ok
    - Apply this patch and import some more records with items. The log
      should now be similar to this:
      id;operation;status
      1;insert;ok
      1;insertitem;ok
      2;insert;ok
      2;insertitem;ok

    Signed-off-by: Maksim Sen <maksim.sen at inlibro.com>
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>

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

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit aa7954907b94487f17d13573c19d9fbd7c6677b2)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit c678eccdb6844ed8d8e5a15ff263dfff2827f3fe
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Tue Mar 13 10:15:31 2018 +0000

    Bug 20383: Hide link to plugin management if plugins are not enabled

    This patch adds a check in admin-home.pl to see whether plugins are
    enabled. If plugins are disabled, the link to plugins management is
    hidden even if the user has plugin management permission.

    To test, view the administration home page with plugins enabled and
    disabled via the enable_plugins flag in the Koha configuration file.
    Also test using users with different plugins permissions:

    - Plugins enabled
      - CAN_user_plugins = 1
        -> Plugins link appears
      - CAN_user_plugins = 0
        -> Plugins link hidden

    - Plugins disabled
      - CAN_user_plugins = 1,
      - CAN_user_plugins = 0
        -> Plugins link hidden

    Signed-off-by: Séverine QUEUNE <severine.queune at bulac.fr>

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

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

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 8450d2020bdfd524ebd20f0de8d278497fce0759)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit 55bb8b0c744ce120f9e59b7099759e843c3f77e1
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Mon Mar 12 12:56:08 2018 +0000

    Bug 20372: Correct toolbar markup on some pages

    Differences in markup around some toolbars has the potential to create
    CSS styling problems. This patch changes the markup to be consistent
    with other instances of <div id="toolbar"></div>

    To test, apply the patch and view these pages:

    - Course reserves (main page)
    - Course reserves -> Course detail page
    - Reports -> Dictionary

    On each of these pages the toolbar should look correct.

    Signed-off-by: Claire Gravely <claire.gravely at bsz-bw.de>

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

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

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit c9ee7c8d3b34da110702cfd93e16b11350d19a6d)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit ffb17dd0a694c3949315011bd34e8333eae6711b
Author: Maksim Sen <maksim.sen at inlibro.com>
Date:   Mon Mar 12 16:15:30 2018 -0400

    Bug 20376: Make select/unselect all ignore disabled checkboxes on batch record deletion

    Test Plan:

    0.1 - find at least one biblionumber that has at least one item checked out
    0.2 - find at least one biblionumber that has zero items checked out.
    1 - Go into the module "Tools > Batch record deletion"
    2 - in "List of biblionumbers or authority ids (one per line)" put the values you found in step 0.1 and 0.2
    3 - click continue
    4 - biblionumbers with at least one item checked out should not be clickable.
    5 - biblionumbers with zero items checked out should be clickable.
    6 - click "Select all" and "Clear all", both buttons should change the "checked" status of the enabled AND disabled biblionumber checkboxes.
    7 - apply patch
    8 - click "Select all" and "Clear all", both buttons should change the "checked" status of ONLY the enabled biblionumber checkboxes.

    Signed-off-by: JM Broust <jean-manuel.broust at univ-lyon2.fr>

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

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

    Conflicts:
    	koha-tmpl/intranet-tmpl/prog/en/modules/tools/batch_delete_records.tt

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 336e7faaf451ff29d43a4a557861fbc8511c8e5e)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit 77c752d393c6f2db4eaa80dee6245525140b6f0f
Author: Mark Tompsett <mtompset at hotmail.com>
Date:   Tue Oct 31 02:28:58 2017 +0000

    Bug 19547: Deal with --create-db missing debian.cnf issue

    Followed test plan. Patch functions as described and both commands execute.
    Signed-off-by: Dilan Johnpullé <dilan at calyx.net.au>

    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>

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

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 931db23ee369580bbaf137547accef5a8f47412e)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit a92494aec3c0c50e88e4b7fde39dd1b2e6b36744
Author: Mark Tompsett <mtompset at hotmail.com>
Date:   Mon Oct 30 17:59:43 2017 -0400

    Bug 19547: MariaDB install has no debian.cnf

    This attempts to handle issues arising when running
    koha-create on a system that never had MySQL installed.

    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>

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

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit a85d33314cb48a493bbc26f5350911dc5c6b2c51)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit 72b22d6dba025b819b2be7412a4c2fda9a67cc71
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Fri Mar 16 09:02:36 2018 +0100

    Bug 20430: (See also bug 19436) Button display depends on wrong server count

    This is a QA follow-up of bug 19436, submitted separately for ease of
    backporting it. (The bug was not raised by 19436.)

    Obviously, the number of authority servers not biblio servers should
    be the key factor for showing or hiding the button for new authority
    records via Z39.50/SRU.

    Trivial fix.

    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>

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

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 7852d967d1982de2c19571b8733ea8273458882b)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit 12dd65fda95fbf40f2d58c0a79197b1435b50430
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Tue Mar 6 12:26:19 2018 +0100

    Bug 20341: Show authorized value description for withdrawn like damaged and lost

    Actually catalog detail page shows for damaged and lost items the authorized value description if exists.
    We must do the same for withdrawn information.

    Test plan :
    1) Set for items a subfield linked to items.withdrawn and using the authorized value category 'WITHDRAWN'
    2) Edit the authorized value category to add value '2' and description 'Eaten by dog'
    3) Edit an item to set this withdrawn value
    4) Go to staff interface and look at items in record detail page /cgi-bin/koh/catalogue/detail.pl :
    => Without patch you see 'Withdrawn' and with patch you see 'Eaten by dog'
    5) Go to OPAc and look at items in record detail page /cgi-bin/koha/opac-detail.pl :
    => Without patch you see 'Withdrawn' and with patch you see 'Eaten by dog'

    Signed-off-by: delaye <stephane.delaye at biblibre.com>

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

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

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 830ce35795f7bbd68d6b68c4795cb36f27fe2b6d)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

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

Summary of changes:
 admin/admin-home.pl                                      |    6 ++++++
 authorities/authorities-home.pl                          |    2 +-
 catalogue/detail.pl                                      |    4 ++++
 cataloguing/merge.pl                                     |    8 +++++++-
 debian/scripts/koha-create                               |   13 ++++++++++++-
 .../intranet-tmpl/prog/en/modules/admin/admin-home.tt    |    2 +-
 .../intranet-tmpl/prog/en/modules/catalogue/detail.tt    |   10 +++++++++-
 .../prog/en/modules/course_reserves/course-details.tt    |   14 +++++++++-----
 .../prog/en/modules/course_reserves/course-reserves.tt   |    8 +++-----
 .../intranet-tmpl/prog/en/modules/reports/dictionary.tt  |    4 +---
 .../prog/en/modules/tools/batch_delete_records.tt        |    4 ++--
 .../opac-tmpl/bootstrap/en/includes/item-status.inc      |    7 ++++++-
 misc/migration_tools/bulkmarcimport.pl                   |    4 ++--
 serials/subscription-add.pl                              |    2 +-
 14 files changed, 64 insertions(+), 24 deletions(-)

hooks/post-receive
-- 
main Koha release repository