[Home]

Summary:ASTERISK-05192: [patch] HasVoicemail and HasNewVoicemail applications should be functions
Reporter:Tilghman Lesher (tilghman)Labels:
Date Opened:2005-09-29 16:07:27Date Closed:2008-01-15 15:50:18.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Functions/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20050929__vmcount.diff.txt
Description:HasVoicemail and HasNewVoicemail both optionally set variables and jump to +101 conditionally.  Instead of themselves setting a variable, they should be functions which return a value, instead.

- Fixed the documentation for a case which wasn't possible anyway.
- Deprecated the apps in favor of a new function VMCOUNT(box[@context][|folder].
Comments:By: Kevin P. Fleming (kpfleming) 2005-10-04 21:42:34

Committed to CVS HEAD, thanks!

(that code (not what you wrote) has some of the most convoluted usage of strsep() that I have ever seen <G>)
By: Digium Subversion (svnbot) 2008-01-15 15:50:18.000-0600

Repository: asterisk
Revision: 6731

U   trunk/apps/app_hasnewvoicemail.c

------------------------------------------------------------------------
r6731 | kpfleming | 2008-01-15 15:50:18 -0600 (Tue, 15 Jan 2008) | 2 lines

add VMCOUNT() function to replace HasVoicemail/HasNewVoicemail apps (issue ASTERISK-5192)

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

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