OpignoModuleController::userResult() should use Module Status' user, not the current one

Problem/Motivation

When visiting a path for a module status (module/{opigno_module}/result/{user_module_status}) the controlling class assumes the current user, rather than the user from the UserModuleStatus passed as a parameter. This is not visible until manual scoring is used, in which LPs can be "stuck" as incomplete even though all modules have been scored as passing.

Steps to reproduce

1. Create a new Learning Path with a single module, and a single file upload activity
2. Require the activity to be manually scored
3. Have a user complete the activity
4. Manually score the user as passed
5. The user will show a completed Module Status, but an incomplete Learning Path status

Proposed resolution

Update the controller to rely on the user passed from the $user_module_status parameter.

Remaining tasks

Create patch.

User interface changes

None.

API changes

None.

Data model changes

None.