From: Eileen McNaughton <eileen@fuzion.co.nz>
Date: Mon, 27 Oct 2014 21:26:42 +0000 (+1300)
Subject: CRM-15509 put div outside if statements & ensure fieldsets are enclosing within an... 
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=c78d6d6025891829a33d04ee3cdeda949c9d16a1;p=civicrm-core.git

CRM-15509 put div outside if statements & ensure fieldsets are enclosing within an if statement
---

diff --git a/templates/CRM/Core/BillingBlock.tpl b/templates/CRM/Core/BillingBlock.tpl
index 534e6a5ddc..56cad1d540 100644
--- a/templates/CRM/Core/BillingBlock.tpl
+++ b/templates/CRM/Core/BillingBlock.tpl
@@ -24,9 +24,8 @@
  +--------------------------------------------------------------------+
 *}
 {crmRegion name="billing-block"}
-
-{if $paymentFields}
-  <div id="payment_information">
+<div id="payment_information">
+  {if $paymentFields}
     <fieldset class="billing_mode-group {$paymentTypeName}_info-group">
       <legend>
         {$paymentTypeLabel}
@@ -50,30 +49,31 @@
           </div>
         {/foreach}
       </div>
+    </fieldset>
+  {/if}
+  {if $billingDetailsFields}
+    {if $profileAddressFields}
+      <input type="checkbox" id="billingcheckbox" value="0">
+      <label for="billingcheckbox">{ts}My billing address is the same as above{/ts}</label>
     {/if}
-    {if $billingDetailsFields}
-      {if $profileAddressFields}
-        <input type="checkbox" id="billingcheckbox" value="0">
-        <label for="billingcheckbox">{ts}My billing address is the same as above{/ts}</label>
-      {/if}
-      <fieldset class="billing_name_address-group">
-        <legend>{ts}Billing Name and Address{/ts}</legend>
-        <div class="crm-section billing_name_address-section">
-          {foreach from=$billingDetailsFields item=billingField}
-            <div class="crm-section {$form.$billingField.name}-section">
-              <div class="label">{$form.$billingField.label}</div>
-              {if $form.$billingField.type == 'text'}
-                <div class="content">{$form.$billingField.html}</div>
-              {else}
-                <div class="content">{$form.$billingField.html|crmAddClass:big}</div>
-              {/if}
-              <div class="clear"></div>
-            </div>
-          {/foreach}
-        </div>
-      </fieldset>
-    {/if}
-  </div>
+    <fieldset class="billing_name_address-group">
+      <legend>{ts}Billing Name and Address{/ts}</legend>
+      <div class="crm-section billing_name_address-section">
+        {foreach from=$billingDetailsFields item=billingField}
+          <div class="crm-section {$form.$billingField.name}-section">
+            <div class="label">{$form.$billingField.label}</div>
+            {if $form.$billingField.type == 'text'}
+              <div class="content">{$form.$billingField.html}</div>
+            {else}
+              <div class="content">{$form.$billingField.html|crmAddClass:big}</div>
+            {/if}
+            <div class="clear"></div>
+          </div>
+        {/foreach}
+      </div>
+    </fieldset>
+  {/if}
+</div>
 {if $profileAddressFields}
   <script type="text/javascript">
     {literal}