Review Board 1.7.16


New Application: Originate

Review Request #95 - Created Dec. 17, 2008 and submitted

Russell Bryant
/trunk
14075
Reviewers
asterisk-dev
Asterisk
This patch introduces a new application, Originate.  I made a lot of changes to the original patch, so I figured it would be worth getting another set of eyes on the code before commit.
Tested calls to 589 through 595 and got expected results.

exten => s,1,Answer
exten => s,n,MusicOnHold()

exten => 589,1,Answer
exten => 589,n,Originate(SIP/5004,exten,default,586,1)
exten => 589,n,Playback(demo-congrats)

exten => 590,1,Answer
exten => 590,n,Originate(SIP/5004,exten,default,586)
exten => 590,n,Playback(demo-congrats)

exten => 591,1,Answer
exten => 591,n,Originate(SIP/5004,exten,default)
exten => 591,n,Playback(demo-congrats)

exten => 592,1,Answer
exten => 592,n,Originate(SIP/5004,exten)
exten => 592,n,Playback(demo-congrats)

exten => 593,1,Answer
exten => 593,n,Originate(SIP/5004,app,Playback,demo-congrats)
exten => 593,n,Playback(demo-congrats)

exten => 594,1,Answer
exten => 594,n,Originate(SIP/5004,app,Echo)
exten => 594,n,Playback(demo-congrats)

exten => 595,1,Answer
exten => 595,n,Originate(SIP/5004,app)
exten => 595,n,Playback(demo-congrats)
Review request changed
Updated (Dec. 17, 2008, 9:51 a.m.)
Updated diff to add ORIGINATE_STATUS result variable, per mmichelson's suggestion.
Ship it!
Posted (Dec. 17, 2008, 10:13 a.m.)
Looks good to me. Just fix the whitespace problems that reviewboard highlights in red first.

https://reviewboard.asterisk.org/ runs on a server provided by Digium, Inc. and uses bandwidth donated to the open source Asterisk community by API Digital Communications in Huntsville, AL USA.
Please report problems with this site to asteriskteam@digium.com.