Fix merge resolution mistake
authorEileen McNaughton <emcnaughton@wikimedia.org>
Mon, 6 Jun 2022 02:47:11 +0000 (14:47 +1200)
committerEileen McNaughton <emcnaughton@wikimedia.org>
Tue, 7 Jun 2022 00:23:12 +0000 (12:23 +1200)
CRM/Contact/Import/Parser/Contact.php
tests/phpunit/CRM/Contact/Import/Parser/ContactTest.php

index c1ef99603a234644e213a4b1336a8fc9ddd055ea..d284039b30e6d24694225a1d4d283e7495623c99 100644 (file)
@@ -195,6 +195,7 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Import_Parser {
       // CRM-5854, reset the geocode method to null to prevent geocoding
       CRM_Utils_GeocodeProvider::disableForSession();
     }
+    $relatedContacts = [];
 
     try {
       $params = $this->getMappedRow($values);
@@ -230,18 +231,16 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Import_Parser {
 
       //relationship contact insert
       foreach ($this->getRelatedContactsParams($params) as $key => $field) {
-        $formatting = $field;
-        [$formatting, $field] = $this->processContact($field, $formatting, FALSE);
-
+        [$formatting, $field] = $this->processContact($field, $field, FALSE);
         //format common data, CRM-4062
         $this->formatCommonData($field, $formatting);
-
+        $isUpdate = empty($formatting['id']) ? 'new' : 'updated';
         if (empty($formatting['id']) || $this->isUpdateExistingContacts()) {
           $relatedNewContact = $this->createContact($formatting, $formatting['id']);
           $formatting['id'] = $relatedNewContact->id;
         }
         if (empty($relatedContacts[$formatting['id']])) {
-          $relatedContacts[$formatting['id']] = 'new';
+          $relatedContacts[$formatting['id']] = $isUpdate;
         }
 
         $this->createRelationship($key, $formatting['id'], $primaryContactId);
@@ -258,7 +257,7 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Import_Parser {
       return FALSE;
     }
     $extraFields = ['related_contact_created' => 0, 'related_contact_matched' => 0];
-    foreach ($relatedContacts as $key => $outcome) {
+    foreach ($relatedContacts as $outcome) {
       if ($outcome === 'new') {
         $extraFields['related_contact_created']++;
       }
index a918062cb1e99f53368b80348cf791f5c6f0307d..00c907789639ce3fcc695e0e8607fd26fc5fbe68 100644 (file)
@@ -2076,7 +2076,7 @@ class CRM_Contact_Import_Parser_ContactTest extends CiviUnitTestCase {
       $form->postProcess();
     }
     catch (CRM_Core_Exception_PrematureExitException $e) {
-      $queue = Civi::queue('user_job_' . $userJobID);
+      $queue = Civi::queue('user_job_' . $this->userJobID);
       $runner = new CRM_Queue_Runner([
         'queue' => $queue,
         'errorMode' => CRM_Queue_Runner::ERROR_ABORT,