From d65298efd4670779efd1d0c7993c7cf2c34ba380 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. Signed-off-by: Owen Leonard Signed-off-by: Martin Renvoize --- .../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 78489f7cd88..6af1865973a 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: