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
Versions:Frequency of
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



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.