[asterisk-dev] [Code Review] 4010: General option to musiconhold.conf, to make applications MOH override channels musicclass.

[Kristian Høgh]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4010/
-----------------------------------------------------------

(Updated Sept. 24, 2014, 1:47 p.m.)


Review request for Asterisk Developers.


Bugs: ASTERISK-24276
    https://issues.asterisk.org/jira/browse/ASTERISK-24276


Repository: Asterisk


Description
-------

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.


Diffs (updated)
-----

  /trunk/res/res_musiconhold.c 423782 
  /trunk/configs/samples/musiconhold.conf.sample 423782 

Diff: https://reviewboard.asterisk.org/r/4010/diff/


Testing
-------

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.


Thanks,

Kristian Høgh

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140924/9d47ba2d/attachment-0001.html>