Revert "Fixed nested variable replacement using recursion."

This reverts commit 6e47f4e.

Author: andreachild

src/test/templates/JSResolverOCHTTPS.test.js

@@ -798,46 +798,6 @@ test('should resolve query with nested edge filter and variables', () => {
     });
 });
 
-test('should resolve query with nested edge filter and nested scalar variables', () => {
-    const query = 'query GetNodeAirports($filter: AirportInput, $options: Options, $nestedFilter: AirportInput, $nestedOptions: Options) {\n' +
-        '  getNodeAirports(filter: {country: {eq: $country}}, options: {limit: $limit}) {\n' +
-        '    code\n' +
-        '    airportRoutesOut(filter: {country: {eq: $country}}, options: {limit: $limit}) {\n' +
-        '      code\n' +
-        '    }\n' +
-        '  }\n' +
-        '}';
-    const variables = {
-        "country": "CA",
-        "limit": 6
-    }
-    const result = resolveGraphDBQuery({queryObjOrStr: query, variables: variables});
-
-    expect(result).toMatchObject({
-        query: 'MATCH (getNodeAirports_Airport:`airport`) ' +
-            'WHERE getNodeAirports_Airport.country = $getNodeAirports_Airport_country ' +
-            'WITH getNodeAirports_Airport LIMIT 6\n' +
-            'OPTIONAL MATCH (getNodeAirports_Airport)-[getNodeAirports_Airport_airportRoutesOut_route:route]->(getNodeAirports_Airport_airportRoutesOut:`airport`) ' +
-            'WHERE getNodeAirports_Airport_airportRoutesOut.country = $getNodeAirports_Airport_airportRoutesOut_country\n' +
-            'WITH getNodeAirports_Airport, ' +
-            'CASE WHEN getNodeAirports_Airport_airportRoutesOut IS NULL THEN [] ' +
-            'ELSE COLLECT({' +
-            'code: getNodeAirports_Airport_airportRoutesOut.`code`' +
-            '})[..6] ' +
-            'END AS getNodeAirports_Airport_airportRoutesOut_collect\n' +
-            'RETURN collect({' +
-            'code: getNodeAirports_Airport.`code`, ' +
-            'airportRoutesOut: getNodeAirports_Airport_airportRoutesOut_collect' +
-            '})[..6]',
-        parameters: {
-            getNodeAirports_Airport_country: "CA",
-            getNodeAirports_Airport_airportRoutesOut_country: "CA"
-        },
-        language: 'opencypher',
-        refactorOutput: null
-    });
-});
-
 test('should resolve custom mutation with @graphQuery directive and $input parameter', () => {
     const query = 'mutation MyMutation {\n' +
         '  createAirport(\n' +

templates/JSResolverOCHTTPS.js

@@ -917,59 +917,18 @@ function convertToValueNode(value) {
 
 /**
  * Replaces any variable references in the selection with the actual value from the variables object.
- * @param {object} selection the graphQL selection to replace variable references in
- * @param {object} variables the variables object
+ * @param selection the graphQL selection to replace variable references in
+ * @param variables the variables object
  */
-function replaceVariableArgsWithValues(selection, variables = {}) {
-    if (!selection?.arguments || !variables) {
-        return;
-    }
-
-    selection.arguments.forEach(arg => {
-        if (!arg.value) {
-            return;
-        }
-        replaceVariableInValue(arg.value, variables);
+function replaceVariableArgsWithValues(selection, variables) {
+    selection.arguments?.filter(arg => arg.value?.kind === 'Variable' 
+        && arg.value?.name?.value && variables.hasOwnProperty(arg.value.name.value))
+        .forEach(arg => {
+        // replace variable reference with actual value
+        arg.value = convertToValueNode(variables[arg.value.name.value]);
     });
 }
 
-/**
- * Recursively replaces any variable references in a value node with the variable value
- * @param {Object} valueNode - The value node to process
- * @param {Object} variables - The variables object containing values to substitute
- */
-function replaceVariableInValue(valueNode, variables = {}) {
-    if (!valueNode || !variables) {
-        return;
-    }
-    
-    if (valueNode.kind === 'Variable' &&
-        valueNode.name?.value &&
-        variables.hasOwnProperty(valueNode.name.value)) {
-        // replace the variable with actual value
-        Object.assign(valueNode, convertToValueNode(variables[valueNode.name.value]));
-        return;
-    }
-    
-    if (valueNode.kind === 'ObjectValue' && valueNode.fields) {
-        // replace any variables in the object value fields
-        valueNode.fields.forEach(field => {
-            if (field.value) {
-                replaceVariableInValue(field.value, variables);
-            }
-        });
-        return;
-    }
-    
-    if (valueNode.kind === 'ListValue' && valueNode.values) {
-        // replace any variables in the list values
-        valueNode.values.forEach(value => {
-            replaceVariableInValue(value, variables);
-        });
-        return;
-    }
-}
-
 /**
  * Replaces any fragment references in the selection with the actual fragment selections.
  * @param selection the graphQL selection to replace fragment references in