search

jazkarta / edx-platform

the edX learning management system (LMS) and course authoring tool, Studio
http://code.edx.org/
GNU Affero General Public License v3.0
1 stars 0 forks source link

Fix the `get_current_ccx` method #90

Closed cewing closed 9 years ago

cewing commented 9 years ago

The current implementation uses a threadlocal to determine the active CCX. Instead it should use the matched course_id from the URL. The course_id should be passed as an argument. If it's a CCXLocator, then return the CCX it identifies. If not, return None (no active ccx)

cewing commented 9 years ago

This is now fixed. Correctly handles both strings and CourseKey subclasses. Closing as completed, @pdpinch