models::dependency: Move encodable() method to EncodableDependency

Author: Turbo87

src/models/dependency.rs

@@ -5,7 +5,7 @@ use crate::util::errors::{cargo_err, AppResult};
 
 use crate::models::{Crate, Version};
 use crate::schema::*;
-use crate::views::{EncodableCrateDependency, EncodableDependency};
+use crate::views::EncodableCrateDependency;
 
 pub const WILDCARD_ERROR_MESSAGE: &str = "wildcard (`*`) dependency constraints are not allowed \
      on crates.io. See https://doc.rust-lang.org/cargo/faq.html#can-\
@@ -49,24 +49,6 @@ pub enum DependencyKind {
     // if you add a kind here, be sure to update `from_row` below.
 }
 
-impl Dependency {
-    // `downloads` need only be specified when generating a reverse dependency
-    pub fn encodable(self, crate_name: &str, downloads: Option<i32>) -> EncodableDependency {
-        EncodableDependency {
-            id: self.id,
-            version_id: self.version_id,
-            crate_id: crate_name.into(),
-            req: self.req,
-            optional: self.optional,
-            default_features: self.default_features,
-            features: self.features,
-            target: self.target,
-            kind: self.kind,
-            downloads: downloads.unwrap_or(0),
-        }
-    }
-}
-
 pub fn add_dependencies(
     conn: &PgConnection,
     deps: &[EncodableCrateDependency],

src/views.rs

@@ -98,12 +98,28 @@ pub struct EncodableDependency {
 
 impl EncodableDependency {
     pub fn from_dep(dependency: Dependency, crate_name: &str) -> Self {
-        dependency.encodable(crate_name, None)
+        Self::encode(dependency, 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))
+        Self::encode(dependency, crate_name, Some(rev_dep.crate_downloads))
+    }
+
+    // `downloads` need only be specified when generating a reverse dependency
+    fn encode(dependency: Dependency, crate_name: &str, downloads: Option<i32>) -> Self {
+        Self {
+            id: dependency.id,
+            version_id: dependency.version_id,
+            crate_id: crate_name.into(),
+            req: dependency.req,
+            optional: dependency.optional,
+            default_features: dependency.default_features,
+            features: dependency.features,
+            target: dependency.target,
+            kind: dependency.kind,
+            downloads: downloads.unwrap_or(0),
+        }
     }
 }