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

mri_protocol table's ScannerID is set to 0 in RaisinBread and not nullable #7476

Closed
cmadjar opened this issue Jun 11, 2021 · 0 comments
Closed
Assignees
Labels
Category: Bug PR or issue that aims to report or fix a bug Language: SQL PR or issue that update SQL code

Comments

@cmadjar
Copy link
Collaborator

cmadjar commented Jun 11, 2021

The ScannerID field of the mri_protocol table has been modified to add a foreign key constraint with the ID field of the mri_scanner table.

Historically, we used to set the ScannerID field of the mri_protocol table to 0 when a protocol is valid on all study's scanners. Instead of setting the ScannerID field to 0 when an MRI protocol should be valid across all scanners, we should allow for NULL values in the ScannerID field of the mri_protocol table.

TODOs:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Bug PR or issue that aims to report or fix a bug Language: SQL PR or issue that update SQL code
Projects
None yet
Development

No branches or pull requests

2 participants