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

PUSH 24.1.8 to 25.0-release #9299

Open
wants to merge 4 commits into
base: 25.0-release
Choose a base branch
from
Open

PUSH 24.1.8 to 25.0-release #9299

wants to merge 4 commits into from

Conversation

ridz1208
Copy link
Collaborator

DO NOT SQUASH

CamilleBeau and others added 4 commits February 1, 2024 14:53
This fixes the logic in the inactive users query for the assignee drop down of the issue tracker.

The query was missing brackets in the logic and was querying active users in the inactive users array. This resulted in all users being unset from the assignees object for users who do not have the "access_all_profiles" permission. With brackets, users without the "access_all_profiles" permission should be able to see the appropriate users in the assignee drop down of a new issue.
When mixing numbers with strings in an instrument field value (e.g. text field) results in a DQT column that has a sorting issues. This happened for a "summary" instrument in HBCD project that import results from multiple other instruments. This instrument has to clearly display the value of the field (i.e. numeric) or "No data" (i.e. string).
Update the select query from pselectRow to pselect on line 105 under [tools] detect_duplicated_commentids.php file. pselectRow throws an exception when there are duplicates.
@driusan
Copy link
Collaborator

driusan commented Jun 17, 2024

@ridz1208 there are conflicts in tools/detect_duplicated_commentids.php

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.

5 participants