GS-10658 Update saving posts logic (#2405)

Co-authored-by: nickdpicnic <[email protected]>
rmzn3737 · Dec 8, 2023 · d8656b8 · d8656b8
1 parent 239647a
commit d8656b8
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 4 deletions.
diff --git a/lib/features/posts/comment_chat/comment_chat_presentation_model.dart b/lib/features/posts/comment_chat/comment_chat_presentation_model.dart
@@ -259,7 +259,7 @@ class CommentChatPresentationModel implements CommentChatViewModel {
       );
 
   CommentChatPresentationModel byUpdatingSavedStatus({required bool saved}) => copyWith(
-        feedPost: feedPost.copyWith(context: feedPost.context.copyWith(saved: saved)),
+        feedPost: feedPost.byUpdatingSavedStatus(iSaved: saved),
       );
 
   CommentChatPresentationModel byUpdatingAuthorWithFollow({required bool follow}) {

diff --git a/lib/features/posts/comment_chat/comment_chat_presenter.dart b/lib/features/posts/comment_chat/comment_chat_presenter.dart
@@ -206,19 +206,31 @@ class CommentChatPresenter extends Cubit<CommentChatViewModel> with Subscription
         target: AnalyticsTapTarget.postBookmarkButton,
       ),
     );
+
+    final previousState = _model.feedPost.context.saved;
+
+    void emitStatus({required bool saved}) {
+      _emitAndNotify(_model.byUpdatingSavedStatus(saved: saved));
+      _model.onPostUpdatedCallback?.call(_model.feedPost);
+    }
+
+    emitStatus(saved: !previousState);
+
     _savePostToCollectionUseCase
         .execute(
           input: SavePostInput(
             postId: _model.feedPost.id,
-            save: !_model.feedPost.context.saved,
+            save: !previousState,
           ),
         )
         .observeStatusChanges(
           (result) => tryEmit(_model.copyWith(savingPostResult: result)),
         )
         .doOn(
-          success: (post) => tryEmit(_model.byUpdatingSavedStatus(saved: post.context.saved)),
-          fail: (fail) => navigator.showError(fail.displayableFailure()),
+          success: (post) {
+            emitStatus(saved: post.context.saved);
+          },
+          fail: (fail) => emitStatus(saved: previousState),
         );
   }