Check for strings that may be interpreted as numbers (fixes SheetJS#2609

)
AbhiramTS · Jan 18, 2023 · f2b5fcc · f2b5fcc
1 parent 5b4806b
commit f2b5fcc
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/bits/90_utils.js b/bits/90_utils.js
@@ -110,10 +110,13 @@ function make_csv_row(sheet/*:Worksheet*/, r/*:Range*/, R/*:number*/, cols/*:Arr
 			isempty = false;
 			txt = ''+(o.rawNumbers && val.t == "n" ? val.v : format_cell(val, null, o));
 			for(var i = 0, cc = 0; i !== txt.length; ++i) if((cc = txt.charCodeAt(i)) === fs || cc === rs || cc === 34 || o.forceQuotes) {txt = "\"" + txt.replace(qreg, '""') + "\""; break; }
+			if (parseInt(txt) == txt) txt = "\"" + txt + "\"";
 			if(txt == "ID") txt = '"ID"';
 		} else if(val.f != null && !val.F) {
 			isempty = false;
-			txt = '=' + val.f; if(txt.indexOf(",") >= 0) txt = '"' + txt.replace(qreg, '""') + '"';
+			txt = '=' + val.f;
+			if (parseInt(val.f) == val.f) txt = "\"" + val.f + "\"";
+			if(txt.indexOf(",") >= 0) txt = '"' + txt.replace(qreg, '""') + '"';
 		} else txt = "";
 		/* NOTE: Excel CSV does not support array formulae */
 		row.push(txt);