[asterisk-dev] Presence State in Asterisk 11

[Matthew Jordan]

On Sat, Dec 7, 2013 at 10:13 AM, Paul Belanger <paul.belanger at polybeacon.com
> wrote:

> On 13-12-06 01:21 PM, Mark Michelson wrote:
>
>> Jason Parker <mailto:jason.parker at schmoozecom.com>
>>>
>>> > Thursday, December 05, 2013 3:31 PM
>>> > Presence State is a feature that was added in Asterisk 11, which
>>> allows for
>>> > more information to be captured about the state of a user.  There
>>> appears to
>>> > have been a rather large oversight in the implementation.  There is
>>> currently
>>> > no way to monitor the changes to this information, meaning that
>>> nothing can
>>> > take advantage of it in any realtime manner over AMI.
>>> >
>>> > Given that the patch is trivial, and that presence state is not
>>> otherwise
>>> > useful for any outside system, I'd like to propose
>>> > https://reviewboard.asterisk.org/r/3039/ for inclusion into Asterisk
>>> 11.
>>> > ------------------------------------------------------------
>>> --------------------
>>>
>>> I don't find myself caring enough to object to this. I'd be fine with it
>>> being
>>> in 11.
>>>
>>>
>>>  My only thought is to also bump the AMI version, so libraries also know
> of the functionality change.
>


I'm not sure we ever codified this, but should a change that *should* be
non-breaking require a version bump?

The only way this is potentially harmful is if you:
(1) Have func_presencestate loaded (it is the only thing that raises
AST_EVENT_PRESENCE_STATE)
(2) Use the PRESENCE_STATE function to change a presence in a hint
(3) Have written the worst AMI parser in the world

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.

-- 
Matthew Jordan
Digium, Inc. | Engineering Manager
445 Jan Davis Drive NW - Huntsville, AL 35806 - USA
Check us out at: http://digium.com & http://asterisk.org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20131207/e0e36be9/attachment.html>