diff --git a/CRM/Case/BAO/Case.php b/CRM/Case/BAO/Case.php index 1b1340a9ef1..b4a596da4fb 100644 --- a/CRM/Case/BAO/Case.php +++ b/CRM/Case/BAO/Case.php @@ -1093,7 +1093,8 @@ public static function getCaseActivity($caseID, &$params, $contactID, $context = $caseActivity['no_attachments'] = count($attachmentIDs); } - $caseActivities[$caseActivityId]['links'] = CRM_Case_Selector_Search::addCaseActivityLinks($caseID, $contactID, $userID, $context, $dao); + $caseActivities[$caseActivityId]['links'] + = CRM_Case_Selector_Search::addCaseActivityLinks($caseID, $contactID, $userID, $context, $dao); } $caseActivitiesDT = array(); diff --git a/CRM/Case/Form/ActivityView.php b/CRM/Case/Form/ActivityView.php index b18be74fdc3..af9c5c4b708 100644 --- a/CRM/Case/Form/ActivityView.php +++ b/CRM/Case/Form/ActivityView.php @@ -204,6 +204,17 @@ public function preProcess() { ], ]; CRM_Utils_System::appendBreadCrumb($breadcrumb); + + $this->addButtons([ + [ + 'type' => 'cancel', + 'name' => ts('Done'), + ], + ]); + // Add additional action links + $activityDeleted = CRM_Core_DAO::getFieldValue('CRM_Activity_DAO_Activity', $activityID, 'is_deleted'); + $actionLinks = CRM_Case_Selector_Search::permissionedActionLinks($caseID, $contactID, CRM_Core_Session::getLoggedInContactID(), NULL, $activityTypeID, $activityDeleted, $activityID, FALSE); + $this->assign('actionLinks', $actionLinks); } } diff --git a/CRM/Case/Selector/Search.php b/CRM/Case/Selector/Search.php index e34a3cc5fed..37dc38ce7cc 100644 --- a/CRM/Case/Selector/Search.php +++ b/CRM/Case/Selector/Search.php @@ -493,70 +493,192 @@ public function getExportFileName($output = 'csv') { * @param int $contactID * @param int $userID * @param string $context - * @param \CRM_Core_DAO $dao + * @param \CRM_Activity_BAO_Activity $dao + * @param bool $allowView * - * @return string - * HTML formatted Link + * @return string $linksMarkup */ - public static function addCaseActivityLinks($caseID, $contactID, $userID, $context, $dao) { - // FIXME: Why are we not using CRM_Core_Action for these links? This is too much manual work and likely to get out-of-sync with core markup. - $caseActivityId = $dao->id; - $allowView = CRM_Case_BAO_Case::checkPermission($caseActivityId, 'view', $dao->activity_type_id, $userID); - $allowEdit = CRM_Case_BAO_Case::checkPermission($caseActivityId, 'edit', $dao->activity_type_id, $userID); - $allowDelete = CRM_Case_BAO_Case::checkPermission($caseActivityId, 'delete', $dao->activity_type_id, $userID); - $emailActivityTypeIDs = [ - 'Email' => CRM_Core_PseudoConstant::getKey('CRM_Activity_BAO_Activity', 'activity_type_id', 'Email'), - 'Inbound Email' => CRM_Core_PseudoConstant::getKey('CRM_Activity_BAO_Activity', 'activity_type_id', 'Inbound Email'), - ]; - $url = CRM_Utils_System::url("civicrm/case/activity", - "reset=1&cid={$contactID}&caseid={$caseID}", FALSE, NULL, FALSE - ); - $contextUrl = ''; - if ($context == 'fulltext') { - $contextUrl = "&context={$context}"; - } - $editUrl = "{$url}&action=update{$contextUrl}"; - $deleteUrl = "{$url}&action=delete{$contextUrl}"; - $restoreUrl = "{$url}&action=renew{$contextUrl}"; - $viewTitle = ts('View activity'); + public static function addCaseActivityLinks($caseID, $contactID, $userID, $context, $dao, $allowView = TRUE) { $caseDeleted = CRM_Core_DAO::getFieldValue('CRM_Case_DAO_Case', $caseID, 'is_deleted'); - - $url = ""; - $css = 'class="action-item crm-hover-button"'; - if ($allowView) { - $viewUrl = CRM_Utils_System::url('civicrm/case/activity/view', array('cid' => $contactID, 'aid' => $caseActivityId)); - $url = '' . ts('View') . ''; + $actionLinks = self::actionLinks(); + // Check logged in user for permission. + if (CRM_Case_BAO_Case::checkPermission($dao->id, 'view', $dao->activity_type_id, $userID)) { + $permissions[] = CRM_Core_Permission::VIEW; + } + if (!$allowView) { + unset($actionLinks[CRM_Core_Action::VIEW]); } - $additionalUrl = "&id={$caseActivityId}"; if (!$dao->deleted) { - //hide edit link of activity type email.CRM-4530. - if (!in_array($dao->type, $emailActivityTypeIDs)) { - //hide Edit link if activity type is NOT editable (special case activities).CRM-5871 - if ($allowEdit) { - $url .= '' . ts('Edit') . ' '; - } + // Activity is not deleted, allow user to edit/delete if they have permission + // hide Edit link if: + // 1. User does not have edit permission. + // 2. Activity type is NOT editable (special case activities).CRM-5871 + if (CRM_Case_BAO_Case::checkPermission($dao->id, 'edit', $dao->activity_type_id, $userID)) { + $permissions[] = CRM_Core_Permission::EDIT; } - if ($allowDelete) { - $url .= ' ' . ts('Delete') . ''; + if (in_array($dao->activity_type_id, CRM_Activity_BAO_Activity::getViewOnlyActivityTypeIDs())) { + unset($actionLinks[CRM_Core_Action::UPDATE]); } + if (CRM_Case_BAO_Case::checkPermission($dao->id, 'delete', $dao->activity_type_id, $userID)) { + $permissions[] = CRM_Core_Permission::DELETE; + } + unset($actionLinks[CRM_Core_Action::RENEW]); } - elseif (!$caseDeleted) { - $url = ' ' . ts('Restore') . ''; + $extraMask = 0; + if ($dao->deleted && !$caseDeleted + && (CRM_Case_BAO_Case::checkPermission($dao->id, 'delete', $dao->activity_type_id, $userID))) { + // Case is not deleted but activity is. + // Allow user to restore activity if they have delete permissions + unset($actionLinks[CRM_Core_Action::DELETE]); + $extraMask = CRM_Core_Action::RENEW; } - - //check for operations. - if (CRM_Case_BAO_Case::checkPermission($caseActivityId, 'Move To Case', $dao->activity_type_id)) { - $url .= ' ' . ts('Move To Case') . ' '; + if (!CRM_Case_BAO_Case::checkPermission($dao->id, 'Move To Case', $dao->activity_type_id)) { + unset($actionLinks[CRM_Core_Action::DETACH]); } - if (CRM_Case_BAO_Case::checkPermission($caseActivityId, 'Copy To Case', $dao->activity_type_id)) { - $url .= ' ' . ts('Copy To Case') . ' '; + if (!CRM_Case_BAO_Case::checkPermission($dao->id, 'Copy To Case', $dao->activity_type_id)) { + unset($actionLinks[CRM_Core_Action::COPY]); } + $actionMask = CRM_Core_Action::mask($permissions) | $extraMask; + $values = [ + 'aid' => $dao->id, + 'cid' => $contactID, + 'cxt' => empty($context) ? '' : "&context={$context}", + 'caseid' => $caseID, + ]; + $linksMarkup = CRM_Core_Action::formLink($actionLinks, + $actionMask, + $values, + ts('more'), + FALSE, + 'case.tab.row', + 'Activity', + $dao->id + ); // if there are file attachments we will return how many and, if only one, add a link to it if (!empty($dao->attachment_ids)) { - $url .= implode(' ', CRM_Core_BAO_File::paperIconAttachment('civicrm_activity', $caseActivityId)); + $linksMarkup .= implode(' ', CRM_Core_BAO_File::paperIconAttachment('civicrm_activity', $dao->id)); } + return $linksMarkup; + } - return $url; + /** + * @param int $caseID + * @param int $contactID + * @param int $userID + * @param string $context + * @param int $activityTypeID + * @param int $activityDeleted + * @param int $activityID + * @param bool $allowView + * + * @return array|null + */ + public static function permissionedActionLinks($caseID, $contactID, $userID, $context, $activityTypeID, $activityDeleted, $activityID, $allowView = TRUE) { + $caseDeleted = CRM_Core_DAO::getFieldValue('CRM_Case_DAO_Case', $caseID, 'is_deleted'); + $values = [ + 'aid' => $activityID, + 'cid' => $contactID, + 'cxt' => empty($context) ? '' : "&context={$context}", + 'caseid' => $caseID, + ]; + $actionLinks = self::actionLinks(); + + // Check logged in user for permission. + if (CRM_Case_BAO_Case::checkPermission($activityID, 'view', $activityTypeID, $userID)) { + $permissions[] = CRM_Core_Permission::VIEW; + } + if (!$allowView) { + unset($actionLinks[CRM_Core_Action::VIEW]); + } + if (!$activityDeleted) { + // Activity is not deleted, allow user to edit/delete if they have permission + + // hide Edit link if: + // 1. User does not have edit permission. + // 2. Activity type is NOT editable (special case activities).CRM-5871 + if (CRM_Case_BAO_Case::checkPermission($activityID, 'edit', $activityTypeID, $userID)) { + $permissions[] = CRM_Core_Permission::EDIT; + } + if (in_array($activityTypeID, CRM_Activity_BAO_Activity::getViewOnlyActivityTypeIDs())) { + unset($actionLinks[CRM_Core_Action::UPDATE]); + } + if (CRM_Case_BAO_Case::checkPermission($activityID, 'delete', $activityTypeID, $userID)) { + $permissions[] = CRM_Core_Permission::DELETE; + } + unset($actionLinks[CRM_Core_Action::RENEW]); + } + $extraMask = 0; + if ($activityDeleted && !$caseDeleted + && (CRM_Case_BAO_Case::checkPermission($activityID, 'delete', $activityTypeID, $userID))) { + // Case is not deleted but activity is. + // Allow user to restore activity if they have delete permissions + unset($actionLinks[CRM_Core_Action::DELETE]); + $extraMask = CRM_Core_Action::RENEW; + } + if (!CRM_Case_BAO_Case::checkPermission($activityID, 'Move To Case', $activityTypeID)) { + unset($actionLinks[CRM_Core_Action::DETACH]); + } + if (!CRM_Case_BAO_Case::checkPermission($activityID, 'Copy To Case', $activityTypeID)) { + unset($actionLinks[CRM_Core_Action::COPY]); + } + + $actionMask = CRM_Core_Action::mask($permissions) | $extraMask; + return CRM_Core_Action::filterLinks($actionLinks, $actionMask, $values, 'case.activity', 'Activity', $activityID); + } + + /** + * Get the action links for this page. + * + * @return array + */ + public static function actionLinks() { + // check if variable _actionsLinks is populated + if (!isset(self::$_actionLinks)) { + self::$_actionLinks = [ + CRM_Core_Action::VIEW => [ + 'name' => ts('View'), + 'url' => 'civicrm/case/activity/view', + 'qs' => 'reset=1&cid=%%cid%%&caseid=%%caseid%%&aid=%%aid%%', + 'title' => ts('View'), + ], + CRM_Core_Action::UPDATE => [ + 'name' => ts('Edit'), + 'url' => 'civicrm/case/activity', + 'qs' => 'reset=1&cid=%%cid%%&caseid=%%caseid%%&id=%%aid%%&action=update%%cxt%%', + 'title' => ts('Edit'), + 'icon' => 'fa-pencil', + ], + CRM_Core_Action::DELETE => [ + 'name' => ts('Delete'), + 'url' => 'civicrm/case/activity', + 'qs' => 'reset=1&cid=%%cid%%&caseid=%%caseid%%&id=%%aid%%&action=delete%%cxt%%', + 'title' => ts('Delete'), + 'icon' => 'fa-trash', + ], + CRM_Core_Action::RENEW => [ + 'name' => ts('Restore'), + 'url' => 'civicrm/case/activity', + 'qs' => 'reset=1&cid=%%cid%%&caseid=%%caseid%%&id=%%aid%%&action=renew%%cxt%%', + 'title' => ts('Restore'), + 'icon' => 'fa-undo', + ], + CRM_Core_Action::DETACH => [ + 'name' => ts('Move To Case'), + 'ref' => 'move_to_case_action', + 'title' => ts('Move To Case'), + 'extra' => 'onclick = "Javascript:fileOnCase( \'move\', %%aid%%, %%caseid%%, this ); return false;"', + 'icon' => 'fa-clipboard', + ], + CRM_Core_Action::COPY => [ + 'name' => ts('Copy To Case'), + 'ref' => 'copy_to_case_action', + 'title' => ts('Copy To Case'), + 'extra' => 'onclick = "Javascript:fileOnCase( \'copy\', %%aid%%, %%caseid%%, this ); return false;"', + 'icon' => 'fa-files-o', + ], + ]; + } + return self::$_actionLinks; } } diff --git a/CRM/Report/Form/Activity.php b/CRM/Report/Form/Activity.php index 0f4facd7cff..e9004501c90 100644 --- a/CRM/Report/Form/Activity.php +++ b/CRM/Report/Form/Activity.php @@ -361,9 +361,27 @@ public function __construct() { $this->_groupFilter = TRUE; $this->_tagFilter = TRUE; $this->_tagFilterTable = 'civicrm_activity'; + parent::__construct(); } + public function preProcess() { + // Is "Include Case Activities" selected? If yes, include the case_id as a hidden column + $params = $this->controller->exportValues($this->_name); + if (!empty($params['include_case_activities_value'])) { + $this->_columns['civicrm_case_activity'] = [ + 'dao' => 'CRM_Case_DAO_CaseActivity', + 'fields' => [ + 'case_id' => [ + 'no_display' => TRUE, + 'required' => TRUE, + ], + ], + ]; + } + parent::preProcess(); + } + /** * Adding address fields with dbAlias for order clause. * @@ -502,34 +520,7 @@ public function select($recordType = 'target') { * @todo remove this function & declare the 3 contact tables separately */ public function from() { - $activityContacts = CRM_Activity_BAO_ActivityContact::buildOptions('record_type_id', 'validate'); - $targetID = CRM_Utils_Array::key('Activity Targets', $activityContacts); - - $this->_from = " - FROM civicrm_activity {$this->_aliases['civicrm_activity']} - INNER JOIN civicrm_activity_contact {$this->_aliases['civicrm_activity_contact']} - ON {$this->_aliases['civicrm_activity']}.id = {$this->_aliases['civicrm_activity_contact']}.activity_id AND - {$this->_aliases['civicrm_activity_contact']}.record_type_id = {$targetID} - INNER JOIN civicrm_contact civicrm_contact_target - ON {$this->_aliases['civicrm_activity_contact']}.contact_id = civicrm_contact_target.id - {$this->_aclFrom}"; - - if ($this->isTableSelected('civicrm_email')) { - $this->_from .= " - LEFT JOIN civicrm_email civicrm_email_target - ON {$this->_aliases['civicrm_activity_contact']}.contact_id = civicrm_email_target.contact_id AND - civicrm_email_target.is_primary = 1"; - } - - if ($this->isTableSelected('civicrm_phone')) { - $this->_from .= " - LEFT JOIN civicrm_phone civicrm_phone_target - ON {$this->_aliases['civicrm_activity_contact']}.contact_id = civicrm_phone_target.contact_id AND - civicrm_phone_target.is_primary = 1 "; - } - $this->_aliases['civicrm_contact'] = 'civicrm_contact_target'; - - $this->joinAddressFromContact(); + $this->buildFrom('target'); } /** @@ -834,12 +825,18 @@ public function buildQuery($applyLimit = TRUE) { $this->_where .= " AND {$this->_aclWhere} "; } + $caseJoin = ''; + if (!empty($this->_params['include_case_activities_value'])) { + $caseJoin = "LEFT JOIN civicrm_case_activity {$this->_aliases['civicrm_case_activity']} ON {$this->_aliases['civicrm_activity']}.id = {$this->_aliases['civicrm_case_activity']}.activity_id"; + } + $sql = "{$this->_select} FROM $tempTableName tar INNER JOIN civicrm_activity {$this->_aliases['civicrm_activity']} ON {$this->_aliases['civicrm_activity']}.id = tar.civicrm_activity_id INNER JOIN civicrm_activity_contact {$this->_aliases['civicrm_activity_contact']} ON {$this->_aliases['civicrm_activity_contact']}.activity_id = {$this->_aliases['civicrm_activity']}.id AND {$this->_aliases['civicrm_activity_contact']}.record_type_id = {$sourceID} LEFT JOIN civicrm_contact contact_civireport ON contact_civireport.id = {$this->_aliases['civicrm_activity_contact']}.contact_id + {$caseJoin} {$this->_where} {$groupByFromSelect} {$this->_having} {$this->_orderBy} {$this->_limit}"; CRM_Utils_Hook::alterReportVar('sql', $this, $this); @@ -909,20 +906,36 @@ public function alterDisplay(&$rows) { $cid = $rows[$rowNum]['civicrm_contact_contact_source_id']; } - $actActionLinks = CRM_Activity_Selector_Activity::actionLinks($row['civicrm_activity_activity_type_id'], - CRM_Utils_Array::value('civicrm_activity_source_record_id', $rows[$rowNum]), - FALSE, - $rows[$rowNum]['civicrm_activity_id'] - ); - - $actLinkValues = [ - 'id' => $rows[$rowNum]['civicrm_activity_id'], - 'cid' => $cid, - 'cxt' => $context, - ]; - $actUrl = CRM_Utils_System::url($actActionLinks[CRM_Core_Action::VIEW]['url'], - CRM_Core_Action::replace($actActionLinks[CRM_Core_Action::VIEW]['qs'], $actLinkValues), TRUE - ); + if (empty($this->_params['include_case_activities_value']) || empty($rows[$rowNum]['civicrm_case_activity_case_id'])) { + // Generate a "view activity" link + $actActionLinks = CRM_Activity_Selector_Activity::actionLinks($row['civicrm_activity_activity_type_id'], + CRM_Utils_Array::value('civicrm_activity_source_record_id', $rows[$rowNum]), + FALSE, + $rows[$rowNum]['civicrm_activity_id'] + ); + + $actLinkValues = [ + 'id' => $rows[$rowNum]['civicrm_activity_id'], + 'cid' => $cid, + 'cxt' => $context, + ]; + $actUrl = CRM_Utils_System::url($actActionLinks[CRM_Core_Action::VIEW]['url'], + CRM_Core_Action::replace($actActionLinks[CRM_Core_Action::VIEW]['qs'], $actLinkValues), TRUE + ); + } + else { + // Generate a "view case activity" link + $caseActionLinks = CRM_Case_Selector_Search::actionLinks(); + $caseLinkValues = [ + 'aid' => $rows[$rowNum]['civicrm_activity_id'], + 'caseid' => $rows[$rowNum]['civicrm_case_activity_case_id'], + 'cid' => $cid, + 'cxt' => $context, + ]; + $actUrl = CRM_Utils_System::url($caseActionLinks[CRM_Core_Action::VIEW]['url'], + CRM_Core_Action::replace($caseActionLinks[CRM_Core_Action::VIEW]['qs'], $caseLinkValues), TRUE + ); + } } if (array_key_exists('civicrm_contact_contact_source', $row)) { @@ -1110,31 +1123,7 @@ public function sectionTotals() { * refactor in order to get this under ReportTemplateTests) */ protected function buildAssigneeFrom() { - $activityContacts = CRM_Activity_BAO_ActivityContact::buildOptions('record_type_id', 'validate'); - $assigneeID = CRM_Utils_Array::key('Activity Assignees', $activityContacts); - $this->_from = " - FROM civicrm_activity {$this->_aliases['civicrm_activity']} - INNER JOIN civicrm_activity_contact {$this->_aliases['civicrm_activity_contact']} - ON {$this->_aliases['civicrm_activity']}.id = {$this->_aliases['civicrm_activity_contact']}.activity_id AND - {$this->_aliases['civicrm_activity_contact']}.record_type_id = {$assigneeID} - INNER JOIN civicrm_contact civicrm_contact_assignee - ON {$this->_aliases['civicrm_activity_contact']}.contact_id = civicrm_contact_assignee.id - {$this->_aclFrom}"; - - if ($this->isTableSelected('civicrm_email')) { - $this->_from .= " - LEFT JOIN civicrm_email civicrm_email_assignee - ON {$this->_aliases['civicrm_activity_contact']}.contact_id = civicrm_email_assignee.contact_id AND - civicrm_email_assignee.is_primary = 1"; - } - if ($this->isTableSelected('civicrm_phone')) { - $this->_from .= " - LEFT JOIN civicrm_phone civicrm_phone_assignee - ON {$this->_aliases['civicrm_activity_contact']}.contact_id = civicrm_phone_assignee.contact_id AND - civicrm_phone_assignee.is_primary = 1 "; - } - $this->_aliases['civicrm_contact'] = 'civicrm_contact_assignee'; - $this->joinAddressFromContact(); + $this->buildFrom('assignee'); } /** @@ -1144,30 +1133,60 @@ protected function buildAssigneeFrom() { * refactor in order to get this under ReportTemplateTests) */ protected function buildSourceFrom() { + $this->buildFrom('source'); + } + + /** + * Shared function to build the from clause + * + * @param string $recordType (one of 'source', 'activity', 'target') + */ + protected function buildFrom($recordType) { $activityContacts = CRM_Activity_BAO_ActivityContact::buildOptions('record_type_id', 'validate'); - $sourceID = CRM_Utils_Array::key('Activity Source', $activityContacts); + switch ($recordType) { + case 'target': + $recordTypeID = CRM_Utils_Array::key('Activity Targets', $activityContacts); + break; + + case 'source': + $recordTypeID = CRM_Utils_Array::key('Activity Source', $activityContacts); + break; + + case 'assignee': + $recordTypeID = CRM_Utils_Array::key('Activity Assignees', $activityContacts); + break; + + } + $this->_from = " - FROM civicrm_activity {$this->_aliases['civicrm_activity']} - INNER JOIN civicrm_activity_contact {$this->_aliases['civicrm_activity_contact']} - ON {$this->_aliases['civicrm_activity']}.id = {$this->_aliases['civicrm_activity_contact']}.activity_id AND - {$this->_aliases['civicrm_activity_contact']}.record_type_id = {$sourceID} - INNER JOIN civicrm_contact civicrm_contact_source - ON {$this->_aliases['civicrm_activity_contact']}.contact_id = civicrm_contact_source.id - {$this->_aclFrom}"; + FROM civicrm_activity {$this->_aliases['civicrm_activity']} + INNER JOIN civicrm_activity_contact {$this->_aliases['civicrm_activity_contact']} + ON {$this->_aliases['civicrm_activity']}.id = {$this->_aliases['civicrm_activity_contact']}.activity_id AND + {$this->_aliases['civicrm_activity_contact']}.record_type_id = {$recordTypeID} + INNER JOIN civicrm_contact civicrm_contact_{$recordType} + ON {$this->_aliases['civicrm_activity_contact']}.contact_id = civicrm_contact_{$recordType}.id + {$this->_aclFrom}"; + if (!empty($this->_params['include_case_activities_value'])) { + $this->_from .= " + LEFT JOIN civicrm_case_activity {$this->_aliases['civicrm_case_activity']} + ON {$this->_aliases['civicrm_case_activity']}.activity_id = {$this->_aliases['civicrm_activity']}.id"; + } if ($this->isTableSelected('civicrm_email')) { $this->_from .= " - LEFT JOIN civicrm_email civicrm_email_source - ON {$this->_aliases['civicrm_activity_contact']}.contact_id = civicrm_email_source.contact_id AND - civicrm_email_source.is_primary = 1"; + LEFT JOIN civicrm_email civicrm_email_{$recordType} + ON {$this->_aliases['civicrm_activity_contact']}.contact_id = civicrm_email_{$recordType}.contact_id AND + civicrm_email_{$recordType}.is_primary = 1"; } + if ($this->isTableSelected('civicrm_phone')) { $this->_from .= " - LEFT JOIN civicrm_phone civicrm_phone_source - ON {$this->_aliases['civicrm_activity_contact']}.contact_id = civicrm_phone_source.contact_id AND - civicrm_phone_source.is_primary = 1 "; + LEFT JOIN civicrm_phone civicrm_phone_{$recordType} + ON {$this->_aliases['civicrm_activity_contact']}.contact_id = civicrm_phone_{$recordType}.contact_id AND + civicrm_phone_{$recordType}.is_primary = 1 "; } - $this->_aliases['civicrm_contact'] = 'civicrm_contact_source'; + $this->_aliases['civicrm_contact'] = "civicrm_contact_{$recordType}"; + $this->joinAddressFromContact(); } diff --git a/templates/CRM/Case/Form/ActivityView.tpl b/templates/CRM/Case/Form/ActivityView.tpl index b1ac29b9113..cf4c0e92907 100644 --- a/templates/CRM/Case/Form/ActivityView.tpl +++ b/templates/CRM/Case/Form/ActivityView.tpl @@ -25,6 +25,7 @@ *} {* View Case Activities *}
+
{include file="CRM/common/formButtons.tpl" location="top" linkButtons=$actionLinks}
{if $revs} {strip} @@ -77,7 +78,7 @@
{/if} {/if} -
- {crmButton p='civicrm/case' q="reset=1" class='cancel' icon='times'}{ts}Done{/ts}{/crmButton} -
+
{include file="CRM/common/formButtons.tpl" location="bottom" linkButtons=$actionLinks}
+ +{include file="CRM/Case/Form/ActivityToCase.tpl"} diff --git a/templates/CRM/common/formButtons.tpl b/templates/CRM/common/formButtons.tpl index a264de1eae5..0679f82332e 100644 --- a/templates/CRM/common/formButtons.tpl +++ b/templates/CRM/common/formButtons.tpl @@ -28,6 +28,23 @@ by button and name. crmBtnType grabs type keyword from button name (e.g. 'upload', 'next', 'back', 'cancel') so types of buttons can be styled differently via css. *} {crmRegion name='form-buttons'} +{if $linkButtons} + {foreach from=$linkButtons item=linkButton} + {if $linkButton.accesskey} + {capture assign=accessKey}accesskey="{$linkButton.accessKey}"{/capture} + {else}{assign var="accessKey" value=""} + {/if} + {if $linkButton.icon} + {capture assign=icon} {/capture} + {else}{assign var="icon" value=""} + {/if} + {if $linkButton.ref} + {capture assign=linkname}name="{$linkButton.ref}"{/capture} + {else}{capture assign=linkname}name="{$linkButton.name}"{/capture} + {/if} + {$icon}{$linkButton.title} + {/foreach} +{/if} {foreach from=$form.buttons item=button key=key name=btns} {if $key|substring:0:4 EQ '_qf_'} {if $location}