Fix inaccessible links for checksum users
authorJitendra Purohit <jitendra@fuzion.co.nz>
Fri, 31 Aug 2018 09:24:49 +0000 (14:54 +0530)
committerJitendra Purohit <jitendra@fuzion.co.nz>
Fri, 31 Aug 2018 09:24:49 +0000 (14:54 +0530)
CRM/Contact/Page/View/UserDashBoard.php
templates/CRM/Contact/Page/View/UserDashBoard/GroupContact.tpl
templates/CRM/Contribute/Page/PcpUserDashboard.tpl
templates/CRM/Pledge/Page/UserDashboard.tpl

index d294e002737708bffd32d6490a4c67540cf72ac0..2cc99577e1cc6af3dc2254fc33b2997aa464e4c1 100644 (file)
@@ -74,6 +74,7 @@ class CRM_Contact_Page_View_UserDashBoard extends CRM_Core_Page {
     if (empty($userID) && $this->_contactId && $userChecksum) {
       $this->assign('userChecksum', $userChecksum);
       $validUser = CRM_Contact_BAO_Contact_Utils::validChecksum($this->_contactId, $userChecksum);
+      $this->_isChecksumUser = $validUser;
     }
 
     if (!$this->_contactId) {
@@ -169,7 +170,7 @@ class CRM_Contact_Page_View_UserDashBoard extends CRM_Core_Page {
       $this->assign('pcpInfo', $pcpInfo);
     }
 
-    if (!empty($this->_userOptions['Assigned Activities'])) {
+    if (!empty($this->_userOptions['Assigned Activities']) && empty($this->_isChecksumUser)) {
       // Assigned Activities section
       $dashboardElements[] = array(
         'class' => 'crm-dashboard-assignedActivities',
index 30db0fee691f635ed61aab0123adaa107501f844..e03173988b4284df040df4a7b244ca871ace520c 100644 (file)
@@ -23,6 +23,9 @@
  | see the CiviCRM license FAQ at http://civicrm.org/licensing        |
  +--------------------------------------------------------------------+
 *}
+{if $userChecksum}
+  {assign var=edit value='0'}
+{/if}
 {crmRegion name="crm-contact-userdashboard-groupcontact-pre"}
 {/crmRegion}
 <div id="groupContact">
index 7cb4577645ec505296ce1f06dbec3b447cf3633f..3170776c63022126abe85bd3d8008534bf2b178a 100644 (file)
@@ -37,7 +37,7 @@
     <th>{ts}In Support of{/ts}</th>
     <th>{ts}Campaign Ends{/ts}</th>
     <th>{ts}Status{/ts}</th>
-    <th></th>
+    {if !$userChecksum} <th></th> {/if}
   </tr>
 
   {foreach from=$pcpInfo item=row}
@@ -46,7 +46,9 @@
         <td>{$row.pageTitle}</td>
         <td>{if $row.end_date}{$row.end_date|truncate:10:''|crmDate}{else}({ts}ongoing{/ts}){/if}</td>
         <td>{$row.pcpStatus}</td>
-        <td>{$row.action|replace:'xx':$row.pcpId}</td>
+        {if !$userChecksum}
+          <td>{$row.action|replace:'xx':$row.pcpId}</td>
+        {/if}
   </tr>
   {/foreach}
 </table>
@@ -59,7 +61,6 @@
 </div>
 {/if}
 
-
 {if $pcpBlock}
 {strip}
 {if $pcpInfo} {* Change layout and text if they already have a PCP. *}
index f0cab011a01b8ab9db63ce9dc0fdd340cd9ff8ed..59dfbb4aaeb2a97ac4ca2ea8d93d627748b42249 100644 (file)
     <td class="crm-pledge-pledge_next_pay_date">{$row.pledge_next_pay_date|truncate:10:''|crmDate}</td>
     <td class="crm-pledge-pledge_next_pay_amount">{$row.pledge_next_pay_amount|crmMoney:$row.pledge_currency}</td>
     <td class="crm-pledge-pledge_status crm-pledge-pledge_status_{$row.pledge_status}">{$row.pledge_status}</td>
-    <td>
-      {if $row.pledge_contribution_page_id and ($row.pledge_status_name neq 'Completed') and ( $row.contact_id eq $loggedUserID ) }
-        <a href="{crmURL p='civicrm/contribute/transact' q="reset=1&id=`$row.pledge_contribution_page_id`&pledgeId=`$row.pledge_id`"}">{ts}Make Payment{/ts}</a><br/>
-      {/if}
-      <a class="crm-expand-row" title="{ts}view payments{/ts}" href="{crmURL p='civicrm/pledge/payment' q="action=browse&context=`$context`&pledgeId=`$row.pledge_id`&cid=`$row.contact_id`"}">{ts}Payments{/ts}</a>
-    </td>
+    {if !$userChecksum}
+      <td>
+        {if $row.pledge_contribution_page_id and ($row.pledge_status_name neq 'Completed') and ( $row.contact_id eq $loggedUserID ) }
+          <a href="{crmURL p='civicrm/contribute/transact' q="reset=1&id=`$row.pledge_contribution_page_id`&pledgeId=`$row.pledge_id`"}">{ts}Make Payment{/ts}</a><br/>
+        {/if}
+        <a class="crm-expand-row" title="{ts}view payments{/ts}" href="{crmURL p='civicrm/pledge/payment' q="action=browse&context=`$context`&pledgeId=`$row.pledge_id`&cid=`$row.contact_id`"}">{ts}Payments{/ts}</a>
+      </td>
+    {/if}
    </tr>
   {/foreach}
 </table>