Merge pull request #782 from Ocramius/hotfix/sqlite-offset-with-no-limit-support

Fix: SQLite offset with no limit support

lib/Doctrine/DBAL/Platforms/SqlitePlatform.php

@@ -657,6 +657,18 @@ class SqlitePlatform extends AbstractPlatform
         return $sql;
     }
 
+    /**
+     * {@inheritDoc}
+     */
+    protected function doModifyLimitQuery($query, $limit, $offset)
+    {
+        if (null === $limit && null !== $offset) {
+            return $query . ' LIMIT -1 OFFSET ' . $offset;
+        }
+
+        return parent::doModifyLimitQuery($query, $limit, $offset);
+    }
+
     /**
      * {@inheritDoc}
      */

tests/Doctrine/Tests/DBAL/Platforms/SqlitePlatformTest.php

@@ -288,6 +288,12 @@ class SqlitePlatformTest extends AbstractPlatformTestCase
         $this->assertEquals('SELECT * FROM user LIMIT 10', $sql);
     }
 
+    public function testModifyLimitQueryWithOffsetAndEmptyLimit()
+    {
+        $sql = $this->_platform->modifyLimitQuery('SELECT * FROM user', null, 10);
+        $this->assertEquals('SELECT * FROM user LIMIT -1 OFFSET 10', $sql);
+    }
+
     public function getGenerateAlterTableSql()
     {
         return array(