fix docx rendering with handlebars partials

fix #1047
jsreport · Apr 11, 2023 · d254c48 · d254c48
1 parent fbfc52f
commit d254c48
Show file tree

Hide file tree

Showing 3 changed files with 39 additions and 0 deletions.
diff --git a/packages/jsreport-docx/lib/processDocx.js b/packages/jsreport-docx/lib/processDocx.js
@@ -32,9 +32,20 @@ module.exports = (reporter) => async (inputs, req) => {
 
     const contentToRender = filesToRender.map(f => {
       const xmlStr = new XMLSerializer().serializeToString(f.doc, undefined, (node) => {
+        // we need to decode the xml entities for the attributes for handlebars to work ok
         if (node.nodeType === 2 && node.nodeValue && node.nodeValue.includes('{{')) {
           const str = new XMLSerializer().serializeToString(node)
           return decodeXML(str)
+        } else if (
+          // we need to decode the xml entities in text nodes for handlebars to work ok with partials
+          node.nodeType === 3 && node.nodeValue &&
+          (node.nodeValue.includes('{{>') || node.nodeValue.includes('{{#>'))
+        ) {
+          const str = new XMLSerializer().serializeToString(node)
+
+          return str.replace(/{{#?&gt;/g, (m) => {
+            return decodeXML(m)
+          })
         }
 
         return node

diff --git a/packages/jsreport-docx/test/partial.docx b/packages/jsreport-docx/test/partial.docx
diff --git a/packages/jsreport-docx/test/test.js b/packages/jsreport-docx/test/test.js
@@ -158,6 +158,34 @@ describe('docx', () => {
     ])
   })
 
+  it('handlebars-partials', async () => {
+    const result = await reporter.render({
+      template: {
+        engine: 'handlebars',
+        recipe: 'docx',
+        helpers: `
+          const h = require('handlebars')
+
+          h.registerPartial('test', '{{name}}')
+        `,
+        docx: {
+          templateAsset: {
+            content: fs.readFileSync(
+              path.join(__dirname, 'partial.docx')
+            )
+          }
+        }
+      },
+      data: {
+        name: 'John'
+      }
+    })
+
+    fs.writeFileSync('out.docx', result.content)
+    const text = (await extractor.extract(result.content)).getBody()
+    text.should.containEql('Hello world John')
+  })
+
   it('invoice', async () => {
     const result = await reporter.render({
       template: {