[Home]

Summary:ASTERISK-12800: meetme list <confno> concise fails
Reporter:John Hass (john8675309)Labels:
Date Opened:2008-09-29 13:05:11Date Closed:2009-09-22 08:02:22
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_meetme
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) app_meetme.c.patch
Description:running:
*CLI> meetme list main_bb concise
[Sep 29 13:09:53] WARNING[32384]: app_meetme.c:3343 admin_exec: MeetMeAdmin requires an argument!

I can't seem to get a list of all the conference users.
Comments:By: Eliel Sardanons (eliel) 2008-09-29 13:23:58

Please try this patch, thanks!
By: Leif Madsen (lmadsen) 2008-09-29 13:29:22

Yesterday mvanbaak committed the CLI consistency stuff we were working on during AstriDevCon. This has been done shortly before we commit the res_clialias stuff that file, mvanbaak, and myself have been working on this week.

The command you need is:

meetme list concise main_bb I believe. Thus this patch is not required.
By: John Hass (john8675309) 2008-09-29 13:33:47

*CLI> meetme list main_bb concise
[Sep 29 13:41:03] WARNING[10023]: app_meetme.c:3343 admin_exec: MeetMeAdmin requires an argument!
By: Eliel Sardanons (eliel) 2008-09-29 13:37:32

blitzrage, my patch fix the error reported by the user.
By: Eliel Sardanons (eliel) 2008-09-30 08:18:13

New patch uploaded to fix CLI command 'meetme list [concise] <confno>'.
- Autocomplete was bogus.
- <confno> was not being checked, if the user specified one.
- we were trying to execute a bogus command (with admin_exec) while we were trying to just list conference numbers

Sorry for uploading and deleting patches.
By: John Hass (john8675309) 2008-10-01 20:59:36

I tried this patch it works well but if I type:

meetme list <confno> concise

will display all the conferences (not in concise mode though, even though it is supposed to show the conference members)
By: Michiel van Baak (mvanbaak) 2008-10-02 01:18:44

That is because it is 'meetme list concise <confno>'
By: John Hass (john8675309) 2008-10-02 06:39:59

I still think something is broken:

CLI> meetme list concise main_bb
main_bb!4!0!09:08:09!1!0
CLI> meetme list main_bb
Conf Num       Parties        Marked     Activity  Creation  Locked
main_bb        0004           N/A        09:08:27  Dynamic   No    
* Total number of MeetMe users: 4
By: Eliel Sardanons (eliel) 2008-10-02 06:43:10

What do you think it is broken? what is not working as you expected? the command was renamed to "meetme list [concise] <confno>"
By: John Hass (john8675309) 2008-10-02 06:50:13

Shouldn't meetme list [concise] <confno> show all the members of the conference instead of just listing the conference?  in previous versions I could list concise and see who is talking, muted, and the length that the particular member was in the conference.
By: Eliel Sardanons (eliel) 2008-10-02 06:58:54

yes, you are right, let me check why this was removed.
By: Michiel van Baak (mvanbaak) 2008-10-02 11:58:21

eliel: I found it. It's totally my fault. I broke it with the CLI cleanup branch.
I'll fix it as well. I'm almost there.
By: Michiel van Baak (mvanbaak) 2008-10-02 12:41:56

how about this:

*CLI> meetme list
Conf Num       Parties        Marked     Activity  Creation  Locked
1234           0001      N/A        00:00:07  Static    No    
* Total number of MeetMe users: 1
*CLI> meetme list concise
No such conference: concise.
*CLI> meetme list 1234
User #: 01    <unknown> <no name>            Channel: Console/dsp     (not talking) 00:00:20
1 users in that conference.
*CLI> meetme list 1234 concise
1!!!Console/dsp!!!!!0!00:00:24
*CLI>
By: Michiel van Baak (mvanbaak) 2008-10-02 12:48:59

Ok, more complete example:

*CLI> meetme list
Conf Num       Parties        Marked     Activity  Creation  Locked
1234           0001      N/A        00:00:06  Static    No    
* Total number of MeetMe users: 1
*CLI> meetme list concise
1234!1!0!00:00:10!0!0
*CLI> meetme list 1234
User #: 01    <unknown> <no name>            Channel: Console/dsp     (not talking) 00:00:15
1 users in that conference.
*CLI> meetme list 1234 concise
1!!!Console/dsp!!!!!0!00:00:18
*CLI>
By: Digium Subversion (svnbot) 2008-10-02 12:52:19

Repository: asterisk
Revision: 145915

U   trunk/apps/app_meetme.c

------------------------------------------------------------------------
r145915 | mvanbaak | 2008-10-02 12:52:18 -0500 (Thu, 02 Oct 2008) | 6 lines

fix the 'meetme list', 'meetme list concise', 'meetme list $confno' and 'meetme list $confno concise' CLI commands

(closes issue ASTERISK-12800)
Reported by: john8675309
Help and feedback from eliel, thanks!

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

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