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.

Sign up for GitHub

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

Jump to bottom

[MRI Violations] No column named "TarchiveID" #5421

Closed
johnsaigle opened this issue Oct 30, 2019 · 2 comments
Closed

[MRI Violations] No column named "TarchiveID" #5421

johnsaigle opened this issue Oct 30, 2019 · 2 comments
Assignees
@cmadjar

Comments

@johnsaigle
Copy link
Contributor

TestPlan:

MRI Protocol Check Violations page - check Selection Filter results for each of Tarchive ID, PatientName, CandID and DICOM Series UID [Manual Testing]

There is a filter called TarchiveID but it's not clear what column that corresponds to.

Entering data in the filter causes some rows to disappear so it appears that it's operating on something.

Not sure if this is a UI bug or a DB issue.
@cmadjar
Copy link
Collaborator

cmadjar commented Oct 30, 2019

@johnsaigle when we meet, things will get clearer (I hope). This is not really a bug.

@cmadjar cmadjar self-assigned this Nov 1, 2019
@johnsaigle johnsaigle mentioned this issue Nov 5, 2019
Merged
driusan pushed a commit that referenced this issue Nov 8, 2019
@johnsaigle @driusan
[MRI_Violations] Fixes permission bug; integration test to confirm bu…
1a25972 
…g fix; test plan update (#5546)

Allows users with the 'edit' permission to view the module even if they don't have 'view' permissions.

Adds an integration test to the above + code cleanup.

Rewrites the test plan to be up-to-date and easier to follow.

    Resolves #5419
    Resolves #5421
    Resolves #5422
@driusan
Copy link
Collaborator

driusan commented Nov 8, 2019

Fixed by above

@driusan driusan closed this as completed Nov 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cmadjar cmadjar

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@driusan @cmadjar @johnsaigle