Fix #62

README.md

@@ -30,7 +30,7 @@ a generic, expandable connector, with new providers added as contributed by the
 
 ### Installation
 
-* `fwconsole ma downloadinstall https://github.com/simontelephonics/smsconnector/releases/download/v16.0.17.1/smsconnector-16.0.17.1.tar.gz`
+* `fwconsole ma downloadinstall https://github.com/simontelephonics/smsconnector/releases/download/v16.0.17.2/smsconnector-16.0.17.2.tar.gz`
 * `fwconsole r`
 
 ### Configuration

adaptor/Smsconnector.class.php

@@ -13,6 +13,8 @@ class Smsconnector extends \FreePBX\modules\Sms\AdaptorBase {
 
     public function sendMedia($to,$from,$cnam,$message=null,$files=array(),$time=null,$adaptor=null,$emid=null,$chatId='')
     {
+        $message ??= ''; // string is expected, so don't allow a null
+
         // Store in database
         $retval = array();
         try 
@@ -41,6 +43,8 @@ public function sendMedia($to,$from,$cnam,$message=null,$files=array(),$time=nul
 
     public function sendMessage($to,$from,$cnam,$message,$time=null,$adaptor=null,$emid=null,$chatId='')
     {
+        $message ??= ''; // string is expected, so don't allow a null
+
         // Clean up 'to' number if we got one with a +. Only seems to apply to mobile app. UCP
         // and desktop apps format number before invoking this module.
         $to = ltrim($to, '+');

module.xml

@@ -2,7 +2,7 @@
 	<rawname>smsconnector</rawname>
 	<repo>unsupported</repo>
 	<name>SMS Connector</name>
-	<version>16.0.17.1</version>
+	<version>16.0.17.2</version>
 	<category>Connectivity</category>
 	<Publisher>Simon Telephonics LLC</Publisher>
 	<info>http://wiki.freepbx.org/</info>
@@ -45,6 +45,7 @@
 		</table>
 	</database>
 	<changelog>
+		*16.0.17.2* Bug fix: #62 set message to empty string if null
 		*16.0.17.1* Bug fix: replace any incoming null/unset message bodies with empty string
 		*16.0.17* Adds provider Bandwidth.com
 		*16.0.16.1* Update i18n, code clean-up, number formatting enhancements