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

Pulldata treats ".” and "_” as the same symbol in entity properties #6461

Closed
dbemke opened this issue Oct 17, 2024 · 4 comments
Closed
Milestone

Comments

@dbemke
Copy link

dbemke commented Oct 17, 2024

ODK Collect version

the master version 5dad7f4

Android version

10, 14

Device used

Redmi 9T, Pixel 7a

Problem description

It seems that pulldata treats ".” and "_” as the same symbol in properties. Opening an entities update form (using pulldata) with properties "circumference_cm” and "circumference.cm” causes an error.
errorPulldata

Steps to reproduce the problem

  1. Create a project with trees registration form with properties "circumference_cm” and "circumference.cm”, trees update pulldata form or use user "one” in https://staging.getodk.cloud/#/projects/101/app-users
    trees_update_pulldata.xlsx.txt
  2. Download the project and open trees update pulldata form.

Expected behavior

I'm not sure what's the expected behavior (e.g. OData also converts some punctuation marks to "_") but maybe in Collect it shouldn't ?

@lognaturel
Copy link
Member

I believe that in the short term #6451 will address this. Maybe you can check it again when merged?

@dbemke
Copy link
Author

dbemke commented Oct 18, 2024

I believe that in the short term #6451 will address this. Maybe you can check it again when merged?

I checked it and in #6451 the issue occurs.

@lognaturel lognaturel added this to the v2024.3 milestone Oct 22, 2024
@lognaturel
Copy link
Member

I think that may mean that the pulldata database is still being built in addition to the entities database even though it's not used. I'm somewhat surprised by this so we should probably see what's going on.

@lognaturel
Copy link
Member

Closing in favor of the more general #6471

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

No branches or pull requests

3 participants