Multi record import screen fix

diff --git a/CRM/Core/BAO/Mapping.php b/CRM/Core/BAO/Mapping.php
index 56c1d9e02034591b8e1c82a6404eab339f88058c..309972bb6aaf35c420a29862b5c77056d3506823 100644 (file)
--- a/CRM/Core/BAO/Mapping.php
+++ b/CRM/Core/BAO/Mapping.php
@@ -151,6 +151,7 @@ class CRM_Core_BAO_Mapping extends CRM_Core_DAO_Mapping {
         civicrm_api3('OptionValue', 'create', [
           'option_group_id' => 'mapping_type',
           'label' => $mappingType,
+          'name' => $mappingType,
           'value' => max(array_keys($mappingValues['values'])) + 1,
           'is_reserved' => 1,
         ]);

diff --git a/tests/phpunit/CRM/Core/BAO/MappingTest.php b/tests/phpunit/CRM/Core/BAO/MappingTest.php
index 50dea9ef5916cc9e2e2d8d050fe846c086023bad..fb71da022eaf7391cd7e59acbae0923e1cd3637b 100644 (file)
--- a/tests/phpunit/CRM/Core/BAO/MappingTest.php
+++ b/tests/phpunit/CRM/Core/BAO/MappingTest.php
@@ -219,4 +219,11 @@ class CRM_Core_BAO_MappingTest extends CiviUnitTestCase {
     ];
   }
 
+  /**
+   * Ensure getCreateMappingValues() doesn't return an error when there are spaces in the name.
+   */
+  public function testGetCreateMappingValues() {
+    CRM_Core_BAO_Mapping::getCreateMappingValues("Import Multi value custom data");
+  }
+
 }