[Home]

Summary:ASTERISK-04686: [patch] Fix SENDTEXTSTATUS for app_sendtext
Reporter:Olle Johansson (oej)Labels:
Date Opened:2005-07-26 14:14:14Date Closed:2008-01-15 15:44:49.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_sendtext
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) sendtext.txt
( 1) sendtext2.txt
Description:This patch:
* Adds SENDTEXTSTATUS status output
* Removes the ability for sendtext to return -1 (hangup). Do we *really* want sendtext to be able to hangup a call?
If accepted, the later part needs to be added to UPGRADE.txt
Comments:By: Olle Johansson (oej) 2005-07-26 14:15:24

Another alternative would be to add an option for "DO NOT HANG UP FOR ANYTHING"
By: Olle Johansson (oej) 2005-07-26 14:39:54

Kevin just made this patch not up to date...
By: Olle Johansson (oej) 2005-07-26 15:00:48

updated with option_priority_jumping
By: Kevin P. Fleming (kpfleming) 2005-07-26 18:10:06

Please use ast_goto_if_exists(), it has a specific advantage over open-coding the the match/jump.

I don't think we should drop the ability to make the app 'jump'... are you willing to add a 'j' option to enable the old behavior?

(p.s. 'deprecated' not 'depreciated')
By: Olle Johansson (oej) 2005-07-27 03:01:32

I am not removing the jump at all, but removing the hangup.
By: Kevin P. Fleming (kpfleming) 2005-08-22 17:54:08

Committed to CVS HEAD with minor mods, thanks!
By: Digium Subversion (svnbot) 2008-01-15 15:44:49.000-0600

Repository: asterisk
Revision: 6359

U   trunk/apps/app_sendtext.c

------------------------------------------------------------------------
r6359 | kpfleming | 2008-01-15 15:44:48 -0600 (Tue, 15 Jan 2008) | 2 lines

add text status variable output to SendText() application (issue ASTERISK-4686)

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

http://svn.digium.com/view/asterisk?view=rev&revision=6359