Fix pagination code

Author: jtgeibel

src/controllers/helpers/pagination.rs

@@ -17,8 +17,8 @@ use std::str::FromStr;
 use std::sync::Arc;
 
 const MAX_PAGE_BEFORE_SUSPECTED_BOT: u32 = 10;
-const DEFAULT_PER_PAGE: u32 = 10;
-const MAX_PER_PAGE: u32 = 100;
+const DEFAULT_PER_PAGE: i64 = 10;
+const MAX_PER_PAGE: i64 = 100;
 
 #[derive(Debug, Clone, PartialEq, Eq)]
 pub(crate) enum Page {
@@ -30,7 +30,7 @@ pub(crate) enum Page {
 #[derive(Debug, Clone)]
 pub(crate) struct PaginationOptions {
     pub(crate) page: Page,
-    pub(crate) per_page: u32,
+    pub(crate) per_page: i64,
 }
 
 impl PaginationOptions {
@@ -42,9 +42,9 @@ impl PaginationOptions {
         }
     }
 
-    pub(crate) fn offset(&self) -> Option<u32> {
+    pub(crate) fn offset(&self) -> Option<i64> {
         if let Page::Numeric(p) = self.page {
-            Some((p - 1) * self.per_page)
+            Some((p - 1) as i64 * self.per_page)
         } else {
             None
         }
@@ -238,10 +238,9 @@ where
         out.push_sql("SELECT *, COUNT(*) OVER () FROM (");
         self.query.walk_ast(out.reborrow())?;
         out.push_sql(") t LIMIT ");
-        out.push_bind_param::<BigInt, _>((self.options.per_page as &i64))?;
+        out.push_bind_param::<BigInt, _>(&self.options.per_page)?;
         if let Some(offset) = self.options.offset() {
-            out.push_sql(" OFFSET ");
-            out.push_bind_param::<BigInt, _>(&(offset as i64))?;
+            out.push_sql(format!(" OFFSET {}", offset).as_str());
         }
         Ok(())
     }