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

Some files that won't convert #2

Closed
nathanathan opened this issue Feb 19, 2013 · 3 comments
Closed

Some files that won't convert #2

nathanathan opened this issue Feb 19, 2013 · 3 comments

Comments

@nathanathan
Copy link
Contributor

xlsx files exported from google docs seem to cause errors. I was able to convert this workbook when it was in it's original excel format.

Also, I'm getting an apparently unrelated error when I try to convert this xlsx file generated by excel.

@redchair123
Copy link

(both are clearly bugs and I'm currently testing a patch)

Quick summary of what happened:

In the first case (from gdocs), neither the core properties nor the worksheet dimensions were emitted by google docs (both of which I assumed were included in every xlsx workbook).

The fix is to have the parser deduce the dimensions if its not provided (and to use sheet names from the workbook manifest if the properties are not provided)

In the second case (from excel), there are empty cells in places like E1,F1,D2,E2,F2,... and according to the schema those cells shouldn't have been generated in the first place (in the XLS format there's a special type of cell for the cells that haven't been specified). In this case, I forced it to emit undefined (and updated sheet_to_row_object_array to ignore undefined elements)

@redchair123
Copy link

@nathanathan thanks a lot and keep the issues coming :) Also, can you tell me what version of excel you used for the second file?

@nathanathan
Copy link
Contributor Author

Thanks!
The excel version I was using: 14.0.6129.5000 (32-bit)

redchair123 added a commit that referenced this issue Mar 10, 2013
redchair123 added a commit that referenced this issue Oct 10, 2013
SheetJSDev added a commit that referenced this issue Dec 6, 2013
godu pushed a commit to CoorpAcademy/js-xlsx that referenced this issue Aug 13, 2015
Add syntax highlighting to readme examples
clarketm pushed a commit to clarketm/js-xlsx that referenced this issue Nov 27, 2016
Fixes SheetJS#2

Also added a quick patch to handle unknown types
@SheetJS SheetJS locked and limited conversation to collaborators Feb 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants