@robocoder opened this Issue on December 1st 2010 Contributor

This looks wrong:

// increase limit by offset when visitor paginates
if(!empty($offset))
{
    $limit += (int)$offset;
}
...
    LIMIT ".(int)$limit;

Since it doesn't look like the $offset is set in the current code, this probably won't be a compat buster: remove the += statement, and change the LIMIT clause to:

    LIMIT ".(int)$offset.",".(int)$limit;
@robocoder commented on December 1st 2010 Contributor

Or use:

    LIMIT ".(int)$limit." OFFSET ".(int)$offset;

which is compatible with PostgreSQL syntax.

@robocoder commented on December 1st 2010 Contributor

(In [3389]) fixes #1850

This Issue was closed on December 1st 2010
Powered by GitHub Issue Mirror