|Summary:||ASTERISK-07305: [patch] [post-1.4] MailboxExists() should be a dialplan function|
|Reporter:||Jared Smith (jsmith)||Labels:|
|Date Opened:||2006-07-07 19:23:00||Date Closed:||2006-09-26 11:56:40|
|Environment:||Attachments:||( 0) 20060708__bug7503.diff.txt|
|Description:||The MailboxExists() application complains that priority + 101 doesn't exist, even when you don't use the 'j' option (to tell it to jump to priority + 101). I realize this is pretty trivial, but it's annoying nonetheless.|
Also, shouldn't MailboxExists() really be a dialplan function and not an application? This seems to me to be a perfect candidate for a function and not an application.
****** STEPS TO REPRODUCE ******
Set your verbosity to a reasonable level
Create mailbox 1234@test
Add an extension to your dialplan like this:
exten => 1234,1,MailboxExists(1234@test)
Watch the asterisk console complain "app_voicemail.c:5691 vm_box_exists: VM box 1234@test exists, but extension 1234, priority 103 doesn't exist"
|Comments:||By: Tilghman Lesher (tilghman) 2006-07-08 01:27:10|
In this case, it is being controlled by the option
in extensions.conf, so it should be working exactly correct. As far as converting it to a dialplan function, you're correct. Patch converts the application into a function on trunk.
By: Tilghman Lesher (tilghman) 2006-07-08 01:28:44
Patch needs testing prior to commit.
By: Jared Smith (jsmith) 2006-07-08 08:01:15
It may be controlled the priorityjumping option in the general section of extensions.conf, but in this case I have priorityjumping explicitly set to "no", and it still complains.
I'll try to test Corydon76's patch this morning... if I don't get to it today, it'll be around July 17th before I get back to it, as I'm going on vacation.
By: Tilghman Lesher (tilghman) 2006-07-08 11:43:08
I don't see how that's possible. Either you're using the j flag to the application to make it jump or else the global priorityjumping option is set to true. These are the only two ways that that code could execute.
By: Serge Vecher (serge-v) 2006-08-08 11:12:21
By: Serge Vecher (serge-v) 2006-08-21 14:48:53
By: Jared Smith (jsmith) 2006-08-22 14:26:12
Go ahead and close the bug.
By: Tilghman Lesher (tilghman) 2006-09-26 11:56:40
Added to trunk in 43659.