#23 remove control characters from strings

protobi · Jun 8, 2022 · e5b2861 · e5b2861
1 parent 86b4dc2
commit e5b2861
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 4 deletions.
diff --git a/lib/msexcel-builder.js b/lib/msexcel-builder.js
diff --git a/lib/msexcel-builder.js.map b/lib/msexcel-builder.js.map
diff --git a/package.json b/package.json
@@ -6,7 +6,7 @@
     "xlsx",
     "excel"
   ],
-  "version": "0.4.9",
+  "version": "0.4.10",
   "repository": {
     "type": "git",
     "url": "git://github.com/protobi/msexcel-builder.git"

diff --git a/src/msexcel-builder.coffee b/src/msexcel-builder.coffee
@@ -855,7 +855,8 @@ class Sheet
               @[key] col, row, str[key]
           else console.error('Ignoring ', key, col, row, str[key])
       else if  typeof str == 'string'
-        if str != null and str != ''
+        if str != null and str != '' and str != undefined
+          str = str.replace(/[\u0000-\u001F\u007F-\u009F\u200b]/g, "")
           @data[row][col].v = @book.ss.str2id('' + str)
           return @data[row][col].dataType = 'string'