Should calling this method without an old_course_id be an error? Why do we have code that does it?