LoggingConfigForm: only display the Syslog facility option if logging to Syslog

This commit is contained in:
Alexander A. Klimov 2016-11-04 17:19:35 +01:00
parent f27e8c059d
commit 1045d91389

View File

@ -94,29 +94,31 @@ class LoggingConfigForm extends Form
) )
); );
if (Platform::isWindows()) { if (! isset($formData['logging_log']) || $formData['logging_log'] === 'syslog') {
/* @see https://secure.php.net/manual/en/function.openlog.php */ if (Platform::isWindows()) {
$this->addElement( /* @see https://secure.php.net/manual/en/function.openlog.php */
'hidden', $this->addElement(
'logging_facility', 'hidden',
array( 'logging_facility',
'value' => 'user', array(
'disabled' => true 'value' => 'user',
) 'disabled' => true
); )
} else { );
$facilities = array_keys(SyslogWriter::$facilities); } else {
$this->addElement( $facilities = array_keys(SyslogWriter::$facilities);
'select', $this->addElement(
'logging_facility', 'select',
array( 'logging_facility',
'required' => true, array(
'label' => $this->translate('Facility'), 'required' => true,
'description' => $this->translate('The syslog facility to utilize.'), 'label' => $this->translate('Facility'),
'value' => 'user', 'description' => $this->translate('The syslog facility to utilize.'),
'multiOptions' => array_combine($facilities, $facilities) 'value' => 'user',
) 'multiOptions' => array_combine($facilities, $facilities)
); )
);
}
} }
} elseif (isset($formData['logging_log']) && $formData['logging_log'] === 'file') { } elseif (isset($formData['logging_log']) && $formData['logging_log'] === 'file') {
$this->addElement( $this->addElement(