fix: check input type before calling string match on it

ghiscoding · ghiscoding · commit 9361aa653c67 · 2024-02-10T13:12:02.000-05:00
diff --git a/packages/excel-builder-vanilla/src/Excel/SharedStrings.ts b/packages/excel-builder-vanilla/src/Excel/SharedStrings.ts
@@ -42,7 +42,7 @@ export class SharedStrings {
 
     while (l--) {
       const clone = template.cloneNode(true);
-      if (strings[l]?.match(/\s+/)) {
+      if (typeof strings[l] === 'string' && strings[l].match(/\s+/)) {
         clone.firstChild!.setAttribute('xml:space', 'preserve');
       }
       clone.firstChild!.firstChild!.nodeValue = strings[l];

Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ export class SharedStrings {`
`42`	`42`
`43`	`43`	`while (l--) {`
`44`	`44`	`const clone = template.cloneNode(true);`
`45`		`- if (strings[l]?.match(/\s+/)) {`
	`45`	`+ if (typeof strings[l] === 'string' && strings[l].match(/\s+/)) {`
`46`	`46`	`clone.firstChild!.setAttribute('xml:space', 'preserve');`
`47`	`47`	`}`
`48`	`48`	`clone.firstChild!.firstChild!.nodeValue = strings[l];`