[Home]

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:41Date Closed:2008-01-15 15:32:14.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents: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