visible-tag needs to respect the do_show-value in case websrv_avail set do_show to false already

Signed-off-by: Michael Kaufmann (d00p) <d00p@froxlor.org>
This commit is contained in:
Michael Kaufmann (d00p)
2015-03-02 13:40:38 +01:00
parent fe718ef67f
commit 27514d32de
3 changed files with 8 additions and 6 deletions

View File

@@ -43,8 +43,9 @@ function buildFormEx($form, $part = '') {
}
// visible = Settings::Get('phpfpm.enabled') for example would result in false if not enabled
// and therefore not shown as intended
if (isset($groupdetails['visible'])) {
// and therefore not shown as intended. Only check if do_show is still true as it might
// be false due to websrv_avail
if (isset($groupdetails['visible']) && $do_show) {
$do_show = $groupdetails['visible'];
}

View File

@@ -55,8 +55,9 @@ function getFormFieldOutput($fieldname, $fielddata) {
}
// visible = Settings::Get('phpfpm.enabled') for example would result in false if not enabled
// and therefore not shown as intended
if (isset($fielddata['visible'])) {
// and therefore not shown as intended. Only check if do_show is still true as it might
// be false due to websrv_avail
if (isset($fielddata['visible']) && $do_show) {
$do_show = $fielddata['visible'];
}

View File

@@ -49,7 +49,7 @@ function getFormOverviewGroupOutput($groupname, $groupdetails) {
if($fielddetails['type'] == 'option')
{
$options_array = $fielddetails['option_options'];
$options_array = $fielddetails['option_options'];
$options = '';
foreach($options_array as $value => $vtitle)
{
@@ -82,7 +82,7 @@ function getFormOverviewGroupOutput($groupname, $groupdetails) {
if (!in_array($websrv, $groupdetails['websrv_avail'])) {
$do_show = false;
}
}
}
if ($do_show) {
eval("\$group = \"" . getTemplate("settings/settings_overviewgroup") . "\";");