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

Invalid variable name error when trying to export a form with dynamic names/labels to .sav #1803

Closed
FrankApiyo opened this issue Apr 2, 2020 · 2 comments · Fixed by #1806

Comments

@FrankApiyo
Copy link
Member

FrankApiyo commented Apr 2, 2020

Problem description

.sav exports for form with dynamic name and labels in its choice sheet fail. The following error is logged to sentry: https://sentry.onalabs.org/ona/onadata/issues/70562/?query=is%3Aunresolved%20sav

Steps to reproduce the behavior

You need a form with select one or select mutlitple fields whose names/labels are dynamic and look something as follows: #3:_${Q19member1}. Collect data with this form and then try to export to .sav

Additional Information

Here's information regarding valid variable names in spss: https://www.ibm.com/support/knowledgecenter/SSLVMB_23.0.0/spss/base/syn_variables_variable_names.html. There's no mention of the { character. It probably should not be in a variable name.

Aha! Link: https://ona.aha.io/features/PROD-158

@DavisRayM DavisRayM changed the title Invalid variable name errro when trying to export a form with dynamic names/labels to .sav Invalid variable name error when trying to export a form with dynamic names/labels to .sav Apr 2, 2020
@DavisRayM
Copy link
Contributor

What would be the ideal way to handle dynamic labels when creating the sav export ? Would we need to just escape the special characters ? or would we need to replace the dynamic string with the correct value ?

CC: @ukanga

@DavisRayM DavisRayM self-assigned this Apr 6, 2020
@DavisRayM
Copy link
Contributor

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

Successfully merging a pull request may close this issue.

2 participants