[koha-commits] main Koha release repository branch master updated. v19.05.00-692-gd818799

Fri Sep 27 15:22:49 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  d8187991a40f58ab815fd1e75d6a3671ca72a668 (commit)
       via  d0a06e365b693de6609998b3d649d9ef4d083880 (commit)
       via  8171c9d05dc3adf0014a727e304f666606f7f751 (commit)
       via  0aeb5998481a28fb72c8f184bd3f85f7491dc858 (commit)
       via  64148cf4ac6f20787a809b36243672430faa6e8f (commit)
       via  4b642386d8983a12c1a79924ae1fac180b10c3d0 (commit)
       via  fa97bc90df532720287eaf296c15f7facdaa730d (commit)
       via  adb232c8f0f4bf8574ca3dce9548196430c57fb0 (commit)
       via  8766689d34c63416f38a8248367483c573c5036e (commit)
      from  148f35963d5eae1f9eabc0c7cb0b69cd7056b020 (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 d8187991a40f58ab815fd1e75d6a3671ca72a668
Author: Lucas Gass <lucas at bywatersolutions.com>
Date:   Sun Sep 22 02:01:23 2019 +0000

    Bug 23683: Course reserves public notes on specific items should allow for HTML

    Signed-off-by: Lisette Scheer <lisetteslatah at gmail.com>
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

commit d0a06e365b693de6609998b3d649d9ef4d083880
Author: Katrin Fischer <katrin.fischer.83 at web.de>
Date:   Fri Jul 12 12:40:22 2019 +0000

    Bug 23101: Hide action buttons on contracts if user lacks permission

    When the user doesn't have superlibrarian, full acq or
    contracts_manage permission, don't show buttons for editing
    and deleting contracts.

    To test:
    - Create a vendor with a few contracts
    - Create a staff user with
      - superlibrarian = can view edit/delete contracts
      - full acq perms = same
      - without manage_contracts = can view, but action buttons are gone
    - Make sure the sorting of the contracts table works in all cases

    Signed-off-by: Holly Cooper <hc at interleaf.ie>
    Signed-off-by: Bouzid Fergani <bouzid.fergani at inlibro.com>
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

commit 8171c9d05dc3adf0014a727e304f666606f7f751
Author: Tomas Cohen Arazi <tomascohen at theke.io>
Date:   Fri Sep 13 11:26:03 2019 -0300

    Bug 23607: Make /patrons/:patron_id/account staff only

    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

commit 0aeb5998481a28fb72c8f184bd3f85f7491dc858
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Tue Sep 17 15:41:12 2019 -0400

    Bug 23625: Make new 'required' attributes match the currently used syntax in other templates

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

commit 64148cf4ac6f20787a809b36243672430faa6e8f
Author: Eric Phetteplace <phette23 at gmail.com>
Date:   Mon Sep 16 12:53:32 2019 -0700

    Bug 23625: ArticleRequestsMandatoryFields* only affects field labels, does not make inputs required

    Test plan:

    1. Enable article requests ( syspref: ArticleRequests => Enable, Circ and fine rules ALL/ALL (or a given category/itemtype): Article requests => Yes )
    2. Set one or more fields to require in the ArticleRequestsMandatoryFields, ArticleRequestsMandatoryFieldsItemOnly, and/or ArticleRequestsMandatoryFieldsRecordOnly settings (all these settings are affected in the same manner)
    3. Search for a title and select the "Request Article" button from the OPAC search results
    4. Authenticate as a patron
    5. Neglect to fill out at least one of the mandatory fields
    6. Select the "Place Request" button at the bottom of the form
    7. The form submits despite the empty mandatory fields
    8. Apply patch
    9. Repeat steps 3, 5, & 6
    10 . The form should refuse to submit, show a browser-native message about the missing required fields.

    Sponsored-by: California College of the Arts
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

commit 4b642386d8983a12c1a79924ae1fac180b10c3d0
Author: Nick Clemens <nick at bywatersolutions.com>
Date:   Fri Sep 6 11:13:27 2019 +0000

    Bug 22602: Fix OverDrive circulation when coming from another site

    On bug 21078 we caught an error and returned, this makes us skip some variable setting that we need.

    This patch moves that code into its own function.

    To test:
    0 - Have OverDrive circulation setup and working
    1 - Place a link to your kohadev installation on another site
    2 - Follow that link to the opac
    3 - Note all items show as 'Place hold' even if available copies
    4 - Apply patch
    5 - Repeat 1&2
    6 - Note the items availability shows correctly

    Signed-off-by: Elizabeth Quinn <EQuinn at chplnj.org>
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

commit fa97bc90df532720287eaf296c15f7facdaa730d
Author: Nick <nick at bywatersolutions.com>
Date:   Fri Sep 20 16:44:53 2019 +0000

    Bug 23651: Add address of restricted page to syspref description

    To test:
    1 - Look at the restricted page syspref
    2 - Check that the url is listed
    3 - Add some content to the page
    4 - Save the prefs
    5 - Verify the address listed takes you to the page on the opac

    Signed-off-by: Lisette Scheer <lisetteslatah at gmail.com>
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

commit adb232c8f0f4bf8574ca3dce9548196430c57fb0
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Wed May 31 16:54:32 2017 +0200

    Bug 18710: Wrong subfield modified in batch item modification

    In Tools > Batch item modification, one can modify or delete a subfield.
    When deleting a subfield et modifying another, the wrong subfield gets modified.

    This is caused by the fact that disabled inputs are not posted in form.
    So for a subfield to delete :
    Input name=field_value is not posted. But input name=subfield is posted with subfield code.
    So these 2 arrays does not have the same size :
        my @subfields = $input->multi_param('subfield');
        my @values    = $input->multi_param('field_value');

    For exemple, deleting $2, not changing $u and modidying $v will modify $u with value for $v o_O

    This patch correts but setting disabled all inputs of a deleted subfield : field_value, tag, subfield and mandatory. Like it was not present in the form.

    Test plan :
    - Go to Tools > Batch item modification
    - Enter a barcode and submit
    - Lets say there are subfields : $2 (not mandatory), $u and $v
    - Check the subfield $2 to be deleted
    => $2 input is disabled
    - Enter a text in $v
    - Submit the form
    => Without patch, the text for $v gets into $u
    => With patch, the text for $v gets into $v
    - Check the subfield $2 is well deleted

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

commit 8766689d34c63416f38a8248367483c573c5036e
Author: Tomas Cohen Arazi <tomascohen at theke.io>
Date:   Mon Sep 16 17:22:17 2019 -0300

    Bug 23627: Reduce Koha::Biblio->get_coins noise if no 245

    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    Signed-off-by: Maryse Simard <maryse.simard at inlibro.com>
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

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

Summary of changes:
 Koha/Biblio.pm                                     |    2 +-
 api/v1/swagger/paths/patrons_account.json          |    2 --
 .../prog/en/modules/acqui/supplier.tt              |   18 ++++++++++-----
 .../prog/en/modules/admin/preferences/opac.pref    |    2 +-
 .../en/modules/course_reserves/course-details.tt   |    2 +-
 .../prog/en/modules/tools/batchMod-edit.tt         |    6 ++---
 .../bootstrap/en/modules/opac-course-details.tt    |    2 +-
 .../bootstrap/en/modules/opac-request-article.tt   |   23 +++++++++++++-------
 koha-tmpl/opac-tmpl/bootstrap/js/overdrive.js      |   17 +++++++++------
 t/db_dependent/api/v1/patrons_accounts.t           |    4 ++--
 10 files changed, 47 insertions(+), 31 deletions(-)

hooks/post-receive
-- 
main Koha release repository