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

Joshua C. Colp asteriskteam at digium.com
Mon Jan 14 06:19:58 CST 2019


Joshua C. Colp has submitted this change and it was merged. ( https://gerrit.asterisk.org/10848 )

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; Approved for Submit
  Kevin Harwell: Looks good to me, approved



diff --git a/apps/app_voicemail.c b/apps/app_voicemail.c
index fb5ee88..d132e2b 100644
--- a/apps/app_voicemail.c
+++ b/apps/app_voicemail.c
@@ -7091,6 +7091,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/10848
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-MessageType: merged
Gerrit-Change-Id: I8efbace06e6122ea0793f7bdb073d4378e8274ca
Gerrit-Change-Number: 10848
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/20190114/c19affb7/attachment.html>


More information about the asterisk-code-review mailing list