Merge pull request #329 from ThomWilhelm/CRM-12289

Only send SMS if it is a mobile number, do not delete parent activity as...

CRM/Activity/BAO/Activity.php

@@ -1515,7 +1515,12 @@ LEFT JOIN   civicrm_case_activity ON ( civicrm_case_activity.activity_id = tbl.a
       $tokenHtml = CRM_Utils_Token::replaceContactTokens($html, $values, TRUE, $messageToken, FALSE, $escapeSmarty);
       $tokenHtml = CRM_Utils_Token::replaceHookTokens($tokenHtml, $values, $categories, TRUE, $escapeSmarty);
 
-      $smsParams['To'] = $values['phone'];
+      // Only send if the phone is of type mobile
+      if($values['phone_type_id'] == 2) {
+        $smsParams['To'] = $values['phone'];
+      }else{
+        $smsParams['To'] = '';
+      }
 
       if (self::sendSMSMessage($contactId,
           $tokenText,

tools/extensions/org.civicrm.sms.clickatell/org_civicrm_sms_clickatell.php

@@ -284,12 +284,10 @@ class org_civicrm_sms_clickatell extends CRM_SMS_Provider {
         return $send[1];
       }
       else {
-        // delete any parent activity & throw error
-        if (CRM_Utils_Array::value('parent_activity_id', $header)) {
-          $params = array('id' => $header['parent_activity_id']);
-          CRM_Activity_BAO_Activity::deleteActivity($params);
-        }
-        return PEAR::raiseError($response['data']);
+        // TODO: Should add a failed activity instead.
+      	
+        CRM_Core_Error::debug_log_message($response['data']);
+        return;
       }
     }
   }