For some launch container types, trying to open a student's LTI submission for review as an instructor does not work. Instead of a LtiSubmissionReviewRequest Moodle makes an LtiResourceLinkRequest.

For example, with launch container 'Existing window', when I click on a link to a student submission in a user report, Moodle opens `/mod/lti/view.php?id=1292947&action=gradeReport&user=174015`, which then redirects to `/mod/lti/launch.php?id=1292947`. The `action=gradeReport&user=174015` part, which triggers LtiSubmissionReviewRequest has disappeared.

A similar issue happens with container type 'New window'. Moodle redirects to `/mod/lti/launch.php?id=1239690&triggerview=0&action=gradeReport&user=212559`, where the escaping of ampersands as `&` breaks the URL parsing.

There is no such issue with container types 'Embed' and 'Embed, without blocks'.

I suspect the cause is some faulty/incomplete launch URL handling logic in `/mod/lti/view.php`.