<div dir="ltr"><div class="gmail_default" style="color:rgb(102,0,0)">These changes have been completed and tested (minus recording which already has a user supplied id in the filename).  Here are the reviewboard posts:</div>
<div class="gmail_default" style="color:rgb(102,0,0)"><br></div><div class="gmail_default" style="color:rgb(102,0,0)">Main code (large, sorry):  <a href="https://reviewboard.asterisk.org/r/3191/">https://reviewboard.asterisk.org/r/3191/</a></div>
<div class="gmail_default" style="color:rgb(102,0,0)"><br></div><div class="gmail_default" style="color:rgb(102,0,0)">AMI originate test: <a href="https://reviewboard.asterisk.org/r/3243/">https://reviewboard.asterisk.org/r/3243/</a></div>
<div class="gmail_default" style="color:rgb(102,0,0)"><br></div><div class="gmail_default" style="color:rgb(102,0,0)">Starpy change required for AMI originate test: <a href="https://reviewboard.asterisk.org/r/3242/">https://reviewboard.asterisk.org/r/3242/</a></div>
<div class="gmail_default" style="color:rgb(102,0,0)"><br></div><div class="gmail_default" style="color:rgb(102,0,0)">ARI tests (originate, bridge, playback, snoop): <a href="https://reviewboard.asterisk.org/r/3277/">https://reviewboard.asterisk.org/r/3277/</a> (includes upgrades to ari.py to provide full test-config.yaml based ARI testing)</div>
<div class="gmail_default" style="color:rgb(102,0,0)"><br></div><div class="gmail_default" style="color:rgb(102,0,0)">Incidental testsuite improvements discovered along the way:</div><div class="gmail_default" style="color:rgb(102,0,0)">
<br></div><div class="gmail_default" style="color:rgb(102,0,0)">Better yaml error reporting: <a href="https://reviewboard.asterisk.org/r/3287/">https://reviewboard.asterisk.org/r/3287/</a></div><div class="gmail_default" style="color:rgb(102,0,0)">
<br></div><div class="gmail_default" style="color:rgb(102,0,0)">Fix for version check zombies: <a href="https://reviewboard.asterisk.org/r/3258/">https://reviewboard.asterisk.org/r/3258/</a></div><div class="gmail_default" style="color:rgb(102,0,0)">
<br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Mar 3, 2014 at 2:27 PM, Paul Belanger <span dir="ltr"><<a href="mailto:paul.belanger@polybeacon.com" target="_blank">paul.belanger@polybeacon.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Mon, Mar 3, 2014 at 2:47 PM, Scott Griepentrog<br>
<<a href="mailto:sgriepentrog@digium.com">sgriepentrog@digium.com</a>> wrote:<br>
><br>
> As an update, the following plan is being implemented (resulting from a combination of comments on app-dev and on reviewboard):<br>
><br>
> For each object type (channels, bridge, playback, recording, snoop) there will be a:<br>
><br>
> POST /objecttype/specifiedid<br>
><br>
> operation that will create the object with that ID.  The<br>
><br>
> POST /objecttype<br>
><br>
> will also create the object, but will use the previously existing unique id creation method.  It will also have the optional objectId query parameter to allow it to be specified (since there are already two methods defined in the code, and it makes reusing existing libraries more convenient by adding a param instead of changing the URI processing).<br>

><br>
> For the channels object type, there is a query parameter otherChannelId which provides the name for the second channel when originating with a Local channel.  If otherChannelId is not specified, but a uniqueid for the first channel is given, the second channel will be assigned the same value with a ;2 suffix (matching the channel name convention).<br>

><br>
> The AMI implementation of channel origination matches ARI except that there is one origination method with two optional parameters, ChannelId and OtherChannelId.<br>
><br>
> Also: in both AMI and ARI, the otherChannelId value is ignored if channelId is not specified.<br>
><br>
</div>+1<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Paul Belanger | PolyBeacon, Inc.<br>
Jabber: <a href="mailto:paul.belanger@polybeacon.com">paul.belanger@polybeacon.com</a> | IRC: pabelanger (Freenode)<br>
Github: <a href="https://github.com/pabelanger" target="_blank">https://github.com/pabelanger</a> | Twitter: <a href="https://twitter.com/pabelanger" target="_blank">https://twitter.com/pabelanger</a><br>
</font></span><div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
asterisk-app-dev mailing list<br>
<a href="mailto:asterisk-app-dev@lists.digium.com">asterisk-app-dev@lists.digium.com</a><br>
<a href="http://lists.digium.com/cgi-bin/mailman/listinfo/asterisk-app-dev" target="_blank">http://lists.digium.com/cgi-bin/mailman/listinfo/asterisk-app-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><img alt="Digium logo" src="https://my.digium.com/images/graphics/digium_RGB_signature.gif" width="288" height="50" style="color:rgb(0,0,0);font-family:Arial,Helvetica,sans-serif;font-size:12px"><div>
Scott Griepentrog<br>Digium, Inc · Software Developer<br>445 Jan Davis Drive NW · Huntsville, AL 35806 · US<br>direct/fax: +1 256 428 6239 · mobile: +1 317 507 4029<br>Check us out at: <a href="http://digium.com" target="_blank">http://digium.com</a> · <a href="http://asterisk.org" target="_blank">http://asterisk.org</a><br>
</div></div>
</div>