[mri_violations] Fix links to BrainBrowser from the MRI violations module (#8392)

The links to BrainBrowser from the MRI violations module were not working for two reasons:

- queried the wrong table name for the protocol violations (mri_violations_log should have been used instead of mri_violation_log)
- the FileDownloadHandler class was using the basename of the file instead of the relative path which resulted in BrainBrowser not finding the file on the file system.

These bugs were found when testing #7473 but have been present since 24.0.

Author: cmadjar

modules/brainbrowser/php/imageinfo.class.inc

@@ -185,7 +185,7 @@ class ImageInfo extends \NDB_Page
                                      '' as Visit_label,
                                      MincFile as File,
                                      LogID as FileID
-                              FROM mri_violation_log
+                              FROM mri_violations_log
                               WHERE LogID IN (" . join(",", $params) .")";
                 break;
             case 'CandidateError':

php/libraries/FilesDownloadHandler.php

@@ -67,7 +67,9 @@ public function handle(ServerRequestInterface $request) : ResponseInterface
             );
         }
         //Use basename to remove path traversal characters.
-        $filename = basename(strval($request->getAttribute('filename')));
+        $filename = \Utility::resolvePath(
+            strval($request->getAttribute('filename'))
+        );
 
         if (empty($filename)) {
             return new \LORIS\Http\Response\JSON\BadRequest(