chore: fix some bugs

frontend/app_flowy/lib/startup/deps_resolver.dart

@@ -157,9 +157,10 @@ void _resolveGridDeps(GetIt getIt) {
     ),
   );
 
-  getIt.registerFactoryParam<GridHeaderBloc, String, List<Field>>(
-    (gridId, fields) => GridHeaderBloc(
-      data: GridHeaderData(gridId: gridId, fields: fields),
+  getIt.registerFactoryParam<GridHeaderBloc, String, GridFieldCache>(
+    (gridId, fieldCache) => GridHeaderBloc(
+      gridId: gridId,
+      fieldCache: fieldCache,
     ),
   );
 

frontend/app_flowy/lib/startup/tasks/app_widget.dart

@@ -113,7 +113,7 @@ class ApplicationBlocObserver extends BlocObserver {
   // ignore: unnecessary_overrides
   void onTransition(Bloc bloc, Transition transition) {
     // Log.debug("[current]: ${transition.currentState} \n\n[next]: ${transition.nextState}");
-    // Log.debug("${transition.nextState}");
+    Log.debug("${transition.nextState}");
     super.onTransition(bloc, transition);
   }
 
@@ -123,9 +123,9 @@ class ApplicationBlocObserver extends BlocObserver {
     super.onError(bloc, error, stackTrace);
   }
 
-  // @override
-  // void onEvent(Bloc bloc, Object? event) {
-  //   Log.debug("$event");
-  //   super.onEvent(bloc, event);
-  // }
+  @override
+  void onEvent(Bloc bloc, Object? event) {
+    // Log.debug("$event");
+    super.onEvent(bloc, event);
+  }
 }

frontend/app_flowy/lib/workspace/application/grid/cell_bloc/date_cell_bloc.dart

@@ -62,7 +62,11 @@ class DateCellBloc extends Bloc<DateCellEvent, DateCellState> {
 
     _fieldListener.updateFieldNotifier.addPublishListener((result) {
       result.fold(
-        (field) => add(DateCellEvent.didReceiveFieldUpdate(field)),
+        (field) {
+          if (!isClosed) {
+            add(DateCellEvent.didReceiveFieldUpdate(field));
+          }
+        },
         (err) => Log.error(err),
       );
     });

frontend/app_flowy/lib/workspace/application/grid/cell_bloc/selection_editor_bloc.dart

@@ -144,7 +144,11 @@ class SelectOptionEditorBloc extends Bloc<SelectOptionEditorEvent, SelectOptionE
 
     _fieldListener.updateFieldNotifier.addPublishListener((result) {
       result.fold(
-        (field) => add(SelectOptionEditorEvent.didReceiveFieldUpdate(field)),
+        (field) {
+          if (!isClosed) {
+            add(SelectOptionEditorEvent.didReceiveFieldUpdate(field));
+          }
+        },
         (err) => Log.error(err),
       );
     });

frontend/app_flowy/lib/workspace/application/grid/data.dart

@@ -1,8 +0,0 @@
-import 'package:flowy_sdk/protobuf/flowy-grid-data-model/grid.pb.dart';
-
-class GridHeaderData {
-  final String gridId;
-  final List<Field> fields;
-
-  GridHeaderData({required this.gridId, required this.fields});
-}

frontend/app_flowy/lib/workspace/application/grid/field/field_cell_bloc.dart

@@ -47,7 +47,11 @@ class FieldCellBloc extends Bloc<FieldCellEvent, FieldCellState> {
   void _startListening() {
     _fieldListener.updateFieldNotifier.addPublishListener((result) {
       result.fold(
-        (field) => add(FieldCellEvent.didReceiveFieldUpdate(field)),
+        (field) {
+          if (!isClosed) {
+            add(FieldCellEvent.didReceiveFieldUpdate(field));
+          }
+        },
         (err) => Log.error(err),
       );
     });

frontend/app_flowy/lib/workspace/application/grid/grid_bloc.dart

@@ -41,10 +41,7 @@ class GridBloc extends Bloc<GridEvent, GridState> {
             emit(state.copyWith(rows: value.rows, listState: value.listState));
           },
           didReceiveFieldUpdate: (_DidReceiveFieldUpdate value) {
-            emit(state.copyWith(
-              rows: _rowCache.rows,
-              fields: value.fields,
-            ));
+            emit(state.copyWith(rows: _rowCache.rows, fields: value.fields));
           },
         );
       },
@@ -56,19 +53,17 @@ class GridBloc extends Bloc<GridEvent, GridState> {
     await _gridService.closeGrid();
     await _fieldListener.stop();
     await _gridListener.stop();
+    fieldCache.dispose();
     return super.close();
   }
 
   void _startListening() {
-    fieldCache.addListener((fields) {
-      _rowCache.updateFields(fields);
-    });
-
     _fieldListener.updateFieldsNotifier.addPublishListener((result) {
       result.fold(
         (changeset) {
           fieldCache.applyChangeset(changeset);
-          add(GridEvent.didReceiveFieldUpdate(List.from(fieldCache.fields)));
+          _rowCache.updateFields(fieldCache.unmodifiableFields);
+          add(GridEvent.didReceiveFieldUpdate(fieldCache.clonedFields));
         },
         (err) => Log.error(err),
       );
@@ -111,12 +106,12 @@ class GridBloc extends Bloc<GridEvent, GridState> {
     return Future(
       () => result.fold(
         (fields) {
-          fieldCache.fields = fields.items;
-          _rowCache.updateWithBlock(grid.blockOrders);
+          fieldCache.clonedFields = fields.items;
+          _rowCache.updateWithBlock(grid.blockOrders, fieldCache.unmodifiableFields);
 
           emit(state.copyWith(
             grid: Some(grid),
-            fields: fieldCache.fields,
+            fields: fieldCache.clonedFields,
             rows: _rowCache.rows,
             loadingState: GridLoadingState.finish(left(unit)),
           ));

frontend/app_flowy/lib/workspace/application/grid/grid_header_bloc.dart

@@ -1,6 +1,3 @@
-import 'package:app_flowy/workspace/application/grid/data.dart';
-import 'package:app_flowy/workspace/application/grid/field/grid_listenr.dart';
-import 'package:flowy_sdk/log.dart';
 import 'package:flowy_sdk/protobuf/flowy-grid-data-model/grid.pb.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:freezed_annotation/freezed_annotation.dart';
@@ -12,17 +9,13 @@ part 'grid_header_bloc.freezed.dart';
 
 class GridHeaderBloc extends Bloc<GridHeaderEvent, GridHeaderState> {
   final FieldService _fieldService;
-  final GridFieldCache _fieldCache;
-  final GridFieldsListener _fieldListener;
+  final GridFieldCache fieldCache;
 
   GridHeaderBloc({
-    required GridHeaderData data,
-  })  : _fieldListener = GridFieldsListener(gridId: data.gridId),
-        _fieldService = FieldService(gridId: data.gridId),
-        _fieldCache = GridFieldCache(),
-        super(GridHeaderState.initial(data.fields)) {
-    _fieldCache.fields = data.fields;
-
+    required String gridId,
+    required this.fieldCache,
+  })  : _fieldService = FieldService(gridId: gridId),
+        super(GridHeaderState.initial(fieldCache.clonedFields)) {
     on<GridHeaderEvent>(
       (event, emit) async {
         await event.map(
@@ -30,7 +23,6 @@ class GridHeaderBloc extends Bloc<GridHeaderEvent, GridHeaderState> {
             _startListening();
           },
           didReceiveFieldUpdate: (_DidReceiveFieldUpdate value) {
-            value.fields.retainWhere((field) => field.visibility);
             emit(state.copyWith(fields: value.fields));
           },
         );
@@ -39,22 +31,15 @@ class GridHeaderBloc extends Bloc<GridHeaderEvent, GridHeaderState> {
   }
 
   Future<void> _startListening() async {
-    _fieldListener.updateFieldsNotifier.addPublishListener((result) {
-      result.fold(
-        (changeset) {
-          _fieldCache.applyChangeset(changeset);
-          add(GridHeaderEvent.didReceiveFieldUpdate(List.from(_fieldCache.fields)));
-        },
-        (err) => Log.error(err),
-      );
+    fieldCache.listenOnFieldChanged((fields) {
+      if (!isClosed) {
+        add(GridHeaderEvent.didReceiveFieldUpdate(fields));
+      }
     });
-
-    _fieldListener.start();
   }
 
   @override
   Future<void> close() async {
-    await _fieldListener.stop();
     return super.close();
   }
 }
@@ -70,7 +55,8 @@ class GridHeaderState with _$GridHeaderState {
   const factory GridHeaderState({required List<Field> fields}) = _GridHeaderState;
 
   factory GridHeaderState.initial(List<Field> fields) {
-    fields.retainWhere((field) => field.visibility);
+    // final List<Field> newFields = List.from(fields);
+    // newFields.retainWhere((field) => field.visibility);
     return GridHeaderState(fields: fields);
   }
 }

frontend/app_flowy/lib/workspace/application/grid/grid_service.dart

@@ -62,25 +62,23 @@ class GridFieldCache {
     _updateFields(changeset.updatedFields);
   }
 
-  List<Field> get fields => _fieldNotifier.fields;
+  UnmodifiableListView<Field> get unmodifiableFields => UnmodifiableListView(_fieldNotifier.fields);
 
-  set fields(List<Field> fields) {
-    _fieldNotifier.fields = fields;
-  }
+  List<Field> get clonedFields => [..._fieldNotifier.fields];
 
-  set onFieldChanged(void Function(List<Field>) onChanged) {
-    _fieldNotifier.addListener(() => onChanged(fields));
+  set clonedFields(List<Field> fields) {
+    _fieldNotifier.fields = [...fields];
   }
 
-  void addListener(void Function(List<Field>) onFieldChanged) {
-    _fieldNotifier.addListener(() => onFieldChanged(fields));
+  void listenOnFieldChanged(void Function(List<Field>) onFieldChanged) {
+    _fieldNotifier.addListener(() => onFieldChanged(clonedFields));
   }
 
   void _removeFields(List<FieldOrder> deletedFields) {
     if (deletedFields.isEmpty) {
       return;
     }
-    final List<Field> fields = List.from(_fieldNotifier.fields);
+    final List<Field> fields = _fieldNotifier.fields;
     final Map<String, FieldOrder> deletedFieldMap = {
       for (var fieldOrder in deletedFields) fieldOrder.fieldId: fieldOrder
     };
@@ -93,7 +91,7 @@ class GridFieldCache {
     if (insertedFields.isEmpty) {
       return;
     }
-    final List<Field> fields = List.from(_fieldNotifier.fields);
+    final List<Field> fields = _fieldNotifier.fields;
     for (final indexField in insertedFields) {
       if (fields.length > indexField.index) {
         fields.removeAt(indexField.index);
@@ -109,7 +107,7 @@ class GridFieldCache {
     if (updatedFields.isEmpty) {
       return;
     }
-    final List<Field> fields = List.from(_fieldNotifier.fields);
+    final List<Field> fields = _fieldNotifier.fields;
     for (final updatedField in updatedFields) {
       final index = fields.indexWhere((field) => field.id == updatedField.id);
       if (index != -1) {
@@ -119,24 +117,29 @@ class GridFieldCache {
     }
     _fieldNotifier.fields = fields;
   }
+
+  void dispose() {
+    _fieldNotifier.dispose();
+  }
 }
 
 class GridRowCache {
   final String gridId;
-  List<Field> _fields = [];
+  UnmodifiableListView<Field> _fields = UnmodifiableListView([]);
   List<RowData> _rows = [];
 
   GridRowCache({required this.gridId});
 
   List<RowData> get rows => _rows;
 
-  void updateWithBlock(List<GridBlockOrder> blocks) {
+  void updateWithBlock(List<GridBlockOrder> blocks, UnmodifiableListView<Field> fields) {
+    _fields = fields;
     _rows = blocks.expand((block) => block.rowOrders).map((rowOrder) {
       return RowData.fromBlockRow(gridId, rowOrder, _fields);
     }).toList();
   }
 
-  void updateFields(List<Field> fields) {
+  void updateFields(UnmodifiableListView<Field> fields) {
     if (fields.isEmpty) {
       return;
     }

frontend/app_flowy/lib/workspace/application/grid/prelude.dart

@@ -2,7 +2,6 @@ export 'grid_bloc.dart';
 export 'row/row_bloc.dart';
 export 'row/row_service.dart';
 export 'grid_service.dart';
-export 'data.dart';
 export 'grid_header_bloc.dart';
 
 // Field

frontend/app_flowy/lib/workspace/application/grid/row/row_bloc.dart

@@ -80,9 +80,9 @@ class RowBloc extends Bloc<RowEvent, RowState> {
       );
     });
 
-    _fieldCache.addListener((fields) {
+    _fieldCache.listenOnFieldChanged((fields) {
       if (!isClosed) {
-        add(RowEvent.didReceiveFieldUpdate(fields));
+        // add(RowEvent.didReceiveFieldUpdate(fields));
       }
     });
 

frontend/app_flowy/lib/workspace/presentation/plugins/grid/src/grid_page.dart

@@ -77,7 +77,7 @@ class FlowyGrid extends StatelessWidget {
         final child = _wrapScrollView(
           state.fields,
           [
-            _GridHeader(gridId: state.gridId, fields: List.from(state.fields)),
+            _GridHeader(gridId: state.gridId, fields: state.fields),
             _GridRows(),
             const _GridFooter(),
           ],
@@ -150,7 +150,8 @@ class _GridHeader extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
-    return GridHeaderSliverAdaptor(gridId: gridId, fields: fields);
+    final fieldCache = context.read<GridBloc>().fieldCache;
+    return GridHeaderSliverAdaptor(gridId: gridId, fieldCache: fieldCache);
   }
 }
 

frontend/app_flowy/lib/workspace/presentation/plugins/grid/src/widgets/header/grid_header.dart

@@ -14,18 +14,19 @@ import 'field_cell.dart';
 
 class GridHeaderSliverAdaptor extends StatelessWidget {
   final String gridId;
-  final List<Field> fields;
+  final GridFieldCache fieldCache;
 
-  const GridHeaderSliverAdaptor({required this.gridId, required this.fields, Key? key}) : super(key: key);
+  const GridHeaderSliverAdaptor({required this.gridId, required this.fieldCache, Key? key}) : super(key: key);
 
   @override
   Widget build(BuildContext context) {
     return BlocProvider(
-      create: (context) => getIt<GridHeaderBloc>(param1: gridId, param2: fields)..add(const GridHeaderEvent.initial()),
+      create: (context) =>
+          getIt<GridHeaderBloc>(param1: gridId, param2: fieldCache)..add(const GridHeaderEvent.initial()),
       child: BlocBuilder<GridHeaderBloc, GridHeaderState>(
         builder: (context, state) {
           return SliverPersistentHeader(
-            delegate: SliverHeaderDelegateImplementation(gridId: gridId, fields: fields),
+            delegate: SliverHeaderDelegateImplementation(gridId: gridId, fields: state.fields),
             floating: true,
             pinned: true,
           );
@@ -77,6 +78,7 @@ class _GridHeader extends StatelessWidget {
     return BlocBuilder<GridHeaderBloc, GridHeaderState>(
       builder: (context, state) {
         final cells = state.fields
+            .where((field) => field.visibility)
             .map((field) => GridFieldCellContext(gridId: gridId, field: field))
             .map((ctx) => GridFieldCell(ctx, key: ValueKey(ctx.field.id)))
             .toList();