<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/2656/">https://reviewboard.asterisk.org/r/2656/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#e0e0e0" width="100%" cellpadding="8" style="border: 1px gray solid;">
<tr>
<td>
<h1 style="margin-right: 0.2em; padding: 0; font-size: 10pt;">This change has been marked as submitted.</h1>
</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.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers, Matt Jordan and rmudgett.</div>
<div>By jrose.</div>
<p style="color: grey;"><i>Updated July 8, 2013, 6:12 p.m.</i></p>
<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-21877">ASTERISK-21877</a>
</div>
<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;">If ringing was set for the channel in parking and that channel gets swapped out for another channel (in the case of an attended transfer to Park with the 'r' option for example), then that ringing previously wasn't honored and the channel would just play music on hold as normal. This patch makes the swap event check if the ringing option was applied to the swapping channel's holding_participant role so that it can be applied with the rest of the options.
Also, error checking for bridge role application has been added.
Also noteworthy, I noticed that announcements would conflict with the ringing option and cause ringing to stop without being resumed. This is a bridging problem more than a parking problem and it has been noted in ASTERISK-22006 along with a proposed fix that I worked out with Richard.</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;">Attended transfer to Park(,r)
Finalized the transfer after hearing the announcement
Parkee hears ringing afterwards.
I also probed at the bridge role options to make sure they were being applied.
Testing also revealed the issue with ast_bridge_channel_playfile and the ringing indication. This can cause ringing not to play if the call is transferred before the announcement is finished.</pre>
</td>
</tr>
</table>
<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/bridging.c <span style="color: grey">(393530)</span></li>
<li>/trunk/res/parking/parking_bridge.c <span style="color: grey">(393530)</span></li>
<li>/trunk/res/parking/parking_controller.c <span style="color: grey">(393530)</span></li>
<li>/trunk/res/parking/res_parking.h <span style="color: grey">(393530)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/2656/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>