Summary: | ASTERISK-03983: single options column not supported in app_voicemail when using realtime | ||
Reporter: | ruok (ruok) | Labels: | |
Date Opened: | 2005-04-25 13:54:41 | Date Closed: | 2008-01-15 15:32:14.000-0600 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Applications/app_voicemail |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) app_voicemail.patch | |
Description: | The realtime voicemail options parsing assumes each option is its own column in the database. While I'm sure this is preferred, when doing upgrades or working with legacy installs, the options column could still be its own column, not to mention that 'delete' is a sql reserved word. This could easily be resolved by making a call to apply_options() from within apply_option() when a variable of name 'options' is found. | ||
Comments: | By: Kaj J. Niemi (kajtzu) 2005-04-26 04:38:11 Nice feature, works for me. The field separator is "|"... I think this should be documented somewhere else besides apply_options() ;-) By: Kevin P. Fleming (kpfleming) 2005-04-26 20:46:15 I agree, this is clearly the right solution for upgraders. Please get a disclaimer filed so I don't have to ask in the future, but this patch is small enough to go in without one. Thanks! By: Kevin P. Fleming (kpfleming) 2005-04-26 20:47:25 No realtime in stable :-) By: Digium Subversion (svnbot) 2008-01-15 15:32:14.000-0600 Repository: asterisk Revision: 5506 U trunk/apps/app_voicemail.c ------------------------------------------------------------------------ r5506 | kpfleming | 2008-01-15 15:32:13 -0600 (Tue, 15 Jan 2008) | 2 lines support old single-column options for realtime voicemail (bug ASTERISK-3983) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=5506 |