Summary: | ASTERISK-20898: sound_only_one parameter will be ignored in confbridge.conf | ||||
Reporter: | Stephan Schönberg (www.sberg.net) (derlinuxer) | Labels: | |||
Date Opened: | 2013-01-05 05:06:36.000-0600 | Date Closed: | 2013-01-27 14:40:58.000-0600 | ||
Priority: | Minor | Regression? | |||
Status: | Closed/Complete | Components: | Applications/app_confbridge | ||
Versions: | 11.1.0 | Frequency of Occurrence | Constant | ||
Related Issues: |
| ||||
Environment: | centOS 6.3 64Bit | Attachments: | ( 0) asterisk-20898-custom-sounds-ignored.diff | ||
Description: | The Parameter sound_only_one set in confbridge.conf does not have any effect.
The patch for ASTERISK-20404 has not been fully added on asterisk version 11 yet. part of conf file {noformat} ; --- ConfBridge Bridge Profile Options --- [basic_bridge] type=bridge sound_only_person=audioplace/conf_only_person sound_wait_for_leader=audioplace/wait_for_moderator sound_place_into_conference=audioplace/silence-500ms sound_only_one=audioplace/conf_many_person sound_leader_has_left=audioplace/moderator_has_left_conference {noformat} With help of issue ASTERISK-20404 I have modified the source like: edit ./asterisk-source/apps/confbridge/conf_config_parser.c {noformat} * structure of a dynamic profile will need to be altered, a completely new sounds structure must be * created instead of simply holding a reference to the one built by the config file. */ ast_string_field_set(sounds, onlyperson, b_profile->sounds->onlyperson); + ast_string_field_set(sounds, onlyone, b_profile->sounds->onlyone); ast_string_field_set(sounds, hasjoin, b_profile->sounds->hasjoin); ast_string_field_set(sounds, hasleft, b_profile->sounds->hasleft); {noformat} This will fix the issue in my case. But someone has to do it properly in the source code system. And I'm not sure of side effects on this change. | ||||
Comments: | By: Michael L. Young (elguero) 2013-01-05 21:58:43.014-0600 Not sure why the commit for ASTERISK-20404 was blocked from going into branch 11. Here is a patch for 11 which is a bit more complete. Please test it and report back. Thanks. By: Stephan Schönberg (www.sberg.net) (derlinuxer) 2013-01-06 08:39:25.236-0600 The Patch asterisk-20898-custom-sounds-ignored.diff is working. I've tested it with asterisk version 11.1.2 (clean download). thanks |