From bbd224bb3afb51504aa98514ad62f10a4449fb40 Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Fri, 21 Jul 2023 11:38:32 +0000
Subject: [PATCH] Bug 25023: Claims returned dates not formatted according to
 dateformat preference

This patch updates the code for rendering the claims returned table on
the checkout and patron detail pages. Now the dates are formatted using
the global js-date-format.inc code.

To test, apply the patch and make sure the claims returned feature is
enabled by setting a value in the ClaimReturnedLostValue system
preference.

- Check out some items to a patron.
- Under the checkouts tab, mark several checkouts as "Claim returned."
- Open the claims tab.
- The dates in the "Created on" column should be formatted according to
  your dateformat system preference.
  - Change the dateformat preference and return to the checkouts page.
    Confirm that the claims tab shows dates formatted correctly.
- The "created on" and "updated on" columns should sort correctly with
  any dateformat setting. You may have to directly modify the dates in
  the database in order to have the right data for testing this.

Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
---
 .../prog/en/includes/patron-return-claims.inc |  2 ++
 koha-tmpl/intranet-tmpl/prog/js/checkouts.js  | 20 +++++++++++++++----
 2 files changed, 18 insertions(+), 4 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/patron-return-claims.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/patron-return-claims.inc
index 3b90d3497c2..c0b40e68064 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/patron-return-claims.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/patron-return-claims.inc
@@ -15,6 +15,8 @@
             <th class="return-claim-record-title anti-the">Title</th>
             <th class="return-claim-notes">Notes</th>
             <th class="return-claim-created-on">Created on</th>
+            <th class="return-claim-created-on">Created on</th>
+            <th class="return-claim-updated-on">Updated on</th>
             <th class="return-claim-updated-on">Updated on</th>
             <th class="return-claim-resolution">Resolution</th>
             <th class="return-claim-actions NoSort">&nbsp;</th>
diff --git a/koha-tmpl/intranet-tmpl/prog/js/checkouts.js b/koha-tmpl/intranet-tmpl/prog/js/checkouts.js
index 4920eccb9d3..0b373204656 100644
--- a/koha-tmpl/intranet-tmpl/prog/js/checkouts.js
+++ b/koha-tmpl/intranet-tmpl/prog/js/checkouts.js
@@ -985,16 +985,28 @@ $(document).ready(function() {
                         }
                     },
                     {
+                        "mDataProp": "created_on",
+                        "bVisible": false,
+                    },
+                    {
+                        "orderData": 4,
                         "mDataProp": function ( oObj ) {
-                            let created_on = new Date( oObj.created_on );
-                            return created_on.toLocaleDateString();
+                            if ( oObj.created_on ) {
+                                return $date(oObj.created_on, { no_tz_adjust: true });;
+                            } else {
+                                return "";
+                            }
                         }
                     },
                     {
+                        "mDataProp": "updated_on",
+                        "bVisible": false,
+                    },
+                    {
+                        "orderData": 6,
                         "mDataProp": function ( oObj ) {
                             if ( oObj.updated_on ) {
-                                let updated_on = new Date( oObj.updated_on );
-                                return updated_on.toLocaleDateString();
+                                return $date(oObj.updated_on, { no_tz_adjust: true });
                             } else {
                                 return "";
                             }
-- 
2.34.1