[Home]

Summary:ASTERISK-07159: [patch] app_directory does not support ODBC voicemail storage
Reporter:Edwin horton (edhorton)Labels:
Date Opened:2006-06-13 08:09:54Date Closed:2006-06-30 09:11:35
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_directory
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20060618__directory_odbc_storage.diff.txt
( 1) debug
Description:Using ODBC voicemail storage and Asterisk-1.2.9.1. All voicemail functions work well - except.  If a VM user records messages and name, they are properly placed in MySql data base with an index of -1.  VMMain properly plays these messages when a VM message is left for a extension.  However, the application Directory, correctly accepts 3 digits for the last name, but spells the name rather than play back the recorded greeting.  If I leave everything alone, except recompile Asterisk without ODBC storage, all is well.  Of course in this case the audio message files are stored in the regular directory structure, rather than the database.  I think this was not the case in previous Asterisk-1.2.X versions, but I have not tried them all.

****** ADDITIONAL INFORMATION ******

Using Centos 4.3 2.6.9-34.ELsmp.  Using Realtime Static to store SIP.conf and VOICEMAIL.conf.  Realtime extensions. Debug shows all is happy with database lookups.  I did not see the actual request for the greeting file from the database.
Comments:By: Serge Vecher (serge-v) 2006-06-16 13:06:04

edhorton: the problem here is that app_directory does not support ODBC_STORAGE like app_voicemail. It is logically erroneous, since these apps are closely related; but whilst app_voicemail was designed to support ODBC_STORAGE, app_directory was not. As such, there will not be any code change for 1.2 branch. A question remains on how to address this in trunk (development branch), as there are some architectural questions to this.

I suggest that you request some feedback from the asterisk-dev mailing list referencing this bug.

Thanks

By: Tilghman Lesher (tilghman) 2006-06-18 12:59:39

No, it never worked in 1.2, but it probably should have.  I think that qualifies it as a bug, so here's a patch.  Please try this out and report back your results.

By: Tilghman Lesher (tilghman) 2006-06-22 09:41:27

edhorton:  the concept of this patch is now approved for inclusion in 1.2, so it's now vital that you provide feedback on whether this patch works for you.

By: Serge Vecher (serge-v) 2006-06-28 11:55:02

edhorton: what happened here?

By: Edwin horton (edhorton) 2006-06-30 08:36:51

Applied the patch and the app seems to work very well.  I have tested it for all of our voicemail users and verified that the correct context is used for fetching names.  System has been running for two days with no hickups since the new version was installed.  Sorry it took me so long, but I was traveling.

Regards

By: Tilghman Lesher (tilghman) 2006-06-30 09:11:35

Committed to 1.2, merged to trunk.  Thank you for getting back to us about testing.