From 5d5d9db7d9bee9f06674f387e0ce1a42e0fbb8f3 Mon Sep 17 00:00:00 2001 From: Owen Leonard 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. --- .../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 3b90d3497c..c0b40e6806 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 @@ Title Notes Created on + Created on + Updated on Updated on Resolution   diff --git a/koha-tmpl/intranet-tmpl/prog/js/checkouts.js b/koha-tmpl/intranet-tmpl/prog/js/checkouts.js index 4920eccb9d..0b37320465 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.30.2