[REF] [Import] Minor code cleanup

[eileenmcnaughton]

Overview

[REF] [Import] Minor code cleanup

Before

A $value array is constructed if the mapped field has a value for either location_type_id or website_type_id. The array is a subset of the $mappedField array holding the location values & in addition it has the field name & value - eg. it would look like

$value = ['location_type_id' => 1, 'email' => 'e@mail.com'];

or
$value = ['website_type_id' => 1, 'url' => 'https://civicrm.org'];

The construction builds up the array with a whole lotta ifs

After

array foo used instead

Technical Details

This is how the code section winds up looking

Comments

Solid test cover of this function in CRM_Contact_Import_Parser_ContactTest

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