catalyst / moodle-mod_scormremote

A Moodle activity for serving SCORM packages remotely to another LMS across domains
https://moodle.org/plugins/mod_scormremote
Other
9 stars 3 forks source link

Errors reported by error.php require Moodle login #53

Closed leonstr closed 5 months ago

leonstr commented 1 year ago

mod/scormremote/error.php sets the HTTP status with a corresponding message, for example, 402 | Subscription limit reached. But error.php has require_login() at the top (line 30) so the student never sees the error, just the login page of the Moodle site hosting the SCORM.

If I remove the require_login() students see any error messages (401, 402, etc.) as expected.

leonstr commented 5 months ago

Fixed by edc28b0.