Some code cleanups

ftoverride.php

@@ -197,6 +197,14 @@ function ftoverride_civicrm_entityTypes(&$entityTypes) {
  */
 function ftoverride_civicrm_buildForm($formName, &$form) {
   if ('CRM_Contribute_Form_ContributionPage_Settings' == $formName) {
+    $showElement = TRUE;
+    if ($form->getVar('_id')) {
+      $snippet = CRM_Utils_Array::value('snippet', $_GET);
+      if (empty($snippet)) {
+        $showElement = FALSE;
+      }
+    }
+    $form->assign('showElement', $showElement);
     $financialTypes = [];
     CRM_Financial_BAO_FinancialType::getAvailableFinancialTypes($financialTypes, $form->getVar('_action'));
     $form->add(
@@ -266,7 +274,7 @@ function ftoverride_civicrm_buildForm($formName, &$form) {
 function ftoverride_civicrm_pre($op, $objectName, $id, &$params) {
   if (in_array($op, ['create', 'edit'])
     && $objectName == 'ContributionPage'
-    && !empty($params['designation'])
+    && isset($params['designation'])
   ) {
     if ($id) {
       $params['id'] = $id;

templates/CRM/Contribute/Form/ContributionMain/common.tpl

@@ -6,7 +6,7 @@
 {literal}
 <script type="text/javascript">
   CRM.$(function($) {
-    $('div#priceset-div').after($('div.ft_override_designation-section'));
+    $('div#priceset:first').after($('div.ft_override_designation-section'));
   });
 </script>
 {/literal}

templates/CRM/Contribute/Form/ContributionPage/common.tpl

@@ -1,3 +1,4 @@
+{if $showElement}
 <table class="ft_override_designation-block">
   <tr class="crm-contribution-contributionpage-settings-form-block-ft_override_designation">
     <td scope="row" class="label" width="20%">{$form.designation.label}</td>
@@ -11,3 +12,4 @@
   });
 </script>
 {/literal}
+{/if}