| 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-0600 | Date Closed: | 2008-01-15 14:52:04.000-0600 |
| Priority: | Major | Regression? | No |
| Status: | Closed/Complete | Components: | Core/General |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| 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) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=2779 | ||