wordpress 4.9.6 Clickatell 4.1 All was working fine until upgrade to Civi 5.22 and upgrade to SMS API 1.5 (yup in retrospect should have tested separately!)
phone is home-mobile-primary
CiviCRM Stack Exchange is a question and answer site for administrators and users of the CiviCRM Constituent Relationship Management software. It only takes a minute to sign up.
Sign up to join this communitywordpress 4.9.6 Clickatell 4.1 All was working fine until upgrade to Civi 5.22 and upgrade to SMS API 1.5 (yup in retrospect should have tested separately!)
phone is home-mobile-primary
Looks like sites/all/modules/civicrm/CRM/Activity/BAO/Activity.php
is failing in function function sendSMSMessage(
because $doNotSms = TRUE;
on line 3154. Set it to false and you should be Ok. However, the check will not happen, so all numbers (even Do Not SMS #s) will get the SMS.
The other option is to comment out the following block and add an if block:
/*
if ($smsProviderParams['To']) {
// If phone number is specified use it
$toPhoneNumber = trim($smsProviderParams['To']);
Civi::log()->debug(print_r($smsProviderParams, true)."\ntoID: $toID\n");
//echo "<pre>".print_r($smsProviderParams, true)."</pre>";
$doNotSms = FALSE;
}
elseif ($toID) {
*/
if ($toID) {
// No phone number specified, so find a suitable one for the contact
Have raised this as an issue with the extension maintainers:
https://github.com/veda-consulting/org.civicrm.sms.clickatell/issues/20