Remove never-true fields, and impossible property handling from previously shared function

[eileenmcnaughton]

Overview

Follow up cleanup on #23542

Before

Function now only called from one place which passes NULL for $ufGroupID and FALSE for $skipCustom

Plus customRecordValues only ever set in the Profile path - which no longer shares this function

After

poof

Technical Details

Comments

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).