Summary:ASTERISK-00637: [patch] advanced directory context handling/remove deprecated app_voicemail compat
Reporter:km (km)Labels:
Date Opened:2003-12-07 13:34:10.000-0600Date Closed:2008-01-15 14:52:04.000-0600
Versions:Frequency of
Environment:Attachments:( 0) app_dir.patch
( 1) app_dir.patch
Description:This patch does two things:

A) Allows the user to specify an additional context as an argument.  This additional context is where the extension will actually be executed.

B) It removes the checking required when we had both app_voicemail and app_voicemail2 -- since we only have the newer voicemail app, the older code was deprecated.
Comments:By: km (km) 2003-12-07 14:53:29.000-0600

download the larger patch, it's got two fixes to it: option_verbose actually exists now and it makes argument parsing a bit easier...

By: atacomm (atacomm) 2003-12-18 17:41:25.000-0600

patch works great, been running for 2 weeks, no problems.

By: Tilghman Lesher (tilghman) 2003-12-21 10:13:24.000-0600

I'd object to the patch namely because it breaks existing usage of Directory().  You should make the second context optional and default to using the same context as the first argument, if the second argument is missing.

For that matter, we should probably make both arguments optional, and use default for both if both are missing.

By: atacomm (atacomm) 2003-12-21 10:47:37.000-0600

Actually the second context is optional, km designed the second context to fill in with the first context if its not supplied.

By: atacomm (atacomm) 2004-01-10 19:41:21.000-0600

Can we get this into 0.7.0?

By: Olle Johansson (oej) 2004-03-21 16:01:02.000-0600

Corydon - ok now or still something missing?

By: twisted (twisted) 2004-04-18 11:13:27

I would like to see this in CVS as well, as the current operation of directory() requires you to have your mailboxes and extensions in the dialplan all within the same context.  I'd like to be able to change that, and I think others would too.   Check for the ability to patch against current cvs-head/stable and report back pls.

By: Mark Spencer (markster) 2004-04-26 09:44:03

Great idea, not thrilled with the implementation.  I've done my own, re-open this bug if it doesn't work for you.

By: Digium Subversion (svnbot) 2008-01-15 14:52:04.000-0600

Repository: asterisk
Revision: 2779

U   trunk/apps/app_directory.c

r2779 | markster | 2008-01-15 14:52:03 -0600 (Tue, 15 Jan 2008) | 2 lines

Allow two contexts to be specified for directory application (reimplementation fo bug ASTERISK-637)