check component key in both double and single quotes

Author: akgagnat

packages/shared-components/src/formio/overrides/utils-overrides/utils-overrides.test.js

@@ -116,9 +116,16 @@ describe('utils-overrides', () => {
       });
     });
 
-    it('does not add null check to composite component key', () => {
-      const code = "show = util.isBornBeforeYear(1964, 'container.fnr', submission);";
-      expect(UtilsOverrides.sanitizeJavaScriptCode(code)).toBe(code);
+    describe('composite component key inside expression', () => {
+      it('is not null checked when wrapped in single qoutes', () => {
+        const code = "show = utils.isBornBeforeYear(1964, 'container.fnr', submission);";
+        expect(UtilsOverrides.sanitizeJavaScriptCode(code)).toBe(code);
+      });
+
+      it('is not null checked when wrapped in double qoutes', () => {
+        const code = 'show = utils.isBornBeforeYear(1964, "container.fnr", submission);';
+        expect(UtilsOverrides.sanitizeJavaScriptCode(code)).toBe(code);
+      });
     });
   });
 

packages/shared-domain/src/utils/formio/sanitize-javascript-code.js

@@ -39,7 +39,7 @@ function mapChainedLookups(text) {
    */
   const arrayOfChainedLookups = text.match(/((\w+\.)+\w+\b)(?![(.])/g) || [];
   [...new Set(arrayOfChainedLookups)]
-    .filter((exp) => !text.includes(`'${exp}'`))
+    .filter((exp) => !text.includes(`'${exp}'`) && !text.includes(`"${exp}"`))
     .forEach((chainedLookup) => (mappedString = addNullChecksToChainedLookup(chainedLookup, mappedString)));
   return mappedString;
 }