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

[ticket/14972] Fix find_key_index method

Browse source 
PHPBB3-14972
This commit is contained in:
rxu 2017-05-09 01:07:38 +07:00
parent e336a62fa3
commit 67a65e3788
1 changed files with 37 additions and 50 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

87
phpBB/phpbb/template/context.php
View file

@ -142,7 +142,7 @@ class context
*/ */
protected function set_num_rows(&$loop_data) protected function set_num_rows(&$loop_data)
{ {
$s_num_rows = sizeof($loop_data); $s_num_rows = count($loop_data);
foreach ($loop_data as &$mod_block) foreach ($loop_data as &$mod_block)
{ {
foreach ($mod_block as $sub_block_name => &$sub_block) foreach ($mod_block as $sub_block_name => &$sub_block)
@ -194,16 +194,16 @@ class context
{ {
// Nested block. // Nested block.
$blocks = explode('.', $blockname); $blocks = explode('.', $blockname);
$blockcount = sizeof($blocks) - 1; $blockcount = count($blocks) - 1;
$str = &$this->tpldata; $str = &$this->tpldata;
for ($i = 0; $i < $blockcount; $i++) for ($i = 0; $i < $blockcount; $i++)
{ {
$str = &$str[$blocks[$i]]; $str = &$str[$blocks[$i]];
$str = &$str[sizeof($str) - 1]; $str = &$str[count($str) - 1];
} }
$s_row_count = isset($str[$blocks[$blockcount]]) ? sizeof($str[$blocks[$blockcount]]) : 0; $s_row_count = isset($str[$blocks[$blockcount]]) ? count($str[$blocks[$blockcount]]) : 0;
$vararray['S_ROW_COUNT'] = $vararray['S_ROW_NUM'] = $s_row_count; $vararray['S_ROW_COUNT'] = $vararray['S_ROW_NUM'] = $s_row_count;
// Assign S_FIRST_ROW // Assign S_FIRST_ROW
@ -231,7 +231,7 @@ class context
else else
{ {
// Top-level block. // Top-level block.
$s_row_count = (isset($this->tpldata[$blockname])) ? sizeof($this->tpldata[$blockname]) : 0; $s_row_count = (isset($this->tpldata[$blockname])) ? count($this->tpldata[$blockname]) : 0;
$vararray['S_ROW_COUNT'] = $vararray['S_ROW_NUM'] = $s_row_count; $vararray['S_ROW_COUNT'] = $vararray['S_ROW_NUM'] = $s_row_count;
// Assign S_FIRST_ROW // Assign S_FIRST_ROW
@ -285,7 +285,7 @@ class context
{ {
// For nested block, $blockcount > 0, for top-level block, $blockcount == 0 // For nested block, $blockcount > 0, for top-level block, $blockcount == 0
$blocks = explode('.', $blockname); $blocks = explode('.', $blockname);
$blockcount = sizeof($blocks) - 1; $blockcount = count($blocks) - 1;
$block = $this->tpldata; $block = $this->tpldata;
for ($i = 0; $i <= $blockcount; $i++) for ($i = 0; $i <= $blockcount; $i++)
@ -296,17 +296,17 @@ class context
if (strpos($blocks[$i], '[]') === $pos) if (strpos($blocks[$i], '[]') === $pos)
{ {
$index = sizeof($block[$name]) - 1; $index = count($block[$name]) - 1;
} }
else else
{ {
$index = min((int) substr($blocks[$i], $pos + 1, -1), sizeof($block[$name]) - 1); $index = min((int) substr($blocks[$i], $pos + 1, -1), count($block[$name]) - 1);
} }
} }
else else
{ {
$name = $blocks[$i]; $name = $blocks[$i];
$index = sizeof($block[$name]) - 1; $index = count($block[$name]) - 1;
} }
$block = $block[$name]; $block = $block[$name];
$block = $block[$index]; $block = $block[$index];
@ -355,39 +355,26 @@ class context
{ {
// For nested block, $blockcount > 0, for top-level block, $blockcount == 0 // For nested block, $blockcount > 0, for top-level block, $blockcount == 0
$blocks = explode('.', $blockname); $blocks = explode('.', $blockname);
$blockcount = sizeof($blocks) - 1; $blockcount = count($blocks) - 1;
$block = $this->tpldata; $block = $this->tpldata;
for ($i = 0; $i < $blockcount; $i++) for ($i = 0; $i < $blockcount; $i++)
{ {
if (($pos = strpos($blocks[$i], '[')) !== false) $pos = strpos($blocks[$i], '[');
{ $name = ($pos !== false) ? substr($blocks[$i], 0, $pos) : $blocks[$i];
$name = substr($blocks[$i], 0, $pos);
if (strpos($blocks[$i], '[]') === $pos)
{
$index = sizeof($block[$name]) - 1;
}
else
{
$index = min((int) substr($blocks[$i], $pos + 1, -1), sizeof($block[$name]) - 1);
}
}
else
{
$name = $blocks[$i];
$index = sizeof($block[$name]) - 1;
}
if (!isset($block[$name])) if (!isset($block[$name]))
{ {
return false; return false;
} }
$block = $block[$name];
if (!isset($block[$index])) $index = (!$pos || strpos($blocks[$i], '[]') === $pos) ? (count($block[$name]) - 1) : (min((int) substr($blocks[$i], $pos + 1, -1), count($block[$name]) - 1));
if (!isset($block[$name][$index]))
{ {
return false; return false;
} }
$block = $block[$index]; $block = $block[$name][$index];
} }
if (!isset($block[$blocks[$i]])) if (!isset($block[$blocks[$i]]))
@ -397,9 +384,9 @@ class context
$block = $block[$blocks[$i]]; // Traverse the last block $block = $block[$blocks[$i]]; // Traverse the last block
// Change key to zero (change first position) if false and to last position if true // Change key to zero (change first position) if false and to last position if true
if ($key === false || $key === true) if (is_bool($key))
{ {
return ($key === false) ? 0 : sizeof($block) - 1; return (!$key) ? 0 : count($block) - 1;
} }
// Get correct position if array given // Get correct position if array given
@ -416,7 +403,7 @@ class context
} }
} }
return (is_int($key) && ((0 <= $key) && ($key < sizeof($block)))) ? $key : false; return (is_int($key) && ((0 <= $key) && ($key < count($block)))) ? $key : false;
} }
/** /**
@ -453,7 +440,7 @@ class context
// For nested block, $blockcount > 0, for top-level block, $blockcount == 0 // For nested block, $blockcount > 0, for top-level block, $blockcount == 0
$blocks = explode('.', $blockname); $blocks = explode('.', $blockname);
$blockcount = sizeof($blocks) - 1; $blockcount = count($blocks) - 1;
$block = &$this->tpldata; $block = &$this->tpldata;
for ($i = 0; $i < $blockcount; $i++) for ($i = 0; $i < $blockcount; $i++)
@ -464,17 +451,17 @@ class context
if (strpos($blocks[$i], '[]') === $pos) if (strpos($blocks[$i], '[]') === $pos)
{ {
$index = sizeof($block[$name]) - 1; $index = count($block[$name]) - 1;
} }
else else
{ {
$index = min((int) substr($blocks[$i], $pos + 1, -1), sizeof($block[$name]) - 1); $index = min((int) substr($blocks[$i], $pos + 1, -1), count($block[$name]) - 1);
} }
} }
else else
{ {
$name = $blocks[$i]; $name = $blocks[$i];
$index = sizeof($block[$name]) - 1; $index = count($block[$name]) - 1;
} }
$block = &$block[$name]; $block = &$block[$name];
$block = &$block[$index]; $block = &$block[$index];
@ -496,7 +483,7 @@ class context
// Change key to zero (change first position) if false and to last position if true // Change key to zero (change first position) if false and to last position if true
if ($key === false || $key === true) if ($key === false || $key === true)
{ {
$key = ($key === false) ? 0 : sizeof($block); $key = ($key === false) ? 0 : count($block);
} }
// Get correct position if array given // Get correct position if array given
@ -526,9 +513,9 @@ class context
if ($mode == 'insert') if ($mode == 'insert')
{ {
// Make sure we are not exceeding the last iteration // Make sure we are not exceeding the last iteration
if ($key >= sizeof($block)) if ($key >= count($block))
{ {
$key = sizeof($block); $key = count($block);
unset($block[($key - 1)]['S_LAST_ROW']); unset($block[($key - 1)]['S_LAST_ROW']);
$vararray['S_LAST_ROW'] = true; $vararray['S_LAST_ROW'] = true;
} }
@ -543,7 +530,7 @@ class context
$vararray['S_BLOCK_NAME'] = $name; $vararray['S_BLOCK_NAME'] = $name;
// Re-position template blocks // Re-position template blocks
for ($i = sizeof($block); $i > $key; $i--) for ($i = count($block); $i > $key; $i--)
{ {
$block[$i] = $block[$i-1]; $block[$i] = $block[$i-1];
@ -561,12 +548,12 @@ class context
if ($mode == 'change') if ($mode == 'change')
{ {
// If key is out of bounds, do not change anything // If key is out of bounds, do not change anything
if ($key > sizeof($block) || $key < 0) if ($key > count($block) || $key < 0)
{ {
return false; return false;
} }
if ($key == sizeof($block)) if ($key == count($block))
{ {
$key--; $key--;
} }
@ -580,26 +567,26 @@ class context
if ($mode == 'delete') if ($mode == 'delete')
{ {
// If we are exceeding last iteration, do not delete anything // If we are exceeding last iteration, do not delete anything
if ($key > sizeof($block) || $key < 0) if ($key > count($block) || $key < 0)
{ {
return false; return false;
} }
// If we are positioned at the end, we remove the last element // If we are positioned at the end, we remove the last element
if ($key == sizeof($block)) if ($key == count($block))
{ {
$key--; $key--;
} }
// We are deleting the last element in the block, so remove the block // We are deleting the last element in the block, so remove the block
if (sizeof($block) === 1) if (count($block) === 1)
{ {
$block = null; // unset($block); does not work on references $block = null; // unset($block); does not work on references
return true; return true;
} }
// Re-position template blocks // Re-position template blocks
for ($i = $key; $i < sizeof($block)-1; $i++) for ($i = $key; $i < count($block)-1; $i++)
{ {
$block[$i] = $block[$i+1]; $block[$i] = $block[$i+1];
$block[$i]['S_ROW_COUNT'] = $block[$i]['S_ROW_NUM'] = $i; $block[$i]['S_ROW_COUNT'] = $block[$i]['S_ROW_NUM'] = $i;
@ -610,7 +597,7 @@ class context
// Set first and last elements again, in case they were removed // Set first and last elements again, in case they were removed
$block[0]['S_FIRST_ROW'] = true; $block[0]['S_FIRST_ROW'] = true;
$block[sizeof($block)-1]['S_LAST_ROW'] = true; $block[count($block)-1]['S_LAST_ROW'] = true;
return true; return true;
} }
@ -631,13 +618,13 @@ class context
{ {
// Nested block. // Nested block.
$blocks = explode('.', $blockname); $blocks = explode('.', $blockname);
$blockcount = sizeof($blocks) - 1; $blockcount = count($blocks) - 1;
$str = &$this->tpldata; $str = &$this->tpldata;
for ($i = 0; $i < $blockcount; $i++) for ($i = 0; $i < $blockcount; $i++)
{ {
$str = &$str[$blocks[$i]]; $str = &$str[$blocks[$i]];
$str = &$str[sizeof($str) - 1]; $str = &$str[count($str) - 1];
} }
unset($str[$blocks[$blockcount]]); unset($str[$blocks[$blockcount]]);