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:00Date Closed:2006-09-26 11:56:40
Versions:Frequency of
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

jsmith: *ping*

By: Serge Vecher (serge-v) 2006-08-21 14:48:53

ping again

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.