}
$transaction = new CRM_Core_Transaction();
- if (is_array(CRM_Utils_Array::value('source_record_id', $params))) {
+ $sqlWhereParams = $where = [];
+ if (isset($params['source_record_id']) && is_array($params['source_record_id'])) {
$sourceRecordIds = implode(',', $params['source_record_id']);
}
else {
- $sourceRecordIds = CRM_Utils_Array::value('source_record_id', $params);
+ $sourceRecordIds = $params['source_record_id'] ?? NULL;
}
+ if ($sourceRecordIds) {
+ $where[] = 'source_record_id IN ( %1 )';
+ $sqlWhereParams[1] = [$sourceRecordIds, 'CommaSeparatedIntegers'];
+ }
$result = NULL;
if (!$moveToTrash) {
if (!isset($params['id'])) {
- if (is_array($params['activity_type_id'])) {
- $activityTypes = implode(',', $params['activity_type_id']);
- }
- else {
- $activityTypes = $params['activity_type_id'];
+ if (!empty($params['activity_type_id'])) {
+ $where[] = 'activity_type_id IN ( %2 )';
+ $sqlWhereParams[2] = [implode(',', (array) $params['activity_type_id']), 'CommaSeparatedIntegers'];
}
-
- $query = "DELETE FROM civicrm_activity WHERE source_record_id IN ({$sourceRecordIds}) AND activity_type_id IN ( {$activityTypes} )";
- $dao = CRM_Core_DAO::executeQuery($query);
+ $query = "DELETE FROM civicrm_activity WHERE " . implode(' AND ', $where);
+ $dao = CRM_Core_DAO::executeQuery($query, $sqlWhereParams);
}
else {
$activity = new CRM_Activity_DAO_Activity();
$activity->case_id = CRM_Case_BAO_Case::getCaseIdByActivityId($activity->id);
// CRM-13994 delete activity entity_tag
- $query = "DELETE FROM civicrm_entity_tag WHERE entity_table = 'civicrm_activity' AND entity_id = {$activity->id}";
- $dao = CRM_Core_DAO::executeQuery($query);
+ $query = "DELETE FROM civicrm_entity_tag WHERE entity_table = 'civicrm_activity' AND entity_id = %1";
+ $dao = CRM_Core_DAO::executeQuery($query, [1 => [$activity->id, 'Positive']]);
}
}
else {
}
else {
// at worst, take source for recently viewed display
- $recentContactId = CRM_Utils_Array::value('source_contact_id', $params);
+ $recentContactId = $params['source_contact_id'] ?? NULL;
}
if (isset($params['assignee_contact_id'])) {
// if the subject contains a ‘[case #…]’ string, file that activity on the related case (CRM-5916)
$matches = [];
- $subjectToMatch = CRM_Utils_Array::value('subject', $params);
+ $subjectToMatch = $params['subject'] ?? NULL;
if (preg_match('/\[case #([0-9a-h]{7})\]/', $subjectToMatch, $matches)) {
$key = CRM_Core_DAO::escapeString(CIVICRM_SITE_KEY);
$hash = $matches[1];
foreach ($result as $id => $activity) {
$activities[$id] = [
'activity_id' => $activity['id'],
- 'activity_date_time' => CRM_Utils_Array::value('activity_date_time', $activity),
- 'subject' => CRM_Utils_Array::value('subject', $activity),
- 'assignee_contact_name' => CRM_Utils_Array::value('assignee_contact_sort_name', $activity, []),
- 'source_contact_id' => CRM_Utils_Array::value('source_contact_id', $activity),
- 'source_contact_name' => CRM_Utils_Array::value('source_contact_sort_name', $activity),
+ 'activity_date_time' => $activity['activity_date_time'] ?? NULL,
+ 'subject' => $activity['subject'] ?? NULL,
+ 'assignee_contact_name' => $activity['assignee_contact_sort_name'] ?? [],
+ 'source_contact_id' => $activity['source_contact_id'] ?? NULL,
+ 'source_contact_name' => $activity['source_contact_sort_name'] ?? NULL,
];
$activities[$id]['activity_type_name'] = CRM_Core_PseudoConstant::getName('CRM_Activity_BAO_Activity', 'activity_type_id', $activity['activity_type_id']);
$activities[$id]['activity_type'] = CRM_Core_PseudoConstant::getLabel('CRM_Activity_BAO_Activity', 'activity_type_id', $activity['activity_type_id']);
- $activities[$id]['target_contact_count'] = CRM_Utils_Array::value('target_contact_count', $activity, 0);
+ $activities[$id]['target_contact_count'] = $activity['target_contact_count'] ?? 0;
if (!empty($activity['target_contact_count'])) {
$displayedTarget = civicrm_api3('ActivityContact', 'get', [
'activity_id' => $id,
}
// case related fields
elseif ($apiKey == 'case_id' && !$isBulkActivity) {
- $activities[$id][$expectedName] = CRM_Utils_Array::value($apiKey, $activity);
+ $activities[$id][$expectedName] = $activity[$apiKey] ?? NULL;
// fetch case subject for case ID found
if (!empty($activity['case_id'])) {
}
else {
// @todo this generic assign could just be handled in array declaration earlier.
- $activities[$id][$expectedName] = CRM_Utils_Array::value($apiKey, $activity);
+ $activities[$id][$expectedName] = $activity[$apiKey] ?? NULL;
if ($apiKey == 'campaign_id') {
- $activities[$id]['campaign'] = CRM_Utils_Array::value($activities[$id][$expectedName], $allCampaigns);
+ $activities[$id]['campaign'] = $allCampaigns[$activities[$id][$expectedName]] ?? NULL;
}
}
}
$smsProviderParams['To'] = '';
}
- $doNotSms = CRM_Utils_Array::value('do_not_sms', $contact, 0);
+ $doNotSms = $contact['do_not_sms'] ?? 0;
if ($doNotSms) {
$errMsgs[] = PEAR::raiseError('Contact Does not accept SMS', NULL, PEAR_ERROR_RETURN);
// To get primary mobile phonenumber, if not get the first mobile phonenumber
if (!empty($toPhoneNumbers)) {
$toPhoneNumberDetails = reset($toPhoneNumbers);
- $toPhoneNumber = CRM_Utils_Array::value('phone', $toPhoneNumberDetails);
+ $toPhoneNumber = $toPhoneNumberDetails['phone'] ?? NULL;
// Contact allows to send sms
}
}
*
* @param $activityID
* Current activity id.
- *
+ * @deprecated
* @return int
* $params count of prior activities otherwise false.
* @throws \CRM_Core_Exception
$followupParams['activity_type_id'] = $params['followup_activity_type_id'];
// Get Subject of Follow-up Activiity, CRM-4491
- $followupParams['subject'] = CRM_Utils_Array::value('followup_activity_subject', $params);
- $followupParams['assignee_contact_id'] = CRM_Utils_Array::value('followup_assignee_contact_id', $params);
+ $followupParams['subject'] = $params['followup_activity_subject'] ?? NULL;
+ $followupParams['assignee_contact_id'] = $params['followup_assignee_contact_id'] ?? NULL;
// Create target contact for followup.
if (!empty($params['target_contact_id'])) {
'is_deleted' => 0,
'is_current_revision' => 1,
'is_test' => 0,
- 'contact_id' => CRM_Utils_Array::value('contact_id', $params),
- 'activity_date_time' => CRM_Utils_Array::value('activity_date_time', $params),
+ 'contact_id' => $params['contact_id'] ?? NULL,
+ 'activity_date_time' => $params['activity_date_time'] ?? NULL,
'check_permissions' => 1,
'options' => [
- 'offset' => CRM_Utils_Array::value('offset', $params, 0),
+ 'offset' => $params['offset'] ?? 0,
],
];
// Format the params.
$params['offset'] = ($params['page'] - 1) * $params['rp'];
$params['rowCount'] = $params['rp'];
- $params['sort'] = CRM_Utils_Array::value('sortBy', $params);
+ $params['sort'] = $params['sortBy'] ?? NULL;
$params['caseId'] = NULL;
- $context = CRM_Utils_Array::value('context', $params);
+ $context = $params['context'] ?? NULL;
$showContactOverlay = !CRM_Utils_String::startsWith($context, "dashlet");
$activityTypeInfo = civicrm_api3('OptionValue', 'get', [
'option_group_id' => "activity_type",
*/
public static function sendToAssignee($activity, $mailToContacts, $params = []) {
if (!CRM_Utils_Array::crmIsEmptyArray($mailToContacts)) {
- $clientID = CRM_Utils_Array::value('client_id', $params);
- $caseID = CRM_Utils_Array::value('case_id', $params);
+ $clientID = $params['client_id'] ?? NULL;
+ $caseID = $params['case_id'] ?? NULL;
$ics = new CRM_Activity_BAO_ICalendar($activity);
$attachments = CRM_Core_BAO_File::getEntityFile('civicrm_activity', $activity->id);