[asterisk-dev] [Code Review] 2680: ARI: Channels - MOH start and stop

[jrose]

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

(Updated July 18, 2013, 4:45 p.m.)


Review request for Asterisk Developers, David Lee and Matt Jordan.


Bugs: ASTERISK-21974
    https://issues.asterisk.org/jira/browse/ASTERISK-21974


Repository: Asterisk


Description
-------

This solves half of the issue.

mohStartChannel(channel, mohclass) - Play music on hold of the specified class to the requested channel. May be interrupted by other media requests such as playOnChannel.
mohStop(channel) - Stop playing music on hold for the requested channel.

This patch is pretty simple, so I don't think there will be an issue with it.


Diffs (updated)
-----

  /trunk/include/asterisk/stasis_app.h 394458 
  /trunk/res/res_stasis_http_channels.c 394458 
  /trunk/res/stasis/control.c 394458 
  /trunk/res/stasis_http/resource_channels.h 394458 
  /trunk/res/stasis_http/resource_channels.c 394458 
  /trunk/rest-api/api-docs/channels.json 394458 

Diff: https://reviewboard.asterisk.org/r/2680/diff/


Testing
-------

Started music on hold without the mohclass arg and with the mohclass arg. Observed how it would react if mohstart were used with one class and then switched to another. The music on hold would restart with the new class as anticipated.

Tested stop music on hold with and without music on hold playing. If it's playing, it stops as one would anticipate and if it's not playing, nothing happens. Nothing too irregular there.

Tested both of these operations while the channel is in a bridge. Didn't work. Oh no. Well, I talked with Matt about it after finding out the same thing happens with all media playback operations in ARI and he says this is a known bug outside the scope of this problem..


Thanks,

jrose

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130718/2f8b6fb5/attachment.htm>