Implemented fixes for LDEV-6302 and added comprehensive test cases for the StructToSorted() function.

core/src/main/java/lucee/runtime/functions/struct/StructToSorted.java

@@ -34,7 +34,7 @@ public static Struct call(PageContext pc, Struct base) throws PageException {
 	}
 
 	public static Struct call(PageContext pc, Struct base, Object sortTypeOrSortFunc) throws PageException {
-		if(Decision.isSimpleValue(sortTypeOrSortFunc)) call(pc, base, Caster.toString(sortTypeOrSortFunc), "asc", false);
+		if(Decision.isSimpleValue(sortTypeOrSortFunc)) return call(pc, base, Caster.toString(sortTypeOrSortFunc), "asc", false);
 		return _call(pc, base, Caster.toFunction(sortTypeOrSortFunc));
 	}
 

test/tickets/LDEV6302.cfc

@@ -0,0 +1,49 @@
+component extends="org.lucee.cfml.test.LuceeTestCase" label="LDEV-6302"{
+
+	function beforeAll() {
+
+		// Struct with mixed-case keys for text and textNoCase tests
+		variables.textStruct = {
+			Zulu: { label: "Zulu" },
+			alpha: { label: "Alpha" },
+			MIKE: { label: "Mike" }
+		};
+
+		// Struct for numeric test
+		variables.numericStruct = {
+			"10": "ten",
+			"2": "two",
+			"1": "one"
+		};
+
+	}
+
+	function run( testResults, testBox ) {
+		describe( title="LDEV-6302: struct.toSorted/StructToSorted single-argument string defaults sortOrder to asc", body=function() {
+			it( title="member toSorted('text') with one argument defaults to ascending order", body=function( currentSpec ) {
+				expect(
+					structKeyList(duplicate(variables.textStruct).toSorted("text"), ",")
+				   ).toBe("alpha,MIKE,Zulu");
+			} );
+
+			it( title="member toSorted('textNoCase') with one argument defaults to ascending order", body=function( currentSpec ) {
+				expect(
+					structKeyList(duplicate(variables.textStruct).toSorted("textNoCase"), ",")
+				).toBe("alpha,MIKE,Zulu");
+			} );
+
+			it( title="member toSorted('numeric') with one argument defaults to ascending order", body=function( currentSpec ) {
+				expect(
+					structKeyList(duplicate(variables.numericStruct).toSorted("numeric"), ",")
+				).toBe("1,2,10");
+			} );
+
+			it( title="StructToSorted(base,'text') with one argument defaults to ascending order", body=function( currentSpec ) {
+				expect(
+					structKeyList(StructToSorted(duplicate(variables.textStruct), "text"), ",")
+				   ).toBe("alpha,MIKE,Zulu");
+			} );
+		} );
+	}
+
+}