[Home]

Summary:ASTERISK-00485: [patch] app_voicemail2 exit priority+101 when no mailbox is found
Reporter:screel (screel)Labels:
Date Opened:2003-11-05 15:54:40.000-0600Date Closed:2008-01-15 14:37:39.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) vm101.diff
( 1) vm101-documentation.diff
Description:When Voicemail2() is called with a mailbox that does not exist, it currently exits with: No entry in voicemail config file.  The patch causes it to still produce the warning, but to jump to priority + 101.

There is no provision (that I saw) to see if a mailbox exists before sending a caller into voicemail.  If a mailbox does not exist for a user, we may want to send to a general mailbox, back to an operator, or do who knows what else.
Comments:By: Brian West (bkw918) 2003-11-13 15:51:27.000-0600

Has anyone tested this?  This seems like a good addition to app_voicemail[2] ... Also can you submit a new diff against app_voicemail.c since app_voicemail2.c is now app_voicemail.c and the old voicemail is gone.

bkw

By: Mark Spencer (markster) 2003-11-13 17:55:30.000-0600

Fixed, would be nice if there was some documentation.

By: screel (screel) 2003-11-14 09:39:33.000-0600

Patch to include documentation in descrip_vm (as requested)

By: Brian West (bkw918) 2003-11-24 15:48:28.000-0600

Docs added

By: Digium Subversion (svnbot) 2008-01-15 14:37:39.000-0600

Repository: asterisk
Revision: 1746

U   trunk/apps/app_voicemail.c

------------------------------------------------------------------------
r1746 | markster | 2008-01-15 14:37:38 -0600 (Tue, 15 Jan 2008) | 2 lines

If no vm box, go to 101 if appropriate (bug ASTERISK-485)

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

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