<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7616">View Change</a></p><p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p>(3 comments)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/7616/2/main/features.c">File main/features.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7616/2/main/features.c@791">Patch Set #2, Line 791:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;"> if (ast_test_flag(ast_channel_flags(chana), AST_FLAG_MOH)) {<br>          ast_moh_stop(chana);<br>  }<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Undo this and see the third comment.</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7616/2/main/features.c@810">Patch Set #2, Line 810:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">       if (ast_test_flag(ast_channel_flags(chanb), AST_FLAG_MOH)) {<br>          ast_moh_stop(chanb);<br>  }<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Undo this and see the third comment.</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7616/2/main/features.c@821">Patch Set #2, Line 821:</a> <code style="font-family:monospace,monospace">    if (ast_bridge_add_channel(bridge, chana, NULL, playtone & PLAYTONE_CHANNEL1, xfer_cfg_a ? xfer_cfg_a->xfersound : NULL)) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">The main/bridge.c:ast_bridge_add_channel() function should unconditionally call ast_moh_stop() just like the main/features.c:do_bridge_masquerade() function did in v11.  This will not only fix the MOH problem with the AMI Bridge action but also fix the same problem with the Bridge dialplan application and make use the right MOH class for the AMI Park action when the channel gets parked.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/7616">change 7616</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/7616"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I4b129c5a20c15e63968842460ac5a1a85903cf9f </div>
<div style="display:none"> Gerrit-Change-Number: 7616 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Ivan Poddubny <ivan.poddubny@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 18 Dec 2017 20:39:01 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>