Summary: | ASTERISK-14240: [patch] Fix sms support | ||
Reporter: | Artem Makhutov (artem) | Labels: | |
Date Opened: | 2009-05-31 10:19:53 | Date Closed: | 2009-07-15 13:25:12 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | 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 |