Summary:	ASTERISK-15316: Voicemail messages flagged as urgent do not get emailed
Reporter:	Dave Hawkes (hevad)	Labels:
Date Opened:	2009-12-15 10:36:16.000-0600	Date Closed:	2010-02-26 12:49:39.000-0600
Priority:	Major	Regression?	No
Status:	Closed/Complete	Components:	Applications/app_voicemail
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:
Description:	When a user marks a voicemail as urgent and voicemail is using regular file based storage an email is not generated and sent to the voicemail recipient. If the message is not marked as urgent then an email is sent as expected. ****** ADDITIONAL INFORMATION ****** None.
Comments:	By: Dave Hawkes (hevad) 2009-12-15 12:28:10.000-0600 I believe this is connected to an existing bug: https://issues.asterisk.org/view.php?id=15654 By: Leif Madsen (lmadsen) 2009-12-15 15:49:45.000-0600 Does the patch on that issue you mention resolve this issue? By: Dave Hawkes (hevad) 2009-12-15 19:51:07.000-0600 Yes, It fixes this email problem, though the message waiting counts are not correct on attached snom phones. One urgent message displays as 2 new. Also the patch did not apply cleanly to current SVN for some reason, though it was trivial to fix. By: Tilghman Lesher (tilghman) 2010-02-23 02:08:02.000-0600 I have updated the patch on ASTERISK-14597, and further testing is requested. By: Digium Subversion (svnbot) 2010-02-26 12:42:00.000-0600 Repository: asterisk Revision: 249187 U   trunk/apps/app_voicemail.c ------------------------------------------------------------------------ r249187 | tilghman | 2010-02-26 12:41:57 -0600 (Fri, 26 Feb 2010) | 18 lines Cleanups to fix bugs in the VM count API functions. - Urgent voicemails were not attached, because the attachment code looked in the wrong folder. - Urgent voicemails were sometimes counted twice when displaying the count of new messages. - Backends were inconsistent as to which voicemails each API counted. - Unit tests added to verify behavior in the future. (closes issue ASTERISK-14597) Reported by: tomo1657 Patches:       20100225__issue15654.diff.txt uploaded by tilghman (license 14) Tested by: tilghman (closes issue ASTERISK-15316) Reported by: hevad Review: https://reviewboard.asterisk.org/r/525/ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=249187 By: Digium Subversion (svnbot) 2010-02-26 12:48:59.000-0600 Repository: asterisk Revision: 249189 _U  branches/1.6.1/ U   branches/1.6.1/apps/app_voicemail.c ------------------------------------------------------------------------ r249189 | tilghman | 2010-02-26 12:48:58 -0600 (Fri, 26 Feb 2010) | 24 lines Merged revisions 249187 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........  r249187 | tilghman | 2010-02-26 12:41:57 -0600 (Fri, 26 Feb 2010) | 18 lines    Cleanups to fix bugs in the VM count API functions.    - Urgent voicemails were not attached, because the attachment code looked in the wrong folder.  - Urgent voicemails were sometimes counted twice when displaying the count of new messages.  - Backends were inconsistent as to which voicemails each API counted.    (closes issue ASTERISK-14597)   Reported by: tomo1657   Patches:         20100225__issue15654.diff.txt uploaded by tilghman (license 14)   Tested by: tilghman    (closes issue ASTERISK-15316)   Reported by: hevad    Review: https://reviewboard.asterisk.org/r/525/ ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=249189 By: Digium Subversion (svnbot) 2010-02-26 12:49:38.000-0600 Repository: asterisk Revision: 249190 _U  branches/1.6.2/ U   branches/1.6.2/apps/app_voicemail.c ------------------------------------------------------------------------ r249190 | tilghman | 2010-02-26 12:49:37 -0600 (Fri, 26 Feb 2010) | 24 lines Merged revisions 249187 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........  r249187 | tilghman | 2010-02-26 12:41:57 -0600 (Fri, 26 Feb 2010) | 18 lines    Cleanups to fix bugs in the VM count API functions.    - Urgent voicemails were not attached, because the attachment code looked in the wrong folder.  - Urgent voicemails were sometimes counted twice when displaying the count of new messages.  - Backends were inconsistent as to which voicemails each API counted.    (closes issue ASTERISK-14597)   Reported by: tomo1657   Patches:         20100225__issue15654.diff.txt uploaded by tilghman (license 14)   Tested by: tilghman    (closes issue ASTERISK-15316)   Reported by: hevad    Review: https://reviewboard.asterisk.org/r/525/ ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=249190