views: Implement from_dep/reverse_dep() methods for EncodableDependency

Author: Turbo87

src/controllers/krate/metadata.rs

@@ -247,7 +247,7 @@ pub fn reverse_dependencies(req: &mut dyn RequestExt) -> EndpointResult {
     let (rev_deps, total) = krate.reverse_dependencies(&*conn, pagination_options)?;
     let rev_deps: Vec<_> = rev_deps
         .into_iter()
-        .map(|dep| dep.encodable(&krate.name))
+        .map(|dep| EncodableDependency::from_reverse_dep(dep, &krate.name))
         .collect();
 
     let version_ids: Vec<i32> = rev_deps.iter().map(|dep| dep.version_id).collect();

src/controllers/version/metadata.rs

@@ -26,7 +26,7 @@ pub fn dependencies(req: &mut dyn RequestExt) -> EndpointResult {
     let deps = version.dependencies(&conn)?;
     let deps = deps
         .into_iter()
-        .map(|(dep, crate_name)| dep.encodable(&crate_name, None))
+        .map(|(dep, crate_name)| EncodableDependency::from_dep(dep, &crate_name))
         .collect();
 
     #[derive(Serialize)]

src/models/dependency.rs

@@ -67,13 +67,6 @@ impl Dependency {
     }
 }
 
-impl ReverseDependency {
-    pub fn encodable(self, crate_name: &str) -> EncodableDependency {
-        self.dependency
-            .encodable(crate_name, Some(self.crate_downloads))
-    }
-}
-
 pub fn add_dependencies(
     conn: &PgConnection,
     deps: &[EncodableCrateDependency],

src/views.rs

@@ -3,7 +3,8 @@ use std::collections::HashMap;
 
 use crate::github;
 use crate::models::{
-    Badge, Category, CreatedApiToken, DependencyKind, Keyword, Owner, Team, User, VersionDownload,
+    Badge, Category, CreatedApiToken, Dependency, DependencyKind, Keyword, Owner,
+    ReverseDependency, Team, User, VersionDownload,
 };
 use crate::util::rfc3339;
 
@@ -95,6 +96,17 @@ pub struct EncodableDependency {
     pub downloads: i32,
 }
 
+impl EncodableDependency {
+    pub fn from_dep(dependency: Dependency, crate_name: &str) -> Self {
+        dependency.encodable(crate_name, None)
+    }
+
+    pub fn from_reverse_dep(rev_dep: ReverseDependency, crate_name: &str) -> Self {
+        let dependency = rev_dep.dependency;
+        dependency.encodable(crate_name, Some(rev_dep.crate_downloads))
+    }
+}
+
 #[derive(Serialize, Deserialize, Debug)]
 pub struct EncodableVersionDownload {
     pub version: i32,