Add deprecation warnings to old-style Location classes

Isn't the developer getting this warning going to use `locator.CourseLocator.from_string` not `locator.CourseLocator._from_string` ?