What does the user's display preferences have to do with this function taking a date time string? This method's for converting to that format, not from it, so it still doesn't make sense to take a string.

If this is to take a string it should be a static method, not an instance once.