Change FactsetId to use OldIdentifier instead of string

This is pretty ugly, what is it actually used for on the other side, can we just get rid of it now?