[Asterisk-code-review] manager: Remove AMI "Queues" action. (asterisk[master])

Benjamin Keith Ford asteriskteam at digium.com
Tue Jul 11 09:43:53 CDT 2017


Hello Jenkins2, Joshua Colp, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/5929

to look at the new patch set (#2).

Change subject: manager: Remove AMI "Queues" action.
......................................................................

manager: Remove AMI "Queues" action.

When performing the "Queues" action via AMI, it outputs the same
text that the Asterisk CLI outputs when running a "queue show"
command, which does not conform with the AMI spec. "QueueStatus"
already does what the "Queues" action should do, so instead of
correcting the output, the "Queues" action will be removed and
"QueueStatus" should be used instead.

The AMI version was also updated. Since this is a removal of
functionality, the MAJOR part of the version number was
incremented.

ASTERISK-27073
Reported by: Brian

Change-Id: Id11743859758255b69cc3a557750d7a56c6d16f8
---
M apps/app_queue.c
M include/asterisk/manager.h
2 files changed, 1 insertion(+), 26 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/29/5929/2
-- 
To view, visit https://gerrit.asterisk.org/5929
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id11743859758255b69cc3a557750d7a56c6d16f8
Gerrit-Change-Number: 5929
Gerrit-PatchSet: 2
Gerrit-Owner: Benjamin Keith Ford <bford at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20170711/02a9be2d/attachment.html>


More information about the asterisk-code-review mailing list