Don't create orphaned person records when updating

Since we were incorrectly altering the records with `accepts_nested_attributes`, updating both separately in a transaction seems to make more sense.

It also cleans up the form since we can just use flat user attributes.