The Person class has what seems to be duplicative data fields. In addition to the standard creator, changer, and voider information it also collects person-specific versions of those, e.g.,
There is a comment in the copy constructor for Person which seems to indicate some differentiation between a copy of the Person object (that might be shared by a Patient and User etc.) and an instance of the Person object. My best guess is that this functionality exists to allow the creation of a non-voided Patient from a voided Person using the appropriate constructor.
I was wondering if we need to maintain these fields to support any actual functionality or is it safe to deprecate them?