Merge pull request #34 from JS-AK/feat/template/updates

fix: added check hasArraysInReplacements in substitute

Author: JS-AK

src/lib/template/template-memory.ts

@@ -256,8 +256,9 @@ export class TemplateMemory {
 			const TABLE_REGEX = /\$\{table:([a-zA-Z0-9_]+)\.([a-zA-Z0-9_]+)\}/g;
 
 			const hasTablePlaceholders = TABLE_REGEX.test(sharedStringsContent) || TABLE_REGEX.test(sheetContent);
+			const hasArraysInReplacements = Utils.foundArraysInReplacements(replacements);
 
-			if (hasTablePlaceholders) {
+			if (hasTablePlaceholders && hasArraysInReplacements) {
 				const result = this.#expandTableRows(sheetContent, sharedStringsContent, replacements);
 
 				sheetContent = result.sheet;

src/lib/template/utils/found-arrays-in-replacements.ts

@@ -0,0 +1,30 @@
+/**
+ * Recursively checks if an object contains any arrays in its values.
+ *
+ * @param {Record<string, unknown>} replacements - The object to check for arrays.
+ * @returns {boolean} True if any arrays are found, false otherwise.
+ */
+
+export function foundArraysInReplacements(replacements: Record<string, unknown>): boolean {
+	let isFound = false;
+
+	for (const key in replacements) {
+		const value = replacements[key];
+
+		if (Array.isArray(value)) {
+			isFound = true;
+
+			return isFound;
+		}
+
+		if (typeof value === "object" && value !== null) {
+			isFound = foundArraysInReplacements(value as Record<string, unknown>);
+
+			if (isFound) {
+				return isFound;
+			}
+		}
+	}
+
+	return isFound;
+}

src/lib/template/utils/found-arrays-in-replacements.unit.spec.ts

@@ -0,0 +1,29 @@
+import { describe, expect, it } from "vitest";
+
+import { foundArraysInReplacements } from "./found-arrays-in-replacements.js";
+
+describe("foundArraysInReplacements", () => {
+	it("should return true if the replacements object contains arrays at the top level", () => {
+		const replacements = { key: ["value1", "value2"] };
+
+		const result = foundArraysInReplacements(replacements);
+
+		expect(result).toBe(true);
+	});
+
+	it("should return true if the replacements object contains arrays at the second level", () => {
+		const replacements = { key: { subkey: ["value1", "value2"] } };
+
+		const result = foundArraysInReplacements(replacements);
+
+		expect(result).toBe(true);
+	});
+
+	it("should return false if the replacements object does not contain arrays", () => {
+		const replacements = { key: "value" };
+
+		const result = foundArraysInReplacements(replacements);
+
+		expect(result).toBe(false);
+	});
+});

src/lib/template/utils/index.ts

@@ -8,6 +8,7 @@ export * from "./column-index-to-letter.js";
 export * from "./compare-columns.js";
 export * from "./escape-xml.js";
 export * from "./extract-xml-declaration.js";
+export * from "./found-arrays-in-replacements.js";
 export * from "./get-by-path.js";
 export * from "./get-max-row-number.js";
 export * from "./get-rows-above.js";