Include "versions" array in reverse dependencies query result

Author: Turbo87

src/krate.rs

@@ -1543,19 +1543,33 @@ fn modify_owners(req: &mut Request, add: bool) -> CargoResult<Response> {
 
 /// Handles the `GET /crates/:crate_id/reverse_dependencies` route.
 pub fn reverse_dependencies(req: &mut Request) -> CargoResult<Response> {
+    use diesel::expression::dsl::any;
+
     let name = &req.params()["crate_id"];
     let conn = req.db_conn()?;
     let krate = Crate::by_name(name).first::<Crate>(&*conn)?;
     let (offset, limit) = req.pagination(10, 100)?;
     let (rev_deps, total) = krate.reverse_dependencies(&*conn, offset, limit)?;
-    let rev_deps = rev_deps
+    let rev_deps: Vec<_> = rev_deps
         .into_iter()
         .map(|dep| dep.encodable(&krate.name))
         .collect();
 
+    let version_ids: Vec<i32> = rev_deps.iter().map(|dep| dep.version_id).collect();
+
+    let versions = versions::table
+        .filter(versions::id.eq(any(version_ids)))
+        .inner_join(crates::table)
+        .select((versions::all_columns, crates::name))
+        .load::<(Version, String)>(&*conn)?
+        .into_iter()
+        .map(|(version, krate_name)| version.encodable(&krate_name))
+        .collect();
+
     #[derive(RustcEncodable)]
     struct R {
         dependencies: Vec<EncodableDependency>,
+        versions: Vec<EncodableVersion>,
         meta: Meta,
     }
     #[derive(RustcEncodable)]
@@ -1564,6 +1578,7 @@ pub fn reverse_dependencies(req: &mut Request) -> CargoResult<Response> {
     }
     Ok(req.json(&R {
         dependencies: rev_deps,
+        versions,
         meta: Meta { total: total },
     }))
 }

src/tests/krate.rs

@@ -60,6 +60,7 @@ struct Deps {
 #[derive(RustcDecodable)]
 struct RevDeps {
     dependencies: Vec<EncodableDependency>,
+    versions: Vec<EncodableVersion>,
     meta: CrateMeta,
 }
 #[derive(RustcDecodable)]
@@ -1636,6 +1637,9 @@ fn reverse_dependencies() {
     assert_eq!(deps.dependencies.len(), 1);
     assert_eq!(deps.meta.total, 1);
     assert_eq!(deps.dependencies[0].crate_id, "c1");
+    assert_eq!(deps.versions.len(), 1);
+    assert_eq!(deps.versions[0].krate, "c2");
+    assert_eq!(deps.versions[0].num, "1.1.0");
 
     // c1 has no dependent crates.
     req.with_path("/api/v1/crates/c2/reverse_dependencies");