xlsx-template-browser 0.1.4 → 0.1.6

package/lib/excelHelpers.js

@@ -7,7 +7,7 @@ export const valueToString = (value) => {
   if (!value) return ''
   if (Array.isArray(value)) return value.map(valueToString).toString()
   if (typeof value === 'object') value = JSON.stringify(value)
-  return value.toString()
+  return escapeXml(value.toString())
 }
 
 /**
@@ -37,7 +37,7 @@ export const guessDataType = (value) => {
     return { cellType: 'n', value: dateToExcel(value) }
   }
   if (typeof value === 'boolean') return { cellType: 'b', value: value + 0 }
-  return { cellType: 's', value: valueToString(value )}
+  return { cellType: 's', value: valueToString(value) }
 }
 
 /**
@@ -61,10 +61,14 @@ export const createSharedString = (text, template) => {
  * @param {string|number} options.row - The row reference for the cell.
  * @param {string|number} options.column - The column reference (either string or numeric) for the cell.
  * @param {Element} options.template - (Optional) An existing template to clone for the cell.
- * @param {string} options.cellType - The type of the cell (e.g., 's', 'n', 'b') for Excel formatting.
+ * @param {string} options.cellType - The type of the cell (e.g., 's', 'n', 'b', 'f') for Excel formatting.
  * @returns {Element} - The created cell element.
  */
 export const createCell = ({ value, row, column, template, cellType }) => {
+  // If the cell is not a string, then skip
+  if (!cellType && !value) {
+    return template
+  }
   // Clone initial node with the style or create a new one
   const cell = template ? template.cloneNode() : document.createElement('c')
   // Create a cell reference
@@ -111,3 +115,24 @@ export const getExcelColumnIndex = (columnName) => {
   return index
 }
 
+/**
+ * Escapes unsafe XML characters in a given string to ensure its safe inclusion
+ * in XML documents, preventing parsing issues.
+ *
+ * @param {string} unsafe - The input string containing XML-unsafe characters.
+ * @returns {string} - A new string with XML-unsafe characters replaced by their
+ * corresponding HTML entities.
+ */
+
+const unsafeChars = {
+  '<': '&lt;',
+  '>': '&gt;',
+  '&': '&amp;',
+  '\'': '&apos;',
+  '"': '&quot;'
+}
+
+export const escapeXml = (unsafe) => {
+  return unsafe.replace(/[<>&'"]/g, (c) =>  { return unsafeChars[c] })
+}
+

package/lib/main.js

@@ -50,6 +50,16 @@ const replace = async (template, data) => {
   // We need to replace only text values with the new ones, therefore we process shared strings first
   // Get all string items
   xml.querySelectorAll('si').forEach((si, i) => {
+    // Get all rich format tags
+    const r = si.querySelector('r')
+    if (r) {
+      const xmlString = si.innerHTML
+      const newString = xmlString.replace(globalAccessorRegex, s => valueToString(getByNotation(data, s.replace(/\$\{|}/g, ''))))
+      si.innerHTML = newString
+      newSharedStrings.push(si)
+      valuesToReplace[i] = { value: newSharedStrings.length - 1, cellType: 's' }
+      return false
+    }
     // Get all text tags
     const t = si.querySelector('t')
     if (!t) return false
@@ -69,7 +79,8 @@ const replace = async (template, data) => {
     let { accessor, table } = match.groups
     // If we have table values, we need to process them in a separate way
     const isTable = typeof table === 'string'
-    const value = getByNotation(data, accessor)
+    let value = getByNotation(data, accessor)
+    if (!value) return false
     // Save tables for further processing
     if (isTable) {
       valuesToReplace[i] = { isTable, value: value.map(guessDataType) }
@@ -125,6 +136,12 @@ const replace = async (template, data) => {
         let newIndex = index + cellOffset
         // Get the cell value tag
         let v = c.querySelector('v') || {}
+        // Check if the cell contains string
+        let f = c.getAttribute('t')
+        if (!f || f !== 's') {
+          newCells[newIndex] = Object.assign({}, { template: c })
+          return false
+        }
         // Get the new value to replace
         let newValue = valuesToReplace[v.textContent] || {}
         if (!Array.isArray(newValue)) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "xlsx-template-browser",
-  "version": "0.1.4",
+  "version": "0.1.6",
   "description": "On-the-fly generation of .xlsx (Excel) files directly within the browser using templates constructed in Excel.",
   "main": "./lib/index",
   "files": [