Summary:	ASTERISK-02518: Has(New)VoiceMail does not assume INBOX if no folder specified
Reporter:	kdau (kdau)	Labels:
Date Opened:	2004-10-02 19:26:14	Date Closed:	2008-01-15 15:09:00.000-0600
Priority:	Minor	Regression?	No
Status:	Closed/Complete	Components:	Core/General
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) 20041003__fix_2559.diff.txt ( 1) app_hasnewvoicemail-error.out ( 2) jamesfix.diff.txt
Description:	Contrary to documentation and the apparent intent of the code, the HasVoiceMail and HasNewVoiceMail applications do not assume the INBOX folder if none is specified in their invocation. Reliably repeatable on Gentoo Linux with 1.0.1 and CVS head from today. ****** ADDITIONAL INFORMATION ****** Relevant verbose console output from errant invocation attached. Error message suggests folder variable stays null.
Comments:	By: kdau (kdau) 2004-10-02 19:28:13 Invocation that caused the error was:  exten => s,1,HasNewVoicemail(100${c_line}) Where $c_line was set to "3". By: James Golovich (jamesgolovich) 2004-10-03 01:36:51 Fixed in CVS By: Tilghman Lesher (tilghman) 2004-10-03 08:52:34 I think there's a better fix to be had (i.e. using strsep correctly).  See attached patch. By: Mark Spencer (markster) 2004-10-03 09:08:48 James' fix is just fine.  Short and to the point. By: Mark Spencer (markster) 2004-10-03 09:09:46 Oops need to attach patch for Russell to put in 1.0 though... By: Russell Bryant (russell) 2004-10-03 12:42:13 added to the 1.0 branch, thanks! By: Digium Subversion (svnbot) 2008-01-15 15:09:00.000-0600 Repository: asterisk Revision: 3899 U   branches/v1-0/apps/app_hasnewvoicemail.c ------------------------------------------------------------------------ r3899 | russell | 2008-01-15 15:08:59 -0600 (Tue, 15 Jan 2008) | 2 lines Has(New)VoiceMail does not assume INBOX if no folder specified (bug ASTERISK-2518) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=3899