Notices on import - consolidate mapping variables

This consolidates 2 variables mappingName and loadedMapping into
one (savedMappingName) - loadedMapping is only ever used as
a boolean so presence or otherwise of savedMappingName is enough.

This addresses enotices and simplifies the code

diff --git a/CRM/Activity/Import/Form/MapField.php b/CRM/Activity/Import/Form/MapField.php
index a7c2d2b49bd9c4c8643b4a8329d1b66ab4073be9..215ec9eccaca5815b74f1f0bb328ef8501d43b7f 100644 (file)
--- a/CRM/Activity/Import/Form/MapField.php
+++ b/CRM/Activity/Import/Form/MapField.php
@@ -84,14 +84,13 @@ class CRM_Activity_Import_Form_MapField extends CRM_Import_Form_MapField {
       // Get an array of the name values for mapping fields associated with this mapping_id.
       $mappingName = CRM_Core_BAO_Mapping::getMappingFieldValues($savedMapping, 'name');
 
-      $this->assign('loadedMapping', $savedMapping);
       $this->set('loadedMapping', $savedMapping);
 
       $params = ['id' => $savedMapping];
       $temp = [];
       $mappingDetails = CRM_Core_BAO_Mapping::retrieve($params, $temp);
 
-      $this->assign('savedName', $mappingDetails->name);
+      $this->assign('savedMappingName', $mappingDetails->name);
 
       $this->add('hidden', 'mappingId', $savedMapping);
 

diff --git a/CRM/Activity/Import/Form/Preview.php b/CRM/Activity/Import/Form/Preview.php
index 3b3de7d38e49c0a9d8663d3658bceb6313a8818f..b12ffef3660149169966f08f52293cbc576c1d4a 100644 (file)
--- a/CRM/Activity/Import/Form/Preview.php
+++ b/CRM/Activity/Import/Form/Preview.php
@@ -39,9 +39,8 @@ class CRM_Activity_Import_Form_Preview extends CRM_Import_Form_Preview {
       $mapDAO = new CRM_Core_DAO_Mapping();
       $mapDAO->id = $mappingId;
       $mapDAO->find(TRUE);
-      $this->assign('loadedMapping', $mappingId);
-      $this->assign('savedName', $mapDAO->name);
     }
+    $this->assign('savedMappingName', $mappingId ? $mapDAO->name : NULL);
 
     if ($skipColumnHeader) {
       $this->assign('skipColumnHeader', $skipColumnHeader);

diff --git a/CRM/Contact/Import/Form/Preview.php b/CRM/Contact/Import/Form/Preview.php
index 48f484eb00362ae32242f3e2b6691c1b48b692c4..2e8028a01ecb943c19af54acd481c6353449ab02 100644 (file)
--- a/CRM/Contact/Import/Form/Preview.php
+++ b/CRM/Contact/Import/Form/Preview.php
@@ -49,9 +49,8 @@ class CRM_Contact_Import_Form_Preview extends CRM_Import_Form_Preview {
       $mapDAO = new CRM_Core_DAO_Mapping();
       $mapDAO->id = $mappingId;
       $mapDAO->find(TRUE);
-      $this->assign('loadedMapping', $mappingId);
-      $this->assign('savedName', $mapDAO->name);
     }
+    $this->assign('savedMappingName', $mappingId ? $mapDAO->name : NULL);
 
     $this->assign('rowDisplayCount', 2);
 

diff --git a/CRM/Contribute/Import/Form/Preview.php b/CRM/Contribute/Import/Form/Preview.php
index e45de43a25b8a2d446fa429d0f64342d3c75faa6..41cd6f0a115f9d9adaac8329248286d9a9c7a44a 100644 (file)
--- a/CRM/Contribute/Import/Form/Preview.php
+++ b/CRM/Contribute/Import/Form/Preview.php
@@ -41,9 +41,8 @@ class CRM_Contribute_Import_Form_Preview extends CRM_Import_Form_Preview {
       $mapDAO = new CRM_Core_DAO_Mapping();
       $mapDAO->id = $mappingId;
       $mapDAO->find(TRUE);
-      $this->assign('loadedMapping', $mappingId);
-      $this->assign('savedName', $mapDAO->name);
     }
+    $this->assign('savedMappingName', $mappingId ? $mapDAO->name : NULL);
 
     if ($skipColumnHeader) {
       $this->assign('skipColumnHeader', $skipColumnHeader);

diff --git a/CRM/Custom/Import/Form/Preview.php b/CRM/Custom/Import/Form/Preview.php
index 76a5765705cecb80e95bc05063b52e401fe644ae..0abfd721953c38e1b06faf0216f338b2c77967d6 100644 (file)
--- a/CRM/Custom/Import/Form/Preview.php
+++ b/CRM/Custom/Import/Form/Preview.php
@@ -29,9 +29,8 @@ class CRM_Custom_Import_Form_Preview extends CRM_Import_Form_Preview {
       $mapDAO = new CRM_Core_DAO_Mapping();
       $mapDAO->id = $mappingId;
       $mapDAO->find(TRUE);
-      $this->assign('loadedMapping', $mappingId);
-      $this->assign('savedName', $mapDAO->name);
     }
+    $this->assign('savedMappingName', $mappingId ? $mapDAO->name : NULL);
 
     if ($skipColumnHeader) {
       $this->assign('skipColumnHeader', $skipColumnHeader);

diff --git a/CRM/Event/Import/Form/MapField.php b/CRM/Event/Import/Form/MapField.php
index df28b3141fbffa169ebbb324330cd7071899a07f..bf7472a1af65fc453f4974710af66476a557f27f 100644 (file)
--- a/CRM/Event/Import/Form/MapField.php
+++ b/CRM/Event/Import/Form/MapField.php
@@ -104,11 +104,6 @@ class CRM_Event_Import_Form_MapField extends CRM_Import_Form_MapField {
 
       //mapping is to be loaded from database
 
-      $params = array('id' => $savedMapping);
-      $temp = [];
-      $mappingDetails = CRM_Core_BAO_Mapping::retrieve($params, $temp);
-
-      $this->assign('loadedMapping', $mappingDetails->name);
       $this->set('loadedMapping', $savedMapping);
 
       $getMappingName = new CRM_Core_DAO_Mapping();
@@ -119,7 +114,7 @@ class CRM_Event_Import_Form_MapField extends CRM_Import_Form_MapField {
         $mapperName = $getMappingName->name;
       }
 
-      $this->assign('savedName', $mapperName);
+      $this->assign('savedMappingName', $mapperName);
 
       $this->add('hidden', 'mappingId', $savedMapping);
 

diff --git a/CRM/Event/Import/Form/Preview.php b/CRM/Event/Import/Form/Preview.php
index 629ed8e5cf84433e8ab0c5d2b9df29366ee4422d..ccec35738ac2e8284a9132163164c9288b8d2097 100644 (file)
--- a/CRM/Event/Import/Form/Preview.php
+++ b/CRM/Event/Import/Form/Preview.php
@@ -42,9 +42,8 @@ class CRM_Event_Import_Form_Preview extends CRM_Import_Form_Preview {
       $mapDAO = new CRM_Core_DAO_Mapping();
       $mapDAO->id = $mappingId;
       $mapDAO->find(TRUE);
-      $this->assign('loadedMapping', $mappingId);
-      $this->assign('savedName', $mapDAO->name);
     }
+    $this->assign('savedMappingName', $mappingId ? $mapDAO->name : NULL);
 
     if ($skipColumnHeader) {
       $this->assign('skipColumnHeader', $skipColumnHeader);

diff --git a/CRM/Import/Form/MapField.php b/CRM/Import/Form/MapField.php
index 41fe9a6da53d457e14c8e582144196f2b10757cc..db40c55970bb8910344b0bdd6f5f60e0f4e1a5f2 100644 (file)
--- a/CRM/Import/Form/MapField.php
+++ b/CRM/Import/Form/MapField.php
@@ -157,8 +157,6 @@ abstract class CRM_Import_Form_MapField extends CRM_Core_Form {
 
       $mappingName = (string) civicrm_api3('Mapping', 'getvalue', ['id' => $savedMappingID, 'return' => 'name']);
       $this->set('loadedMapping', $savedMapping);
-      $this->assign('loadedMapping', $mappingName);
-      $this->assign('savedName', $mappingName);
       $this->add('hidden', 'mappingId', $savedMappingID);
 
       $this->addElement('checkbox', 'updateMapping', ts('Update this field mapping'), NULL);
@@ -166,7 +164,7 @@ abstract class CRM_Import_Form_MapField extends CRM_Core_Form {
       $this->add('text', 'saveMappingName', ts('Name'));
       $this->add('text', 'saveMappingDesc', ts('Description'));
     }
-
+    $this->assign('savedMappingName', $mappingName ?? NULL);
     $this->addElement('checkbox', 'saveMapping', $saveDetailsName, NULL, ['onclick' => "showSaveDetails(this)"]);
   }
 

diff --git a/CRM/Member/Import/Form/MapField.php b/CRM/Member/Import/Form/MapField.php
index 0b1ee0e9384085d1102355ab76ecd279545d6677..c06c9afeebf29dabff856474fa4ff76827907b18 100644 (file)
--- a/CRM/Member/Import/Form/MapField.php
+++ b/CRM/Member/Import/Form/MapField.php
@@ -117,11 +117,6 @@ class CRM_Member_Import_Form_MapField extends CRM_Import_Form_MapField {
 
       //mapping is to be loaded from database
 
-      $params = array('id' => $savedMapping);
-      $temp = [];
-      $mappingDetails = CRM_Core_BAO_Mapping::retrieve($params, $temp);
-
-      $this->assign('loadedMapping', $mappingDetails->name);
       $this->set('loadedMapping', $savedMapping);
 
       $getMappingName = new CRM_Core_DAO_Mapping();
@@ -132,7 +127,7 @@ class CRM_Member_Import_Form_MapField extends CRM_Import_Form_MapField {
         $mapperName = $getMappingName->name;
       }
 
-      $this->assign('savedName', $mapperName);
+      $this->assign('savedMappingName', $mapperName);
 
       $this->add('hidden', 'mappingId', $savedMapping);
 

diff --git a/CRM/Member/Import/Form/Preview.php b/CRM/Member/Import/Form/Preview.php
index 03484335300d078951cb405311acb75a14f6ee80..0409888425b261d9842426ab03a552a2952da345 100644 (file)
--- a/CRM/Member/Import/Form/Preview.php
+++ b/CRM/Member/Import/Form/Preview.php
@@ -42,9 +42,8 @@ class CRM_Member_Import_Form_Preview extends CRM_Import_Form_Preview {
       $mapDAO = new CRM_Core_DAO_Mapping();
       $mapDAO->id = $mappingId;
       $mapDAO->find(TRUE);
-      $this->assign('loadedMapping', $mappingId);
-      $this->assign('savedName', $mapDAO->name);
     }
+    $this->assign('savedMappingName', $mappingId ? $mapDAO->name : NULL);
 
     if ($skipColumnHeader) {
       $this->assign('skipColumnHeader', $skipColumnHeader);

diff --git a/templates/CRM/Activity/Import/Form/MapTable.tpl b/templates/CRM/Activity/Import/Form/MapTable.tpl
index 97f3b7a2f37d44ccf84f343e732e24352210bfcb..9990cb38bf0d6370388f30773e5910a58d6b22e6 100644 (file)
--- a/templates/CRM/Activity/Import/Form/MapTable.tpl
+++ b/templates/CRM/Activity/Import/Form/MapTable.tpl
@@ -13,13 +13,13 @@
  <div id="map-field">
     {strip}
     <table>
-    {if $loadedMapping}
-        <tr class="columnheader-dark"><th colspan="4">{ts 1=$savedName}Saved Field Mapping: %1{/ts}</td></tr>
+    {if $savedMappingName}
+      <tr class="columnheader-dark"><th colspan="4">{ts 1=$savedMappingName}Saved Field Mapping: %1{/ts}</td></tr>
     {/if}
         <tr class="columnheader">
             {section name=rows loop=$rowDisplayCount}
        {if $skipColumnHeader }
-                   { if $smarty.section.rows.iteration == 1 }
+                   {if $smarty.section.rows.iteration == 1}
                      <th>{ts}Column Headers{/ts}</th>
                    {else}
                      <th>{ts 1=$smarty.section.rows.iteration}Import Data (row %1){/ts}</th>
@@ -61,7 +61,7 @@
     {if $wizard.currentStepName != 'Preview'}
     <div>
 
-      {if $loadedMapping}
+      {if $savedMappingName}
           <span>{$form.updateMapping.html} &nbsp;&nbsp; {$form.updateMapping.label}</span>
       {/if}
       <span>{$form.saveMapping.html} &nbsp;&nbsp; {$form.saveMapping.label}</span>

diff --git a/templates/CRM/Contact/Import/Form/MapTable.tpl b/templates/CRM/Contact/Import/Form/MapTable.tpl
index 20162b4cbc874299e5f24fd4dcd78476c34e25a1..5d6f22a44bd5ea9e90d11e8b00b93a50dad5bbf4 100644 (file)
--- a/templates/CRM/Contact/Import/Form/MapTable.tpl
+++ b/templates/CRM/Contact/Import/Form/MapTable.tpl
@@ -13,8 +13,8 @@
  <div id="map-field">
     {strip}
     <table class="selector">
-    {if $loadedMapping}
-        <tr class="columnheader-dark"><th colspan="4">{ts 1=$savedName}Saved Field Mapping: %1{/ts}</td></tr>
+    {if $savedMappingName}
+        <tr class="columnheader-dark"><th colspan="4">{ts 1=$savedMappingName}Saved Field Mapping: %1{/ts}</td></tr>
     {/if}
         <tr class="columnheader">
       {if $showColNames}
@@ -23,7 +23,7 @@
           {assign var="totalRowsDisplay" value=$rowDisplayCount}
       {/if}
             {section name=rows loop=$totalRowsDisplay}
-                { if $smarty.section.rows.iteration == 1 and $showColNames}
+                {if $smarty.section.rows.iteration == 1 and $showColNames}
                   <td>{ts}Column Names{/ts}</td>
                 {elseif $showColNames}
                   <td>{ts 1=$smarty.section.rows.iteration-1}Import Data (row %1){/ts}</td>
@@ -111,7 +111,7 @@
     {if $wizard.currentStepName != 'Preview'}
     <div>
 
-      {if $loadedMapping}
+      {if $savedMappingName}
           <span>{$form.updateMapping.html} &nbsp;&nbsp; {$form.updateMapping.label}</span>
       {/if}
       <span>{$form.saveMapping.html} &nbsp;&nbsp; {$form.saveMapping.label}</span>

diff --git a/templates/CRM/Contribute/Import/Form/MapTable.tpl b/templates/CRM/Contribute/Import/Form/MapTable.tpl
index 4490a735b2538c3804d83bd6b98eda4cbf04b463..3a8cad5515bbd4345a45bb92072c04719de592a5 100644 (file)
--- a/templates/CRM/Contribute/Import/Form/MapTable.tpl
+++ b/templates/CRM/Contribute/Import/Form/MapTable.tpl
@@ -12,13 +12,13 @@
  <div id="map-field">
     {strip}
     <table>
-    {if $loadedMapping}
-        <tr class="columnheader-dark"><th colspan="4">{ts 1=$savedName}Saved Field Mapping: %1{/ts}</td></tr>
-    {/if}
+      {if $savedMappingName}
+        <tr class="columnheader-dark"><th colspan="4">{ts 1=$savedMappingName}Saved Field Mapping: %1{/ts}</td></tr>
+      {/if}
         <tr class="columnheader">
             {section name=rows loop=$rowDisplayCount}
        {if $skipColumnHeader }
-                   { if $smarty.section.rows.iteration == 1 }
+                   {if $smarty.section.rows.iteration == 1}
                      <th>{ts}Column Headers{/ts}</th>
                    {else}
                      <th>{ts 1=$smarty.section.rows.iteration}Import Data (row %1){/ts}</th>
@@ -63,7 +63,7 @@
     {if $wizard.currentStepName != 'Preview'}
     <div>
 
-      {if $loadedMapping}
+      {if $savedMappingName}
           <span>{$form.updateMapping.html} &nbsp;&nbsp; {$form.updateMapping.label}</span>
       {/if}
       <span>{$form.saveMapping.html} &nbsp;&nbsp; {$form.saveMapping.label}</span>

diff --git a/templates/CRM/Event/Import/Form/MapTable.tpl b/templates/CRM/Event/Import/Form/MapTable.tpl
index 204fc97c65f61de4c9e3883886e55d88482134b6..6d5bf9558d22594b1c73eb7c2d1580580ae08d13 100644 (file)
--- a/templates/CRM/Event/Import/Form/MapTable.tpl
+++ b/templates/CRM/Event/Import/Form/MapTable.tpl
@@ -12,8 +12,8 @@
 <div id="map-field">
     {strip}
     <table>
-    {if $loadedMapping}
-        <tr class="columnheader-dark"><th colspan="4">{ts 1=$savedName}Saved Field Mapping: %1{/ts}</td></tr>
+    {if $savedMappingName}
+        <tr class="columnheader-dark"><th colspan="4">{ts 1=$savedMappingName}Saved Field Mapping: %1{/ts}</td></tr>
     {/if}
         <tr class="columnheader">
             {section name=rows loop=$rowDisplayCount}
@@ -59,7 +59,7 @@
     {if $wizard.currentStepName != 'Preview'}
     <div>
 
-      {if $loadedMapping}
+      {if $savedMappingName}
           <span>{$form.updateMapping.html} &nbsp;&nbsp; {$form.updateMapping.label}</span>
       {/if}
       <span>{$form.saveMapping.html} &nbsp;&nbsp; {$form.saveMapping.label}</span>

diff --git a/templates/CRM/Member/Import/Form/MapTable.tpl b/templates/CRM/Member/Import/Form/MapTable.tpl
index d64ac9f23b1e86845134c403244ee00943c74eb4..8a8cbfa47e4a4a1d6b5a18d66884228a8dff0b71 100644 (file)
--- a/templates/CRM/Member/Import/Form/MapTable.tpl
+++ b/templates/CRM/Member/Import/Form/MapTable.tpl
@@ -12,8 +12,8 @@
  <div id="map-field">
     {strip}
     <table>
-    {if $loadedMapping}
-        <tr class="columnheader-dark"><th colspan="4">{ts 1=$savedName}Saved Field Mapping: %1{/ts}</td></tr>
+    {if $savedMappingName}
+        <tr class="columnheader-dark"><th colspan="4">{ts 1=$savedMappingName}Saved Field Mapping: %1{/ts}</td></tr>
     {/if}
         <tr class="columnheader">
             {section name=rows loop=$rowDisplayCount}
@@ -59,7 +59,7 @@
     {if $wizard.currentStepName != 'Preview'}
     <div>
 
-      {if $loadedMapping}
+      {if $savedMappingName}
           <span>{$form.updateMapping.html} &nbsp;&nbsp; {$form.updateMapping.label}</span>
       {/if}
       <span>{$form.saveMapping.html} &nbsp;&nbsp; {$form.saveMapping.label}</span>