Update based on comments from @eileenmcnaughton

civicrm · Aug 28, 2017 · f9aa1e8 · f9aa1e8
1 parent 44f817d
commit f9aa1e8
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/CRM/Activity/BAO/Activity.php b/CRM/Activity/BAO/Activity.php
@@ -672,7 +672,7 @@ public static function getActivities($params, $getCount = FALSE) {
     $activities = array();
 
     // fetch all active activity types
-    $activityTypes = CRM_Activity_BAO_Activity::buildOptions('activity_type_id', 'get');
+    $activityTypes = CRM_Core_OptionGroup::values('activity_type');
 
     // Activity.Get API params
     $activityParams = array(
@@ -2030,9 +2030,6 @@ public static function getContactActivity($contactId) {
 
     $dao = CRM_Core_DAO::executeQuery($query);
 
-    $activityTypes = CRM_Activity_BAO_Activity::buildOptions('activity_type_id', 'get');
-    $activityStatuses = CRM_Activity_BAO_Activity::buildOptions('activity_status_id', 'get');
-
     while ($dao->fetch()) {
       $activities[$dao->activity_id]['id'] = $dao->activity_id;
       $activities[$dao->activity_id]['activity_type_id'] = $dao->activity_type_id;
@@ -2041,8 +2038,8 @@ public static function getContactActivity($contactId) {
       $activities[$dao->activity_id]['activity_date_time'] = $dao->activity_date_time;
       $activities[$dao->activity_id]['details'] = $dao->details;
       $activities[$dao->activity_id]['status_id'] = $dao->status_id;
-      $activities[$dao->activity_id]['activity_name'] = $activityTypes[$dao->activity_type_id];
-      $activities[$dao->activity_id]['status'] = $activityStatuses[$dao->status_id];
+      $activities[$dao->activity_id]['activity_name'] = CRM_Core_PseudoConstant::getLabel('CRM_Activity_BAO_Activity', 'activity_type_id', $dao->activity_type_id);
+      $activities[$dao->activity_id]['status'] = CRM_Core_PseudoConstant::getLabel('CRM_Activity_BAO_Activity', 'activity_status_id', $dao->status_id);
 
       // set to null if not set
       if (!isset($activities[$dao->activity_id]['source_contact_id'])) {