dev/core#1187 Fix bug where import will not do 2 phone types of the same location

[eileenmcnaughton]

Overview

Fixes a bug whereby one phone number is not imported if 2 of the same location type but different phone types are in an import

Before

Import as described in https://lab.civicrm.org/dev/core/issues/1187 does not result in 2 phones

After

2 phones created

Technical Details

The issue is the indexing is squishing them into one

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

[eileenmcnaughton]

@jitendrapurohit are you able to review this?

[jitendrapurohit]

Replicated this issue and confirmed that the PR changes fixes the import of multiple phone values with same loc type.

@eileenmcnaughton This can be merged.

[mattwire]

Merging based on @jitendrapurohit review

[eileenmcnaughton]

Thanks @jitendrapurohit @mattwire