<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/2254/">https://reviewboard.asterisk.org/r/2254/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On January 17th, 2013, 9:37 a.m., <b>Matt Jordan</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I don't see any problem with this going into trunk.
As for Asterisk 11, the best approach is probably to see if anyone has any objections on the asterisk-dev list (which this review will go out to, but sometimes people don't read all of the code reviews).</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I understand this is 'a feature request' which we shy away from in almost all cases for released branches. However, it's an example of an 'extremely low risk' change that one could call a grey area bug fix. State information is fundamental to Asterisk for features like this and not having such state information really does border on a grey are bug/feature. Maybe better stated as 'an oversight' for not being included in the first place. In comparison to the much more invasive feature done back in 1.4 adding device state information to queue members to get around a 'grey area bug' this one is much lower risk.
Also, if I understood correctly at Astricon, there is no more phones branch and new Digium phone features will be done in the main 11 branch. If that understanding is correct, this would definitely be a phone oriented feature. The fact that other phones beyond Digium can take advantage of it is not really relevant, and the super low invasiveness of the patch would make it much less risk then many other changes that have been made for Digium phones.
In any event, this is part of the Asterisk RPM with the FreePBX distro and is already being deployed by the thousands and quickly more. Our goal is to get it in the format that is approved by this board. Beyond that, I guess we can weigh the tradeoff between accepting it into 11 while 11 is very young or having a divergent feature that will eventually be on tens of thousands of Asterisk installations but inconsistent with the official branch.</pre>
<br />
<p>- p_lindheimer</p>
<br />
<p>On December 27th, 2012, 12:46 p.m., p_lindheimer wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers and p_lindheimer.</div>
<div>By p_lindheimer.</div>
<p style="color: grey;"><i>Updated Dec. 27, 2012, 12:46 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Add a Queue Pause Device State that allows hints to be generated and thus useful BLF buttons on phones. The format is:
qpause:qname:agent
so an example hint that could be made from this, for agent 702 (Local) in queue 434 would look like this:
exten => *46*702*434,hint,qpause:434:Local/702@from-queue/n
This has been implemented in FreePBX 2.11 (currently in beta) and is required for full functionality, it has been tested using version 11 and 10 and the patch is included on the version of Asterisk included in the FreePBX Distro for both versions 10 and 11.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">tested on latest versions 10 and 11 using FreePBX 2.11</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/trunk/apps/app_queue.c <span style="color: grey">(378206)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/2254/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>