diff options
author | Richard Mudgett <rmudgett@digium.com> | 2011-06-29 18:07:26 +0000 |
---|---|---|
committer | Richard Mudgett <rmudgett@digium.com> | 2011-06-29 18:07:26 +0000 |
commit | 54763625c679c10ee293b349c0d0e2d6c1071ada (patch) | |
tree | b08bd2b41bb8afdb0df0c4c9c85a63e7ea5c27b1 /apps | |
parent | 1da33048130cd99e0410e9328f830b31dd46c923 (diff) |
Merged revisions 325610 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8
........
r325610 | rmudgett | 2011-06-29 13:05:15 -0500 (Wed, 29 Jun 2011) | 18 lines
Response to QueueRule manager command does not contain ActionID if it was specified.
* Add ActionID support as documented for the QueueRule AMI action.
* Remove documentation for ActionID with the Queues AMI action. The
output does not follow normal AMI response output and there is no place to
put an ActionID header.
(closes issue AST-602)
Reported by: Vlad Povorozniuc
Patches:
jira_ast_602_v1.8.patch (license #5621) patch uploaded by rmudgett
Tested by: Vlad Povorozniuc, rmudgett
Review: https://reviewboard.asterisk.org/r/1295/
JIRA SWP-3575
........
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@325611 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'apps')
-rw-r--r-- | apps/app_queue.c | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/apps/app_queue.c b/apps/app_queue.c index b9beafdd5..69bee045c 100644 --- a/apps/app_queue.c +++ b/apps/app_queue.c @@ -695,7 +695,6 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$") Queues. </synopsis> <syntax> - <xi:include xpointer="xpointer(/docs/manager[@name='Login']/syntax/parameter[@name='ActionID'])" /> </syntax> <description> </description> @@ -7259,9 +7258,15 @@ static int manager_queues_show(struct mansession *s, const struct message *m) static int manager_queue_rule_show(struct mansession *s, const struct message *m) { const char *rule = astman_get_header(m, "Rule"); + const char *id = astman_get_header(m, "ActionID"); struct rule_list *rl_iter; struct penalty_rule *pr_iter; + astman_append(s, "Response: Success\r\n"); + if (!ast_strlen_zero(id)) { + astman_append(s, "ActionID: %s\r\n", id); + } + AST_LIST_LOCK(&rule_lists); AST_LIST_TRAVERSE(&rule_lists, rl_iter, list) { if (ast_strlen_zero(rule) || !strcasecmp(rule, rl_iter->name)) { @@ -7275,6 +7280,10 @@ static int manager_queue_rule_show(struct mansession *s, const struct message *m } AST_LIST_UNLOCK(&rule_lists); + /* + * Two blank lines instead of one because the Response and + * ActionID headers used to not be present. + */ astman_append(s, "\r\n\r\n"); return RESULT_SUCCESS; |