[asterisk-dev] [Code Review] hints with 2+ devices that include ONHOLD

[dimas at dataart.com]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.digium.com/r/254/#review798
-----------------------------------------------------------



/trunk/main/devicestate.c
<http://reviewboard.digium.com/r/254/#comment1982>

    The check in this position means ONHOLD status has a "priority" over BUSY/RING/INUSE. I'm not sure it is good choice - to me at least INUSE/BUSY are more "strong" than INHOLD. Because of that I would put this check last.


- dimas


On 2009-05-26 12:55:56, David Vossel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.digium.com/r/254/
> -----------------------------------------------------------
> 
> (Updated 2009-05-26 12:55:56)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> This is a community patch I cleaned up and tested.  Below is the reporter's, p_lidheimer, description of the issue pulled directly from issue #15057.
> 
> ----------------------------
> If you have a hint with two (or more) devices or a device and a custom devstate that includes ONHOLD, you get unexpected results. Two examples:
> 
> exten => 222,hint,SIP/222&SIP/211
> exten => 333,hint,Custom:DND333&SIP/333
> 
> The following results are inconsistent as they should all return ONHOLD:
> 
> ONHOLD & IDLE = IDLE
> ONHOLD & UNAVAILABLE => UNAVAILABLE
> ONHOLD & UNKNOWN => IDLE
> ONHOLD & INVALID => IDLE
> 
> Per Russell Bryant's response on the mailing list:
> 
> "I would agree that this is a bug. I would expect that combination of
> states to result in an OnHold state."
> 
> 
> This addresses bug 15057.
>     https://issues.asterisk.org/view.php?id=15057
> 
> 
> Diffs
> -----
> 
>   /trunk/include/asterisk/devicestate.h 196415 
>   /trunk/main/devicestate.c 196415 
> 
> Diff: http://reviewboard.digium.com/r/254/diff
> 
> 
> Testing
> -------
> 
> I have tested the patch and can verify the situations in the description all result in ONHOLD.
> 
> 
> Thanks,
> 
> David
> 
>