Summary: | ASTERISK-03941: [patch] Make jump to n+101 optional on Dial command | ||
Reporter: | Jeffrey C. Ollie (jcollie) | Labels: | |
Date Opened: | 2005-04-14 15:23:31 | Date Closed: | 2008-01-15 15:31:39.000-0600 |
Priority: | Major | Regression? | No |
Status: | Closed/Complete | Components: | Core/General |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) nojump-head.patch.txt ( 1) nojump-stable.patch.txt | |
Description: | Now that the Dial command will return the status of the Dial in ${DIALSTATUS} it's not necessary to jump to n+101 on Busy. In fact, with complex dial plans, having Dial jump to n+101 can actually add unneeded complexity. The attached patches add a 'n' (for nojump) option to the Dial command. For example, in the following snippet, control would go to priority 2 after the Dial command. Remove the 'n' option and the control would flow to priority 102. [special] exten => 1,1,Busy exten => 2,1,Dial(Local/1@special,,n) exten => 2,2,NoOp(${CONTEXT} ${EXTEN} ${PRIORITY} ${DIALSTATUS}) exten => 2,102,NoOp(${CONTEXT} ${EXTEN} ${PRIORITY} ${DIALSTATUS}) ****** ADDITIONAL INFORMATION ****** Disclaimer on file. | ||
Comments: | By: Brian West (bkw918) 2005-04-14 16:26:21 I agree with this patch. I also think we need something similar like EXITSTATUS... maybe just jumping to labels? /b By: Michael Jerris (mikej) 2005-04-14 17:57:43 why not a global dialplan setting to disable all n+101's, not just dial? Obviously we would need "EXITSTATUS" to go along with that. By: Mark Spencer (markster) 2005-04-15 00:38:59 Added to CVS, thanks! By: Russell Bryant (russell) 2005-04-15 02:10:53 not included in 1.0 since it is a new feature By: Digium Subversion (svnbot) 2008-01-15 15:31:39.000-0600 Repository: asterisk Revision: 5468 U trunk/apps/app_dial.c ------------------------------------------------------------------------ r5468 | markster | 2008-01-15 15:31:39 -0600 (Tue, 15 Jan 2008) | 2 lines Add 'n' option to dial for "nojump" (bug ASTERISK-3941) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=5468 |