officeparser 5.2.1 → 5.2.2

package/officeParser.js

@@ -235,12 +235,19 @@ function parseExcel(file, callback, config) {
                     && cNode.getElementsByTagName("v")[0].childNodes[0].nodeValue != ''
             }
 
-            /** Find text nodes with t tags in sharedStrings xml file. If the sharedStringsFile is not present, we return an empty array. */
-            const sharedStringsXmlTNodesList = xmlContentFilesObject.sharedStringsFile != undefined ? parseString(xmlContentFilesObject.sharedStringsFile).getElementsByTagName("t")
-                                                                                                    : [];
+            /** Find text nodes with t tags in sharedStrings.xml file. If the sharedStringsFile is not present, we return an empty array. */
+            const sharedStringsXmlSiNodesList = xmlContentFilesObject.sharedStringsFile != undefined
+                ? parseString(xmlContentFilesObject.sharedStringsFile).getElementsByTagName("si")
+                : [];
+
             /** Create shared string array. This will be used as a map to get strings from within sheet files. */
-            const sharedStrings = Array.from(sharedStringsXmlTNodesList)
-                                    .map(tNode => tNode.childNodes[0]?.nodeValue ?? '');
+            const sharedStrings = Array.from(sharedStringsXmlSiNodesList)
+                .map(siNode => {
+                    // Concatenate all <t> nodes within the <si> node
+                    return Array.from(siNode.getElementsByTagName("t"))
+                        .map(tNode => tNode.childNodes[0]?.nodeValue ?? '') // Extract text content from each <t> node
+                        .join(''); // Combine all <t> node text into a single string
+                });
 
             // Parse Sheet files
             xmlContentFilesObject.sheetFiles.forEach(sheetXmlContent => {
@@ -261,13 +268,14 @@ function parseExcel(file, callback, config) {
                                 /** Flag whether this node's value represents an index in the shared string array */
                                 const isIndexInSharedStrings = cNode.getAttribute("t") == "s";
                                 /** Find value nodes represented by v tags */
-                                const value = parseInt(cNode.getElementsByTagName("v")[0].childNodes[0].nodeValue, 10);
+                                const value = cNode.getElementsByTagName("v")[0].childNodes[0].nodeValue;
+                                const valueAsIndex = Number(value);
                                 // Validate text
-                                if (isIndexInSharedStrings && value >= sharedStrings.length)
+                                if (isIndexInSharedStrings && (valueAsIndex != parseInt(value, 10) || valueAsIndex >= sharedStrings.length))
                                     throw ERRORMSG.fileCorrupted(file);
 
                                 return isIndexInSharedStrings
-                                        ? sharedStrings[value]
+                                        ? sharedStrings[valueAsIndex]
                                         : value;
                             }
                             // Should not reach here. If we do, it means we are not filtering out items that we are not ready to process.
@@ -372,13 +380,19 @@ function parseOpenOffice(file, callback, config) {
             function traversal(node, xmlTextArray, isFirstRecursion) {
                 if (!node.childNodes || node.childNodes.length == 0) {
                     if (node.parentNode.tagName.indexOf('text') == 0 && node.nodeValue) {
+                        // If the corresponding value is of type float, we take the value from office:value attribute.
+                        // However, it is not on the parentNode but rather grandparentNode.
+                        const value = node.parentNode.parentNode?.getAttribute('office:value-type') == 'float'
+                                        ? Number(node.parentNode.parentNode.getAttribute('office:value'))
+                                        : node.nodeValue;
+
                         if (isNotesNode(node.parentNode) && (config.putNotesAtLast || config.ignoreNotes)) {
-                            notesText.push(node.nodeValue);
+                            notesText.push(value);
                             if (allowedTextTags.includes(node.parentNode.tagName) && !isFirstRecursion)
                                 notesText.push(config.newlineDelimiter ?? "\n");
                         }
                         else {
-                            xmlTextArray.push(node.nodeValue);
+                            xmlTextArray.push(value);
                             if (allowedTextTags.includes(node.parentNode.tagName) && !isFirstRecursion)
                                 xmlTextArray.push(config.newlineDelimiter ?? "\n");
                         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "officeparser",
-  "version": "5.2.1",
+  "version": "5.2.2",
   "description": "A Node.js library to parse text out of any office file. Currently supports docx, pptx, xlsx, odt, odp, ods, pdf files.",
   "main": "officeParser.js",
   "files": [