INGInious icon indicating copy to clipboard operation
INGInious copied to clipboard

[frontend/courses] registration to an already registered course makes the app crash

Open anthonygego opened this issue 1 year ago • 0 comments
trafficstars

Describe the bug If the course registration page is accessed when it is not relevant (no registration possible or user already registered), an Exception is raised.

The issue comes from these lines :

https://github.com/UCL-INGI/INGInious/blob/fcb729ede7d23a44df27bc328591c22bacc860cb/inginious/frontend/pages/course_register.py#L29-L30

as the method should actually returns a tuple. It should date from the web.py to flask migration when redirect were made with raise statement, while flask actually treat them as typical Response objects.

INGInious installation details

  • Version v0.8+

To Reproduce Steps to reproduce the behavior:

  1. Copy the course registration link : /register/<course_id>
  2. Register to the course.
  3. Access the registration link once registered.

Expected behavior The initial redirection or any other feedback message.

Reported by: @Donaschmi

anthonygego avatar Aug 27 '24 13:08 anthonygego