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

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


Joshua Colp has uploaded a new change for review.

  https://gerrit.asterisk.org/4130

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/30/4130/1

diff --git a/apps/app_queue.c b/apps/app_queue.c
index bf5f954..51741d3 100644
--- a/apps/app_queue.c
+++ b/apps/app_queue.c
@@ -2671,6 +2671,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/4130
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I64be73d5044ce38dd02408bd0e53de965ef65905
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
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