Merge pull request #3 from bureson/master

Complain about empty sheets
larsthorup · Jan 15, 2020 · e011bc8 · e011bc8
2 parents 01cd6c6 + 4a1a1a6
commit e011bc8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "sheet-reader",
-  "version": "1.0.16",
+  "version": "1.0.17",
   "description": "Easily use a spreadsheet as an input format for your app",
   "scripts": {
     "lint": "semistandard --verbose",

diff --git a/sheet-reader.js b/sheet-reader.js
@@ -26,6 +26,7 @@ function parseWorkbook (workbook) {
   return R.fromPairs(workbook.SheetNames.map(sheetKey => {
     const workbookSheet = workbook.Sheets[sheetKey];
     const workbookSheetRangeString = workbookSheet['!ref'];
+    if (!workbookSheetRangeString) throw new Error(`Sheet "${sheetKey}" appears to be empty`);
     const workbookSheetRange = parseRange(workbookSheetRangeString);
     const workSheet = XLSX.utils.sheet_to_json(workbookSheet);
     const tolerance = 1000;