Attachment 165331 Details for Bug 30123 – Bug 30123: Set desk default to logged-in-desk-id on sety library page

[patch] Bug 30123: Set desk default to logged-in-desk-id on sety library page

Bug-30123-Set-desk-default-to-logged-in-desk-id-on.patch (text/plain), 4.04 KB, created by Brendan Lawlor on 2024-04-22 16:53:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Brendan Lawlor

Created: 2024-04-22 16:53:38 UTC

Size: 4.04 KB

patch

obsolete

>From b38ed0add0e2489481e390a8763ba1a333d24046 Mon Sep 17 00:00:00 2001
>From: Brendan Lawlor <blawlor@clamsnet.org>
>Date: Mon, 22 Apr 2024 16:50:59 +0000
>Subject: [PATCH] Bug 30123: Set desk default to logged-in-desk-id on sety
> library page
>
>This patch refactors the javascript that runs on the set library page. If the logged in user branch code is the same as the seleceted library, default the desk selection to the current logged in desk id. Otherwise set the default desk selection to the first option for the current branch.
>
>Test plan:
>
>1. Turn on UseCirculationDesks
>2. Create a few desks with unique names for a few different libraries
>3. Set your library and desk
>4. Reload set-library.pl and notice that the desk always defaults to the last option for the library selected
>5. Apply patch and restart_all
>6. Reload set-library.pl and notice that the desk selection defaults to your current logged in desk
>7. Try changing the library selection to libraries with and without desks
>8. Notice that the desk selection defaults to the first option available for the selected library
>9. Notice that the desk selection defaults to the --- no desk for libraries without desks
>10. Notice the desk selection defaults to the currenty logged in desk if you select the library you are currently logged into
>---
> .../prog/en/includes/html_helpers.inc         |  2 +-
> .../intranet-tmpl/prog/js/desk_selection.js   | 27 +++++--------------
> 2 files changed, 8 insertions(+), 21 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc
>index 6830e06dc7..5b514f5736 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc
>@@ -18,7 +18,7 @@
>             <option class="[% d.branchcode | html %]" value="[% d.desk_id | html %]" >[% d.desk_name | html %]</option>
>           [% END %]
>         [% ELSE %]
>-            <option class="[% d.branchcode | html %]" value="[% d.desk_id | html %]" disabled hidden>[% d.desk_name | html %]</option>
>+            <option class="[% d.branchcode | html %]" value="[% d.desk_id | html %]" disabled style="display: none">[% d.desk_name | html %]</option>
>         [% END %]
>     [% END %]
> [% END %]
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/desk_selection.js b/koha-tmpl/intranet-tmpl/prog/js/desk_selection.js
>index ab42768d13..f6e5136a0d 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/desk_selection.js
>+++ b/koha-tmpl/intranet-tmpl/prog/js/desk_selection.js
>@@ -1,22 +1,4 @@
> $(document).ready(function() {
>-    $("#desk_id").children().each(function() {
>-        var selectedBranch = $("#branch"). children("option:selected"). val();
>-        if ($(this).attr('id') === "nodesk") { //set no desk by default, should be first element
>-            $(this).prop("selected", true);
>-            $(this).prop("disabled", false);
>-            $(this).show();
>-        }
>-        else if ($(this).hasClass(selectedBranch)) {
>-            $('#nodesk').prop("disabled", true); // we have desk, no need for nodesk option
>-            $('#nodesk').hide();
>-            $(this).prop("disabled", false);
>-            $(this).show();
>-            $(this).prop("selected", true)
>-        } else {
>-            $(this).prop("disabled", true);
>-            $(this).hide();
>-        }
>-    });
> 
>     $("#branch").on("change", function() {
> 
>@@ -32,11 +14,16 @@ $(document).ready(function() {
>                 $('#nodesk').hide();
>                 $(this).prop("disabled", false);
>                 $(this).show();
>-                $(this).prop("selected", true)
>+                if ( selectedBranch == $(".logged-in-branch-code").html() ) {
>+                    $("#desk_id").val($(".logged-in-desk-id").html());
>+                } 
>+                else {
>+                    $("#desk_id").val($("#desk_id option:not([disabled]):first").val());
>+                }
>             } else {
>                 $(this).prop("disabled", true);
>                 $(this).hide();
>             }
>         });
>     });
>-}) ;
>+});
>\ No newline at end of file
>-- 
>2.39.2

Actions: View | Diff | Splinter Review

Attachments on bug 30123: 164389 | 164390 | 164846 | 165331 | 165339 | 165358