A few more fixups

Author: jtgeibel

src/controllers/crate_owner_invitation.rs

@@ -125,7 +125,7 @@ fn prepare_list(
             crate_owner_invitations::invited_user_id,
         ))
         // We fetch one element over the page limit to then detect whether there is a next page.
-        .limit(pagination.per_page as i64 + 1);
+        .limit(pagination.per_page + 1);
 
     // Load and paginate the results.
     let mut raw_invitations: Vec<CrateOwnerInvitation> = match pagination.page {

src/controllers/helpers/pagination.rs

@@ -236,7 +236,7 @@ where
         out.push_sql(") t LIMIT ");
         out.push_bind_param::<BigInt, _>(&self.options.per_page)?;
         if let Some(offset) = self.options.offset() {
-            out.push_sql(format!(" OFFSET {}", offset).as_str());
+            out.push_sql(format!(" OFFSET {offset}").as_str());
         }
         Ok(())
     }

src/controllers/krate/search.rs

@@ -263,7 +263,7 @@ pub fn search(req: &mut dyn RequestExt) -> EndpointResult {
     let (total, next_page, prev_page, data, conn) = if supports_seek && !explicit_page {
         // Equivalent of:
         // `WHERE name > (SELECT name FROM crates WHERE id = $1) LIMIT $2`
-        query = query.limit(pagination.per_page as i64);
+        query = query.limit(pagination.per_page);
         if let Some(seek) = seek {
             let crate_name: String = crates::table
                 .find(seek)

src/models/token/scopes.rs

@@ -6,7 +6,7 @@ use diesel::sql_types::Text;
 use std::io::Write;
 
 #[derive(Clone, Copy, Debug, PartialEq, Eq, AsExpression)]
-#[sql_type = "Text"]
+#[diesel(sql_type = Text)]
 pub enum EndpointScope {
     PublishNew,
     PublishUpdate,