Merge branch 'bugfix/LF-3060/fix-changing-input-parameters' into 'mas…

…ter'

Fixed a bug that could cause the context input parameter containing environment variables to change

See merge request lfor/fhirpath.js!18

CHANGELOG.md

@@ -3,6 +3,11 @@
 This log documents significant changes for each release.  This project follows
 [Semantic Versioning](http://semver.org/).
 
+## [3.13.4] - 2024-06-13
+### Fixed
+- a bug that could cause the context input parameter containing environment
+  variables to change.
+
 ## [3.13.3] - 2024-05-24
 ### Changed
 - Added separate TypeScript type definition files for the main file and each

package.json

@@ -1,6 +1,6 @@
 {
   "name": "fhirpath",
-  "version": "3.13.3",
+  "version": "3.13.4",
   "description": "A FHIRPath engine",
   "main": "src/fhirpath.js",
   "types": "src/fhirpath.d.ts",

src/fhirpath.js

@@ -686,7 +686,7 @@ function applyParsedPath(resource, parsedPath, context, model, options) {
   // Set up default standard variables, and allow override from the variables.
   // However, we'll keep our own copy of dataRoot for internal processing.
   let vars = {context: dataRoot, ucum: 'http://unitsofmeasure.org'};
-  let ctx = {dataRoot, processedVars: vars, vars: context || {}, model};
+  let ctx = {dataRoot, processedVars: vars, vars: {...context}, model};
   if (options.traceFn) {
     ctx.customTraceFn = options.traceFn;
   }

test/api.test.js

@@ -260,5 +260,16 @@ describe('evaluate type() on a FHIRPath evaluation result', () => {
       'Quantity', 'Quantity', 'Quantity', 'Quantity', 'Date'
     ]);
   });
+
+  it('should not change the context input parameter containing environment variables', () => {
+    const originalVars = {a: 'abc'};
+    const vars = _.cloneDeep(originalVars);
+    expect(fhirpath.evaluate(
+      {},
+      '%a = \'abc\'',
+      vars
+    )).toStrictEqual([true]);
+    expect(originalVars).toStrictEqual(vars);
+  })
 });