From 575d1703656f56560fe36ab87931fd7b99b7d7e2 Mon Sep 17 00:00:00 2001 From: Ludovic Arnaud Date: Thu, 6 Mar 2003 03:04:29 +0000 Subject: [PATCH] The last one for today. As of now, sql_query_limit($sql, 0) will _not_ limit the number of rows that are affected. git-svn-id: file:///svn/phpbb/trunk@3608 89ea8834-ac86-4346-8a33-228a782c2dd0 --- phpBB/db/mysql.php | 7 ++++++- phpBB/mcp.php | 4 +++- 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/phpBB/db/mysql.php b/phpBB/db/mysql.php index fe18cffa89..8433deb7d8 100644 --- a/phpBB/db/mysql.php +++ b/phpBB/db/mysql.php @@ -210,7 +210,6 @@ class sql_db } // 20030406 Ashe: switched up $total and $offset as per MySQL manual - // Note for other DBALs: if $total == -1 we only want to set an offset (no pun intended) function sql_query_limit($query, $total, $offset = 0, $expire_time = 0) { if ($query != '') @@ -218,6 +217,12 @@ class sql_db $this->query_result = false; $this->num_queries++; + // if $total is set to 0 we do not want to limit the number of rows + if ($total == 0) + { + $total = -1; + } + $query .= ' LIMIT ' . ((!empty($offset)) ? "$offset, $total" : $total); return $this->sql_query($query, $expire_time); diff --git a/phpBB/mcp.php b/phpBB/mcp.php index 5032dc5a4d..7df646c6a9 100644 --- a/phpBB/mcp.php +++ b/phpBB/mcp.php @@ -641,6 +641,8 @@ switch ($mode) AND p.poster_id = u.user_id $limit_posts_time ORDER BY $sort_order"; + + $start = ($posts_per_page == 0) ? 0 : $start; $result = $db->sql_query_limit($sql, $posts_per_page, $start); $i = 0; @@ -941,7 +943,7 @@ switch ($mode) $limit_posts_time ORDER BY $sort_order"; } - $result = $db->sql_query_limit($sql, -1, $start); + $result = $db->sql_query_limit($sql, 0, $start); $post_id_list = array(); while ($row = $db->sql_fetchrow($result))