Skip to content

Commit d43887b

Browse files
committed
correctly filter by 90 day for top downloads
1 parent e90a6be commit d43887b

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

src/controllers/krate/downloads.rs

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,16 +34,17 @@ pub async fn downloads(state: AppState, Path(crate_name): Path<String>) -> AppRe
3434
.load(&mut conn)
3535
.await?;
3636

37-
let top_downloaded_versions: Vec<(i32,)> = VersionDownload::belonging_to(&versions)
37+
let top_downloaded_versions: Vec<i32> = VersionDownload::belonging_to(&versions)
38+
.filter(version_downloads::date.gt(date(now - 90.days())))
3839
.group_by(version_downloads::version_id)
39-
.select((version_downloads::version_id,))
40+
.select(version_downloads::version_id)
4041
.order(sum(version_downloads::downloads).desc())
4142
.limit(5)
4243
.load(&mut conn)
4344
.await?;
4445
let (top_five, rest): (Vec<_>, Vec<_>) = versions
4546
.iter()
46-
.partition(|v| top_downloaded_versions.contains(&(v.id,)));
47+
.partition(|v| top_downloaded_versions.contains(&v.id));
4748

4849
let downloads = VersionDownload::belonging_to(&top_five)
4950
.filter(version_downloads::date.gt(date(now - 90.days())))

0 commit comments

Comments
 (0)