Change FactsetId to use OldIdentifier instead of string

I am not sure what else you can do. Ultimately I would like to move as much of this validation upfront as we can, but it doesn't seem like it can be avoided here.