diff options
| author | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2010-09-17 21:06:03 +0000 |
|---|---|---|
| committer | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2010-09-17 21:06:03 +0000 |
| commit | 7d52d677b23c6101488428cf7247995409d1df34 (patch) | |
| tree | 09e16083abc90ab1834cb3d75bca0b1c1f8307d0 /apps | |
| parent | 5003ce4525a3cab697c8ab118a2b26c4b666c14a (diff) | |
Blank columns should get set on reload, not ignored.
(closes issue #16893)
Reported by: haakon
Patches:
20100818__issue16893.diff.txt uploaded by tilghman (license 14)
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@287386 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'apps')
| -rw-r--r-- | apps/app_queue.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/apps/app_queue.c b/apps/app_queue.c index b0eb0217b..c7231672e 100644 --- a/apps/app_queue.c +++ b/apps/app_queue.c @@ -1300,10 +1300,10 @@ static struct call_queue *find_queue_by_name_rt(const char *queuename, struct as } else tmp_name = v->name; - if (!ast_strlen_zero(v->value)) { - /* Don't want to try to set the option if the value is empty */ - queue_set_param(q, tmp_name, v->value, -1, 0); - } + /* NULL values don't get returned from realtime; blank values should + * still get set. If someone doesn't want a value to be set, they + * should set the realtime column to NULL, not blank. */ + queue_set_param(q, tmp_name, v->value, -1, 0); } if (q->strategy == QUEUE_STRATEGY_ROUNDROBIN) |