chore: fix media type option

Author: appflowy

collab-database/src/fields/type_option/date_type_option.rs

@@ -101,9 +101,8 @@ impl TypeOptionCellReader for DateTypeOption {
     }
   }
 
-  fn numeric_cell(&self, cell: &Cell) -> Option<f64> {
-    let cell_data = DateCellData::from(cell);
-    cell_data.timestamp.map(|timestamp| timestamp as f64)
+  fn numeric_cell(&self, _cell: &Cell) -> Option<f64> {
+    None
   }
 
   fn convert_raw_cell_data(&self, text: &str) -> String {

collab-database/src/fields/type_option/media_type_option.rs

@@ -43,13 +43,10 @@ impl TypeOptionCellReader for MediaTypeOption {
   }
 
   fn convert_raw_cell_data(&self, text: &str) -> String {
-    let data = MediaCellData::from(text.to_string());
-    data
-      .files
-      .into_iter()
-      .map(|file| file.name)
-      .collect::<Vec<_>>()
-      .join(", ")
+    match serde_json::from_str::<MediaCellData>(text) {
+      Ok(value) => value.to_string(),
+      Err(_) => "".to_string(),
+    }
   }
 }
 
@@ -136,21 +133,6 @@ impl From<MediaCellData> for Cell {
   }
 }
 
-impl From<String> for MediaCellData {
-  fn from(s: String) -> Self {
-    if s.is_empty() {
-      return MediaCellData { files: vec![] };
-    }
-
-    let files = s
-      .split(", ")
-      .map(|file: &str| serde_json::from_str::<MediaFile>(file).unwrap_or_default())
-      .collect::<Vec<_>>();
-
-    MediaCellData { files }
-  }
-}
-
 impl ToString for MediaCellData {
   fn to_string(&self) -> String {
     self

collab-database/src/fields/type_option/mod.rs

@@ -143,6 +143,7 @@ pub trait TypeOptionCellReader {
   }
 
   /// Returns the numeric value of the cell. If the value is not numeric, returns `None`.
+  /// Currently, it's used to calculate the sum of the numeric cell values.
   fn numeric_cell(&self, cell: &Cell) -> Option<f64>;
 
   /// Convert the value stored in given key:[CELL_DATA] into a readable text