diff options
| author | David M <davidmj@users.sourceforge.net> | 2007-06-18 13:08:48 +0000 |
|---|---|---|
| committer | David M <davidmj@users.sourceforge.net> | 2007-06-18 13:08:48 +0000 |
| commit | 5f62e8feda7a3b406ff8f3b7d82171491f34ce5a (patch) | |
| tree | ccf7d2cd8ac56d0fa0dc496a80ed73b70c11ae15 /phpBB | |
| parent | 2599387b28db8ae42993c6272750cbd5e6b56d9c (diff) | |
| download | forums-5f62e8feda7a3b406ff8f3b7d82171491f34ce5a.tar forums-5f62e8feda7a3b406ff8f3b7d82171491f34ce5a.tar.gz forums-5f62e8feda7a3b406ff8f3b7d82171491f34ce5a.tar.bz2 forums-5f62e8feda7a3b406ff8f3b7d82171491f34ce5a.tar.xz forums-5f62e8feda7a3b406ff8f3b7d82171491f34ce5a.zip | |
#11313
git-svn-id: file:///svn/phpbb/trunk@7775 89ea8834-ac86-4346-8a33-228a782c2dd0
Diffstat (limited to 'phpBB')
| -rw-r--r-- | phpBB/docs/CHANGELOG.html | 1 | ||||
| -rw-r--r-- | phpBB/includes/functions_admin.php | 20 | ||||
| -rw-r--r-- | phpBB/includes/mcp/mcp_queue.php | 2 |
3 files changed, 22 insertions, 1 deletions
diff --git a/phpBB/docs/CHANGELOG.html b/phpBB/docs/CHANGELOG.html index 46719dfea8..0e6dc600cd 100644 --- a/phpBB/docs/CHANGELOG.html +++ b/phpBB/docs/CHANGELOG.html @@ -282,6 +282,7 @@ p a { <li>[Fix] Custom BBCodes properly handle lowercasing of parameterized tags (Bug #12377)</li> <li>[Fix] Update the forum_id sequence for PostgreSQL during conversion (Bug #11927)</li> <li>[Fix] Allow for multiple tags containing URL and LOCAL_URL tokens (Bug #12473)</li> + <li>[Fix] Properly display forum list in the MCP Queue (Bug #11313)</li> </ul> diff --git a/phpBB/includes/functions_admin.php b/phpBB/includes/functions_admin.php index 185b0d35d2..f522670e5d 100644 --- a/phpBB/includes/functions_admin.php +++ b/phpBB/includes/functions_admin.php @@ -257,11 +257,31 @@ function get_forum_list($acl_list = 'f_list', $id_only = true, $postable_only = $result = $db->sql_query($sql, $expire_time); $forum_rows = array(); + + $right = $padding = 0; + $padding_store = array('0' => 0); + while ($row = $db->sql_fetchrow($result)) { + if ($row['left_id'] < $right) + { + $padding++; + $padding_store[$row['parent_id']] = $padding; + } + else if ($row['left_id'] > $right + 1) + { + // Ok, if the $padding_store for this parent is empty there is something wrong. For now we will skip over it. + // @todo digging deep to find out "how" this can happen. + $padding = (isset($padding_store[$row['parent_id']])) ? $padding_store[$row['parent_id']] : $padding; + } + + $right = $row['right_id']; + $row['padding'] = $padding; + $forum_rows[] = $row; } $db->sql_freeresult($result); + unset($padding_store); } $rowset = array(); diff --git a/phpBB/includes/mcp/mcp_queue.php b/phpBB/includes/mcp/mcp_queue.php index 277fd58927..d3af2caa24 100644 --- a/phpBB/includes/mcp/mcp_queue.php +++ b/phpBB/includes/mcp/mcp_queue.php @@ -274,7 +274,7 @@ class mcp_queue $forum_options = '<option value="0"' . (($forum_id == 0) ? ' selected="selected"' : '') . '>' . $user->lang['ALL_FORUMS'] . '</option>'; foreach ($forum_list_approve as $row) { - $forum_options .= '<option value="' . $row['forum_id'] . '"' . (($forum_id == $row['forum_id']) ? ' selected="selected"' : '') . '>' . $row['forum_name'] . '</option>'; + $forum_options .= '<option value="' . $row['forum_id'] . '"' . (($forum_id == $row['forum_id']) ? ' selected="selected"' : '') . '>' . str_repeat(' ', $row['padding']) . $row['forum_name'] . '</option>'; } $sort_days = $total = 0; |