Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cfspreadsheet action="read" format="csv" ignores empty fields in xls/xlsx file #53

Open
xouqoa opened this issue Jul 18, 2013 · 0 comments

Comments

@xouqoa
Copy link
Contributor

xouqoa commented Jul 18, 2013

If you read an Excel file which has empty cells and use the CSV format, the output from the CSV function ignores any empty fields.

I believe the cause of this is in the parseRowData function in Spreadsheet.cfc, line #2869.

With the change below, it should pick up the empty ones. (Untested, not sure how to replace the file once the extension is installed, and stupid IIS won't let me upload the zip file locally on my machine.)

-- var elements = listToArray( arguments.line, arguments.delimiter );

++ var elements = listToArray( arguments.line, arguments.delimiter, true );

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant