fillOdtElementTemplate now takes an array of nodes as argument for the situation of calling it on the middleContent of a block with several elements so they're evaluated together

scripts/odf/templating/fillOdtElementTemplate.js

@@ -14,7 +14,7 @@ class TemplateDOMBranch{
     /** @type {Node} */
     #leafNode
 
-    // ancestors with this.#ancestors[0] === this.#startNode and this.#ancestors.at(-1) === this.#leafNode
+    // ancestors with this.#ancestors[0] === this.#branchBaseNode and this.#ancestors.at(-1) === this.#leafNode
     /** @type {Node[]} */
     #ancestors
 
@@ -235,9 +235,12 @@ class TemplateBlock{
         }
         //console.log('[fillBlockContentTemplate] after startChild')
 
-        for(const content of this.#middleContent){
+        
-            fillOdtElementTemplate(content, compartement)
+        // if content consists of several parts of an {#each}{/each}
-        }
+        // when arriving to the {/each}, it will be alone (and imbalanced)
+        // and will trigger an error
+        fillOdtElementTemplate(Array.from(this.#middleContent), compartement)
+
         //console.log('[fillBlockContentTemplate] after middleContent')
 
         const endChild = this.endBranch.at(1)
@@ -554,14 +557,22 @@ function fillEachBlock(startNode, iterableExpression, itemExpression, endNode, c
 const IF = ifStartMarkerRegex.source
 const EACH = eachStartMarkerRegex.source
 
+/** @typedef {Element | DocumentFragment | Document} RootElementArgument */
+
+
 /**
  * 
- * @param {Element | DocumentFragment | Document} rootElement 
+ * @param {RootElementArgument | RootElementArgument[]} rootElements
  * @param {Compartment} compartment 
  * @returns {void}
  */
-export default function fillOdtElementTemplate(rootElement, compartment) {
+export default function fillOdtElementTemplate(rootElements, compartment) {
-    //console.log('[fillTemplatedOdtElement]', rootElement.nodeType, rootElement.nodeName, rootElement.textContent)
+    
+    if(!Array.isArray(rootElements)){
+        rootElements = [rootElements]
+    }
+
+    //console.log('[fillTemplatedOdtElement]', rootElements.length, rootElements[0].nodeType, rootElements[0].nodeName, rootElements[0].textContent)
     //console.log('[fillTemplatedOdtElement]', rootElement.documentElement && rootElement.documentElement.textContent)
 
     let currentlyOpenBlocks = []
@@ -585,6 +596,8 @@ export default function fillOdtElementTemplate(rootElement, compartment) {
     // Traverse "in document order"
 
 
+    for(const rootElement of rootElements){
+
         // @ts-ignore
         traverse(rootElement, currentNode => {
             //console.log('currentlyOpenBlocks', currentlyOpenBlocks)
@@ -626,8 +639,9 @@ export default function fillOdtElementTemplate(rootElement, compartment) {
 
                     //console.log('isEachClosingBlock', isEachClosingBlock)
 
-                if(!eachOpeningMarkerNode)
+                    if(!eachOpeningMarkerNode){
                         throw new Error(`{/each} found without corresponding opening {#each x as y}`)
+                    }
 
                     if(currentlyOpenBlocks.at(-1) !== EACH)
                         throw new Error(`{/each} found while the last opened block was not an opening {#each x as y}`)
@@ -637,8 +651,12 @@ export default function fillOdtElementTemplate(rootElement, compartment) {
 
                         // found an {#each} and its corresponding {/each}
                         // execute replacement loop
+                        //console.log('start of fillEachBlock')
+
                         fillEachBlock(eachOpeningMarkerNode, eachBlockIterableExpression, eachBlockItemExpression, eachClosingMarkerNode, compartment)
 
+                        //console.log('end of fillEachBlock')
+
                         eachOpeningMarkerNode = undefined
                         eachBlockIterableExpression = undefined
                         eachBlockItemExpression = undefined
@@ -648,6 +666,7 @@ export default function fillOdtElementTemplate(rootElement, compartment) {
                         // ignore because it will be treated as part of the outer {#each}
                     }
 
+                    //console.log('popping currentlyOpenBlocks')
                     currentlyOpenBlocks.pop()
                 }
 
@@ -768,4 +787,6 @@ export default function fillOdtElementTemplate(rootElement, compartment) {
                 }
             }
         })
+
+    }
 }

tests/fill-odt-template/each.js

@@ -344,3 +344,32 @@ Année
 `.trim())
 
 });
+
+
+test('nested each without common ancestor for inner each', async t => {
+    const templatePath = join(import.meta.dirname, '../fixtures/nested-each-without-common-ancestor-for-inner-each.odt')
+    const templateContent = `{#each liste_espèces_par_impact as élément}
+{#each élément.liste_espèces as espèce}
+{/each}
+{/each}
+`
+
+	const data = {
+        liste_espèces_par_impact: [
+            {}
+        ]
+    }
+
+    const odtTemplate = await getOdtTemplate(templatePath)
+
+    const templateTextContent = await getOdtTextContent(odtTemplate)
+
+    t.deepEqual(templateTextContent, templateContent, 'reconnaissance du template')
+
+    const odtResult = await fillOdtTemplate(odtTemplate, data)
+
+    const odtResultTextContent = await getOdtTextContent(odtResult)
+    t.deepEqual(odtResultTextContent, ``)
+
+
+});