[asterisk-bugs] [Asterisk 0015236]: [patch] Fix sms support

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15236 
====================================================================== 
Reported By:                Artem
Assigned To:                mnicholson
====================================================================== 
Project:                    Asterisk
Issue ID:                   15236
Category:                   Addons/chan_mobile
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     feedback
Asterisk Version:           1.6.1.0 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2009-05-31 10:19 CDT
Last Modified:              2009-06-02 04:07 CDT
====================================================================== 
Summary:                    [patch] Fix sms support
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.
====================================================================== 

---------------------------------------------------------------------- 
 (0105876) Artem (reporter) - 2009-06-02 04:07
 https://issues.asterisk.org/view.php?id=15236#c105876 
---------------------------------------------------------------------- 
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 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-06-02 04:07 Artem          Note Added: 0105876                          
======================================================================