[ticket/305] Simplify method get_global_id()
B3P-305
This commit is contained in:
@@ -443,20 +443,7 @@ class fetch_posts
|
|||||||
{
|
{
|
||||||
if (!empty($this->where_string) || ($row = $this->cache->get('_forum_id_first_forum_post')) === false)
|
if (!empty($this->where_string) || ($row = $this->cache->get('_forum_id_first_forum_post')) === false)
|
||||||
{
|
{
|
||||||
$sql = 'SELECT forum_id
|
$row = $this->get_first_forum_id();
|
||||||
FROM ' . FORUMS_TABLE . '
|
|
||||||
WHERE forum_type = ' . FORUM_POST . '
|
|
||||||
' . str_replace('t.', '', $this->where_string) . '
|
|
||||||
ORDER BY forum_id';
|
|
||||||
$result = $this->db->sql_query_limit($sql, 1);
|
|
||||||
$row = $this->db->sql_fetchrow($result);
|
|
||||||
$this->db->sql_freeresult($result);
|
|
||||||
|
|
||||||
if (empty($this->where_string))
|
|
||||||
{
|
|
||||||
// Cache first forum ID for one day = 86400 s
|
|
||||||
$this->cache->put('_forum_id_first_forum_post', $row, 86400);
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
if (empty($row))
|
if (empty($row))
|
||||||
@@ -469,6 +456,31 @@ class fetch_posts
|
|||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Gets the first forum_id of FORUM_POST type forums
|
||||||
|
*
|
||||||
|
* @return array Database row of query
|
||||||
|
*/
|
||||||
|
protected function get_first_forum_id()
|
||||||
|
{
|
||||||
|
$sql = 'SELECT forum_id
|
||||||
|
FROM ' . FORUMS_TABLE . '
|
||||||
|
WHERE forum_type = ' . FORUM_POST . '
|
||||||
|
' . str_replace('t.', '', $this->where_string) . '
|
||||||
|
ORDER BY forum_id';
|
||||||
|
$result = $this->db->sql_query_limit($sql, 1);
|
||||||
|
$row = $this->db->sql_fetchrow($result);
|
||||||
|
$this->db->sql_freeresult($result);
|
||||||
|
|
||||||
|
if (empty($this->where_string))
|
||||||
|
{
|
||||||
|
// Cache first forum ID for one day = 86400 s
|
||||||
|
$this->cache->put('_forum_id_first_forum_post', $row, 86400);
|
||||||
|
}
|
||||||
|
|
||||||
|
return $row;
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Resets constraints that might have been set before
|
* Resets constraints that might have been set before
|
||||||
*
|
*
|
||||||
|
|||||||
Reference in New Issue
Block a user