Change FactsetId to use OldIdentifier instead of string

The `FactsetId` is only used for error reporting. I could change it so its not a `FactsetId` and just a `String` so we can place in the error message?