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
Priority:	Trivial	Regression?	No
Status:	Closed/Complete	Components:	Applications/app_voicemail
Versions:		Frequency of Occurrence
Related Issues:
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 [general] priorityjumping=yes 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.