Summary: | ASTERISK-02097: [patch] add/remove members to/from queues on cli | ||
Reporter: | Tilghman Lesher (tilghman) | Labels: | |
Date Opened: | 2004-07-23 17:25:05 | Date Closed: | 2008-01-15 15:03:44.000-0600 |
Priority: | Major | Regression? | No |
Status: | Closed/Complete | Components: | 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 |