[Home]

Summary:ASTERISK-02272: [patch] When playing a message longer than X minutes, say the duration in minutes
Reporter:Jean-Philippe Lord (jplord)Labels:
Date Opened:2004-08-25 13:57:10Date Closed:2008-01-15 15:12:11.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Applications/app_voicemail
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20040901appvoicemail.diff
( 1) 20041002appvoicemail.diff
( 2) 20041002r1appvoicemail.diff
( 3) 20041027appvoicemail.diff
( 4) vm-minutes.gsm
Description:This is to inform the user of a long message... just before the message play, say for example "3 minutes" if the message last 3 minutes and 40 seconds.
Comments:By: twisted (twisted) 2004-08-25 17:49:50

I'm assuming this is a request, since there is no patch attached.

By: Jean-Philippe Lord (jplord) 2004-09-01 22:10:33

Here's a first attempt at a patch. - Disclaimer on file -
This patch will enable voicemailmain to say the duration of a message in rounded down minutes. For example, if a message last 2minutes and 35 seconds, before hearing the message, you will hear 2 minutes... which basically tells you that the caller speaks for a long long time... and if you don't feel like hearing all that... just call back ! (by the way, you can dial 321 while message is playing to callback using the callerid number !)

A new gsm file is required and needs to be added to /var/lib/asterisk/sounds or the usual dir you use for sounds.



1. 20040901appvoicemail.diff
patches apps/app_voicemail.c to add the duration playback and 2 parameter to set it up. sayduration=yes/no and saydurationm=2 for 2 minutes. Can be customized globally or per user un voicemail.conf.

patches configs/voicemail.conf.sample. It adds 2 new parameter:
sayduration=yes/no (on by default)
saydurationm=2 (2 by default)
and also add a parameter that was missed in the sample for cidinternalcontexts which allow you to setup voicemailmain to say the recorded name of the user who left the message when its coming from an internal context you setup.

By: Jean-Philippe Lord (jplord) 2004-09-01 22:10:59

Please change title to [patch]

By: twisted (twisted) 2004-10-01 12:02:26

Status update?  Also, the default action needs to be the current default action of app_voicemail, which would be the duration turned OFF.

By: Jean-Philippe Lord (jplord) 2004-10-02 10:55:30

I've been using this for quite awhile now and it works great. I've fixed the default and made a new diff that works with the current CVS.

By: twisted (twisted) 2004-10-27 16:26:27

Is this still valid with current cvs?  If not, we need a new patch, and forgive me if i've forgotton, but do you have a disclaimer on file?

Update required to continue.

--Housekeeping

By: Jean-Philippe Lord (jplord) 2004-10-27 19:23:29

New Patch 20041027appvoicemail.diff works with current CVS.
vm-minutes.gsm has been taken from asterisk-sounds CVS.

Been using this for 2 months now and works great.
Disclaimer is on file.

By: Jean-Philippe Lord (jplord) 2004-10-30 10:10:03

Reminder sent to twisted

Hi ! Please see the updated information.

By: twisted (twisted) 2004-10-30 11:41:54

I see it.

By: Mark Spencer (markster) 2004-10-30 16:46:53

Added to CVS, thanks!

By: Russell Bryant (russell) 2004-10-31 14:44:58.000-0600

not in 1.0

By: Digium Subversion (svnbot) 2008-01-15 15:12:11.000-0600

Repository: asterisk
Revision: 4130

U   trunk/apps/app_voicemail.c
U   trunk/configs/voicemail.conf.sample
A   trunk/sounds/vm-minutes.gsm
U   trunk/sounds.txt

------------------------------------------------------------------------
r4130 | markster | 2008-01-15 15:12:10 -0600 (Tue, 15 Jan 2008) | 2 lines

Add voicemail to say # of minutes remaining (bug ASTERISK-2272)

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

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