Fixing code using lookbehind + lookahead, for Safari (#314)

* Replacing lookbehind + lookahead with safari-compatible code * Adding a simple unit test Co-authored-by: Ole Martin Handeland <[email protected]>
Altinn · Jul 4, 2022 · 9f10ce5 · 9f10ce5
1 parent 786b839
commit 9f10ce5
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/src/altinn-app-frontend/src/utils/databindings.test.ts b/src/altinn-app-frontend/src/utils/databindings.test.ts
@@ -8,6 +8,7 @@ import {
   getKeyWithoutIndex,
   mapFormData,
   removeGroupData,
+  getKeyIndex,
 } from './databindings';
 
 describe('utils/databindings.ts', () => {
@@ -111,6 +112,12 @@ describe('utils/databindings.ts', () => {
     ];
   });
 
+  describe('getKeyIndex', () => {
+    it('should work', () => {
+      expect(getKeyIndex('Group[1].Group2[0].group2prop')).toEqual([1, 0]);
+    });
+  });
+
   describe('flattenObject', () => {
     it('should return property of type number as a string', () => {
       testObj.aNumber = 43;

diff --git a/src/altinn-app-frontend/src/utils/databindings.ts b/src/altinn-app-frontend/src/utils/databindings.ts
@@ -72,8 +72,8 @@ export function getKeyWithoutIndex(keyWithIndex: string): string {
  * as an array => [0, 1]
  */
 export function getKeyIndex(keyWithIndex: string): number[] {
-  const match = keyWithIndex.match(/(?<=\[)\d+(?=])]/g) || [];
-  return match.map((n) => parseInt(n, 10));
+  const match = keyWithIndex.match(/\[\d+]/g) || [];
+  return match.map((n) => parseInt(n.replace('[', '').replace(']', ''), 10));
 }
 
 /**