<table cellspacing="0" cellpadding="0" border="0" ><tr><td valign="top" style="font: inherit;">Thanks Russell,<br><br>I filed a bug and created a proposed patch:<br><br>http://bugs.digium.com/view.php?id=15057<br><br>the 1.4 patch is tested and seems to work. The trunk patch should be ok but I don't have a system to build and test it against at the moment.<br><br>--- On <b>Thu, 5/7/09, Russell Bryant <i><russell@digium.com></i></b> wrote:<br><blockquote style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; padding-left: 5px;">From: Russell Bryant <russell@digium.com><br>Subject: Re: [asterisk-dev] expected hint value when multiple devices are combined (onHold)<br>To: p_lindheimer@yahoo.com, "Asterisk Developers Mailing List" <asterisk-dev@lists.digium.com><br>Date: Thursday, May 7, 2009, 6:17 PM<br><br><pre>On May 7, 2009, at 4:27 PM, Philippe Lindheimer wrote:<br><br>> OnHold & Idle = Idle (NOT WHAT ONE WOULD
EXPECT)<br>> <br>> I conjecture that one would expect any state that 'raises' the<br>level of the combination above an idle state would return that state, and<br>basically the 'less available' (busier so to speak) that the state is,<br>it would take precedence. Most of the behavior appears that way with the<br>exception that Idle is taking precedence of onHold which seems very<br>counter-intuitive.<br>> <br>> Before filing this as a "bug" I would like to get the feedback<br>of others what would be the proper value of such a combination.<br><br><br>I would agree that this is a bug. I would expect that combination of states to<br>result in an OnHold state.<br><br>The fix should be relatively easy. The responsible code is in<br>ast_extension_state2() of main/pbx.c in Asterisk 1.4.<br><br>In Asterisk trunk, the same logic was moved into some common code. Search for<br>ast_devstate_aggregate in main/devicestate.c.<br><br>--<br>Russell
Bryant<br>Digium, Inc. | Senior Software Engineer, Open Source Team Lead<br>445 Jan Davis Drive NW - Huntsville, AL 35806 - USA<br>Check us out at: www.digium.com & www.asterisk.org<br><br><br><br><br></pre></blockquote></td></tr></table><br>