<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/1193/">https://reviewboard.asterisk.org/r/1193/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On December 2nd, 2011, 1:03 p.m., <b>jrose</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I think this is ready as-is. It's trunk only since these are new AMI actions, so go ahead and document it in CHANGES as well.</pre>
</blockquote>
<p>On December 5th, 2011, 9:19 a.m., <b>jrose</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">By the way, if there isn't any feedback on this within a week or so, I'll go ahead and commit it myself.</pre>
</blockquote>
<p>On December 5th, 2011, 10:38 a.m., <b>Olle E Johansson</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">We really have tried to avoid separate start/stop actions for a long time. I really would like this to follow the syntax we have followed for some releases.</pre>
</blockquote>
<p>On December 5th, 2011, 1:16 p.m., <b>wdoekes</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">While I do prefer to not pollute the Action namespace -- I too like a separate Operation parameter -- I find no evidence of this being standard:
The only "Operation" parameter I can find is:
- Filter -> Operation (Add)
Having separate actions is far more common:
- Confbridge*
- DAHDIDND*
- Meetme*
- Qeueue*
- *Monitor</pre>
</blockquote>
<p>On December 5th, 2011, 1:30 p.m., <b>Olle E Johansson</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Most of these are old, except ConfBridge - I must have missed that review. We haven't used "Operation" as much as "status: on/off" and others. There are several examples of that. We had an agreement a long time ago that this was the recommended architecture for new manager actions.</pre>
</blockquote>
<p>On December 5th, 2011, 1:48 p.m., <b>jrose</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Filter only seems to have one operation as well. If this is meant to be the current standard, I can't really see any examples of it having been followed.
Since this is only going into trunk, I don't really see any problem with committing the patch though. There is still plenty of time before Asterisk11 even starts being talked about for release purposes, so naming issues like this are malleable for the time being, and the patch to change it to work like you are describing would be fairly trivial.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The StopMixMonitor and MixMonitor AMI actions are consistent with the Dialplan Application's they mimic. If StopMixMonitor ever gets any arguments passed in, then we have a giant mess with the AMI actions if we use the same action for both starting and stopping.
With that said. I am backing the use of the StopMixMonitor and MixMonitor AMI actions.
</pre>
<br />
<p>- David</p>
<br />
<p>On April 21st, 2011, 9:29 a.m., telecos82 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.</div>
<div>By telecos82.</div>
<p style="color: grey;"><i>Updated April 21, 2011, 9:29 a.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;">Add MixMonitor and StopMixMonitor AMI actions to allow starting and stopping MixMonitor from AMI.</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;">Tested with SIP, IAX and DAHDI channels.</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/19155">19155</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>http://svn.asterisk.org/svn/asterisk/branches/1.8/apps/app_mixmonitor.c <span style="color: grey">(314406)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/1193/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>