[asterisk-bugs] [Asterisk 0011678]: [patch] Notification email should use the voicemail's metadata

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=11678 
====================================================================== 
Reported By:                jamessan
Assigned To:                lmadsen
====================================================================== 
Project:                    Asterisk
Issue ID:                   11678
Category:                   Applications/app_voicemail/NewFeature
Reproducibility:            always
Severity:                   feature
Priority:                   normal
Status:                     ready for testing
Asterisk Version:           1.4.22 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2008-01-04 10:15 CST
Last Modified:              2009-03-27 12:56 CDT
====================================================================== 
Summary:                    [patch] Notification email should use the
voicemail's metadata
Description: 
When a user has voicemail notification enabled, the email that is sent
builds various variables (VM_DATE, VM_CALLERID, VM_CIDNAME, VM_CIDNUM) on
the fly instead of using the information from msgXXXX.txt.  This isn't
normally that different for a notification email triggered by someone
directly leaving a voicemail but it can differ significantly when it is for
a voicemail that is being forwarded.  Also, using the voicemail metadata
would make the notification email consistent with what is presented to the
user when they access the voicemail via dialing in to the voicemail app. 
For example, A leaves a voicemail for B.  B forwards the voicemail to C. 
C's notification email will say that they received an email from B whereas
when they listen to the voicemail envelope it will be listed as originating
from A.
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
parent of           0012745 [branch] RT should adapt to existing co...
====================================================================== 

---------------------------------------------------------------------- 
 (0102300) lmadsen (administrator) - 2009-03-27 12:56
 http://bugs.digium.com/view.php?id=11678#c102300 
---------------------------------------------------------------------- 
OK, I tested again. No crashing!

However, when I forward a message, I don't get the ORIG_* variables
populated it seems. Here is the template I'm using to send:

emailbody=Current variables for this message are:\n\n* ${VM_NAME}\n*
${VM_DUR}\n* ${VM_MSGNUM}\n* ${VM_MAILBOX}\n* ${VM_CALLERID}\n*
${VM_CIDNUM}\n* ${VM_CIDNAME}\n* ${VM_DATE}\n\n* ${ORIG_VM_DATE}\n*
${ORIG_VM_CALLERID}\n* ${ORIG_VM_CIDNUM}\n* ${ORIG_VM_CIDNAME}

And here is the result of a forwarded message, left for Bob, and Bob
forwarded to Alice (600 and 601 respectively):

* Alice
* 0:06
* 1
* 601
* "Leif Madsen" <4164790259>
* 4164790259
* Leif Madsen
* Friday, March 27, 2009 at 01:52:47 PM

*
*
*
* 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-03-27 12:56 lmadsen        Note Added: 0102300                          
======================================================================