From 04f6b0b86cf43ffedd96c59dcce4972831507ce9 Mon Sep 17 00:00:00 2001
From: JoramWilander <jwawilander@gmail.com>
Date: Thu, 25 Feb 2016 14:14:52 -0500
Subject: Fix deleting threads

---
 web/react/components/delete_post_modal.jsx | 51 ++++--------------------------
 web/react/components/rhs_thread.jsx        |  8 +++--
 2 files changed, 12 insertions(+), 47 deletions(-)

diff --git a/web/react/components/delete_post_modal.jsx b/web/react/components/delete_post_modal.jsx
index f8e3e406a..beeb91be0 100644
--- a/web/react/components/delete_post_modal.jsx
+++ b/web/react/components/delete_post_modal.jsx
@@ -21,9 +21,6 @@ export default class DeletePostModal extends React.Component {
         this.handleDelete = this.handleDelete.bind(this);
         this.handleToggle = this.handleToggle.bind(this);
         this.handleHide = this.handleHide.bind(this);
-        this.onListenerChange = this.onListenerChange.bind(this);
-
-        this.selectedList = null;
 
         this.state = {
             show: false,
@@ -35,11 +32,9 @@ export default class DeletePostModal extends React.Component {
 
     componentDidMount() {
         ModalStore.addModalListener(ActionTypes.TOGGLE_DELETE_POST_MODAL, this.handleToggle);
-        PostStore.addSelectedPostChangeListener(this.onListenerChange);
     }
 
     componentWillUnmount() {
-        PostStore.removeSelectedPostChangeListener(this.onListenerChange);
         ModalStore.removeModalListener(ActionTypes.TOGGLE_DELETE_POST_MODAL, this.handleToggle);
     }
 
@@ -56,40 +51,15 @@ export default class DeletePostModal extends React.Component {
             this.state.post.channel_id,
             this.state.post.id,
             () => {
-                var selectedList = this.selectedList;
-
-                if (selectedList && selectedList.order && selectedList.order.length > 0) {
-                    var selectedPost = selectedList.posts[selectedList.order[0]];
-                    if ((selectedPost.id === this.state.post.id && !this.state.root_id) || selectedPost.root_id === this.state.post.id) {
-                        AppDispatcher.handleServerAction({
-                            type: ActionTypes.RECEIVED_SEARCH,
-                            results: null
-                        });
-
-                        AppDispatcher.handleServerAction({
-                            type: ActionTypes.RECEIVED_POST_SELECTED,
-                            postId: null
-                        });
-                    } else if (selectedPost.id === this.state.post.id && this.state.root_id) {
-                        if (selectedPost.root_id && selectedPost.root_id.length > 0 && selectedList.posts[selectedPost.root_id]) {
-                            selectedList.order = [selectedPost.root_id];
-                            delete selectedList.posts[selectedPost.id];
-
-                            AppDispatcher.handleServerAction({
-                                type: ActionTypes.RECEIVED_POST_SELECTED,
-                                postId: selectedPost.root_id
-                            });
-
-                            AppDispatcher.handleServerAction({
-                                type: ActionTypes.RECEIVED_SEARCH,
-                                results: null
-                            });
-                        }
-                    }
-                }
-
                 PostStore.deletePost(this.state.post);
                 AsyncClient.getPosts(this.state.post.channel_id);
+
+                if (this.state.post.id === PostStore.getSelectedPostId()) {
+                    AppDispatcher.handleServerAction({
+                        type: ActionTypes.RECEIVED_POST_SELECTED,
+                        postId: null
+                    });
+                }
             },
             (err) => {
                 AsyncClient.dispatchError(err, 'deletePost');
@@ -112,13 +82,6 @@ export default class DeletePostModal extends React.Component {
         this.setState({show: false});
     }
 
-    onListenerChange() {
-        var newList = PostStore.getSelectedPost();
-        if (!Utils.areObjectsEqual(this.selectedList, newList)) {
-            this.selectedList = newList;
-        }
-    }
-
     render() {
         if (!this.state.post) {
             return null;
diff --git a/web/react/components/rhs_thread.jsx b/web/react/components/rhs_thread.jsx
index 5e782b09b..292624846 100644
--- a/web/react/components/rhs_thread.jsx
+++ b/web/react/components/rhs_thread.jsx
@@ -91,9 +91,11 @@ export default class RhsThread extends React.Component {
         });
     }
     onPostChange() {
-        const selected = PostStore.getSelectedPost();
-        const posts = PostStore.getSelectedPostThread();
-        this.setState({posts, selected});
+        if (this.mounted) {
+            const selected = PostStore.getSelectedPost();
+            const posts = PostStore.getSelectedPostThread();
+            this.setState({posts, selected});
+        }
     }
     onUserChange() {
         const profiles = JSON.parse(JSON.stringify(UserStore.getProfiles()));
-- 
cgit v1.2.3-1-g7c22