The original `remap_namespace` function had a `module.save()` after performing these actions (it'd be `course.save()` in this case).  I don't see a call to save() anywhere; does that not matter anymore?  Also there was some more code in remap_namespace (L:516 through L:545 in the original diff) that was not moved here; is that functionality no longer necessary?  (The answer to both these questions may well be "yes", I just want to be sure we're not going to have a regression from throwing out such a large function)