Merge pull request #20828 from colemanw/tplFix

Fix undefined tpl vars
civicrm · Jul 12, 2021 · 3c2d2df · 3c2d2df
2 parents 4247f8c + d40167a
commit 3c2d2df
Show file tree

Hide file tree

Showing 6 changed files with 9 additions and 13 deletions.
diff --git a/CRM/Admin/Page/Access.php b/CRM/Admin/Page/Access.php
@@ -25,12 +25,8 @@ class CRM_Admin_Page_Access extends CRM_Core_Page {
    */
   public function run() {
     $urlParams = CRM_Utils_System::getCMSPermissionsUrlParams();
-    if (isset($urlParams['ufAccessURL'])) {
-      $this->assign('ufAccessURL', $urlParams['ufAccessURL']);
-    }
-    if (isset($urlParams['jAccessParams'])) {
-      $this->assign('jAccessParams', $urlParams['jAccessParams']);
-    }
+    $this->assign('ufAccessURL', $urlParams['ufAccessURL'] ?? NULL);
+    $this->assign('jAccessParams', $urlParams['jAccessParams'] ?? NULL);
     return parent::run();
   }
 

diff --git a/templates/CRM/ACL/Form/WordPress/Permissions.tpl b/templates/CRM/ACL/Form/WordPress/Permissions.tpl
@@ -38,7 +38,7 @@
 
       <td style="height: 2.6em;">
         {$row.label}
-        {if $row.desc}
+        {if !empty($row.desc)}
           <br/><span class="description">{$row.desc}</span>
         {/if}
       </td>

diff --git a/templates/CRM/Contact/Form/Search/ResultTasks.tpl b/templates/CRM/Contact/Form/Search/ResultTasks.tpl
@@ -39,7 +39,7 @@
   <table class="form-layout-compressed">
   <tr>
     <td class="font-size12pt" style="width: 30%;">
-        {if $savedSearch.name}{$savedSearch.name} ({ts}smart group{/ts}) - {/if}
+        {if !empty($savedSearch.name)}{$savedSearch.name} ({ts}smart group{/ts}) - {/if}
         {ts count=$pager->_totalItems plural='%count Contacts'}%count Contact{/ts}
     </td>
 

diff --git a/templates/CRM/Contact/Form/Selector.tpl b/templates/CRM/Contact/Form/Selector.tpl
@@ -23,10 +23,10 @@
       {/if}
       {foreach from=$columnHeaders item=header}
         <th scope="col">
-        {if $header.sort}
+        {if !empty($header.sort)}
           {assign var='key' value=$header.sort}
           {$sort->_response.$key.link}
-        {else}
+        {elseif !empty($header.name)}
           {$header.name}
         {/if}
         </th>
@@ -78,7 +78,7 @@
                 {$row.status}</td>
             {/if}
             <td>{$row.contact_type}</td>
-            <td><a href="{crmURL p='civicrm/contact/view' q="reset=1&cid=`$row.contact_id`&key=`$qfKey`&context=`$context`"}">{if $row.is_deleted}<del>{/if}{$row.sort_name}{if $row.is_deleted}</del>{/if}</a></td>
+            <td><a href="{crmURL p='civicrm/contact/view' q="reset=1&cid=`$row.contact_id`&key=`$qfKey`&context=`$context`"}">{if !empty($row.is_deleted)}<del>{/if}{$row.sort_name}{if !empty($row.is_deleted)}</del>{/if}</a></td>
             {if $action eq 512 or $action eq 256}
               {if !empty($columnHeaders.street_address)}
           <td><span title="{$row.street_address|escape}">{$row.street_address|mb_truncate:22:"...":true}{privacyFlag field=do_not_mail condition=$row.do_not_mail}</span></td>

diff --git a/templates/CRM/Custom/Form/Group.tpl b/templates/CRM/Custom/Form/Group.tpl
@@ -73,7 +73,7 @@
       {crmButton p='civicrm/admin/custom/group/field' q="action=browse&reset=1&gid=$gid" icon="th-list"}{ts}Custom Fields for this Set{/ts}{/crmButton}
     </div>
 {/if}
-{$initHideBlocks}
+{if !empty($initHideBlocks)}{$initHideBlocks}{/if}
 {literal}
 <script type="text/Javascript">
 CRM.$(function($) {

diff --git a/templates/CRM/common/pagerAToZ.tpl b/templates/CRM/common/pagerAToZ.tpl
@@ -11,7 +11,7 @@
 <div id="alpha-filter">
     <ul>
     {foreach from=$aToZ item=letter}
-        <li {if $letter.class}class="{$letter.class}"{/if}>{$letter.item}</li>
+        <li {if !empty($letter.class)}class="{$letter.class}"{/if}>{$letter.item}</li>
     {/foreach}
     </ul>
 </div>