[Asterisk-code-review] app voicemail: Fix Channel variable VM MESSAGEFILE for "urge... (asterisk[13])

Friendly Automation asteriskteam at digium.com
Fri Jan 11 18:28:12 CST 2019


Friendly Automation has submitted this change and it was merged. ( https://gerrit.asterisk.org/10845 )

Change subject: app_voicemail: Fix Channel variable VM_MESSAGEFILE for "urgent" voicemail
......................................................................

app_voicemail: Fix Channel variable VM_MESSAGEFILE for "urgent" voicemail

If a voicemail is marked "urgent" then the VM_MESSAGEFILE channel variable is
not updated correctly since urgent messages are in a different directory. The
fix is to update the channel variable when the path to the urgent message is
created.

ASTERISK-28225

Change-Id: I8efbace06e6122ea0793f7bdb073d4378e8274ca
---
M apps/app_voicemail.c
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Joshua C. Colp: Looks good to me, but someone else must approve
  Kevin Harwell: Looks good to me, approved
  Friendly Automation: Approved for Submit



diff --git a/apps/app_voicemail.c b/apps/app_voicemail.c
index 1dd7aa0..06b2720 100644
--- a/apps/app_voicemail.c
+++ b/apps/app_voicemail.c
@@ -7056,6 +7056,7 @@
 						RENAME(dir, msgnum, vmu->mailbox, vmu->context, urgdir, x, sfn, dfn);
 						/* Notification must happen for this new message in Urgent folder, not INBOX */
 						ast_copy_string(fn, dfn, sizeof(fn));
+						pbx_builtin_setvar_helper(chan, "VM_MESSAGEFILE", fn);
 						msgnum = x;
 					}
 #endif

-- 
To view, visit https://gerrit.asterisk.org/10845
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: merged
Gerrit-Change-Id: I8efbace06e6122ea0793f7bdb073d4378e8274ca
Gerrit-Change-Number: 10845
Gerrit-PatchSet: 1
Gerrit-Owner: Bryan Boatright <ast-bugs at omega71.com>
Gerrit-Reviewer: Friendly Automation (1000185)
Gerrit-Reviewer: Joshua C. Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190111/485e4466/attachment-0001.html>


More information about the asterisk-code-review mailing list