From 92e44801159ad4c5d3d136dafcb8a36e0ddd4ec6 Mon Sep 17 00:00:00 2001 From: eileen Date: Fri, 6 Sep 2019 11:42:21 +1200 Subject: [PATCH] Move calc for do-not-import default to processor --- CRM/Contact/Import/Form/MapField.php | 5 ++--- CRM/Import/ImportProcessor.php | 3 +++ 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/CRM/Contact/Import/Form/MapField.php b/CRM/Contact/Import/Form/MapField.php index 6d46ae4fde..833ff73720 100644 --- a/CRM/Contact/Import/Form/MapField.php +++ b/CRM/Contact/Import/Form/MapField.php @@ -852,13 +852,12 @@ class CRM_Contact_Import_Form_MapField extends CRM_Import_Form_MapField { */ public function loadSavedMapping($processor, $mappingName, $i, $defaults, $js, $hasColumnNames) { $formName = $processor->getFormName(); - if (isset($mappingName[$i])) { + if ($processor->getFieldName($i)) { + $defaults["mapper[$i]"] = $processor->getSavedQuickformDefaultsForColumn($i); if ($mappingName[$i] != ts('- do not import -')) { - $defaults["mapper[$i]"] = $processor->getSavedQuickformDefaultsForColumn($i); $js .= $processor->getQuickFormJSForField($i); } else { - $defaults["mapper[$i]"] = []; for ($k = 1; $k < 4; $k++) { $js .= "{$formName}['mapper[$i][$k]'].style.display = 'none';\n"; } diff --git a/CRM/Import/ImportProcessor.php b/CRM/Import/ImportProcessor.php index 929be20551..6ecca5a737 100644 --- a/CRM/Import/ImportProcessor.php +++ b/CRM/Import/ImportProcessor.php @@ -545,6 +545,9 @@ class CRM_Import_ImportProcessor { * @throws \CiviCRM_API3_Exception */ public function getSavedQuickformDefaultsForColumn($column) { + if ($this->getFieldName($column) === ts('- do not import -')) { + return []; + } if ($this->getValidRelationshipKey($column)) { if ($this->getWebsiteTypeID($column)) { return [$this->getValidRelationshipKey($column), $this->getFieldName($column), $this->getWebsiteTypeID($column)]; -- 2.25.1