[Asterisk-code-review] app queue: Added initialization for "context" parameter (asterisk[11])

Joshua Colp asteriskteam at digium.com
Mon Oct 17 08:15:05 CDT 2016


Joshua Colp has uploaded a new change for review.

  https://gerrit.asterisk.org/4127

Change subject: app_queue: Added initialization for "context" parameter
......................................................................

app_queue: Added initialization for "context" parameter

When using Asterisk Realtime Architecture, empty fields are skipped and the
default values are used. If the "context" parameter in queue was set and then
cleared from the database, the old value remains in memory and it continues
to be used. This change initialize the "context" parameter with an empty value,
allowing clearing the parameter.

ASTERISK-26462 #close

Change-Id: I64be73d5044ce38dd02408bd0e53de965ef65905
---
M apps/app_queue.c
1 file changed, 3 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/27/4127/1

diff --git a/apps/app_queue.c b/apps/app_queue.c
index 4010693..991dfd4 100644
--- a/apps/app_queue.c
+++ b/apps/app_queue.c
@@ -2120,6 +2120,9 @@
 	q->retry = DEFAULT_RETRY;
 	q->timeout = DEFAULT_TIMEOUT;
 	q->maxlen = 0;
+
+	ast_string_field_set(q, context, "");
+
 	q->announcefrequency = 0;
 	q->minannouncefrequency = DEFAULT_MIN_ANNOUNCE_FREQUENCY;
 	q->announceholdtime = 1;

-- 
To view, visit https://gerrit.asterisk.org/4127
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I64be73d5044ce38dd02408bd0e53de965ef65905
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 11
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Leandro Dardini <ldardini at gmail.com>



More information about the asterisk-code-review mailing list