Summary:ASTERISK-08717: moh-class set with SetMusicOnHold() overwrites the one set with 'm' option of Dial()
Reporter:Christian Benke (christianbee)Labels:
Date Opened:2007-02-02 09:55:51.000-0600Date Closed:2007-06-30 09:19:59
Versions:Frequency of
Description:After upgrading from 1.2.9 to 1.2.14 i experienced a change in the way musiconhold is played - While in 1.2.9 musiconhold was playing continuous on sequential extensions after a timeout, it is restarted for every extension in 1.2.14:

;music starts
exten => 902,1,Dial(SIP/phone1@proxy.com|5|m(mymusic))
;music starts again
exten => 902,n,Dial(SIP/phone2@proxy.com|5|m(mymusic))
;and again
exten => 902,n,Dial(SIP/phone3@proxy.com|5|m(mymusic))

This scenario would e.g. apply when a company has several deskphones and after some time, when the employee doesn't pickup, the secretaries phone should start ringing too. Since it is very usual that the musiconhold-file contains a message like 'Welcome to our company, we'll be with you in a few moments' it would be restarted everytime a extensionsjump occurs which looks quite unprofessional for the caller.
Therefore i think that this change in behaviour is a bug, it should at least be configurable in voicemail.conf if you want the file to continue playing in the same call or if it should be started from the beginning/a random file should be picked.

This behaviour applies to 1.2.14 and 1.4.
Comments:By: Christian Benke (christianbee) 2007-02-02 10:02:56.000-0600

uhm, probably a duplicate of 8672, sry. will test 1.4 rev53088 now...

By: Christian Benke (christianbee) 2007-02-02 16:00:23.000-0600

issue doesn't appear in 1.2 rev 53117 and 1.4 rev 53114, but in 1.4 rev 53114 there's another related bug, when a default moh-class is set with SetMusicOnHold or Set(CHANNEL(musicclass)= it overwrites the parameter given in the m-option of the dial-command, e.g.:

exten => 902,n,SetMusicOnHold(moh_hold)
exten => 902,n,Dial(SIP/phone1@proxy.com|60|m(moh_ringing))
exten => 902,n,Hangup

doesn't play "moh_ringing" to the caller but "moh_hold".

By: Joshua C. Colp (jcolp) 2007-02-09 13:40:12.000-0600

Fixed in 1.4 as of revision 53749 and trunk as of revision 53750.