diff options
author | Harrison Healey <harrisonmhealey@gmail.com> | 2017-07-10 17:36:26 -0400 |
---|---|---|
committer | Saturnino Abril <saturnino.abril@gmail.com> | 2017-07-11 05:36:26 +0800 |
commit | d307162d1621a5c422f8da19d8ff94b16e80abf0 (patch) | |
tree | ec94251714e87ef936fa36085e24d04887f2e9ab /webapp/actions/post_actions.jsx | |
parent | 0c910cc8c5a5487a129663431c6cd4d52640bd66 (diff) | |
download | chat-d307162d1621a5c422f8da19d8ff94b16e80abf0.tar.gz chat-d307162d1621a5c422f8da19d8ff94b16e80abf0.tar.bz2 chat-d307162d1621a5c422f8da19d8ff94b16e80abf0.zip |
PLT-7039 Use loadProfilesAndStatusesForPosts from the redux library (Resubmit) (#6892)
* PLT-7039 Use loadProfilesAndStatusesForPosts from the redux library
* Switched mattermost-redux to webapp-4.0 branch
Diffstat (limited to 'webapp/actions/post_actions.jsx')
-rw-r--r-- | webapp/actions/post_actions.jsx | 30 |
1 files changed, 4 insertions, 26 deletions
diff --git a/webapp/actions/post_actions.jsx b/webapp/actions/post_actions.jsx index a32fee616..f319d59e3 100644 --- a/webapp/actions/post_actions.jsx +++ b/webapp/actions/post_actions.jsx @@ -21,7 +21,6 @@ import store from 'stores/redux_store.jsx'; const dispatch = store.dispatch; const getState = store.getState; -import {getProfilesByIds} from 'mattermost-redux/actions/users'; import * as PostActions from 'mattermost-redux/actions/posts'; import {getMyChannelMember} from 'mattermost-redux/actions/channels'; @@ -57,7 +56,7 @@ function completePostReceive(post, websocketMessageProps) { PostActions.getPostThread(post.root_id)(dispatch, getState).then( (data) => { dispatchPostActions(post, websocketMessageProps); - loadProfilesForPosts(data.posts); + PostActions.getProfilesAndStatusesForPosts(data.posts, dispatch, getState); } ); @@ -125,7 +124,7 @@ export function getFlaggedPosts() { is_pinned_posts: false }); - loadProfilesForPosts(data.posts); + PostActions.getProfilesAndStatusesForPosts(data.posts, dispatch, getState); } ).catch( () => {} //eslint-disable-line no-empty-function @@ -149,34 +148,13 @@ export function getPinnedPosts(channelId = ChannelStore.getCurrentId()) { is_pinned_posts: true }); - loadProfilesForPosts(data.posts); + PostActions.getProfilesAndStatusesForPosts(data.posts, dispatch, getState); } ).catch( () => {} //eslint-disable-line no-empty-function ); } -export function loadProfilesForPosts(posts) { - const profilesToLoad = {}; - for (const pid in posts) { - if (!posts.hasOwnProperty(pid)) { - continue; - } - - const post = posts[pid]; - if (!UserStore.hasProfile(post.user_id)) { - profilesToLoad[post.user_id] = true; - } - } - - const list = Object.keys(profilesToLoad); - if (list.length === 0) { - return; - } - - getProfilesByIds(list)(dispatch, getState); -} - export function addReaction(channelId, postId, emojiName) { PostActions.addReaction(postId, emojiName)(dispatch, getState); } @@ -260,7 +238,7 @@ export function performSearch(terms, isMentionSearch, success, error) { is_mention_search: isMentionSearch }); - loadProfilesForPosts(data.posts); + PostActions.getProfilesAndStatusesForPosts(data.posts, dispatch, getState); if (success) { success(data); |