kobotoolbox / kpi / 29816039201 / 5

82%

master: 76%

push

github

fix(xlsForm): keep leading "=" in labels as text instead of a formula DEV-1235 (#7282)

### 📣 Summary

Question and choice labels that start with an equals sign (e.g. `=foo`)
now display and export correctly instead of turning into `0` or a
spreadsheet error.

### 📖 Description

If a label began with `=`, it was mistaken for a spreadsheet formula.
The deployed form showed `0` in place of the label in Enketo, and
exporting the form to XLSForm produced an error (`Err:520`) where the
label should be. Labels (and other cells) starting with `=` are now kept
as plain text everywhere.

### 💭 Notes

- Root cause: `Asset.to_xlsx_io()` builds the workbook with
`xlsxwriter`, whose generic `write()` routes any `=`-prefixed string to
`write_formula()`. The deploy path feeds the same in-memory xlsx to
pyxform, so one bug corrupted **both** the XLSForm export and the
deployed XForm (pyxform reads the formula's cached `0`).
- Fix: pass `{'strings_to_formulas': False}` to the
`xlsxwriter.Workbook`. XLSForm cells are never Excel formulas, so this
is safe; it's workbook-scoped, so it also covers `+`/`-`/`@` prefixes
and the survey, choices and settings sheets.
- Added regression test
`test_to_xlsx_io_preserves_leading_equals_in_label`.

### 👀 Preview steps

1. ℹ️ have an account and a project
2. add a text question with the label `=foo`, then deploy the form
3. 🔴 [on main] open the form in Enketo → the label shows `0`; export the
form to XLSForm → the cell shows `Err:520`
4. 🟢 [on PR] open the form in Enketo → the label shows `=foo`; export
the form to XLSForm → the cell shows `=foo`

4565 of 12627 branches covered (36.15%)

18974 of 37393 relevant lines covered (50.74%)

0.51 hits per line