WIP: initial attempt at a solution for skipping override checks via _linea…

[cguardia]

…ge when ccx is not enabled

[amir-qayyum-khan]

@cguardia here is coverage report, need to look into coverage of field_overrides.py (75 %)

[cpennington]

@cewing: my gut is no, that filtering the providers and not doing the lineage is going to be significantly better. That said, we'd want to be sure that we aren't filtering the list of providers more than one per request (for instance), and that we aren't requiring each provider to instantiate the course on their own.

For instance, we could put the set of provider classes in cls.provider_classes, but produce a filtered version of that list each time we instantiate the block. And we can cache (using RequestCache, for instance, or a thread-local object) the mapping of course -> enabled providers, so that we don't have to call the provider API for every course for every block.

[amir-qayyum-khan]

@pdpinch list is update

there are coverage issues

• lms/djangoapps/ccx/overrides.py (85.7%): Missing lines 38,42-43,67
• lms/djangoapps/ccx/utils.py (50.0%): Missing lines 266,271
• lms/djangoapps/ccx/views.py (77.1%): Missing lines 93,113,117,157,161-162,197,268-269,274,378-379,394,437,494,533
• lms/djangoapps/course_structure_api/v0/views.py (98.0%): Missing lines 435,564
• lms/djangoapps/courseware/module_render.py (79.3%): Missing lines 761,815,818-819,1025-1026

[cguardia]

Made this into a WIP PR on edx:

https://github.com/edx/edx-platform/pull/8519