makary/phpbb
1
0
Fork 0
mirror of https://github.com/phpbb/phpbb.git synced 2025-06-28 22:28:51 +00:00
Code Issues Projects Releases Packages Wiki Activity

check for not set values in acp_board to correctly grab those input elements not populated if empty (checkboxes and multiple select fields)

Browse source 
git-svn-id: file:///svn/phpbb/branches/phpBB-3_0_0@9944 89ea8834-ac86-4346-8a33-228a782c2dd0
This commit is contained in:
Meik Sievertsen 2009-08-10 11:17:24 +00:00
parent d327154a5a
commit dea4eb6a0b
1 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
phpBB/includes/acp/acp_board.php
View file

@ -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... // 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) 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; continue;
} }
@ -450,6 +450,12 @@ class acp_board
continue; 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 // Erm, we spotted an array
if ($null['type'] == 'select_multiple' && $submit && isset($_REQUEST['config'][$config_name])) if ($null['type'] == 'select_multiple' && $submit && isset($_REQUEST['config'][$config_name]))
{ {