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

fix: Investigate the reason of differences in events x enrollments in analytics query[2.39-DHIS2-16227-backport] #17750

Merged
merged 2 commits into from
Jun 10, 2024

Conversation

d-bernat
Copy link
Contributor

@d-bernat d-bernat commented Jun 10, 2024

Backport
There is the problem with sorting of the repeatable stages. The issue was pinpointed to the event report date in the repeatable stage. The capture app allowed users to add new events in the repeatable stage. The report date (found in the executiondate column of the event table in the tracker database) is treated as a date without a time section. When analytics queries for repeatable stage events using an position index, the executiondate is crucial for ensuring the correct outcome, as it serves as the sorting criteria. Sorting on non-unique data can yield inconsistent results. This is the scenario described in the ticket. Report dates of events within the repeatable stage are not unique, and sorting may yield different outcomes.

@d-bernat d-bernat changed the title fix: Investigate the reason of differences in events x enrollments in analytics query[2.40-DHIS2-16227] [DRAFT]fix: Investigate the reason of differences in events x enrollments in analytics query[2.40-DHIS2-16227] Jun 10, 2024
@d-bernat d-bernat changed the title [DRAFT]fix: Investigate the reason of differences in events x enrollments in analytics query[2.40-DHIS2-16227] [DRAFT]fix: Investigate the reason of differences in events x enrollments in analytics query[2.39-DHIS2-16227-backport] Jun 10, 2024
@d-bernat d-bernat changed the title [DRAFT]fix: Investigate the reason of differences in events x enrollments in analytics query[2.39-DHIS2-16227-backport] fix: Investigate the reason of differences in events x enrollments in analytics query[2.39-DHIS2-16227-backport] Jun 10, 2024
Copy link

sonarcloud bot commented Jun 10, 2024

@d-bernat d-bernat merged commit d600ed1 into 2.39 Jun 10, 2024
9 checks passed
@d-bernat d-bernat deleted the 2.39-DHIS2-16227-backport branch June 10, 2024 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants