<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Dec 7, 2013 at 10:13 AM, 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 13-12-06 01:21 PM, Mark Michelson wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

Jason Parker <mailto:<a href="mailto:jason.parker@schmoozecom.com" target="_blank">jason.parker@<u></u>schmoozecom.com</a>><div class="im"><br>
> Thursday, December 05, 2013 3:31 PM<br>
> Presence State is a feature that was added in Asterisk 11, which allows for<br>
> more information to be captured about the state of a user.  There appears to<br>
> have been a rather large oversight in the implementation.  There is currently<br>
> no way to monitor the changes to this information, meaning that nothing can<br>
> take advantage of it in any realtime manner over AMI.<br>
><br>
> Given that the patch is trivial, and that presence state is not otherwise<br>
> useful for any outside system, I'd like to propose<br>
> <a href="https://reviewboard.asterisk.org/r/3039/" target="_blank">https://reviewboard.asterisk.<u></u>org/r/3039/</a> for inclusion into Asterisk 11.<br></div>
> ------------------------------<u></u>------------------------------<u></u>--------------------<div class="im"><br>
I don't find myself caring enough to object to this. I'd be fine with it being<br>
in 11.<br>
<br>
<br>
</div></blockquote></blockquote>
My only thought is to also bump the AMI version, so libraries also know of the functionality change.<span class=""><font color="#888888"><br>
</font></span></blockquote><div><br><br></div><div>I'm not sure we ever codified this, but should a change that *should* be non-breaking require a version bump?<br><br></div><div>The only way this is potentially harmful is if you:<br>
</div><div>(1) Have func_presencestate loaded (it is the only thing that raises AST_EVENT_PRESENCE_STATE)<br></div><div>(2) Use the PRESENCE_STATE function to change a presence in a hint<br></div><div>(3) Have written the worst AMI parser in the world<br>
<br></div><div>I'm not saying that we shouldn't bump the version, but this is one of those things that can only occur if you are using a feature provided by a loadable module and are a blithering idiot.<br></div></div>
<br>-- <br><div dir="ltr"><div>Matthew Jordan<br></div><div>Digium, Inc. | Engineering Manager</div><div>445 Jan Davis Drive NW - Huntsville, AL 35806 - USA</div><div>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></div>
</div>
</div></div>