fix: avoid updating PostNotifier state if text is not changed (#459)

poppingmoon · Oct 29, 2024 · 1dfc491 · 1dfc491
1 parent 04f1cd7
commit 1dfc491
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/lib/provider/post_notifier_provider.dart b/lib/provider/post_notifier_provider.dart
@@ -282,7 +282,13 @@ class PostNotifier extends _$PostNotifier {
   }
 
   void setCw(String? cw) {
-    state = state.copyWith(cw: cw?.isNotEmpty ?? false ? cw : null);
+    if ((cw == null || cw.isEmpty) && state.cw != null) {
+      state = state.copyWith(cw: null);
+    } else if (cw != state.cw) {
+      state = state.copyWith(cw: cw);
+    } else {
+      return;
+    }
     _scheduleSave();
   }
 
@@ -462,7 +468,13 @@ class PostNotifier extends _$PostNotifier {
   }
 
   void setText(String? text) {
-    state = state.copyWith(text: text?.isNotEmpty ?? false ? text : null);
+    if ((text == null || text.isEmpty) && state.text != null) {
+      state = state.copyWith(text: null);
+    } else if (text != state.text) {
+      state = state.copyWith(text: text);
+    } else {
+      return;
+    }
     _scheduleSave();
   }
 

diff --git a/lib/provider/post_notifier_provider.g.dart b/lib/provider/post_notifier_provider.g.dart