Online membership receipt simplification

Rather than having one section in an IF for the scenario where we have a separate payment situation
but there is no contribution payment, consolidate this scenario with the general 'we
have a membership but no contribution in the receipt

diff --git a/xml/templates/message_templates/membership_online_receipt_html.tpl b/xml/templates/message_templates/membership_online_receipt_html.tpl
index bae92aa780b88c5bfea1e5954ca5afad681585b3..0381eca2005074cd1cf18d833fb99258b4de82bc 100644 (file)
--- a/xml/templates/message_templates/membership_online_receipt_html.tpl
+++ b/xml/templates/message_templates/membership_online_receipt_html.tpl
@@ -72,7 +72,7 @@
         <th {$headerStyle}>{ts}Membership Fee{/ts}</th>
       </tr>
 
-      {if !$useForMember and isset($membership_amount) and !empty($is_quick_config)}
+      {if !$isShowLineItems && {contribution.total_amount|boolean}}
         <tr>
           <td {$labelStyle}>
             {ts 1=$membership_name}%1 Membership{/ts}
@@ -177,20 +177,6 @@
             {contribution.total_amount} {if isset($amount_level)} - {$amount_level}{/if}
         </td>
       </tr>
-    {elseif isset($membership_amount)}
-      <tr>
-        <th {$headerStyle}>
-          {ts}Membership Fee{/ts}
-        </th>
-      </tr>
-      <tr>
-        <td {$labelStyle}>
-          {ts 1=$membership_name}%1 Membership{/ts}
-        </td>
-        <td {$valueStyle}>
-          {$membership_amount|crmMoney}
-        </td>
-      </tr>
     {/if}
 
     {if !empty($receive_date)}