This implementation shouldn't be needed, but maybe it works?

ortus-boxlang · Dec 7, 2024 · 3a87b2d · 3a87b2d
1 parent 7843933
commit 3a87b2d
Showing 1 changed file with 20 additions and 11 deletions.
diff --git a/src/main/java/ortus/boxlang/modules/compat/bifs/system/GetClientVariablesList.java b/src/main/java/ortus/boxlang/modules/compat/bifs/system/GetClientVariablesList.java
@@ -24,7 +24,7 @@
 import ortus.boxlang.runtime.scopes.ArgumentsScope;
 import ortus.boxlang.runtime.scopes.Key;
 import ortus.boxlang.runtime.types.Argument;
-import ortus.boxlang.runtime.types.util.BLCollector;
+import ortus.boxlang.runtime.types.Array;
 import ortus.boxlang.runtime.types.util.ListUtil;
 
 @BoxBIF
@@ -54,16 +54,25 @@ public Object _invoke( IBoxContext context, ArgumentsScope arguments ) {
 		if ( existingClientContext == null ) {
 			return "";
 		}
-		return ListUtil.asString(
-		    existingClientContext
-		        .getClient()
-		        .getClientScope()
-		        .keySet()
-		        .stream()
-		        .filter( key -> !systemProvidedVariables.contains( key ) )
-		        .collect( BLCollector.toArray() ),
-		    ","
-		);
+
+		Array vars = new Array();
+		for ( Key key : existingClientContext.getClient().getClientScope().keySet() ) {
+			if ( !systemProvidedVariables.contains( key ) ) {
+				vars.add( key );
+			}
+		}
+		return ListUtil.asString( vars, "," );
+
+		// return ListUtil.asString(
+		// existingClientContext
+		// .getClient()
+		// .getClientScope()
+		// .keySet()
+		// .stream()
+		// .filter( key -> !systemProvidedVariables.contains( key ) )
+		// .collect( BLCollector.toArray() ),
+		// ","
+		// );
 
 	}
 }