<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/1825/">https://reviewboard.asterisk.org/r/1825/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<p>- Mark</p>
<br />
<p>On March 22nd, 2012, 2:26 p.m., jrose wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers, Mark Michelson and Matt Jordan.</div>
<div>By jrose.</div>
<p style="color: grey;"><i>Updated March 22, 2012, 2:26 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Similar to the F option added to queue, this adds a new option to the bridge application to transfer the party that gets bridged upon (awkwardly worded maybe) to a position in the dialplan specified by the bridge F option from the perspective of the channel that bridged.
Also moves a shared function from a couple of places for app_dial and app_queue into utils.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">A sip phone would dial the following extension:
exten => 045,1,NoOp(BindBape)
exten => 045,2,Answer()
exten => 045,3,Set(GLOBAL(CHANBAPE)=${CHANNEL})
exten => 045,4,Playback(tt-weasels)
exten => 045,5,Wait(30)
exten => 045,6,HangUp()
>From here, a second phone would dial any of these extensions:
exten => 046,1,NoOp(BrideBape)
exten => 046,2,Answer()
exten => 046,3,Bridge(${CHANBAPE}, F(5))
exten => 046,4,Playback(tt-monkeys)
exten => 046,5,NoOp(Finish!)
exten => 046,6,NoOp(Finish More!)
exten => 047,1,NoOp(BridgeBape2)
exten => 047,2,Answer()
exten => 047,3,Bridge(${CHANBAPE}, F)
exten => 047,4,Playback(tt-allbusy)
exten => 047,5,NoOp(BridgeBape2 finish)
exten => 048,1,NoOp(BridgeBape3)
exten => 048,2,Answer()
exten => 048,3,Bridge(${CHANBAPE}, F(049,4))
exten => 048,4,NoOp(Failure)
exten => 050,1,NoOp(BrigeBape4)
exten => 050,2,Answer()
exten => 050,3,Bridge(${CHANBAPE}, F(bridgetestcontext,20,1))
Also this stuff:
exten => 049,1,NoOp(Failure)
exten => 049,2,NoOp(Failure)
exten => 049,3,NoOp(Failure)
exten => 049,4,Playback(tt-weasels)
exten => 049,5,NoOp(BridgeBape3 Success)
[bridgetestcontext]
exten => 20,1,NoOp(Successful bape4)
exten => 20,n,Playback(tt-somethingwrong)
For each of these cases, I just checked to make sure that hanging up the bridger resulted in the bridged party getting sent to the expected position.
There will be automated tests added for bridging in general as well as for this feature into testsuite soon as well.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://issues.asterisk.org/jira/browse/ASTERISK-19282">ASTERISK-19282</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/trunk/main/utils.c <span style="color: grey">(360188)</span></li>
<li>/trunk/apps/app_queue.c <span style="color: grey">(360188)</span></li>
<li>/trunk/include/asterisk/utils.h <span style="color: grey">(360188)</span></li>
<li>/trunk/main/features.c <span style="color: grey">(360188)</span></li>
<li>/trunk/apps/app_dial.c <span style="color: grey">(360188)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/1825/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>