[Home]

Summary:ASTERISK-14147: [patch] Voicemail "greetings only" feature
Reporter:floletarmo (floletarmo)Labels:
Date Opened:2009-05-16 18:15:45Date Closed:2009-12-14 17:16:02.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Applications/app_voicemail/NewFeature
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) voicemail_changes.patch
Description:The expected behaviour of a "Greetings only" Voicemail is to play the user's greeting message and to hang up without recording anything.
This feature, that exists on some PBX, allows users who don't want voice
messages to easely setup a failover announcement.

So why using the Voicemail application instead of playing a recording ?
Mainly for simpleness reasons:
- It gives a unique dialplan entry point to all users (those who wants a
recorder and those who don't) when they want to configure their greeting
messages.
- It keeps the dialplan simple, as the user failover destination is
always the Voicemail application.

As I didn't find such a feature in the voicemail documentation, my first
guess was to set the user maxmsg to zero. Next I looked more closely at app_voicemail.c and realized that zero is not a valid value.

So I wrote a small patch for app_voicemail.c to support maxmsg=0 and act
as a "Greetings only" VM in the appropriate case. I hope you will find this feature interesting :)

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

External references :
http://forums.digium.com/viewtopic.php?t=68773
http://lists.digium.com/pipermail/asterisk-dev/2009-May/038300.html
Comments:By: Digium Subversion (svnbot) 2009-12-14 17:16:01.000-0600

Repository: asterisk
Revision: 234820

U   trunk/CHANGES
U   trunk/apps/app_voicemail.c
U   trunk/configs/voicemail.conf.sample

------------------------------------------------------------------------
r234820 | tilghman | 2009-12-14 17:16:00 -0600 (Mon, 14 Dec 2009) | 7 lines

Allow greetings-only mailboxes for Voicemail.
(closes issue ASTERISK-14147)
Reported by: floletarmo
Patches:
      voicemail_changes.patch uploaded by floletarmo (license 784)
      (with some additional changes by me)

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

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