change order of removals in fillIfBlock to properly remove right/left content

scripts/odf/templating/fillOdtElementTemplate.js

@@ -207,9 +207,9 @@ class TemplateBlock{
         }
 
         //console.group('\n== TemplateBlock ==')
-        //console.log('startBranch', this.startBranch.at(1).nodeName, this.startBranch.at(1).textContent)
+        //this.startBranch.logBranch('startBranch')
         //console.log('middleContent', this.#middleContent.map(n => n.textContent).join(''))
-        //console.log('endBranch', this.startBranch.at(1).nodeName, this.endBranch.at(1).textContent)
+        //this.endBranch.logBranch('endBranch')
         //console.log('common ancestor', this.#commonAncestor.nodeName, '\n')
         //console.groupEnd()
     }
@@ -425,6 +425,8 @@ function findPlacesToFillInString(str, compartment) {
  * @param {Compartment} compartment 
  */
 function fillIfBlock(ifOpeningMarkerNode, ifElseMarkerNode, ifClosingMarkerNode, ifBlockConditionExpression, compartment) {
+    //const docEl = ifOpeningMarkerNode.ownerDocument.documentElement
+    
     const conditionValue = compartment.evaluate(ifBlockConditionExpression)
 
     /** @type {TemplateBlock | undefined} */
@@ -445,22 +447,27 @@ function fillIfBlock(ifOpeningMarkerNode, ifElseMarkerNode, ifClosingMarkerNode,
         thenTemplateBlock = new TemplateBlock(ifOpeningMarkerNode, ifClosingMarkerNode)
     }
 
+    if(conditionValue) {
+        if(elseTemplateBlock){
+            elseTemplateBlock.removeContent()
+        }
+
         thenTemplateBlock.removeMarkersAndEmptyAncestors()
         if(elseTemplateBlock){
             elseTemplateBlock.removeMarkersAndEmptyAncestors()
         }
 
-
-    if(conditionValue) {
         thenTemplateBlock.fillBlockContentTemplate(compartment)
-
-        if(elseTemplateBlock){
-            elseTemplateBlock.removeContent()
-        }
     }
     else{
+        // remove content before removing markers so that right and left content are fully removed
         thenTemplateBlock.removeContent()
         
+        thenTemplateBlock.removeMarkersAndEmptyAncestors()
+        if(elseTemplateBlock){
+            elseTemplateBlock.removeMarkersAndEmptyAncestors()
+        }
+
         if(elseTemplateBlock){
             elseTemplateBlock.fillBlockContentTemplate(compartment)
         }

tests/fill-odt-template/formatting.js

@@ -167,8 +167,8 @@ test('template filling - formatted-start-each-single-paragraph', async t => {
 });
 
 
-test('template filling - formatted ghost if', async t => {
-    const templatePath = join(import.meta.dirname, '../fixtures/reducing.odt')
+test('template filling - formatted ghost if then', async t => {
+    const templatePath = join(import.meta.dirname, '../fixtures/ghost-if.odt')
     const templateContent = `
     Utilisation de sources lumineuses : {#if scientifique.source_lumineuses}Oui{:else}Non{/if}
 `
@@ -188,3 +188,24 @@ test('template filling - formatted ghost if', async t => {
 
 });
 
+
+test('template filling - formatted ghost if else', async t => {
+    const templatePath = join(import.meta.dirname, '../fixtures/ghost-if.odt')
+    const templateContent = `
+    Utilisation de sources lumineuses : {#if scientifique.source_lumineuses}Oui{:else}Non{/if}
+`
+
+    const data = {scientifique: {source_lumineuses: false}}
+
+    const odtTemplate = await getOdtTemplate(templatePath)
+
+    const templateTextContent = await getOdtTextContent(odtTemplate)    
+    t.deepEqual(templateTextContent.trim(), templateContent.trim(), 'reconnaissance du template')
+    let odtResult = await fillOdtTemplate(odtTemplate, data)
+
+    const odtResultTextContent = await getOdtTextContent(odtResult)
+    t.deepEqual(odtResultTextContent.trim(), `
+ Utilisation de sources lumineuses : Non
+`.trim())
+
+});