[Import] Remove silly param
authorEileen McNaughton <emcnaughton@wikimedia.org>
Thu, 19 May 2022 01:49:50 +0000 (13:49 +1200)
committerEileen McNaughton <emcnaughton@wikimedia.org>
Thu, 19 May 2022 04:59:58 +0000 (16:59 +1200)
CRM/Contact/Import/Parser/Contact.php

index 00ae15f24f628bde104eaa1048e34490a9baba94..1a3c3770a029589d3795018e76ee4183c56a210e 100644 (file)
@@ -29,7 +29,6 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Import_Parser {
   use CRM_Contact_Import_MetadataTrait;
 
   protected $_mapperKeys = [];
-  protected $_relationships;
 
   /**
    * Is update only permitted on an id match.
@@ -1059,9 +1058,8 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Import_Parser {
    *   A string containing all the error-fields.
    *
    * @param null $csType
-   * @param null $relationships
    */
-  public static function isErrorInCustomData($params, &$errorMessage, $csType = NULL, $relationships = NULL) {
+  public static function isErrorInCustomData($params, &$errorMessage, $csType = NULL) {
     $dateType = CRM_Core_Session::singleton()->get("dateTypes");
     $errors = [];
 
@@ -1220,18 +1218,10 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Import_Parser {
           }
         }
       }
-      elseif (is_array($params[$key]) && isset($params[$key]["contact_type"])) {
+      elseif (is_array($params[$key]) && isset($params[$key]["contact_type"]) && in_array(substr($key, -3), ['a_b', 'b_a'], TRUE)) {
         //CRM-5125
         //supporting custom data of related contact subtypes
-        $relation = NULL;
-        if ($relationships) {
-          if (array_key_exists($key, $relationships)) {
-            $relation = $key;
-          }
-          elseif (CRM_Utils_Array::key($key, $relationships)) {
-            $relation = CRM_Utils_Array::key($key, $relationships);
-          }
-        }
+        $relation = $key;
         if (!empty($relation)) {
           [$id, $first, $second] = CRM_Utils_System::explode('_', $relation, 3);
           $direction = "contact_sub_type_$second";
@@ -1244,7 +1234,7 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Import_Parser {
           }
         }
 
-        self::isErrorInCustomData($params[$key], $errorMessage, $csType, $relationships);
+        self::isErrorInCustomData($params[$key], $errorMessage, $csType);
       }
     }
     if ($errors) {
@@ -1900,8 +1890,6 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Import_Parser {
    */
   protected function setFieldMetadata() {
     $this->setImportableFieldsMetadata($this->getContactImportMetadata());
-    // Probably no longer needed but here for now.
-    $this->_relationships = $this->getRelationships();
   }
 
   /**
@@ -2831,7 +2819,7 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Import_Parser {
     }
 
     //checking error in custom data
-    $this->isErrorInCustomData($params, $errorMessage, $csType, $this->_relationships);
+    $this->isErrorInCustomData($params, $errorMessage, $csType);
 
     //checking error in core data
     $this->isErrorInCoreData($params, $errorMessage);