From 0a64595b731269d8d9553fe90e06f2d88d637f83 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Fri, 5 Jan 2024 12:10:33 +0100 Subject: [PATCH] Bug 35329: Move patron search to modal - funds This is a tricky one. We can have several modals per page, patron-search.inc needs to be adjusted to not use ids but classes. Also we need to declare JS variables with 'var' instead of 'let' (which does not allow redefinition of the same variable). Test plan: Create or modify a fund "Select owner" and "Add users" to open the modals. --- .../prog/en/includes/av-build-dropbox.inc | 12 +- .../prog/en/includes/patron-search.inc | 261 ++++++++++-------- .../prog/en/includes/patronfields.inc | 2 +- .../prog/en/modules/admin/aqbudgets.tt | 37 +-- .../prog/en/modules/members/member.tt | 6 +- 5 files changed, 174 insertions(+), 144 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/av-build-dropbox.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/av-build-dropbox.inc index 263a190a150..78b4bcbbf57 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/av-build-dropbox.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/av-build-dropbox.inc @@ -12,7 +12,11 @@ [% DEFAULT class = '' size = 20 %] [% IF avs.count %] - + [% ELSE %] + [% ELSE %] - + [% IF no_id %] + + [% ELSE %] + + [% END %] [% END %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/patron-search.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/patron-search.inc index 306ab20fad3..7392b3e6545 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/patron-search.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/patron-search.inc @@ -7,11 +7,13 @@ [% USE Asset %] [% USE To %] +[% SET search_results_block_id = 'searchresults' %] + [%# Display a simple form %] [% BLOCK patron_search_filters_simple %] -
+
Enter patron card number or partial name:
- +
[% END %] @@ -27,13 +29,14 @@ [%# - search_type: select 'contains' or 'starts with' %] [%- searchtype = searchtype || Koha.Preference('DefaultPatronSearchMethod') -%] [% BLOCK patron_search_filters %] -
+ +

Search for patron

  1. - +
  2. [% FOR f IN filters %] @@ -41,7 +44,7 @@ [% CASE 'branch' %]
  3. - [% SET libraries = Branches.all( only_from_group => 1 ) %] [% IF libraries.size != 1 %] @@ -54,7 +57,7 @@ [% CASE 'category' %]
  4. - [% FOREACH category IN Categories.limited.unblessed %] @@ -64,12 +67,12 @@ [% CASE 'sort1' %]
  5. - [% PROCESS 'av-build-dropbox.inc' name="sort1_filter", category="Bsort1", empty=1, size = 20 %] + [% PROCESS 'av-build-dropbox.inc' no_id => 1, name="sort1_filter", category="Bsort1", empty=1, size = 20 %]
  6. [% CASE 'sort2' %]
  7. - [% PROCESS 'av-build-dropbox.inc' name="sort2_filter", category="Bsort2", empty=1, size = 20 %] + [% PROCESS 'av-build-dropbox.inc' no_id => 1, name="sort2_filter", category="Bsort2", empty=1, size = 20 %]
  8. [% CASE 'search_field' %]
  9. @@ -78,7 +81,7 @@ [% CASE 'search_type' %]
  10. - [% IF searchtype == "starts_with" %] @@ -93,7 +96,7 @@
- +
@@ -135,14 +138,14 @@ [% END %] -

Patrons found for:

+

Patrons found for: