Attachment 154987 Details for Bug 34660 – Bug 34660: Make deliveries table on housebound.tt a DataTable

[patch] Bug 34660: Make deliveries table on housebound.tt a DataTable

Bug-34660-Make-deliveries-table-on-houseboundtt-a-.patch (text/plain), 6.36 KB, created by ByWater Sandboxes on 2023-08-30 20:17:29 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: ByWater Sandboxes

Created: 2023-08-30 20:17:29 UTC

Size: 6.36 KB

patch

obsolete

>From 0a12a74febf8847d521a24f6d10cc54c688de354 Mon Sep 17 00:00:00 2001
>From: Lucas Gass <lucas@bywatersolutions.com>
>Date: Wed, 30 Aug 2023 16:16:43 +0000
>Subject: [PATCH] Bug 34660: Make deliveries table on housebound.tt a DataTable
>
>To test:
>1. Turn on the housebound module ( HouseboundModule )
>2. From a staff account add at leats one staff member as a Deliverer and a Chooser.
>3. From a patron account click on the 'Housebound' tab. Add some deliveries.
>4. Notice they sort with the oldest deilivery first.
>5. Apply patch and reload the page.
>6. Now the table is a DataTable and can be dymacially sorted.
>
>Signed-off-by: Rhonda Kuiper <rkuiper@roundrocktexas.gov>
>---
> .../prog/en/modules/members/housebound.tt     | 58 ++++++++++++-------
> 1 file changed, 36 insertions(+), 22 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/housebound.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/housebound.tt
>index 67daa86150..23775159b6 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/housebound.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/housebound.tt
>@@ -387,30 +387,38 @@
>                     <h3>Deliveries</h3>
>                     [% housebound_visits = housebound_profile.housebound_visits %]
>                     [% IF  housebound_visits.count > 0 %]
>-                        <table border="0" width="100%" cellpadding="3" cellspacing="0">
>-                            <tr>
>-                                <th>ID</th><th>Date</th><th>Chooser</th><th>Deliverer</th><th class="noExport">Actions</th>
>-                            </tr>
>-                            [% FOREACH entry IN housebound_visits %]
>+                        <table id="deliveries">
>+                            <thead>
>                                 <tr>
>-                                    <td>[% entry.id | html %]</td>
>-                                    <td>[% entry.appointment_date | $KohaDates %] ([% entry.day_segment | html %])</td>
>-                                    <td>
>-                                        <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% entry.chooser.borrowernumber | uri %]">
>-                                        [% INCLUDE 'patron-title.inc' patron = entry.chooser invert_name = 0 %]
>-                                        </a>
>-                                    </td>
>-                                    <td>
>-                                        <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% entry.deliverer.borrowernumber | uri %]">
>-                                        [% INCLUDE 'patron-title.inc' patron = entry.deliverer invert_name = 0 %]
>-                                        </a>
>-                                    </td>
>-                                    <td class="actions">
>-                                        <a class="btn btn-default btn-xs" href="/cgi-bin/koha/members/housebound.pl?method=visit_update_or_create&visit_id=[% entry.id | html %]&borrowernumber=[% patron.borrowernumber | html %]"><i class="fa-solid fa-pencil" aria-hidden="true"></i> Edit</a>
>-                                        <a class="delete btn btn-default btn-xs" href="/cgi-bin/koha/members/housebound.pl?method=visit_delete&visit_id=[% entry.id | html %]&borrowernumber=[% patron.borrowernumber | html %]"><i class="fa fa-trash-can"></i> Delete</a>
>-                                    </td>
>+                                    <th>ID</th>
>+                                    <th>Date</th>
>+                                    <th>Chooser</th>
>+                                    <th>Deliverer</th>
>+                                    <th class="noExport">Actions</th>
>                                 </tr>
>-                            [% END %]
>+                            </thead>
>+                            <tbody>
>+                                [% FOREACH entry IN housebound_visits %]
>+                                    <tr>
>+                                        <td>[% entry.id | html %]</td>
>+                                        <td>[% entry.appointment_date | $KohaDates %] ([% entry.day_segment | html %])</td>
>+                                        <td>
>+                                            <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% entry.chooser.borrowernumber | uri %]">
>+                                            [% INCLUDE 'patron-title.inc' patron = entry.chooser invert_name = 0 %]
>+                                            </a>
>+                                        </td>
>+                                        <td>
>+                                            <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% entry.deliverer.borrowernumber | uri %]">
>+                                            [% INCLUDE 'patron-title.inc' patron = entry.deliverer invert_name = 0 %]
>+                                            </a>
>+                                        </td>
>+                                        <td class="actions">
>+                                            <a class="btn btn-default btn-xs" href="/cgi-bin/koha/members/housebound.pl?method=visit_update_or_create&visit_id=[% entry.id | html %]&borrowernumber=[% patron.borrowernumber | html %]"><i class="fa-solid fa-pencil" aria-hidden="true"></i> Edit</a>
>+                                            <a class="delete btn btn-default btn-xs" href="/cgi-bin/koha/members/housebound.pl?method=visit_delete&visit_id=[% entry.id | html %]&borrowernumber=[% patron.borrowernumber | html %]"><i class="fa fa-trash-can"></i> Delete</a>
>+                                        </td>
>+                                    </tr>
>+                                [% END %]
>+                            </tbody>
>                         </table>
>                     [% END %]
>                     <div class="action">
>@@ -435,12 +443,18 @@
> 
> [% MACRO jsinclude BLOCK %]
>     [% INCLUDE 'calendar.inc' %]
>+    [% INCLUDE 'datatables.inc' %]
>+    [% INCLUDE 'columns_settings.inc' %]
>     <script>
>         $(document).ready(function() {
>             $("a.delete").click(function(){
>                 return confirm(_("Are you sure you want to delete this delivery?"));
>             });
>         });
>+        KohaTable("deliveries", {
>+            "sPaginationType": "full",
>+            "autoWidth": false,
>+        });
>     </script>
>     [% INCLUDE 'str/members-menu.inc' %]
>     [% Asset.js("js/members-menu.js") | $raw %]
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 34660: 154975 | 154987 | 155800