changed


On Thu, May 8, 2014 at 1:32 PM, Calen Pennington
<notifications@github.com>wrote:

> In lms/djangoapps/instructor/views/tools.py:
>
> > @@ -208,14 +208,16 @@ def dump_student_extensions(course, student):
> >          student_id=student.id)
> >      for module in query:
> >          state = json.loads(module.state)
> > -        if module.module_state_key not in units:
> > +        # temporary hack: module_state_key is missing the run but units are not. fix module_state_key
> > +        module_loc = course.id.make_usage_key(module.module_state_key.category, module.module_state_key.name)
>
> I guess I preferred the map_into_course route because it seems like it's
> making it more clear which data was wrong/missing. You could also use
> map_into_course(module.course_id), so that it's obvious that you want the
> course that the module was recorded in.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/3645/files#r12441179>
> .
>