From a59cecb1a1ca241d109336fbca2ae52317cc65ab Mon Sep 17 00:00:00 2001
From: eileen <emcnaughton@wikimedia.org>
Date: Thu, 6 Apr 2017 11:05:48 +1200
Subject: [PATCH] Add deprecation comments, CRM-19987

---
 CRM/Activity/BAO/Activity.php                   | 3 +++
 CRM/Contact/Form/Task/SMSCommon.php             | 1 +
 tests/phpunit/CRM/Activity/BAO/ActivityTest.php | 1 +
 3 files changed, 5 insertions(+)

diff --git a/CRM/Activity/BAO/Activity.php b/CRM/Activity/BAO/Activity.php
index 05e5d09de9b7..52d192476f73 100644
--- a/CRM/Activity/BAO/Activity.php
+++ b/CRM/Activity/BAO/Activity.php
@@ -1705,6 +1705,8 @@ public static function &importableFields($status = FALSE) {
   }
 
   /**
+   * @deprecated - use the api instead.
+   *
    * Get the Activities of a target contact.
    *
    * @param int $contactId
@@ -1714,6 +1716,7 @@ public static function &importableFields($status = FALSE) {
    *   array of activity fields
    */
   public static function getContactActivity($contactId) {
+    // @todo remove this function entirely.
     $activities = array();
     $activityContacts = CRM_Core_OptionGroup::values('activity_contacts', FALSE, FALSE, FALSE, NULL, 'name');
     $sourceID = CRM_Utils_Array::key('Activity Source', $activityContacts);
diff --git a/CRM/Contact/Form/Task/SMSCommon.php b/CRM/Contact/Form/Task/SMSCommon.php
index 6ab8bc815b37..a4ab022d269c 100644
--- a/CRM/Contact/Form/Task/SMSCommon.php
+++ b/CRM/Contact/Form/Task/SMSCommon.php
@@ -198,6 +198,7 @@ public static function buildQuickForm(&$form) {
 
         if (CRM_Utils_System::getClassName($form) == 'CRM_Activity_Form_Task_SMS') {
           //to check for "if the contact id belongs to a specified activity type"
+          // @todo use the api instead - function is deprecated.
           $actDetails = CRM_Activity_BAO_Activity::getContactActivity($contactId);
           if (self::RECIEVED_SMS_ACTIVITY_SUBJECT !=
             CRM_Utils_Array::retrieveValueRecursive($actDetails, 'subject')
diff --git a/tests/phpunit/CRM/Activity/BAO/ActivityTest.php b/tests/phpunit/CRM/Activity/BAO/ActivityTest.php
index 006307aa8092..06cd104a3efd 100644
--- a/tests/phpunit/CRM/Activity/BAO/ActivityTest.php
+++ b/tests/phpunit/CRM/Activity/BAO/ActivityTest.php
@@ -81,6 +81,7 @@ public function testGetContactActivity() {
       'subject', 'Database check for created activity.'
     );
 
+    // @todo - remove this deprecated functions
     $activities = CRM_Activity_BAO_Activity::getContactActivity($targetContactId);
 
     $this->assertEquals($activities[$activityId]['subject'], 'Scheduling Meeting', 'Verify activity subject is correct.');