Skip to content

Commit 740bf40

Browse files
authored
Merge pull request #1930 from stevengreens10/patch-2
Made options argument unnecessary in saveTable
2 parents d3344b4 + 34a4a96 commit 740bf40

File tree

1 file changed

+11
-4
lines changed

1 file changed

+11
-4
lines changed

src/io/files.js

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1083,7 +1083,8 @@ p5.prototype.saveBytes = function () {
10831083

10841084
};
10851085

1086-
// object, filename, options --> saveJSON, saveStrings, saveTable
1086+
// object, filename, options --> saveJSON, saveStrings,
1087+
10871088
// filename, [extension] [canvas] --> saveImage
10881089

10891090
/**
@@ -1372,15 +1373,21 @@ function escapeHelper(content) {
13721373
*
13731374
*/
13741375
p5.prototype.saveTable = function (table, filename, options) {
1375-
var pWriter = this.createWriter(filename, options);
1376+
var ext;
1377+
if(options === undefined){
1378+
ext = filename.substring(filename.lastIndexOf(".")+1,filename.length);
1379+
}else{
1380+
ext = options;
1381+
}
1382+
var pWriter = this.createWriter(filename, ext);
13761383

13771384
var header = table.columns;
13781385

13791386
var sep = ','; // default to CSV
1380-
if (options === 'tsv') {
1387+
if (ext === 'tsv') {
13811388
sep = '\t';
13821389
}
1383-
if (options !== 'html') {
1390+
if (ext !== 'html') {
13841391
// make header if it has values
13851392
if (header[0] !== '0') {
13861393
for (var h = 0; h < header.length; h++) {

0 commit comments

Comments
 (0)