1064: You have an error in your SQL syntax near '' - mysql

Question

1064: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 4SELECT a.* FROM ws_items a WHERE 1 AND enabled=1 AND visible=1 ORDER BY created DESC LIMIT 0,
I searched the website, but none matches my query. I have no idea on how to use mysql, but I really need to solve this problem...
Maybe someone here knows the solution
if ($params["page_name"]=="index")
{
$filter = "";
if ($_SESSION['user_data']['type']!="admin")
{
$filter = " AND `enabled`=1 AND `visible`=1 ";
}
$sql = "SELECT a.* FROM
`".DB_PREFIX."items` a WHERE 1 $filter
ORDER BY `created` DESC
LIMIT ".($_SESSION['itemsPerPage']*($page-1)).",".$_SESSION['itemsPerPage'];
$items = $DB->getAll($sql);
for($i=0; $i<count($items); $i++)
{
$items[$i][third] = "";
$items[$i][thumb_rating] = rating_bar($items[$i]['id'], '5', 'static');
if($i%3==2)
$items[$i][third] = 'style="margin-right:0;"';
}
$smarty->assign("items", $items);
$sql = "SELECT COUNT(a.id) as `total` FROM
`".DB_PREFIX."items` a WHERE 1 $filter
";
$items = $DB->get($sql);
$total = $items['total'];
$smarty->assign("pagination", $DB->show_pagination($total, $page, $_SESSION['itemsPerPage']));
$output = $this->display(__FILE__, 'latestwallpapers.tpl');
}
return $output;
}

Answer 1

What I think is happening is that
$_SESSION['itemsPerPage']
is null and when multiplying by $page - 1 is being converted to 0 so it's evaluating to 0 in the query, but when concatenated at the end is just producing nothing which is causing the syntax error in the LIMIT clause:
DESC LIMIT 0,

Answer 2

If you have a safe way of creating $filter that prevents SQL injection, then try changing it to:
$sql = "SELECT a.* FROM
`".DB_PREFIX."items` a WHERE 1=1 $filter
ORDER BY `created` DESC
LIMIT ".($_SESSION['itemsPerPage']*($page-1)).",".($_SESSION['itemsPerPage']+0);
If your query ends with LIMIT 0 then it won't return any rows and if it ends LIMIT 0, then it is indeed syntactically incorrect.
You need to make sure that you have a value for the offset.
I have tweaked the query to add zero to the offset, so it should now end with LIMIT 0,0 but beware this won't return any rows. I guess your session variable doesn't have a value.
Why not try var_dump($_SESSION['itemsPerPage']); as suggested by #VMai?
EDIT: I bet you $_SESSION['itemsPerPage'] is set after the first time you reference it. Hence the first time you load the page it fails, then when you refresh, that value is there and it works.