- implement fixes/changes in issue #44 to correct zero becomming empty

- test syntax fixes

Author: DaSpawn

lib/worksheet.js

@@ -226,7 +226,7 @@ XlsxStreamReaderWorkSheet.prototype._handleWorkSheetNode = function (nodeData) {
           var index = parseInt(workingVal)
           workingVal = self.workBook._getSharedString(index)
 
-          self.workingRow.values[cellNum] = workingVal || ''
+          self.workingRow.values[cellNum] = (workingVal || workingVal === 0) ? workingVal : ''
 
           workingCell = {}
           break
@@ -269,7 +269,7 @@ XlsxStreamReaderWorkSheet.prototype._handleWorkSheetNode = function (nodeData) {
             }
           }
 
-          self.workingRow.values[cellNum] = workingVal || ''
+          self.workingRow.values[cellNum] = (workingVal || workingVal === 0) ? workingVal : ''
 
           workingCell = {}
       }

test/issue_44_empty_0.xlsx

@@ -125,6 +125,22 @@ describe('The xslx stream parser', function () {
       rows.length || done(new Error('Read nothing'))
     })
   })
+  it('parse 0 as 0', function (done) {
+    const workBookReader = new XlsxStreamReader()
+    fs.createReadStream(path.join(__dirname, 'issue_44_empty_0.xlsx')).pipe(workBookReader)
+    const rows = []
+    workBookReader.on('worksheet', function (workSheetReader) {
+      workSheetReader.on('end', function () {
+        assert.strictEqual(rows[1][1], 0)
+        assert.strictEqual(rows[1][2], 1)
+        done()
+      })
+      workSheetReader.on('row', function (r) {
+        rows.push(r.values)
+      })
+      workSheetReader.process()
+    })
+  })
 })
 
 function consumeXlsxFile (cb) {

test/test.js

@@ -7,6 +7,6 @@ module.exports = {
     fs: require('fs')
   },
   output: {
-    filename: 'xlsx-stream-reader.bundle.js'       
+    filename: 'xlsx-stream-reader.bundle.js'
   }
-};
+}