Trailing white space causes bug.

KeeTraxx · Sep 23, 2015 · 0c47aef · 0c47aef
1 parent eeaee07
commit 0c47aef
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 2 deletions.
diff --git a/lib/Sheet.js b/lib/Sheet.js
@@ -150,7 +150,7 @@ function Sheet(basedir, file, contents, sharedStrings) {
             if (cell.$.t == 's') {
                 return sharedStrings[parseInt(cell.v)].t[0];
             } else if(cell.f && cell.v) {
-                return cell.v[0];
+                return cell.v[0].hasOwnProperty('_') ? cell.v[0]._ : cell.v[0];
             } else if (cell.f) {
                 // formula
                 return '=' + cell.f[0];

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "kexcel",
-  "version": "2.0.12",
+  "version": "2.0.13",
   "description": "Create or open and then edit existing XLSX files",
   "keywords": [
     "xlsx",

diff --git a/test/functionvalues.js b/test/functionvalues.js
@@ -18,4 +18,8 @@ describe('Return values instead of functions', function () {
         workbook.getSheet(0).getCellValue(1,1).should.equal('42');
     });
 
+    it('Trailing white space test', function () {
+        workbook.getSheet(0).getCellValue(2,1).should.equal('test with trailing white space ');
+    });
+
 });
diff --git a/test/input-files/42.xlsx b/test/input-files/42.xlsx