@smile-cdr/fhirts 2.2.4 → 2.2.5

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 2.2.5
+
+* Fixed issue with `ResourceUtils.getValuesAtResourcePath` not being able to get an array with values if path exists for a top level element and is an array.
+
 ## 2.2.4
 
 ### Updates (R3)

package/GETTINGSTARTED.md

@@ -12,8 +12,8 @@
 - [Utilities](#utilities)
    - [PatchUtils](#patchutils)
    - [QueryBuilder](#querybuilder)
-   - [BundleUtils](#bundleutilities)
-   - [ResourceUtils](#resourceutilities)
+   - [BundleUtils](#bundleutils)
+   - [ResourceUtils](#resourceutils)
    - [Bundle Utilities (static BundleUtils)](#bundleutilities)
    - [Resource Utilities (static ResourceUtils)](#resourceutilities)
 
@@ -88,7 +88,7 @@ Read more about resource narrowing here : https://www.typescriptlang.org/docs/ha
 - All of the above mentioned classes are currently in preliminary phase and will be refined in future as per needs.
 - The above utlity classes include common functionalities used by front end applications using FHIR. 
 
-#### PatchUtils 
+#### PatchUtils
 - Published in `v2.2.0`.
 - Example usage demonstrated below.
 ```js
@@ -118,7 +118,7 @@ console.log(patchParameters)
          }
 	   ]
    }
- * 
+ */ 
 ```
 
 #### QueryBuilder
@@ -148,9 +148,9 @@ import { BundleUtils } from '@smile-cdr/fhirts';
  
 const bundleUtils = new BundleUtils();
 // returns arrayof Claim resources from Bundle.entry 
-const claimsList = bundleUtils.getResourcesFromBundle(Bundle.entry, 'Claim');
+const claimsList = bundleUtils.getResources(Bundle.entry, 'Claim');
 // returns a single resource with ID 123 from Bundle.entry
-const resource = bundleUtils.getResourceFromBundle(Bundle.entry, '123'); 
+const resource = bundleUtils.getResource(Bundle.entry, '123'); 
 ```
 
 #### ResourceUtils

package/dist/library/ResourceUtils/ResourceUtils.js

@@ -72,7 +72,7 @@ class ResourceUtils {
                     let resultSet = [];
                     for (let subPathIndex = 0; subPathIndex < resourcePathValue.length; subPathIndex++) {
                         const subPathValue = resourcePathValue[subPathIndex];
-                        if (this.isPrimitive(subPathValue)) {
+                        if (this.isPrimitive(subPathValue) || pathSections.length === 2) {
                             resultSet.push(subPathValue);
                         }
                         else {

package/dist/library/ResourceUtils/ResourceUtils.spec.js

@@ -190,6 +190,28 @@ describe("ResourceUtils", () => {
             expect(pathValues.length).toEqual(1);
             expect(pathValues[0]).toEqual("male");
         });
+        it("should return array with values if path exists for a top level element and is a array", () => {
+            // setup
+            const expected = [
+                {
+                    use: "usual",
+                    family: "van de Heuvel",
+                    given: ["Pieter", "Peter"],
+                    suffix: ["MSc"],
+                },
+                {
+                    use: "usual",
+                    family: "van de Heuvel",
+                    given: ["Pieter"],
+                    suffix: ["MSc"],
+                },
+            ];
+            // execute
+            const actual = resourceUtils.getValuesAtResourcePath(patientPayload, "Patient.name");
+            // validate
+            expect(actual.length).toEqual(2);
+            expect(actual).toEqual(expected);
+        });
         it("should return array with values for a array under object", () => {
             // execute
             const pathValues = resourceUtils.getValuesAtResourcePath(patientPayload, "Patient.name.given");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smile-cdr/fhirts",
-  "version": "2.2.4",
+  "version": "2.2.5",
   "description": "Fhir ts/js library for frontend apps",
   "license": "Apache-2.0",
   "main": "dist/index.js",

package/src/library/ResourceUtils/ResourceUtils.spec.ts

@@ -255,6 +255,32 @@ describe("ResourceUtils", () => {
       expect(pathValues[0]).toEqual("male");
     });
 
+    it("should return array with values if path exists for a top level element and is a array", () => {
+      // setup
+      const expected = [
+        {
+          use: "usual",
+          family: "van de Heuvel",
+          given: ["Pieter", "Peter"],
+          suffix: ["MSc"],
+        },
+        {
+          use: "usual",
+          family: "van de Heuvel",
+          given: ["Pieter"],
+          suffix: ["MSc"],
+        },
+      ];
+      // execute
+      const actual = resourceUtils.getValuesAtResourcePath(
+        patientPayload,
+        "Patient.name"
+      );
+      // validate
+      expect(actual.length).toEqual(2);
+      expect(actual).toEqual(expected);
+    });
+
     it("should return array with values for a array under object", () => {
       // execute
       const pathValues = resourceUtils.getValuesAtResourcePath(patientPayload, "Patient.name.given");

package/src/library/ResourceUtils/ResourceUtils.ts

@@ -82,9 +82,10 @@ export class ResourceUtils {
           let resultSet = [];
           for (let subPathIndex = 0; subPathIndex < resourcePathValue.length; subPathIndex++) {
             const subPathValue = resourcePathValue[subPathIndex];
-            if (this.isPrimitive(subPathValue)) {
+            if (this.isPrimitive(subPathValue) || pathSections.length === 2) {
               resultSet.push(subPathValue);
-            } else {
+            }
+            else {
               resultSet.push(...this.getValuesAtResourcePath(subPathValue,
                 pathSections.slice(index).join(".")));
             }