Stop setting now-unused updateWithId
authorEileen McNaughton <emcnaughton@wikimedia.org>
Thu, 2 Jun 2022 07:05:26 +0000 (19:05 +1200)
committerEileen McNaughton <emcnaughton@wikimedia.org>
Thu, 2 Jun 2022 07:05:53 +0000 (19:05 +1200)
CRM/Contact/Import/Parser/Contact.php

index cde72b26153915f73007ce0492e7ad18b53b4133..2580dcc7749a42e0ba8981cbcea1be28e22ef321 100644 (file)
@@ -29,19 +29,6 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Import_Parser {
   use CRM_Contact_Import_MetadataTrait;
 
   protected $_mapperKeys = [];
-
-  /**
-   * Is update only permitted on an id match.
-   *
-   * Note this historically was true for when id or external identifier was
-   * present. However, CRM-17275 determined that a dedupe-match could over-ride
-   * external identifier.
-   *
-   * @var bool
-   */
-  protected $_updateWithId;
-
-  protected $_externalIdentifierIndex;
   protected $_allExternalIdentifiers = [];
   protected $_parseStreetAddress;
 
@@ -126,21 +113,6 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Import_Parser {
 
     $this->setActiveFields($this->_mapperKeys);
 
-    $this->_externalIdentifierIndex = -1;
-
-    $index = 0;
-    foreach ($this->_mapperKeys as $key) {
-      if ($key == 'external_identifier') {
-        $this->_externalIdentifierIndex = $index;
-      }
-      $index++;
-    }
-
-    $this->_updateWithId = FALSE;
-    if (in_array('id', $this->_mapperKeys) || ($this->_externalIdentifierIndex >= 0 && $this->isUpdateExistingContacts())) {
-      $this->_updateWithId = TRUE;
-    }
-
     $this->_parseStreetAddress = CRM_Utils_Array::value('street_address_parsing', CRM_Core_BAO_Setting::valueOptions(CRM_Core_BAO_Setting::SYSTEM_PREFERENCES_NAME, 'address_options'), FALSE);
   }