From 8c583e9800501b3ca7742fda096470260250cb82 Mon Sep 17 00:00:00 2001
From: Marco van Wieringen <marco.van.wieringen@planets.elm.net>
Date: Tue, 26 Jul 2022 10:47:45 +0200
Subject: [PATCH] Fix datetime-luxon.js

luxon.fromFormat argument text must really be text so if the d variable
is not do not call luxon.fromFormat as then we will get the dreaded
TypeError: input.match is not a function. When the variable is not text
it will not match anyway so return null in that case.
---
 sorting/datetime-luxon.js | 48 +++++++++++++++++++++++----------------
 1 file changed, 29 insertions(+), 19 deletions(-)

diff --git a/sorting/datetime-luxon.js b/sorting/datetime-luxon.js
index f743816..2cfee7c 100644
--- a/sorting/datetime-luxon.js
+++ b/sorting/datetime-luxon.js
@@ -29,13 +29,11 @@
 }(function ($, luxon) {
 
 function strip (d) {
-	if ( typeof d === 'string' ) {
-		// Strip HTML tags and newline characters if possible
-		d = d.replace(/(<.*?>)|(\r?\n|\r)/g, '');
+	// Strip HTML tags and newline characters if possible
+	d = d.replace(/(<.*?>)|(\r?\n|\r)/g, '');
 
-		// Strip out surrounding white space
-		d = d.trim();
-	}
+	// Strip out surrounding white space
+	d = d.trim();
 
 	return d;
 }
@@ -45,25 +43,37 @@ $.fn.dataTable.luxon = function ( format, locale, reverseEmpties ) {
 
 	// Add type detection
 	types.detect.unshift( function ( d ) {
-		d = strip(d);
-
-		// Null and empty values are acceptable
-		if ( d === '' || d === null ) {
+		// Null values are acceptable
+		if ( d === null ) {
 			return 'luxon-'+format;
-		}
+                }
+		if ( typeof d === 'string' ) {
+			d = strip(d);
 
-		return luxon.DateTime.fromFormat( d, format).isValid ?
-			'luxon-'+format :
-			null;
+			// Empty values are acceptable
+			if ( d === '' ) {
+				return 'luxon-'+format;
+			}
+
+			return luxon.DateTime.fromFormat( d, format).isValid ?
+				'luxon-'+format :
+				null;
+		} else {
+			return null;
+                }
 	} );
 
 	// Add sorting method - use an integer for the sorting
 	types.order[ 'luxon-'+format+'-pre' ] = function ( d ) {
-		d = strip(d);
-		
-		return !luxon.DateTime.fromFormat(d, format).isValid ?
-			(reverseEmpties ? -Infinity : Infinity) :
-			parseInt( luxon.DateTime.fromFormat( d, format).ts, 10 );
+		if ( typeof d === 'string' ) {
+			d = strip(d);
+
+			return !luxon.DateTime.fromFormat(d, format).isValid ?
+				(reverseEmpties ? -Infinity : Infinity) :
+				parseInt( luxon.DateTime.fromFormat( d, format).ts, 10 );
+		} else {
+			return null;
+                }
 	};
 };