Summary:	ASTERISK-04476: [patch] ODBC voicemail message storage with stutter dial and light alerts
Reporter:	Nick Wong (nick_kancharla)	Labels:
Date Opened:	2005-06-23 17:29:06	Date Closed:	2005-07-20 19:23:07
Priority:	Major	Regression?	No
Status:	Closed/Complete	Components:	Applications/General
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) voicemail-notification-odbc.patch
Description:	When using ODBC storage for the voicemail messages they aren't in the filesystem long enough for the standard method of detecting voicemails to send an alert to the IAD to turn on the light or enable stutter dial. This patch adds an #IFDEFed alternate function for messagecount() and has_voicemail() that querys the database rather than the file system to see if there are new messages. It also removes the hard-coding of the voicemail tables from the code and reads them from the extconfig.conf file in the standard format: voicemail_messages => odbc,mydatabase,myvoicemailmessagetable This code has been internally tested by our testing department and is already live on our production server. ****** ADDITIONAL INFORMATION ****** This patch also adds two functions to config.c that query the config_maps for a mapping. These functions are: char *get_config_map_database(char *name) char *get_config_map_table(char *name) There was simply no easy way to just get the table name or database name that I could find in a reasonable ammount of time.
Comments:	By: Kevin P. Fleming (kpfleming) 2005-06-23 18:14:10 I don't think that extconfig.conf is really the proper place for this definition, since it holds 'external configuration' information. I can't immediately suggest a better place, except for possibly the [general] section of voicemail.conf. By: Nick Wong (nick_kancharla) 2005-06-24 06:56:59 Well, I put it there because that's where you define the voicemail users table as well. All of the realtime odbc links are established in that file. I can move it somewhere else without much of a problem, but I would imagine that after Realtime becomes more of a standard feature, all of the ODBC links would get their own realtime.conf file. Nick By: Donny Kavanagh (donnyk) 2005-07-18 18:28:07 I've incorporated these changes into 4403.  Can someone link this and 4403 as a related bug.  voicemail table name is defined in voicemail.conf [general] (rathere then extconfig.conf) please see doc/README.odbcstorage By: drmac (drmac) 2005-07-20 09:40:23 i agree, extconfig is mainly for use with realtime. By: Michael Jerris (mikej) 2005-07-20 19:23:07 This appears to now be a duplicate to, and updated in 4403.