Attachment 100740 Details for Bug 23173 – Bug 23173: (follow-up) Display available partners

[patch] Bug 23173: (follow-up) Display available partners

Bug-23173-follow-up-Display-available-partners.patch (text/plain), 3.83 KB, created by Andrew Isherwood on 2020-03-16 12:07:36 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Andrew Isherwood

Created: 2020-03-16 12:07:36 UTC

Size: 3.83 KB

patch

obsolete

>From c0ac7ecbeb1f86a08dfc0a07f2560bc787291f99 Mon Sep 17 00:00:00 2001
>From: Andrew Isherwood <andrew.isherwood@ptfs-europe.com>
>Date: Mon, 16 Mar 2020 12:06:03 +0000
>Subject: [PATCH] Bug 23173: (follow-up) Display available partners
>
>When selecting partners for search during the "Place request with
>partner libraries" workflow, there was no indication of which partners
>could actually be searched (i.e. are attached to a Z target and enabled
>for partner search). We now list them
>---
> .../intranet-tmpl/prog/css/src/staff-global.scss      |  4 ++++
> .../intranet-tmpl/prog/en/modules/ill/ill-requests.tt |  5 ++++-
> .../intranet-tmpl/prog/js/ill-availability-partner.js | 19 ++++++++++++++++++-
> 3 files changed, 26 insertions(+), 2 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss b/koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss
>index 4b03b95468..6837e217d4 100644
>--- a/koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss
>+++ b/koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss
>@@ -3753,6 +3753,10 @@ input.renew {
>         transform: translateY(-50%);
>     }
> 
>+    #generic_confirm_search_count {
>+        margin: 1em 0 1em 10em;
>+    }
>+
>     #generic_confirm_search {
>         display: block;
>         visibility: hidden;
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt
>index c9fe65bcb5..14b7eb8d95 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt
>@@ -150,7 +150,10 @@
>                                             [% END %]
>                                         </select>
>                                         [% IF Koha.Preference('ILLCheckAvailability') %]
>-                                            <button type="button" id="generic_confirm_search">Search selected partners</button>
>+                                            <div id="generic_confirm_search_count">Partners available for searching: <span id="generic_confirm_enabled">none</span></div>
>+                                            <div id="generic_confirm_search">
>+                                                <button type="button">Search selected partners</button>
>+                                            </div>
>                                         [% END %]
>                                     </li>
>                                     <li>
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/ill-availability-partner.js b/koha-tmpl/intranet-tmpl/prog/js/ill-availability-partner.js
>index 07f3664c20..cc5074bbf1 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/ill-availability-partner.js
>+++ b/koha-tmpl/intranet-tmpl/prog/js/ill-availability-partner.js
>@@ -1,8 +1,21 @@
> $(document).ready(function() {
>+    var enabled_names = [];
>+    $('#partners option').each(function() {
>+        var partner = $(this);
>+        var partner_id = partner.data('partner-id');
>+        if (isEnabled(partner_id)) {
>+            enabled_names.push(partner.text().trim());
>+        }
>+    });
>+    $('#generic_confirm_enabled').text(enabled_names.join(', '));
>+
>     $('#partners').change(function() {
>         var selected = [];
>         $('#partners option:selected').each(function() {
>-            selected.push($(this).data('partner-id'));
>+            var partner_id = $(this).data('partner-id');
>+            if (isEnabled(partner_id)) {
>+                selected.push(partner_id);
>+            }
>         });
>         if (selected.length > 0) {
>             $('#generic_confirm_search').css('visibility', 'initial');
>@@ -21,4 +34,8 @@ $(document).ready(function() {
>     $('#partnerSearch').on('hide.bs.modal', function() {
>         $.fn.dataTable.tables({ api: true }).destroy();
>     });
>+
>+    function isEnabled(id) {
>+        return services[0].enabled.indexOf(id.toString()) > -1;
>+    };
> });
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 23173: 91464 | 91465 | 92107 | 92108 | 92539 | 92540 | 92550 | 92551 | 92640 | 92641 | 92693 | 92694 | 92695 | 92715 | 92716 | 92717 | 93146 | 93147 | 93148 | 93909 | 93910 | 93911 | 95268 | 95269 | 95270 | 98521 | 99083 | 100724 | 100725 | 100740 | 102179 | 102180 | 102181 | 102182 | 102183 | 102184 | 102286 | 102287 | 102288 | 102289 | 102290 | 102291 | 102292 | 102293 | 102294