Ticket #4216 (closed Bugs: fixed)

Opened 3 years ago

Last modified 3 years ago

Bandwidth option in IAX Settings fails to display default option

Reported by: drmessano Assigned to: p_lindheimer
Priority: minor Milestone: 2.8
Component: Asterisk IAX Settings Version: 2.8-branch
Keywords: iax settings bandwidth unset Cc:
Confirmation: Need Feedback Distro:
Backend Engine: All Distro Ver:
Backend Ver: SVN Revision (if applicable):

Description

The "Bandwidth" setting under "Audio Codecs" fails to display a default value when nothing is set. This should display a default "unset", since "unset" is being presented as a state, and since "unset" displays when it's actively set as an option.

Change History

04/18/10 21:24:46 changed by p_lindheimer

  • confirmation changed from Unreviewed to Need Feedback.

drmessano,

do you feel like digging into this a bit and seeing what might be up? This code is already in the page.iaxsettings.php file at the top:

$iax_settings['bandwidth'] = isset($_POST['bandwidth']) ? $_POST['bandwidth']:'unset';

does changing it to this fix your issue?:

$iax_settings['bandwidth'] = isset($_POST['bandwidth']) && $_POST['bandwidth'] != '' ? $_POST['bandwidth']:'unset';

05/22/10 11:17:36 changed by p_lindheimer

  • status changed from new to closed.
  • resolution set to fixed.

(In [9666]) fixes #4216 set value in radio button to 'unset' not

05/22/10 12:42:42 changed by p_lindheimer

(In [9671]) Merged revisions 9648-9670 via svnmerge from http://svn.freepbx.org/modules/branches/2.7

........

r9655 | p_lindheimer | 2010-05-20 11:26:28 -0700 (Thu, 20 May 2010) | 1 line

fixes #4210 make sure vmx_option_1_system_default is defined before checking status in validation routine

........

r9664 | p_lindheimer | 2010-05-22 07:20:50 -0700 (Sat, 22 May 2010) | 1 line

fixes #4115 default to AMPUSER if nothing provided at cf prompt

........

r9665 | p_lindheimer | 2010-05-22 07:50:25 -0700 (Sat, 22 May 2010) | 1 line

fixes #4079 better handling of recovering from a previously failed backup module install

........

r9666 | p_lindheimer | 2010-05-22 08:17:35 -0700 (Sat, 22 May 2010) | 1 line

fixes #4216 set value in radio button to 'unset' not

........

r9670 | p_lindheimer | 2010-05-22 09:24:40 -0700 (Sat, 22 May 2010) | 1 line

fixes #4114 avoid incorrect hints being set in CF state changes

........