mirror of
https://github.com/phpbb/phpbb.git
synced 2025-04-13 20:32:11 +02:00
[ticket/16526] Get rid of goto operator
PHPBB3-16526
This commit is contained in:
parent
6cd54639b5
commit
13aa1ff760
@ -446,27 +446,29 @@ function validate_config_vars($config_vars, &$cfg_array, &$error)
|
||||
switch ($validator[$type])
|
||||
{
|
||||
case 'url':
|
||||
$cfg_array[$config_name] = trim($cfg_array[$config_name]);
|
||||
|
||||
if (!empty($cfg_array[$config_name]) && !preg_match('#^' . get_preg_expression('url') . '$#iu', $cfg_array[$config_name]))
|
||||
{
|
||||
$error[] = $language->lang('URL_INVALID', $language->lang($config_definition['lang']));
|
||||
}
|
||||
|
||||
goto validate_string;
|
||||
|
||||
case 'csv':
|
||||
// Validate comma separated values
|
||||
$unfiltered_array = explode(',', $cfg_array[$config_name]);
|
||||
$filtered_array = array_filter($unfiltered_array);
|
||||
if (!empty($filtered_array) && count($unfiltered_array) !== count($filtered_array))
|
||||
if ($validator[$type] == 'url')
|
||||
{
|
||||
$error[] = $language->lang('CSV_INVALID', $language->lang($config_definition['lang']));
|
||||
}
|
||||
$cfg_array[$config_name] = trim($cfg_array[$config_name]);
|
||||
|
||||
if (!empty($cfg_array[$config_name]) && !preg_match('#^' . get_preg_expression('url') . '$#iu', $cfg_array[$config_name]))
|
||||
{
|
||||
$error[] = $language->lang('URL_INVALID', $language->lang($config_definition['lang']));
|
||||
}
|
||||
}
|
||||
else if ($validator[$type] == 'csv')
|
||||
{
|
||||
// Validate comma separated values
|
||||
$unfiltered_array = explode(',', $cfg_array[$config_name]);
|
||||
$filtered_array = array_filter($unfiltered_array);
|
||||
if (!empty($filtered_array) && count($unfiltered_array) !== count($filtered_array))
|
||||
{
|
||||
$error[] = $language->lang('CSV_INVALID', $language->lang($config_definition['lang']));
|
||||
}
|
||||
|
||||
}
|
||||
// no break here
|
||||
|
||||
validate_string:
|
||||
case 'string':
|
||||
$length = utf8_strlen($cfg_array[$config_name]);
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user