[koha-commits] main Koha release repository branch master updated. v19.05.00-378-gd7e6ae3

Fri Jul 19 15:19:37 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  d7e6ae38e431c4d6397c057b0696e03f5a67ce14 (commit)
       via  bc188a9a54e28bd4fa2568dab5e85cf296fa87ab (commit)
       via  ea0127b7fa53335f3e9d3f0170fbc92b2ffd9e3c (commit)
       via  84855d75c1ddc770bf33b952b8f319ad1655a869 (commit)
       via  3cfd78788fe1bda05894a08c22f69ce1baa9482c (commit)
      from  29ecb5b6c29e48cb6964029567bd2b7cc8b6e7ac (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 d7e6ae38e431c4d6397c057b0696e03f5a67ce14
Author: Katrin Fischer <katrin.fischer.83 at web.de>
Date:   Sat Jul 13 14:23:19 2019 +0200

    Bug 22021: Improve item status display when placing holds in staff

    This patch is a bit of a clean-up to bring the item status display
    more in line with the display on the detail page:

    - show descrpition of authorised value for not for loan instead of hardcoded text
    - Show description of authorised value for lost instead of hardcoded text
      This is also a translatability fix, as the text came from the .pl
    - Show description of authorised value damaged instead of hardcoded text
    - Make sequence of status match display on details page:
      lost - damaged - not for loan

    To test:
    - On a record with multiple items
      - Add different status to the items
        damaged, lost, not for loan
      - Make sure you have items with one status and multiple status at the same time
    - Look at how the status display on the detail page
    - Place a hold, compare display
    - Apply patch
    - Repeat

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

commit bc188a9a54e28bd4fa2568dab5e85cf296fa87ab
Author: Nick Clemens <nick at bywatersolutions.com>
Date:   Fri Jul 19 10:49:00 2019 +0000

    Bug 23218: (QA follow-up) Remove dummy value

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

commit ea0127b7fa53335f3e9d3f0170fbc92b2ffd9e3c
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Sat Jun 29 10:18:29 2019 -0500

    Bug 23218: Fix shift in patron's attributes in batch patron modification

    When the "new" button is clicked without selecting a patron's attribute, the update will have unexpected behaviors.

    There is indeed a difference in the number of patron's attributes and the number of values.

    Test plan:
    Go to the batch patron modification
    Enter a cardnumber
    At the bottom of the page, click "new" without selecting an attribute
    On the second line select one, and a value (different than an empty
    string)
    Save
    => The attribute will be updated with an empty string

    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>

commit 84855d75c1ddc770bf33b952b8f319ad1655a869
Author: Pasi Kallinen <pasi.kallinen at koha-suomi.fi>
Date:   Thu Jun 27 14:03:02 2019 +0300

    Bug 23048: Hide non-pickup branches from hold modification select

    When modifying holds in staff client via reserve/request.pl or
    the holds-tab in members/moremember.pl, the pickup library dropdown
    lists libraries not marked for pickup. Hide those branches,
    unless the branch is marked as a pick-up branch for already
    existing hold.

    The reason for showing a non-pickup branch for the holds created
    before the branch was marked as non-pickup: If the non-pickup branch
    wasn't shown, the branch in those holds would get set to the first
    branch in the select if the hold was modified.

    Test plan:
    1) Create a hold for a patron, to be picked up from a certain branch.
    2) Edit the branch and another branch so they are no longer
       pickup locations.
    3) Go to members/moremember.pl for that patron, check the Holds-tab
       Pickup library -drop down. It lists both of the non-pickup branches.
    4) Go to reserve/request.pl for that biblio. The hold-table
       Pickup library -drop down lists both of the non-pickup branches.
    5) Apply patch.
    6) Repeat 3 & 4, drop down no longer lists the non-pickup branch, but
       should still list the branch where the hold was marked to be picked
       from.

    Signed-off-by: Pasi Kallinen <pasi.kallinen at koha-suomi.fi>
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

commit 3cfd78788fe1bda05894a08c22f69ce1baa9482c
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Tue Jul 16 18:05:23 2019 +0000

    Bug 23145: Confirming transfer during checkin clears the table of previously checked-in items

    This patch corrects the checkin template so that checking in an item
    with an existing transfer does not clear the page of previous checkouts.

    To test, apply the patch and check in some items.

     - After each checkin the returned item should appear in the table of
       checkouts.
     - Check in an item which belongs to another branch. You will be asked
       to confirm that the items requires a transfer. When the page reloads
       you should see the same previous checkins in the table.
     - Check in that same item again. You will be again asked to confirm.

       Before this page, confirming the transfer would reload the page and
       clear the list of checkins. Now the page should reload and the table
       of checkins should still show the previously checked-in items.

    Test all the actions when checking in an item which is already in
    transit: OK, Print transfer slip, and Cancel transfer.

    Signed-off-by: George Williams <george at nekls.org>
    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:
 .../intranet-tmpl/prog/en/includes/holds_table.inc |    2 +-
 .../intranet-tmpl/prog/en/modules/circ/returns.tt  |   63 +++++++++++------
 .../prog/en/modules/reserve/request.tt             |   71 ++++++++++----------
 .../prog/en/modules/tools/modborrowers.tt          |    4 +-
 koha-tmpl/intranet-tmpl/prog/js/holds.js           |    5 +-
 reserve/request.pl                                 |   12 ----
 6 files changed, 83 insertions(+), 74 deletions(-)

hooks/post-receive
-- 
main Koha release repository