xlsx-template-browser 0.1.5 → 0.1.8

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())
 }
 
 /**
@@ -27,7 +27,7 @@ const dateToExcel = (value) => {
  *                               or undefined if the data type is not recognized.
  */
 export const guessDataType = (value) => {
-  if (typeof value === 'undefined') return undefined
+  if (typeof value === 'undefined') return { cellType: 's', value: '' }
   if (typeof value === 'number') {
     if (isFinite(value)) return { cellType: 'n', value }
     return undefined
@@ -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) }
 }
 
 /**
@@ -65,6 +65,10 @@ export const createSharedString = (text, template) => {
  * @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
+  }
   // If the cell is formula, then skip
   if (cellType === 'f') {
     // Check precalculated value
@@ -118,3 +122,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/helpers.js

@@ -57,6 +57,7 @@ export const getByNotation = (obj, props) => {
   let accessors = props.match(notationRegex)
   if (!accessors.length) return obj
   accessors = accessors.map(el => /^\d+$/.test(el) ? parseInt(el) : el)
-  return getDeep(obj, accessors)
+  const value = getDeep(obj, accessors)
+  return value || ''
 }
 

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
@@ -70,7 +80,7 @@ const replace = async (template, data) => {
     // If we have table values, we need to process them in a separate way
     const isTable = typeof table === 'string'
     let value = getByNotation(data, accessor)
-    if (!value) return false
+    if (typeof value === 'undefined') return false
     // Save tables for further processing
     if (isTable) {
       valuesToReplace[i] = { isTable, value: value.map(guessDataType) }
@@ -127,13 +137,19 @@ const replace = async (template, data) => {
         // Get the cell value tag
         let v = c.querySelector('v') || {}
         // Check if the cell contains formula, and skip
-        let f = c.querySelector('f')
-        if (f) {
+        let isFormula = c.querySelector('f')
+        if (isFormula) {
           const value = c
           const cellType = 'f'
           newCells[newIndex] = Object.assign({}, { value, cellType }, { template: c })
           return false          
         }
+        // Check if the cell contains string
+        let isString = c.getAttribute('t')
+        if (!isString || isString !== '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.5",
+  "version": "0.1.8",
   "description": "On-the-fly generation of .xlsx (Excel) files directly within the browser using templates constructed in Excel.",
   "main": "./lib/index",
   "files": [