From 52dccfe27beec80003a86f3f73257b87f0118eeb Mon Sep 17 00:00:00 2001
From: Katrin Fischer <katrin.fischer@bsz-bw.de>
Date: Wed, 7 Aug 2024 13:50:37 +0000
Subject: [PATCH] Revert "Bug 36132: Allow to delete multiple patron lists on
 any page"

This reverts commit e1031b88f0172f1db4d43cf05d33d221d347c7bb.

Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
---
 .../prog/en/modules/patron_lists/lists.tt     | 48 +++++++++++--------
 1 file changed, 27 insertions(+), 21 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/lists.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/lists.tt
index b579378a0b..29fd544762 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/lists.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/lists.tt
@@ -1,8 +1,13 @@
 [% USE raw %]
 [% USE Asset %]
+[% PROCESS 'i18n.inc' %]
 [% SET footerjs = 1 %]
 [% INCLUDE 'doc-head-open.inc' %]
-<title>Patron lists &rsaquo; Tools &rsaquo; Koha</title>
+<title>[% FILTER collapse %]
+    [% t("Patron lists") | html %] &rsaquo;
+    [% t("Tools") | html %] &rsaquo;
+    [% t("Koha") | html %]
+[% END %]</title>
 [% INCLUDE 'doc-head-close.inc' %]
 [% INCLUDE 'greybox.inc' %]
 </head>
@@ -27,6 +32,7 @@
     <div class="row">
         <div class="col-sm-10 col-sm-push-2">
             <main>
+                [% INCLUDE 'messages.inc' %]
 
         <div id="toolbar" class="btn-toolbar">
             <div class="btn-group">
@@ -34,7 +40,7 @@
             </div>
         </div>
 
-        <h1>Your patron lists </h1>
+        <h1>Patron lists</h1>
 
         [% IF ( lists ) %]
 
@@ -77,8 +83,8 @@
                                     <ul class="dropdown-menu pull-right" role="menu" aria-labelledby="listactions[% l.patron_list_id | html %]">
                                         <li><a href="/cgi-bin/koha/patron_lists/list.pl?patron_list_id=[% l.patron_list_id | uri %]"><i class="fa fa-user"></i> Add patrons</a></li>
                                         [% UNLESS shared_by_other %]
-                                            <li><a href="/cgi-bin/koha/patron_lists/add-modify.pl?patron_list_id=[% l.patron_list_id | uri %]"><i class="fa fa-pencil"></i> Edit list</a></li>
-                                            <li><a class="delete_patron" href="/cgi-bin/koha/patron_lists/delete.pl?patron_list_id=[% l.patron_list_id | html %]" data-list-name="[% l.name | html %]"><i class="fa fa-trash"></i> Delete list</a></li>
+                                            <li><a href="/cgi-bin/koha/patron_lists/add-modify.pl?patron_list_id=[% l.patron_list_id | uri %]"><i class="fa-solid fa-pencil" aria-hidden="true"></i> Edit list</a></li>
+                                            <li><a class="delete_patron" href="/cgi-bin/koha/patron_lists/delete.pl?patron_list_id=[% l.patron_list_id | html %]" data-list-name="[% l.name | html %]"><i class="fa fa-trash-can"></i> Delete list</a></li>
                                         [% END %]
                                         [% IF ( l.patron_list_patrons_rs.count ) %]
                                             <li class="divider"></li>
@@ -88,14 +94,14 @@
                                             [% IF CAN_user_tools_edit_patrons %]
                                                 <li>
                                                     <a href="/cgi-bin/koha/tools/modborrowers.pl?patron_list_id=[% l.patron_list_id | uri %]&op=show">
-                                                        <i class="fa fa-pencil"></i> Batch edit patrons
+                                                        <i class="fa-solid fa-pencil" aria-hidden="true"></i> Batch edit patrons
                                                     </a>
                                                 </li>
                                             [% END %]
                                             [% IF CAN_user_tools_delete_anonymize_patrons %]
                                                 <li>
                                                     <a href="/cgi-bin/koha/tools/cleanborrowers.pl?step=2&patron_list_id=[% l.patron_list_id | uri %]&checkbox=borrower">
-                                                        <i class="fa fa-trash"></i> Batch delete patrons
+                                                        <i class="fa fa-trash-can"></i> Batch delete patrons
                                                     </a>
                                                 </li>
                                             [% END %]
@@ -141,8 +147,8 @@
 [% MACRO jsinclude BLOCK %]
     [% Asset.js("js/tools-menu.js") | $raw %]
     [% INCLUDE 'datatables.inc' %]
-    <script>
 
+    <script>
         $(document).ready(function() {
             var patronExportModal = $("#patronExportModal");
             var patronExportModalBody = $("#patronExportModal .modal-body");
@@ -151,10 +157,10 @@
 
             $('#patron-lists-table').dataTable($.extend(true, {}, dataTablesDefaults, {
                 "autoWidth": false,
-                "aoColumnDefs": [
-                    {"bSortable": false, "bSearchable": false, 'aTargets': [ 'NoSort' ]}
+                "columnDefs": [
+                    { "orderable": false, "searchable":  false, "targets": [ 'NoSort' ] }
                 ],
-                "sPaginationType": "full"
+                "pagingType": "full"
             } ));
             $(".delete_patron").on("click", function(){
                 $(".dropdown").removeClass("open");
@@ -171,18 +177,18 @@
                 }
             });
 
-        $(document).on("click", ".select_patron", function() {
-            if($(this).is(':checked')){
-                $("#delete_selected_lists").attr("class","btn btn-default btn-sm");
-                selectedPatronLists.push($(this).data("patron-list-id"));
-            }
-            else {
-                selectedPatronLists = selectedPatronLists.filter(item => item !== $(this).data("patron-list-id"));
-                if(selectedPatronLists.length === 0){
-                    $("#delete_selected_lists").attr("class","btn btn-default btn-sm disabled");
+            $(".select_patron").on("click", function() {
+                if($(this).is(':checked')){
+                    $("#delete_selected_lists").attr("class","btn btn-default btn-sm");
+                    selectedPatronLists.push($(this).data("patron-list-id"));
                 }
-            }
-        });
+                else {
+                    selectedPatronLists = selectedPatronLists.filter(item => item !== $(this).data("patron-list-id"));
+                    if(selectedPatronLists.length === 0){
+                        $("#delete_selected_lists").attr("class","btn btn-default btn-sm disabled");
+                    }
+                }
+            });
 
             $(".print_cards").on("click", function(e){
                 e.preventDefault();
-- 
2.39.5