[Home]

Summary:ASTERISK-03726: [patch] revert MOH part of #3815
Reporter:kuj (kuj)Labels:
Date Opened:2005-03-20 21:22:55.000-0600Date Closed:2008-01-15 15:28:08.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_meetme
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) meetdiff.txt
Description:The MOH part (admin/marked user doesn't hear MOH) of ASTERISK-3725 is incorrect and unfortunately breaks other parts of conferencing. It needs to be reverted.

My apologies!

****** ADDITIONAL INFORMATION ******

A marked user didn't hear MOH as the single meetme user (using flags MAwax). This was a stupid user error on my part, though, as using both 'A' (set marked mode) and 'w' (wait for marked user) flags does not make any sense at all. In the code, for deciding whether to send MOH or not, the 'w' flag "trumps" the 'A' flag. Mea culpa.

The unfortunate side effect of the previous patch was that the conference was bridged partways even before any marked user arrived. So it needs to be reverted.
Comments:By: Mark Spencer (markster) 2005-03-20 21:28:22.000-0600

Fixed in CVS head.  thanks!

By: Digium Subversion (svnbot) 2008-01-15 15:28:08.000-0600

Repository: asterisk
Revision: 5221

U   trunk/apps/app_meetme.c

------------------------------------------------------------------------
r5221 | markster | 2008-01-15 15:28:08 -0600 (Tue, 15 Jan 2008) | 2 lines

Revert broken portion of 3815 (bug ASTERISK-3726)

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

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