diff options
author | Meik Sievertsen <acydburn@phpbb.com> | 2009-08-10 11:17:24 +0000 |
---|---|---|
committer | Meik Sievertsen <acydburn@phpbb.com> | 2009-08-10 11:17:24 +0000 |
commit | dea4eb6a0bbe04cc6fdf354a0564e449547064ff (patch) | |
tree | b9866468eabaa4e307ce2b4bf261de5b9b72482f /phpBB | |
parent | d327154a5a5e9360b8ef5407da56e46a6629b26f (diff) | |
download | forums-dea4eb6a0bbe04cc6fdf354a0564e449547064ff.tar forums-dea4eb6a0bbe04cc6fdf354a0564e449547064ff.tar.gz forums-dea4eb6a0bbe04cc6fdf354a0564e449547064ff.tar.bz2 forums-dea4eb6a0bbe04cc6fdf354a0564e449547064ff.tar.xz forums-dea4eb6a0bbe04cc6fdf354a0564e449547064ff.zip |
check for not set values in acp_board to correctly grab those input elements not populated if empty (checkboxes and multiple select fields)
git-svn-id: file:///svn/phpbb/branches/phpBB-3_0_0@9944 89ea8834-ac86-4346-8a33-228a782c2dd0
Diffstat (limited to 'phpBB')
-rw-r--r-- | phpBB/includes/acp/acp_board.php | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/phpBB/includes/acp/acp_board.php b/phpBB/includes/acp/acp_board.php index 7bc94d685e..bd97c29f02 100644 --- a/phpBB/includes/acp/acp_board.php +++ b/phpBB/includes/acp/acp_board.php @@ -440,7 +440,7 @@ class acp_board // We go through the display_vars to make sure no one is trying to set variables he/she is not allowed to... foreach ($display_vars['vars'] as $config_name => $null) { - if (!isset($cfg_array[$config_name]) || strpos($config_name, 'legend') !== false) + if (strpos($config_name, 'legend') !== false) { continue; } @@ -450,6 +450,12 @@ class acp_board continue; } + // It could happen that the cfg array is not set. This happens within feed settings if unselecting all forums in the multiple select fields for example (it is the same as checkbox handling) + if (!isset($cfg_array[$config_name])) + { + $cfg_array[$config_name] = ''; + } + // Erm, we spotted an array if ($null['type'] == 'select_multiple' && $submit && isset($_REQUEST['config'][$config_name])) { |