Kotlin DSL configuration causes incorrect column parsing with xlsx intermediate format about android-gradle-localization-plugin

[koral--]

I mean using:
csvFileURI = "https://docs.google.com/spreadsheets/d/REDACTED/export?format=csv"
Intermediate = between source spreadsheet and result XML.

[koral--]

OK, thanks so there is a bug in XLS(X) parsing. I'll investigate it.

[koral--]

I was able to reproduce this issue.
On spreadsheet used in unit tests, created in 2015 everything works fine. However, a brand new document with copy-pasted content behaves differently.

Some cells eg. never touched rows or columns, are expected to be nulls or not to exist at all. However, they are returned as empty strings with current settings.
Such behavior is permitted by documentation so we need to adapt to this. I'll investigate how to solve it (the naive solution is to filter out empty rows and columns).

For now workaround is to use CSV (in case of Google sheets) or (if formulas are needed or input comes from XLS-only source) to remove empty columns and rows. Not sure how it looks like in Excel, probably it is sufficient to delete empty columns which were touched.

[koral--]

Should be fixed in v1.0.19.
Now rows and columns containing only effectively empty (incl. missing and nulls) cells are ignored.
If column starts with empty cell but contains non-empty row then explicit error is thrown unless column is configured to be ignorable.

[koral--]

Is it reproducible if you change intermediate format to CSV?

[sp0rk]

Do you mean changing xlsFileURI to csvFileURI and hosting the translations differently? I don't understand what you mean by intermediate format. I have not played around with csvGenerationCommand if that's what you mean

[sp0rk]

When changing xls to csv as the intermediate format the issue disappears. So it seems to be only present with xlsFileUri / ?format=xlsx

[sp0rk]

No worries, please ping me back here once you patch this up. Cheers 🍻

[sp0rk]

Wow. Thanks for swift action. Cheers :)