That's a good point. We shouldn't be using `name` unless we're intending to be testing backwards compatibility in `XMLModuleStore`, Otherwise, we should be using `url_name` instead.