[Home]

Summary:ASTERISK-15763: [patch] MusicOnHold ignores realtime when no musiconhold.conf classes configured.
Reporter:Dmitri Anpilogov (dmitri)Labels:
Date Opened:2010-03-08 04:15:37.000-0600Date Closed:2010-06-23 16:55:00
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Resources/res_musiconhold
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) res_musiconhold.patch
Description:Asterisk disables MusicOnHold when no classes configured in musiconhold.conf file, or file is not exists. But it's not correct when realtime is using for moh.
I made a patch to get through of this.
Hope it will be usefull.
Comments:By: Leif Madsen (lmadsen) 2010-03-08 10:03:58.000-0600

Thanks for the submission!

Feel free to request testing from people on the asterisk-users and/or asterisk-dev mailing lists.

Thanks!

By: Atis Lezdins (atis) 2010-05-07 13:10:52

We have been running this on production for several months now, and realtime music-on-hold is working fine.

(Disclaimer: I work at the same company as dmitri)

By: Digium Subversion (svnbot) 2010-06-23 16:53:48

Repository: asterisk
Revision: 272332

U   trunk/res/res_musiconhold.c

------------------------------------------------------------------------
r272332 | tilghman | 2010-06-23 16:53:48 -0500 (Wed, 23 Jun 2010) | 8 lines

If there is realtime configuration, it does not get re-read on reload unless the config file also changes.

(closes issue ASTERISK-15763)
Reported by: dmitri
Patches:
      res_musiconhold.patch uploaded by dmitri (license 1001)
Tested by: atis

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=272332

By: Digium Subversion (svnbot) 2010-06-23 16:54:59

Repository: asterisk
Revision: 272333

_U  branches/1.6.2/
U   branches/1.6.2/res/res_musiconhold.c

------------------------------------------------------------------------
r272333 | tilghman | 2010-06-23 16:54:59 -0500 (Wed, 23 Jun 2010) | 15 lines

Merged revisions 272332 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r272332 | tilghman | 2010-06-23 16:53:49 -0500 (Wed, 23 Jun 2010) | 8 lines
 
 If there is realtime configuration, it does not get re-read on reload unless the config file also changes.
 
 (closes issue ASTERISK-15763)
  Reported by: dmitri
  Patches:
        res_musiconhold.patch uploaded by dmitri (license 1001)
  Tested by: atis
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=272333