[MRI violations] Fix multiple rows for file protocol violations not resolvable

[cmadjar]

Brief summary of changes

This fixes a bug where the violation could not be resolved when there is more than one Protocol Violation returned for a file/timerun combination. The fixes include the following:

• only show one row per pname/timeRun/mincFile/scanType in the main menu filter table when there are multiple Protocol Violation for a given file (the list of violations will be displayed when clicking on the Protocol Violation link.
• when selecting a resolution, loop through all entries in mri_violations_log matching the timeRun/mincFile and update the resolution status for all matching entries in the violations_resolved table.

Testing instructions

For the raisinbread dataset, the following images match what this PR is fixing:

• assembly/300170/V1/mri/native/demo_300170_V1_dwi65_001.mnc
• assembly/300170/V1/mri/native/demo_300170_V1_dwi25_001.mnc

In current 25 code, should see two rows displayed in the main menu filter with the same content for the example images listed above and it should not be possible to save the resolution status for those files.

In code pulled from this PR, only one row per file will be displayed in the main menu filter and the resolution status will save. Look into the violations_resolved table and you should see 2 entries added for the file that was resolved.

Link(s) to related issue(s)

• Resolves [mri_violations] Cannot resolve any MRI violation #8626

[nicolasbrossard]

Code looks good. The test detailed in the PR description works fin on my VM after I pulled Cecile's code. Approving.

[cmadjar]

@driusan ready for you :-)