Add background-geopoint question type which exposes xforms-value-changed event with odk:setgeopoint action
#726
Commits on Oct 19, 2024
-
add new XLSForm question type called background-geopoint which saves …
…an odk:setgeopoint action in response to an xforms-value-changed event
-
-
-
consolidate validation for trigger values with validation for section…
… name uniqueness in survey class
-
add more tests for background-geopoint questions when used with other…
… setvalue question types and also when used in groups and repeats
-
chg: improvements for the addition of background-geopoint
- builder.py: - de-duplicate elif call to create_question_from_dict - use relevant class dictionaries directly instead of passing around - replace regex with strip() to avoid replacing whitespace everywhere in the question name (whitespace invalid in question names). - question.py: - avoid calling self.get_root() repeatedly as it iterates the form (up the tree from the current question). - remove `if` block in nest_set_nodes since it is only called when there are items to be processed. - question_types.py: - add validation functions for workbook_to_json. used external strings for re-use, easier testing, and (maybe someday) i18n. - xls2json.py: - move background-geopoint questions to workbook_to_json from survey.py and question.py to allow row references in errors - test_background_geopoint.py: - change the test class name to match unittest idiom / test discovery - remove `note` question from all tests since this is not relevant to the feature being tested or any specific edge cases of interest. - remove xpath expressions testing only the presence of groups, since these are tested already as part of the xpath to find the inputs within the groups. - readability improvements: - structured test method names - state the overall test case expectation in docstring - indicate the topic of the xpaths - wrap long xpaths and use single-quotes for predicates - shorten test fixture names and tidy up markdown formatting
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.