<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/1028/">https://reviewboard.asterisk.org/r/1028/</a>
     </td>
    </tr>
   </table>
   <br />


<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.orgrb/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 Paul Belanger.</div>
<div>By haakon.</div>





<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 information about the reason a call was not answered by the queue-member.
In queue_log a RINGNOANSWER is generated, with no information about the reason.

This patch gives information about the hangup reason of call-attempts to queue members.
This way you can check which calls that was having technical dificulties, and which was just ignored by the member.

The patch changes the behaviour of queue log by adding HANGUPCAUSE as the last parameter to queue_log &quot;RINGNOANSWER&quot;, and adding &quot;Reason:&quot; parameter to AMI-event AgentRingNoAnswer..

Both additions give you a textual representation of the hangup-cause.
(had to add cause2shortstr() in channel.c for this)</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;">Called into queues with several valid and invalid memebers and checked queue_log and AMI for correct hangupcauses.

queue_log example output:
1290380780|1290378912.0|queuetest|DAHDI/g0/12345678|RINGNOANSWER|0|NORMAL_CIRCUIT_CONGESTION</pre>
  </td>
 </tr>
</table>



<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>


 <a href="https://issues.asterisk.org/view.php?id=16925">16925</a>


</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>trunk/UPGRADE.txt <span style="color: grey">(295785)</span></li>

 <li>trunk/apps/app_queue.c <span style="color: grey">(295785)</span></li>

 <li>trunk/include/asterisk/channel.h <span style="color: grey">(295785)</span></li>

 <li>trunk/main/channel.c <span style="color: grey">(295785)</span></li>

</ul>

<p><a href="https://reviewboard.asterisk.org/r/1028/diff/" style="margin-left: 3em;">View Diff</a></p>




  </td>
 </tr>
</table>




  </div>
 </body>
</html>