<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/3816/">https://reviewboard.asterisk.org/r/3816/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers.</div>
<div>By Mark Michelson.</div>
<p style="color: grey;"><i>Updated July 17, 2014, 6:01 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">This fixes some bugs found for DTMF-initiated attended transfers:
1) Fixes a bridge reference leak
2) Gathers transfer parties before performing bridge merges on success and threeway cases.
3) Gathers parties for cases where the transferer is not in any bridge.
This also fixes a copy-paste error in stasis_bridges where the wrong channel was having its snapshot grabbed for a publication.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
Asterisk
</div>
<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;">Transfer publications in Stasis have, in the past given transferer channels and the bridges they belong to. The theory behind this is that if given that information, and you have kept track of who all has entered and left the associated bridges, you can derive who the transferee channels are and who the transfer target channels are. In practice, however, it's more useful to present this information directly in the publication.
This changeset adds transferee channel snapshots to blind and attended transfer publications and a transfer target channel snapshot to the blind transfer publication. We only provide such snapshots if there is a single transferee or transfer target. If transferring multiple parties or transferring to multiple parties, these snapshots are omitted.
One thing to check for is whether consumers of the transfer events have been properly updated to make use of the new information being added in this review (I'm looking at you, ARI!).</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;">Testing on this is pretty paltry at the moment. All I have done is to perform manual tests of nominal cases. I can confirm that AMI now prints transferee channel information for both blind and attended transfers. I can also confirm that AMI prints transfer target information for attended transfers.
Due to the looming deadline of the feature freeze, plus the stipulation that the code only needs to be up for review to be a candidate for inclusion, I have posted this without any testsuite tests. This will not be committed without either modifying existing tests or adding new tests, though.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/trunk/res/stasis/app.c <span style="color: grey">(418633)</span></li>
<li>/trunk/main/stasis_bridges.c <span style="color: grey">(418633)</span></li>
<li>/trunk/main/cel.c <span style="color: grey">(418633)</span></li>
<li>/trunk/main/bridge_basic.c <span style="color: grey">(418633)</span></li>
<li>/trunk/main/bridge.c <span style="color: grey">(418633)</span></li>
<li>/trunk/include/asterisk/stasis_bridges.h <span style="color: grey">(418633)</span></li>
<li>/trunk/apps/app_queue.c <span style="color: grey">(418633)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/3816/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>