[Home]

Summary:ASTERISK-14240: [patch] Fix sms support
Reporter:Artem Makhutov (artem)Labels:
Date Opened:2009-05-31 10:19:53Date Closed:2009-07-15 13:25:12
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Addons/chan_mobile
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) sms.patch
Description:Currently SMS is completly broken. AT+CNMI=3,1,0,0,0 is not supported and should be changed to AT+CNMI=2,1,0,0,0.

This patch fixes the problem.
Comments:By: Matthew Nicholson (mnicholson) 2009-06-01 14:00:07

What phones has this been tested with?

By: Artem Makhutov (artem) 2009-06-02 04:07:56

Hello,

I could only fully test it using a Nokia 6810. This is the only phone I have that supports Text-Based SMS over Bluetooth.

All other phones I have (Sony Ericsson K800i, Nokia E51, Sony Ericsson K700i, Sony Ericsson K710i, Siemens) do support AT+CNMI=2,1,0,0,0 but not AT+CNMI=3,1,0,0,0.

Accoring to all AT-Command references the 1st parameter of the AT+CNMI command must me 2. The 3 is not defined anywhere. So I believe that this was a typo in the source.

Regards, Artem

By: Digium Subversion (svnbot) 2009-06-03 18:49:05

Repository: asterisk-addons
Revision: 943

U   trunk/channels/chan_mobile.c

------------------------------------------------------------------------
r943 | mnicholson | 2009-06-03 18:49:05 -0500 (Wed, 03 Jun 2009) | 8 lines

Fix sms support by setting the correct mode for receiving messages.

(closes issue ASTERISK-14240)
Reported by: Artem
Patches:
     sms.patch uploaded by Artem (license 800)
Tested by: Artem

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk-addons?view=rev&revision=943

By: Digium Subversion (svnbot) 2009-06-03 18:52:07

Repository: asterisk-addons
Revision: 944

_U  branches/1.6.2/
U   branches/1.6.2/channels/chan_mobile.c

------------------------------------------------------------------------
r944 | mnicholson | 2009-06-03 18:52:06 -0500 (Wed, 03 Jun 2009) | 15 lines

Merged revisions 943 via svnmerge from
https://origsvn.digium.com/svn/asterisk-addons/trunk

........
 r943 | mnicholson | 2009-06-03 18:49:04 -0500 (Wed, 03 Jun 2009) | 8 lines
 
 Fix sms support by setting the correct mode for receiving messages.
 
 (closes issue ASTERISK-14240)
 Reported by: Artem
 Patches:
       sms.patch uploaded by Artem (license 800)
 Tested by: Artem
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk-addons?view=rev&revision=944