tracker issue : CF-4051876

select a category, or use search below
(searches all categories and all time range)
Title:

[ANeff] Bug for: spreadsheetAddColumn() and spreadsheetAddRow() missing delimiter parameter

| View in Tracker

Status/Resolution/Reason: Closed/Won't Fix/

Reporter/Name(from Bugbase): Aaron Neff / Aaron Neff (Aaron Neff)

Created: 09/08/2015

Components: Language

Versions: 11.0

Failure Type:

Found In Build/Fixed In Build: CF11_Final /

Priority/Frequency: Major / Some users will encounter

Locale/System: English / Win All

Vote Count: 1

spreadsheetAddColumn() and spreadsheetAddRow() are missing the delimiter parameter

This:
SpreadsheetAddColumn(spreadsheetObj, data[, startRow, startColumn[, insert]][, datatype])
SpreadsheetAddRow(spreadsheetObj, data[,startRow, startColumn[, insert]][, datatype])

Should be this:
SpreadsheetAddColumn(spreadsheetObj, data[, delimiter][, startRow, startColumn[, insert]][, datatype])
SpreadsheetAddRow(spreadsheetObj, data[, delimiter][,startRow, startColumn[, insert]][, datatype])

Why?
Answer 1) SpreadsheetAddColumn() must have a delimiter parameter b/c it currently cannot set "a,b" into a single cell.
Answer 2) SpreadsheetAddRow() should then have a delimiter parameter to maintain consistency w/ SpreadsheetAddColumn().
Answer 3) All existing functions that take a list parameter currently also take a delimiter parameter, w/ the exception of the above 2 functions.

----------------------------- Additional Watson Details -----------------------------

Watson Bug ID:	4051876

External Customer Info:
External Company:  
External Customer Name: Aaron Neff
External Customer Email:

Attachments:

Comments:

I forgot spreadsheetAddRows()/spreadsheet.addRows() - but Adam's filed #CF-4051598 for that.
Comment by External U.
5936 | September 08, 2015 05:53:07 PM GMT
For consistency, the SpreadsheetAddRows() syntax could be: SpreadsheetAddRows(spreadsheetObj, data[, delimiter][, startRow, startColumn[, insert]])
Comment by External U.
5937 | September 08, 2015 06:09:30 PM GMT
+1 ..........................
Vote by External U.
5941 | September 13, 2015 03:57:27 PM GMT
IMO, Supporting list in SpreasheetAddRow and many other functions was a bad idea to begin with. To keep the functions clean, instead of adding 'delimiter' and 'ignoreEmptyFields' in every function, a better approach is to use ListToArray function and then pass that array here. Thoughts?
Comment by Rupesh K.
5938 | September 13, 2015 10:15:20 PM GMT
Hi Rupesh, Perfect! That'd solve spreadsheetAddColumn() and spreadsheetAddRow(). And spreadsheetAddRows() could accept an array of arrays? (Adam's idea in #CF-4051598) Thanks!, -Aaron
Comment by External U.
5939 | September 13, 2015 11:12:10 PM GMT
Yes, supporting array of arrays makes sense. We would close this bug and we would fix #CF-4051598 for supporting array of array
Comment by Rupesh K.
5940 | September 13, 2015 11:48:33 PM GMT