[REF] Minor code cleanup.

[eileenmcnaughton]

Overview

Removes a small amount of pointless code

Before

if (!array_key_exists('related_contact', $values)) { is repeated

After

Once we condense the above to one if we find the value in the else is never used

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).

[demeritcowboy]

This looks ok.

The next two ifs right after could also be combined into one if. Well, unless it's a bizzaro function where you need to call it twice to get it to give you displayName, but it doesn't look like it.

The outstanding 5.25-to-master PR still needs to be merged to get the tests to pass.

[seamuslee001]

Jenkins re test this please

[eileenmcnaughton]

@demeritcowboy yep I spotted those next 2 - I may do a follow up

[seamuslee001]

Looks fine to me merging