[asterisk-dev] [Code Review] 3306: app_confbridge: Make explicitly stop MOH if a user is kicked or hangs up while MOH is playing.

[rmudgett]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3306/#review11127
-----------------------------------------------------------


I have already taken care of the two red blobs next to lines I have changed.

- rmudgett


On March 6, 2014, 5:59 p.m., rmudgett wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3306/
> -----------------------------------------------------------
> 
> (Updated March 6, 2014, 5:59 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-23311
>     https://issues.asterisk.org/jira/browse/ASTERISK-23311
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> When MOH is playing to a user in a conference and the user is kicked or hangs up from the conference then the AMI MusicOnHoldSotp events didn't happen.
> 
> This patch should be done for v11+.
> 
> 
> Diffs
> -----
> 
>   /branches/12/res/res_musiconhold.c 410092 
>   /branches/12/main/channel.c 410092 
>   /branches/12/apps/confbridge/conf_state_single_marked.c 410092 
>   /branches/12/apps/confbridge/conf_state_single.c 410092 
>   /branches/12/apps/confbridge/conf_state_inactive.c 410092 
>   /branches/12/apps/confbridge/conf_state.c 410092 
> 
> Diff: https://reviewboard.asterisk.org/r/3306/diff/
> 
> 
> Testing
> -------
> 
> Before patch the AMI MusicOnHoldStop event did not happen when the user is kicked or hangs up to leave the conference.
> With the patch the event happens.
> 
> 
> Thanks,
> 
> rmudgett
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140306/144b6479/attachment.html>