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




<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for Asterisk Developers.</div>
<div>By Kristian Høgh.</div>


<p style="color: grey;"><i>Updated Sept. 22, 2014, 12:23 p.m.</i></p>







<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/jira/browse/ASTERISK-24276">ASTERISK-24276</a>


</div>



<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
Asterisk
</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;">General option to musiconhold.conf, to make applications MOH override channels musicclass.
Defaults to no, as it is current behavior.


We use a single trunk for multiple users. The default musicclass from sip.conf is therefore the same for all calls.
The trunk is connected to a kamailio server, where all in- and outgoing calls are send/received.
Each customer (identified by custom headers) can choose a musicclass for MOH, which we apply to the channel.
When a user dials an extension, which point to a queue, we want the user to receive the queue music.
But we still want the user to keep the preferred musicclass, if the call is put on hold later.
Therefore we cant unset musicclass before calling the queue.
Company A prefer musicclass heavy-metal.
That musicclass is therefore applied to incoming calls.
The customer service need to calm people down. Therefore they use other music as queue music.</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;">Whithout this patch or new option appoverridechannel=no or no appoverridechannel set in musiconhold.conf:
- The queue "myqueue", has musicclass=queueclass
- Set(CHANNEL(musiconhold)=mohclass) followed by Queue(myqueue), makes caller hear "mohclass" while waiting for agent to answer.

Whith appoverridechannel=yes in musiconhold.conf:
- The queue "myqueue", has musicclass=queueclass
- Set(CHANNEL(musiconhold)=myclass) followed by Queue(myqueue), makes caller hear "queueclass" while waiting for agent to answer.
</pre>
  </td>
 </tr>
</table>


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

 <li>/trunk/res/res_musiconhold.c <span style="color: grey">(423656)</span></li>

 <li>/trunk/configs/samples/musiconhold.conf.sample <span style="color: grey">(423656)</span></li>

</ul>

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







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




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