diff options
author | Phil Davis <phil.davis@inf.org> | 2016-03-16 18:18:56 +0545 |
---|---|---|
committer | Phil Davis <phil.davis@inf.org> | 2016-03-16 18:18:56 +0545 |
commit | f8088b005e4e045c7a9d7c893f881d6969b42b4c (patch) | |
tree | 3d4de2f84ea0c3bbdee3ff6d069b51ab2f719b38 | |
parent | 0d5c2f413c311231d5e8e3bcc9a70b2fd87517e7 (diff) | |
download | pfsense-f8088b005e4e045c7a9d7c893f881d6969b42b4c.zip pfsense-f8088b005e4e045c7a9d7c893f881d6969b42b4c.tar.gz |
servise_dhcp include btnadvopts in section
Before this change, $section is added to $form before btnadvopts is put into $section. The previous code looked to me like btnadvopts would end up in a $section object that got overwritten by the new Form_Section at line 1165.
This change makes sure that the button is added to the section, and then the section is added to the form.
But actually the old code is working OK. (And the new code also works just the same)
So am I missing something?
-rw-r--r-- | src/usr/local/www/services_dhcp.php | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/usr/local/www/services_dhcp.php b/src/usr/local/www/services_dhcp.php index c65ccd6..eef705d 100644 --- a/src/usr/local/www/services_dhcp.php +++ b/src/usr/local/www/services_dhcp.php @@ -1145,8 +1145,6 @@ $section->addInput(new Form_Input( $pconfig['ldap'] ))->setHelp('Leave blank to disable. Enter a full URI for the LDAP server in the form ldap://ldap.example.com/dc=example,dc=com '); -$form->add($section); - // Advanced Additional options $btnadv = new Form_Button( 'btnadvopts', @@ -1162,6 +1160,8 @@ $section->addInput(new Form_StaticText( $btnadv )); +$form->add($section); + $section = new Form_Section('Additional BOOTP/DHCP Options'); $section->addClass('adnlopts'); |