[Home]

Summary:ASTERISK-02097: [patch] add/remove members to/from queues on cli
Reporter:Tilghman Lesher (tilghman)Labels:
Date Opened:2004-07-23 17:25:05Date Closed:2008-01-15 15:03:44.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Applications/app_queue
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20040727__cli_queue.diff.txt
Description:Adds the following commands to the CLI:
add queue member <interface> to <queue>
remove queue member <interface> from <queue>
Comments:By: twisted (twisted) 2004-07-25 19:08:41

I like this.  Very much :)  - I vote for CVS inclusion Pre-1.0
By: mustdie (mustdie) 2004-07-25 19:36:43

I've test it and it works just great. Please merge it into CVS.
By: Mark Spencer (markster) 2004-07-26 00:40:07

When checking manager stuff, you should be checking for empty's, not NULL.  astman_get_header is guaranteed to not return NULL, but "" instead.

Also, with respect to documentation, '|' is still canonical, ',' is just some syntactic sugar within pbx_config.c
By: Tilghman Lesher (tilghman) 2004-07-26 01:09:27

Okay, suggestions merged.
By: mustdie (mustdie) 2004-07-27 00:50:01

Tested again and works great again. Please merge it into CVS.
By: Mark Spencer (markster) 2004-07-27 23:09:41

Added to CVS
By: Digium Subversion (svnbot) 2008-01-15 15:03:44.000-0600

Repository: asterisk
Revision: 3526

U   trunk/apps/app_queue.c

------------------------------------------------------------------------
r3526 | markster | 2008-01-15 15:03:43 -0600 (Tue, 15 Jan 2008) | 2 lines

Unify queue add/remove from manager and CLI (bug ASTERISK-2097/2123)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=3526