Skip to content

Conversation

@ridz1208
Copy link
Collaborator

DO NOT SQUASH

ridz1208 and others added 26 commits March 15, 2022 09:38
…aces#8048)

The LorisFormDictionaryImpl trait for instruments was using the $this->lorisinstance variable which happens to be null in survey instruments, replaced with $this->loris instanciated in the constructor of NDB_Page and now it works

Resolves aces#8021 
Resolves aces#8020
Fixes a syntax error in the database query for add permission functionality for unversioned files.

Resolves  aces#8023
…ing (aces#8045)

Change mkdir parameters to make the new directory executable. A non-executable directory can not be entered.

Added an if statement around mkdir to avoid warnings when directory already exists for the user.

    Resolves  aces#8035
Update test plan for issue tracker
Fix permission name in DQT test plan.
Fix permission name in user accounts test plan.
Fix behavioural qc test plan.
math function in smarty seems to be broken (deprecated?). Replate it with direct math as supported in smarty 3.

    Resolves  aces#8027
Removed some unrealistic expectations and redundant instructions
Updates the email component to use the pattern property for the email input. Also using the type property for the message to displayed to the user when email isn't matching the pattern property. See here: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/email

edit: I added frontend logic for if by any chance the php validation finds an error not caught by the frontend and so the user will get a notification that email is invalid.

    Resolves  aces#7897
It's not realistic to ask a tester to look at each configuration field to see if it can be changed successfully and it as all the "expected" effect on LORIS. This propose to look at one field of each type to see if the value can be changed and saved.
aces#7978)

The Description field of the annotation.json sidecar is recommended but not required.
…d "Email survey" - fix (aces#8053)

Add a "Create survey" check into the process function to determine if Create or Email survey was clicked.

Fixes aces#8041
…es#7956)

Add site and project filters to behavioural qc module.
Removes lines that are causing timeout & not accomplishing anything as discussed in aces#8039

Resolves aces#8039
Fix some function calls with incorrect arguments in:

- data_integrity/fix_candidate_age.php
- data_integrity/score_instrument.php
- lorisform_parser.php
Fix the datapicker default values for the "Date of Birth" and "Date of Birth Confirm" fields. Updates the testPlan.
This updates delete_timepoint and delete_candidate scripts to run smoothly.

    Resolves  aces#8059
Epochs should not be highlighted (in blue) in the Signal Viewer unless they are marked as visible using the eye button. This fixes bug where all epochs were being shown.

This fix was done by @laemtl on iEEG ATLAS, @jesscall is just pushing it to LORIS.
This changes the candidate_profile template that loads the widgets of the candidate_profile page so that it catches the errors when loading candidate visit information. Specifically, this handles the issue where a candidate is affiliated with multiple sites and the user only has access to one of those sites and the candidate_profile page does not load by filtering out inacccessible timepoints.

    Resolves aces#8040
This adds a test name cache in the getInstance function of the media provisioner to load each instrument only once instead of for each row.

Resolves aces#8033
@ridz1208 ridz1208 added the State: Blocked PR or issue awaiting an external event such as the merge or another PR to proceed label Apr 21, 2022
@ridz1208
Copy link
Collaborator Author

Blocked until @CamilleBeau verifies the effect of the last commit

826575f

@driusan
Copy link
Collaborator

driusan commented May 11, 2022

@CamilleBeau Any update on this?

@CamilleBeau
Copy link
Collaborator

@driusan @ridz1208 Tested 24 on CCNA with the changes from the last commit and media module is working 👍

@driusan driusan merged commit 8fbc01a into aces:main May 17, 2022
@ridz1208 ridz1208 added this to the 25.0.0 milestone May 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

State: Blocked PR or issue awaiting an external event such as the merge or another PR to proceed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants