Skip to content

Commit a3f5833

Browse files
authored
models/krate: Remove PaginationOptions dependency (#10556)
1 parent fbec41b commit a3f5833

File tree

2 files changed

+6
-7
lines changed

2 files changed

+6
-7
lines changed

src/controllers/krate/rev_deps.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,9 @@ pub async fn list_reverse_dependencies(
3030

3131
let krate = path.load_crate(&mut conn).await?;
3232

33-
let (rev_deps, total) = krate
34-
.reverse_dependencies(&mut conn, pagination_options)
35-
.await?;
33+
let offset = pagination_options.offset().unwrap_or_default();
34+
let limit = pagination_options.per_page;
35+
let (rev_deps, total) = krate.reverse_dependencies(&mut conn, offset, limit).await?;
3636

3737
let rev_deps: Vec<_> = rev_deps
3838
.into_iter()

src/models/krate.rs

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ use diesel_async::{AsyncConnection, AsyncPgConnection, RunQueryDsl};
99
use secrecy::SecretString;
1010
use thiserror::Error;
1111

12-
use crate::controllers::helpers::pagination::*;
1312
use crate::models::helpers::with_count::*;
1413
use crate::models::version::TopVersions;
1514
use crate::models::{
@@ -491,17 +490,17 @@ impl Crate {
491490
pub(crate) async fn reverse_dependencies(
492491
&self,
493492
conn: &mut AsyncPgConnection,
494-
options: PaginationOptions,
493+
offset: i64,
494+
limit: i64,
495495
) -> QueryResult<(Vec<ReverseDependency>, i64)> {
496496
use diesel::sql_query;
497497
use diesel::sql_types::{BigInt, Integer};
498498

499-
let offset = options.offset().unwrap_or_default();
500499
let rows: Vec<WithCount<ReverseDependency>> =
501500
sql_query(include_str!("krate_reverse_dependencies.sql"))
502501
.bind::<Integer, _>(self.id)
503502
.bind::<BigInt, _>(offset)
504-
.bind::<BigInt, _>(options.per_page)
503+
.bind::<BigInt, _>(limit)
505504
.load(conn)
506505
.await?;
507506

0 commit comments

Comments
 (0)