dev/core#3664 Remove dataColumnGuessing

diff --git a/CRM/Activity/Import/Form/MapField.php b/CRM/Activity/Import/Form/MapField.php
index 7034eb898b7547b515aa5dcd1558bbfb0f9e05ba..155d6357e424e893a0092c77cb2d946a73c01429 100644 (file)
--- a/CRM/Activity/Import/Form/MapField.php
+++ b/CRM/Activity/Import/Form/MapField.php
@@ -64,9 +64,9 @@ class CRM_Activity_Import_Form_MapField extends CRM_Import_Form_MapField {
   /**
    * Build the form object.
    *
-   * @throws \CiviCRM_API3_Exception
+   * @throws \CRM_Core_Exception
    */
-  public function buildQuickForm() {
+  public function buildQuickForm(): void {
     $savedMappingID = (int) $this->getSubmittedValue('savedMapping');
     $this->buildSavedMappingFields($savedMappingID);
     $this->addFormRule(['CRM_Activity_Import_Form_MapField', 'formRule']);
@@ -75,7 +75,6 @@ class CRM_Activity_Import_Form_MapField extends CRM_Import_Form_MapField {
 
     $defaults = [];
     $headerPatterns = $this->getHeaderPatterns();
-    $dataPatterns = $this->getDataPatterns();
     $fieldMappings = $this->getFieldMappings();
     $columnHeaders = $this->getColumnHeaders();
     $hasHeaders = $this->getSubmittedValue('skipColumnHeader');
@@ -112,9 +111,6 @@ class CRM_Activity_Import_Form_MapField extends CRM_Import_Form_MapField {
           if ($hasHeaders) {
             $defaults["mapper[$i]"] = [$this->defaultFromHeader($columnHeader, $headerPatterns)];
           }
-          else {
-            $defaults["mapper[$i]"] = [$this->defaultFromData($dataPatterns, $i)];
-          }
         }
         // End of load mapping.
       }
@@ -127,10 +123,6 @@ class CRM_Activity_Import_Form_MapField extends CRM_Import_Form_MapField {
             0,
           ];
         }
-        else {
-          // Otherwise guess the default from the form of the data
-          $defaults["mapper[$i]"] = [$this->defaultFromData($dataPatterns, $i), 0];
-        }
       }
 
       $sel->setOptions([$sel1]);

diff --git a/CRM/Contact/Import/Form/MapField.php b/CRM/Contact/Import/Form/MapField.php
index 4c24a7edff16f673a6e05f546d4655feffb16df3..833eb631cc3c6863721288ab644801e6257f9ea8 100644 (file)
--- a/CRM/Contact/Import/Form/MapField.php
+++ b/CRM/Contact/Import/Form/MapField.php
@@ -309,12 +309,6 @@ class CRM_Contact_Import_Form_MapField extends CRM_Import_Form_MapField {
             ];
           }
         }
-        else {
-          // Otherwise guess the default from the form of the data
-          $defaults["mapper[$i]"] = [
-            $this->defaultFromData($this->getDataPatterns(), $i),
-          ];
-        }
         $last_key = array_key_last($defaults["mapper[$i]"]) ?? 0;
       }
       // Call swapOptions on the deepest select element to hide the empty select lists above it.

diff --git a/CRM/Contact/Import/MetadataTrait.php b/CRM/Contact/Import/MetadataTrait.php
index 368df0e11a937353f26eafd0f2897fe1b2b58b59..19db539ed8b6ee65ab0a7769b92d445336543513 100644 (file)
--- a/CRM/Contact/Import/MetadataTrait.php
+++ b/CRM/Contact/Import/MetadataTrait.php
@@ -85,15 +85,6 @@ trait CRM_Contact_Import_MetadataTrait {
     return CRM_Utils_Array::collect('headerPattern', $this->getContactImportMetadata());
   }
 
-  /**
-   * Get an array of header patterns for importable keys.
-   *
-   * @return array
-   */
-  public function getDataPatterns(): array {
-    return CRM_Utils_Array::collect('dataPattern', $this->getContactImportMetadata());
-  }
-
   /**
    * Get an array of header patterns for importable keys.
    *

diff --git a/CRM/Contribute/Import/Form/MapField.php b/CRM/Contribute/Import/Form/MapField.php
index 419d91dd30a582915b815eae87a3c497ec5c4a6e..d4b5a816227cfe9eed1fece07fc381835995e968 100644 (file)
--- a/CRM/Contribute/Import/Form/MapField.php
+++ b/CRM/Contribute/Import/Form/MapField.php
@@ -140,7 +140,6 @@ class CRM_Contribute_Import_Form_MapField extends CRM_Import_Form_MapField {
     $mapperKeys = array_keys($this->_mapperFields);
     $hasHeaders = $this->getSubmittedValue('skipColumnHeader');
     $headerPatterns = $this->getHeaderPatterns();
-    $dataPatterns = $this->getDataPatterns();
     $mapperKeysValues = $this->getSubmittedValue('mapper');
     $columnHeaders = $this->getColumnHeaders();
     $fieldMappings = $this->getFieldMappings();
@@ -214,9 +213,6 @@ class CRM_Contribute_Import_Form_MapField extends CRM_Import_Form_MapField {
           if ($hasHeaders) {
             $defaults["mapper[$i]"] = [$this->defaultFromHeader($columnHeader, $headerPatterns)];
           }
-          else {
-            $defaults["mapper[$i]"] = [$this->defaultFromData($dataPatterns, $i)];
-          }
         }
         //end of load mapping
       }
@@ -237,13 +233,6 @@ class CRM_Contribute_Import_Form_MapField extends CRM_Import_Form_MapField {
             ];
           }
         }
-        else {
-          // Otherwise guess the default from the form of the data
-          $defaults["mapper[$i]"] = [
-            $this->defaultFromData($dataPatterns, $i),
-            0,
-          ];
-        }
         if (!empty($mapperKeysValues) && ($mapperKeysValues[$i][0] ?? NULL) === 'soft_credit') {
           $softCreditField = $mapperKeysValues[$i][1];
           $softCreditTypeID = $mapperKeysValues[$i][2];

diff --git a/CRM/Import/Form/MapField.php b/CRM/Import/Form/MapField.php
index 94feb7729cb0f6ab375c7f7bbb174a1d9a3d0dca..c374e1d9560dd5bdc8939f546d145bf6667b326c 100644 (file)
--- a/CRM/Import/Form/MapField.php
+++ b/CRM/Import/Form/MapField.php
@@ -348,7 +348,6 @@ abstract class CRM_Import_Form_MapField extends CRM_Import_Forms {
     $mapperKeys = array_keys($this->_mapperFields);
     $hasHeaders = $this->getSubmittedValue('skipColumnHeader');
     $headerPatterns = $this->getHeaderPatterns();
-    $dataPatterns = $this->getDataPatterns();
     $fieldMappings = $this->getFieldMappings();
     /* Initialize all field usages to false */
 
@@ -387,9 +386,6 @@ abstract class CRM_Import_Form_MapField extends CRM_Import_Forms {
           if ($hasHeaders) {
             $defaults["mapper[$i]"] = [$this->defaultFromHeader($columnHeader, $headerPatterns)];
           }
-          else {
-            $defaults["mapper[$i]"] = [$this->defaultFromData($dataPatterns, $i)];
-          }
         }
         //end of load mapping
       }
@@ -405,14 +401,6 @@ abstract class CRM_Import_Form_MapField extends CRM_Import_Forms {
             0,
           ];
         }
-        else {
-          // Otherwise guess the default from the form of the data
-          $defaults["mapper[$i]"] = [
-            $this->defaultFromData($dataPatterns, $i),
-            //                     $defaultLocationType->id
-            0,
-          ];
-        }
       }
       $sel->setOptions([$sel1]);
     }

diff --git a/CRM/Import/Forms.php b/CRM/Import/Forms.php
index c9636b25bcff91a516c528a8832d069f97661759..13801eb20b26b3010e5ad23dbc6a84c4ffd30df0 100644 (file)
--- a/CRM/Import/Forms.php
+++ b/CRM/Import/Forms.php
@@ -649,15 +649,6 @@ class CRM_Import_Forms extends CRM_Core_Form {
     return [];
   }
 
-  /**
-   * Get the data patterns to pattern match the incoming data.
-   *
-   * @return array
-   */
-  public function getDataPatterns(): array {
-    return $this->getParser()->getDataPatterns();
-  }
-
   /**
    * Get the data patterns to pattern match the incoming data.
    *

diff --git a/CRM/Import/Parser.php b/CRM/Import/Parser.php
index 086cb4e6281a64779d93e7a61d29105275d26343..8e7199bcb3c8229ef48235b6622ce5143a93c202 100644 (file)
--- a/CRM/Import/Parser.php
+++ b/CRM/Import/Parser.php
@@ -583,19 +583,6 @@ abstract class CRM_Import_Parser implements UserJobInterface {
     return $values;
   }
 
-  /**
-   * @deprecated
-   *
-   * @return array
-   */
-  public function getDataPatterns():array {
-    $values = [];
-    foreach ($this->_fields as $name => $field) {
-      $values[$name] = $field->_dataPattern;
-    }
-    return $values;
-  }
-
   /**
    * Remove single-quote enclosures from a value array (row).
    *

diff --git a/CRM/Member/Import/Form/MapField.php b/CRM/Member/Import/Form/MapField.php
index bee1f3a925518467c6c3d9f698f9a68caf2eb004..ab938e4abb5c4fbdc2b04aa2773d7b14ef0652b7 100644 (file)
--- a/CRM/Member/Import/Form/MapField.php
+++ b/CRM/Member/Import/Form/MapField.php
@@ -23,9 +23,9 @@ class CRM_Member_Import_Form_MapField extends CRM_Import_Form_MapField {
   /**
    * Build the form object.
    *
-   * @return void
+   * @throws \CRM_Core_Exception
    */
-  public function buildQuickForm() {
+  public function buildQuickForm(): void {
     $this->buildSavedMappingFields($this->getSubmittedValue('savedMapping'));
     $this->addFormRule(array('CRM_Member_Import_Form_MapField', 'formRule'), $this);
 
@@ -35,7 +35,6 @@ class CRM_Member_Import_Form_MapField extends CRM_Import_Form_MapField {
     $columnHeaders = $this->getColumnHeaders();
     $hasHeaders = $this->getSubmittedValue('skipColumnHeader');
     $headerPatterns = $this->getHeaderPatterns();
-    $dataPatterns = $this->getDataPatterns();
     // For most fields using the html label is a good thing
     // but for contact ID we really want to specify ID.
     $this->_mapperFields['membership_contact_id'] = ts('Contact ID');
@@ -84,9 +83,6 @@ class CRM_Member_Import_Form_MapField extends CRM_Import_Form_MapField {
           if ($hasHeaders) {
             $defaults["mapper[$i]"] = array($this->defaultFromHeader($columnHeader, $headerPatterns));
           }
-          else {
-            $defaults["mapper[$i]"] = array($this->defaultFromData($dataPatterns, $i));
-          }
         }
         //end of load mapping
       }
@@ -102,14 +98,6 @@ class CRM_Member_Import_Form_MapField extends CRM_Import_Form_MapField {
             0,
           );
         }
-        else {
-          // Otherwise guess the default from the form of the data
-          $defaults["mapper[$i]"] = array(
-            $this->defaultFromData($dataPatterns, $i),
-            //                     $defaultLocationType->id
-            0,
-          );
-        }
       }
       $sel->setOptions([$sel1]);
     }