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

[DQT] MRI importer incorrect parameters? #8829

Closed
wants to merge 2 commits into from

Conversation

ridz1208
Copy link
Collaborator

@ridz1208 ridz1208 commented Jul 6, 2023

Brief summary of changes

Trying to run the mri importer for couchdb gives me this error on demo.loris.ca

PHP Fatal error:  Uncaught TypeError: join(): Argument #2 ($array) must be of type ?array, string given in /var/www/loris/tools/CouchDB_MRI_Importer.php:443
Stack trace:
#0 /var/www/loris/tools/CouchDB_MRI_Importer.php(443): join()
#1 /var/www/loris/tools/CouchDB_MRI_Importer.php(73): CouchDBMRIImporter->updateCandidateDocs()
#2 /var/www/loris/tools/CouchDB_MRI_Importer.php(778): CouchDBMRIImporter->run()

Not sure I understand how it suddently stopped working

@ridz1208 ridz1208 added Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) 24.0.0-bugs Issues or bug fix PRs that were raised during the testing of release 24.0.0 labels Jul 6, 2023
@ridz1208
Copy link
Collaborator Author

ridz1208 commented Jul 6, 2023

@driusan just realized this is already fixed on 25
#8369

@driusan
Copy link
Collaborator

driusan commented Oct 17, 2023

@ridz1208 good for loris 25!

Should this be closed or merged, then? (If the latter it has conflicts and needs to be rebased.)

@sruthymathew123
Copy link
Contributor

Closing this one as per the discussion in the meeting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
24.0.0-bugs Issues or bug fix PRs that were raised during the testing of release 24.0.0 Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants