Summary: | ASTERISK-10587: music on hold ends before new user enters | ||
Reporter: | Andrew Pepler (pep) | Labels: | |
Date Opened: | 2007-10-22 04:12:57 | Date Closed: | 2007-12-12 15:20:04.000-0600 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Applications/app_meetme |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) 11048-app_meetme.diff | |
Description: | If the I or i option is set to announce a new user joining then when the 2nd person enters the conference, the music on hold stops whilst the 1st user's name is being recorded. The music on hold should only stop once the new user is being announced. ****** ADDITIONAL INFORMATION ****** I am using these options with meetme - cdMpsI | ||
Comments: | By: Jason Parker (jparker) 2007-11-01 13:32:43 I looked through the code, and can see why this is happening. I'm looking into a fix now. By: Jason Parker (jparker) 2007-11-01 15:02:24 Could you give this patch a try? By: Jason Parker (jparker) 2007-12-07 09:44:28.000-0600 This needs to be tested, or it will be closed.. By: Andrew Pepler (pep) 2007-12-11 10:00:27.000-0600 Sorry, I will test it this week. By: Andrew Pepler (pep) 2007-12-12 09:04:28.000-0600 I have tested this and it works as expected. Thanks By: Digium Subversion (svnbot) 2007-12-12 15:20:04.000-0600 Repository: asterisk Revision: 92618 _U trunk/ U trunk/apps/app_meetme.c U trunk/channels/ringtone.h ------------------------------------------------------------------------ r92618 | qwell | 2007-12-12 15:20:03 -0600 (Wed, 12 Dec 2007) | 13 lines Merged revisions 92617 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 (closes issue ASTERISK-10587) ........ r92617 | qwell | 2007-12-12 15:15:45 -0600 (Wed, 12 Dec 2007) | 4 lines Don't increment user count until after name has been recorded (if enabled). Issue 11048, tested by pep. ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=92618 |