feat: sort cell (#1593)

* chore: call cell decode data

* chore: cache cell decoded data

* chore: update cache cell data

* chore: cache cell data

* refactor: separate cell type option functionalities

* refactor: add TypeOptionCellDataFilter trait

* chore: remove unused codes

* chore: fix wanrings

* chore: add sort tests

* chore: sort single select and multi select

Co-authored-by: nathan <[email protected]>

Author: appflowy

frontend/app_flowy/lib/plugins/grid/application/cell/cell_service/cell_data_persistence.dart

@@ -63,7 +63,7 @@ class DateCellDataPersistence
 
 CellPathPB _makeCellPath(GridCellIdentifier cellId) {
   return CellPathPB.create()
-    ..gridId = cellId.gridId
+    ..viewId = cellId.gridId
     ..fieldId = cellId.fieldId
     ..rowId = cellId.rowId;
 }

frontend/app_flowy/lib/plugins/grid/application/cell/cell_service/cell_service.dart

@@ -46,7 +46,7 @@ class CellService {
     required GridCellIdentifier cellId,
   }) {
     final payload = CellPathPB.create()
-      ..gridId = cellId.gridId
+      ..viewId = cellId.gridId
       ..fieldId = cellId.fieldId
       ..rowId = cellId.rowId;
     return GridEventGetCell(payload).send();

frontend/app_flowy/lib/plugins/grid/application/cell/select_option_service.dart

@@ -23,7 +23,7 @@ class SelectOptionFFIService {
         return result.fold(
           (option) {
             final cellIdentifier = CellPathPB.create()
-              ..gridId = gridId
+              ..viewId = gridId
               ..fieldId = fieldId
               ..rowId = rowId;
             final payload = SelectOptionChangesetPB.create()
@@ -62,7 +62,7 @@ class SelectOptionFFIService {
 
   Future<Either<SelectOptionCellDataPB, FlowyError>> getOptionContext() {
     final payload = CellPathPB.create()
-      ..gridId = gridId
+      ..viewId = gridId
       ..fieldId = fieldId
       ..rowId = rowId;
 
@@ -87,7 +87,7 @@ class SelectOptionFFIService {
 
   CellPathPB _cellIdentifier() {
     return CellPathPB.create()
-      ..gridId = gridId
+      ..viewId = gridId
       ..fieldId = fieldId
       ..rowId = rowId;
   }

frontend/app_flowy/lib/workspace/presentation/widgets/emoji_picker/src/emoji_lists.dart

@@ -2706,7 +2706,7 @@ final Map<String, String> flags = Map.fromIterables([
   'Flag: Armenia',
   'Flag: Angola',
   'Flag: Antarctica',
-  'Flag: Argentina',
+  'Flag: argentina',
   'Flag: American Samoa',
   'Flag: Austria',
   'Flag: Australia',
@@ -2775,7 +2775,7 @@ final Map<String, String> flags = Map.fromIterables([
   'Flag: Falkland Islands',
   'Flag: Micronesia',
   'Flag: Faroe Islands',
-  'Flag: France',
+  'Flag: france',
   'Flag: Gabon',
   'Flag: United Kingdom',
   'Flag: Grenada',

frontend/app_flowy/packages/appflowy_editor_plugins/lib/src/emoji_picker/src/emoji_lists.dart

@@ -2706,7 +2706,7 @@ final Map<String, String> flags = Map.fromIterables([
   'Flag: Armenia',
   'Flag: Angola',
   'Flag: Antarctica',
-  'Flag: Argentina',
+  'Flag: argentina',
   'Flag: American Samoa',
   'Flag: Austria',
   'Flag: Australia',
@@ -2775,7 +2775,7 @@ final Map<String, String> flags = Map.fromIterables([
   'Flag: Falkland Islands',
   'Flag: Micronesia',
   'Flag: Faroe Islands',
-  'Flag: France',
+  'Flag: france',
   'Flag: Gabon',
   'Flag: United Kingdom',
   'Flag: Grenada',

frontend/rust-lib/Cargo.lock

@@ -45,6 +45,7 @@ futures = "0.3.15"
 atomic_refcell = "0.1.8"
 crossbeam-utils = "0.8.7"
 async-stream = "0.3.2"
+parking_lot = "0.12.1"
 
 [dev-dependencies]
 flowy-test = { path = "../flowy-test" }

frontend/rust-lib/flowy-grid/Cargo.toml

@@ -41,7 +41,7 @@ impl TryInto<CreateSelectOptionParams> for CreateSelectOptionPayloadPB {
 #[derive(Debug, Clone, Default, ProtoBuf)]
 pub struct CellPathPB {
     #[pb(index = 1)]
-    pub grid_id: String,
+    pub view_id: String,
 
     #[pb(index = 2)]
     pub field_id: String,
@@ -50,6 +50,8 @@ pub struct CellPathPB {
     pub row_id: String,
 }
 
+/// Represents as the cell identifier. It's used to locate the cell in corresponding
+/// view's row with the field id.
 pub struct CellPathParams {
     pub view_id: String,
     pub field_id: String,
@@ -60,7 +62,7 @@ impl TryInto<CellPathParams> for CellPathPB {
     type Error = ErrorCode;
 
     fn try_into(self) -> Result<CellPathParams, Self::Error> {
-        let grid_id = NotEmptyStr::parse(self.grid_id).map_err(|_| ErrorCode::GridIdIsEmpty)?;
+        let grid_id = NotEmptyStr::parse(self.view_id).map_err(|_| ErrorCode::GridIdIsEmpty)?;
         let field_id = NotEmptyStr::parse(self.field_id).map_err(|_| ErrorCode::FieldIdIsEmpty)?;
         let row_id = NotEmptyStr::parse(self.row_id).map_err(|_| ErrorCode::RowIdIsEmpty)?;
         Ok(CellPathParams {
@@ -70,15 +72,19 @@ impl TryInto<CellPathParams> for CellPathPB {
         })
     }
 }
+
+/// Represents as the data of the cell.
 #[derive(Debug, Default, ProtoBuf)]
 pub struct CellPB {
     #[pb(index = 1)]
     pub field_id: String,
 
-    // The data was encoded in field_type's data type
+    /// Encoded the data using the helper struct `CellProtobufBlob`.
+    /// Check out the `CellProtobufBlob` for more information.
     #[pb(index = 2)]
     pub data: Vec<u8>,
 
+    /// the field_type will be None if the field with field_id is not found
     #[pb(index = 3, one_of)]
     pub field_type: Option<FieldType>,
 }

frontend/rust-lib/flowy-grid/src/entities/cell_entities.rs

@@ -1,3 +1,4 @@
+use crate::services::filter::FromFilterString;
 use flowy_derive::{ProtoBuf, ProtoBuf_Enum};
 use flowy_error::ErrorCode;
 use grid_rev_model::FilterRevision;
@@ -39,6 +40,18 @@ impl std::convert::TryFrom<u8> for CheckboxFilterConditionPB {
     }
 }
 
+impl FromFilterString for CheckboxFilterPB {
+    fn from_filter_rev(filter_rev: &FilterRevision) -> Self
+    where
+        Self: Sized,
+    {
+        CheckboxFilterPB {
+            condition: CheckboxFilterConditionPB::try_from(filter_rev.condition)
+                .unwrap_or(CheckboxFilterConditionPB::IsChecked),
+        }
+    }
+}
+
 impl std::convert::From<&FilterRevision> for CheckboxFilterPB {
     fn from(rev: &FilterRevision) -> Self {
         CheckboxFilterPB {

frontend/rust-lib/flowy-grid/src/entities/filter_entities/checkbox_filter.rs

@@ -1,3 +1,4 @@
+use crate::services::filter::FromFilterString;
 use flowy_derive::{ProtoBuf, ProtoBuf_Enum};
 use flowy_error::ErrorCode;
 use grid_rev_model::FilterRevision;
@@ -39,6 +40,18 @@ impl std::convert::TryFrom<u8> for ChecklistFilterConditionPB {
     }
 }
 
+impl FromFilterString for ChecklistFilterPB {
+    fn from_filter_rev(filter_rev: &FilterRevision) -> Self
+    where
+        Self: Sized,
+    {
+        ChecklistFilterPB {
+            condition: ChecklistFilterConditionPB::try_from(filter_rev.condition)
+                .unwrap_or(ChecklistFilterConditionPB::IsIncomplete),
+        }
+    }
+}
+
 impl std::convert::From<&FilterRevision> for ChecklistFilterPB {
     fn from(rev: &FilterRevision) -> Self {
         ChecklistFilterPB {