models/version: Extract features() fn (#9550)

Author: Turbo87

src/models/krate.rs

@@ -483,8 +483,7 @@ impl Crate {
 
                 deps.sort();
 
-                let features: BTreeMap<String, Vec<String>> =
-                    serde_json::from_value(version.features).unwrap_or_default();
+                let features = version.features().unwrap_or_default();
                 let (features, features2): (BTreeMap<_, _>, BTreeMap<_, _>) =
                     features.into_iter().partition(|(_k, vals)| {
                         !vals

src/models/version.rs

@@ -3,6 +3,7 @@ use std::collections::BTreeMap;
 use chrono::NaiveDateTime;
 use derive_builder::Builder;
 use diesel::prelude::*;
+use serde::Deserialize;
 
 use crate::util::errors::{bad_request, AppResult};
 
@@ -63,6 +64,16 @@ impl Version {
             None => None,
         }
     }
+
+    /// Deserializes the `features` field from JSON into a `BTreeMap`.
+    ///
+    /// # Returns
+    ///
+    /// * `Ok(BTreeMap<String, Vec<String>>)` - If the deserialization was successful.
+    /// * `Err(serde_json::Error)` - If the deserialization failed.
+    pub fn features(&self) -> Result<BTreeMap<String, Vec<String>>, serde_json::Error> {
+        BTreeMap::<String, Vec<String>>::deserialize(&self.features)
+    }
 }
 
 #[derive(Insertable, Debug, Builder)]