Details
-
Type:
Bug
-
Status: Done/Fixed
-
Priority:
Trivial
-
Resolution: Fixed/Completed
-
Affects Version/s: 4.7.0
-
Fix Version/s: 4.7.20
-
Component/s: None
-
Labels:None
-
Versioning Impact:Patch (backwards-compatible bug fixes)
-
Documentation Required?:None
-
Funding Source:Needs Funding
-
Verified?:No
Description
Import file with two same values of external ids gives a fatal error which affects the whole import process. I think only the affected rows should be skipped.
For eg:
| First Name | Last Name | External Identifier |
|---|---|---|
| test1 | contact1 | fz123 |
| test2 | contact2 | fz123 |
We do have a check for this here https://github.com/civicrm/civicrm-core/blob/master/CRM/Contact/Import/Parser/Contact.php#L509
where import row is skipped when a similar ext id is found in the database. But it seems getFieldValue() doesn't return the contact_id when the second row is in process even though test1 is processed and inserted into the db.