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

Find and fix uses of ccx.overrides.get_current_request #91

Closed cewing closed 9 years ago

cewing commented 9 years ago

If this method is used anywhere, fix it so that it does not rely on the threadlocal _CCX_CONTEXT.

If it is not used anywhere, delete the code.

cguardia commented 9 years ago

It was used, but there was an existing xmodule method that did the same, so using that now. Code removed (fix in ce3833d69af56aa40b69cd2c68f54de9cee2c46f).

cguardia commented 9 years ago

@pdpinch this one is done.