[Home]

Summary:ASTERISK-04479: [patch] externnotify clairfications
Reporter:Matthew Nicholson (mnicholson)Labels:
Date Opened:2005-06-24 13:07:32Date Closed:2008-01-15 15:40:59.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_voicemail
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) extern_notify.diff.txt
Description:In app_voicemail, when run_externnotify is executed, it expects a context and an extension, but everywhere it is called, currently it is passed a context and a context@extension.

run_externnotify("default", "1000@default")

- rather than -

run_externnotify("default", "1000")

Also it appears run_externnotify does not even currently use the context argument.  I have written a patch that fixes all of this.
Comments:By: Kevin P. Fleming (kpfleming) 2005-07-11 22:49:51

Committed to CVS HEAD, thanks!

By: Digium Subversion (svnbot) 2008-01-15 15:40:59.000-0600

Repository: asterisk
Revision: 6100

U   trunk/apps/app_voicemail.c

------------------------------------------------------------------------
r6100 | kpfleming | 2008-01-15 15:40:58 -0600 (Tue, 15 Jan 2008) | 2 lines

correct arguments being passed to external notify function (bug ASTERISK-4479)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=6100