[asterisk-bugs] [Asterisk 0010835]: Add context field to app_meetme.so application for realtime mode

[noreply at bugs.digium.com]

The following issue requires your FEEDBACK. 
====================================================================== 
http://bugs.digium.com/view.php?id=10835 
====================================================================== 
Reported By:                tecnoxarxa
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   10835
Category:                   Applications/app_meetme
Reproducibility:            always
Severity:                   feature
Priority:                   normal
Status:                     feedback
Asterisk Version:            SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             09-27-2007 18:31 CDT
Last Modified:              11-06-2007 13:48 CST
====================================================================== 
Summary:                    Add context field to app_meetme.so application for
realtime mode
Description: 
I add context field to meetme table.

Only need add context field of type varchar(32)

Now, meetme can get same number confroom for distincts contexts.
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
has duplicate       0010822 Add context field to app_meetme.so appl...
====================================================================== 

---------------------------------------------------------------------- 
 Corydon76 - 11-06-07 13:48  
---------------------------------------------------------------------- 
1) You should not be using the channel context for the meetme context.  If
you want a context, you should be inputting it in the arguments, just like
Voicemail does contexts.  The context should default to "default" if not
otherwise specified.

2) Since realtime is in Meetme in 1.4, you need to ensure that existing
tables don't suddenly fail if the context field does not exist.  Note that
ast_update_realtime() returns -1 if the query fails.  Use that to retry the
update without the context field.  You may choose to issue a warning at
this point, too, if you think that not having a context field should be
deprecated.

3) This patch no longer applies to trunk. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
11-06-07 13:48  Corydon76      Note Added: 0073221                          
11-06-07 13:48  Corydon76      Status                   new => feedback     
======================================================================