[Home]

Summary:ASTERISK-09131: [patch] allow a call-specific setting of 'mohclass'
Reporter:vadim (vadim)Labels:
Date Opened:2007-03-28 12:28:08Date Closed:2011-06-07 14:02:40
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Applications/app_queue
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) call-specific-moh.patch
Description:This functionality was needed to  for our client  who provides virtual secretary services, so
hi could implement DID-specific MOH messages, by setting call specific 'mohclass'
variable in the dialplan
Comments:By: Jason Parker (jparker) 2007-03-28 12:33:24

I think this is an interesting addition, and I'm somewhat surprised that it isn't already possible.

Unfortunately, new features can only be put in to svn trunk.  If the patch is any different, would you mind uploading a patch against trunk (it will probably still apply just fine)?
By: vadim (vadim) 2007-03-28 13:04:55

I've just verifyed it applys fine against trunk
By: Caio Begotti (caio1982) 2007-03-28 13:40:58

Vadim, could you write some notes down to instruct us on how we can test the patch and debug stuff? I'd like to report back how it goes but I didn't understand what would change in the dialplan or something.
By: vadim (vadim) 2007-03-29 04:30:37

Here is the small usage example:
===================
exten=> s,1,Answer()
exten=> 999,1,Set(mohclass = my-music)
exten=> 999,2,Queue(my-queue|tn)
exten=> 999,3,Hangup()
============

of course musiconhol.conf should contain definition of my-music class
By: vadim (vadim) 2007-03-29 05:04:22

I've just discovered that

SetMusicOnHold(myclass)

or

Set(CHANNEL(musicclass) = myclass)

seems to offer same functionality as my patch

If some guru can confirm this discovery my patch becomes unnecesary
By: Tilghman Lesher (tilghman) 2007-03-29 15:40:32

That is correct.  SetMusicOnHold does exactly the same thing.
By: Tilghman Lesher (tilghman) 2007-03-29 15:42:21

I take it from your comment that you wish to withdraw this issue.