tracker issue : CF-4159346

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

[ANeff] ER for: spreadsheet.getColumnCount() sheet parameter optional

| View in Tracker

Status/Resolution/Reason: Closed/Fixed/

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

Created: 05/31/2016

Components: Document Management, Office Integration

Versions: 2016

Failure Type: Enhancement Request

Found In Build/Fixed In Build: CF2016_Final /

Priority/Frequency: Normal / Unknown

Locale/System: English / Win All

Vote Count: 0

Listed in the version 2016.0.03.300466 Issues Fixed doc 
Currently spreadsheetGetColumnCount()/spreadsheet.getColumnCount() require the sheet name/number parameter.

It should be optional. It should default to the currently active sheet (i.e. the sheet specified via .setActiveSheet()/.setActiveSheetNumber()).

Repro:

<cfscript>
  ss = spreadsheetNew("mySheet");
  ss.addRows(["one,two,three","four,five,six","seven,eight,nine"], 1, 1);
  writeDump(ss.getColumnCount());
</cfscript>

Actual result: throws "The getColumnCount method was not found."

Expected result: displays mySheet's column count

(then, of course, the doc should be updated to indicate that parameter is optional)

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

Watson Bug ID:	4159346

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

Attachments:

Comments:

Hmm.. that error message is even wrong. It says "The getColumnCount method was not found." Well, obviously .getColumnCount() is implemented. :) ..so that error should probably say "The sheet name or number is required." (or something like that). BUT, no worries b/c this ER here would make that parameter optional - which would resolve the error message since it would no longer occur :) I guess it just reveals an underlying bug in how error messages are handled for member functions. It appears the default behavior is to say 'member function X was not found'. Which.. is wrong. Thanks!, -Aaron
Comment by External U.
2589 | May 31, 2016 03:26:57 AM GMT