Fix undefined tpl vars on contact summary page
authorColeman Watts <coleman@civicrm.org>
Fri, 2 Jul 2021 23:29:24 +0000 (19:29 -0400)
committerColeman Watts <coleman@civicrm.org>
Fri, 2 Jul 2021 23:39:16 +0000 (19:39 -0400)
templates/CRM/Contact/Page/Inline/Demographics.tpl
templates/CRM/Contact/Page/Inline/Phone.tpl
templates/CRM/Contact/Page/View/CustomDataView.tpl
templates/CRM/Custom/Page/CustomDataView.tpl

index f2995abbc69f6f1488b0b00206024427128a590a..a0c2be1de5bdcca9d982d152a6e17af6444ce410 100644 (file)
     <div class="crm-summary-row">
       <div class="crm-label">{ts}Date of Birth{/ts}</div>
       <div class="crm-content crm-contact-birth_date_display">
-         {assign var="date_format" value = $fields.birth_date.smarty_view_format}
-         {$birth_date|crmDate:$date_format}
-          &nbsp;
+        {assign var="date_format" value=$fields.birth_date.smarty_view_format}
+        {if !empty($birth_date)}
+          {$birth_date|crmDate:$date_format}
+        {/if}
       </div>
     </div>
-      {if $is_deceased eq 1}
-        {if $deceased_date}
+      {if !empty($is_deceased)}
+        {if !empty($deceased_date)}
           <div class="crm-summary-row">
             <div class="crm-label">{ts}Date Deceased{/ts}</div>
             <div class="crm-content crm-contact-deceased_date_display">
               {assign var="date_format" value = $fields.birth_date.smarty_view_format}
               {$deceased_date|crmDate:$date_format}
-              {if $birth_date}({ts}Age{/ts} {if $age.y}{ts count=$age.y plural='%count years'}%count year{/ts}{elseif $age.m}{ts count=$age.m plural='%count months'}%count month{/ts}{/if}){/if}
+              {if !empty($birth_date)}({ts}Age{/ts} {if !empty($age.y)}{ts count=$age.y plural='%count years'}%count year{/ts}{elseif !empty($age.m)}{ts count=$age.m plural='%count months'}%count month{/ts}{/if}){/if}
             </div>
           </div>
         {else}
@@ -46,7 +47,7 @@
       {else}
         <div class="crm-summary-row">
           <div class="crm-label">{ts}Age{/ts}</div>
-          <div class="crm-content crm-contact-age_display">{if $age.y}{ts count=$age.y plural='%count years'}%count year{/ts}{elseif $age.m}{ts count=$age.m plural='%count months'}%count month{/ts}{/if}</div>
+          <div class="crm-content crm-contact-age_display">{if !empty($age.y)}{ts count=$age.y plural='%count years'}%count year{/ts}{elseif !empty($age.m)}{ts count=$age.m plural='%count months'}%count month{/ts}{/if}</div>
         </div>
       {/if}
     </div>
index 953fa202e42ebd9a97680388b324f76e92fb7477..2ddfcb73497bd2752537dcc262ddcbf05d6d0821 100644 (file)
       </div>
     {/if}
     {foreach from=$phone item=item}
-      {if $item.phone || $item.phone_ext}
-        <div class="crm-summary-row {if $item.is_primary eq 1}primary{/if}">
+      {if !empty($item.phone) || !empty($item.phone_ext)}
+        <div class="crm-summary-row {if !empty($item.is_primary)}primary{/if}">
           <div class="crm-label">
             {privacyFlag field=do_not_sms condition=$privacy.do_not_sms}
             {privacyFlag field=do_not_phone condition=$privacy.do_not_phone}
             {$item.location_type} {$item.phone_type}
           </div>
           <div class="crm-content crm-contact_phone">
-            {$item.phone}{if $item.phone_ext}&nbsp;&nbsp;{ts}ext.{/ts} {$item.phone_ext}{/if}
+            {$item.phone}{if !empty($item.phone_ext)}&nbsp;&nbsp;{ts}ext.{/ts} {$item.phone_ext}{/if}
           </div>
         </div>
       {/if}
index 68b9eb992b9a44d822caee5d574b7fcf599a39ae..4cfd3cd8ccb46bcdb82279321d167b9c4fb7a8e1 100644 (file)
@@ -12,9 +12,9 @@
 {foreach from=$viewCustomData item=customValues key=customGroupId}
   {assign var="cgcount" value=1}
   {assign var="count" value=$customGroupCount%2}
-  {if ($count eq $side) or $skipTitle }
+  {if ($count eq $side) or !empty($skipTitle)}
     {foreach from=$customValues item=cd_edit key=cvID}
-      <div class="customFieldGroup crm-collapsible{if $cd_edit.collapse_display} collapsed{/if} ui-corner-all {$cd_edit.name} crm-custom-set-block-{$customGroupId}">
+      <div class="customFieldGroup crm-collapsible{if !empty($cd_edit.collapse_display)} collapsed{/if} ui-corner-all {$cd_edit.name} crm-custom-set-block-{$customGroupId}">
         <div class="collapsible-title">
           {$cd_edit.title}
         </div>
index f7f3f870031edfdafdb4799f98ddd80be36b9648..df241801a4141cfe755be05d63cc666793ccdcad 100644 (file)
@@ -31,7 +31,7 @@
       {assign var="showEdit" value=0}
       <tr>
         <td id="{$cd_edit.name}_{$index}" class="section-shown form-item">
-          <div class="crm-accordion-wrapper {if $cd_edit.collapse_display eq 0 or $skipTitle} {else}collapsed{/if}">
+          <div class="crm-accordion-wrapper{if !empty($cd_edit.collapse_display) && empty($skipTitle)} collapsed{/if}">
             {if !$skipTitle}
               <div class="crm-accordion-header">
                 {$cd_edit.title}