From 8f0175e15c4d6bd08ca6795851455468811c3dc9 Mon Sep 17 00:00:00 2001 From: Carlos Tadeu Panato Junior Date: Tue, 17 Jan 2017 21:49:58 +0100 Subject: Move instances of Client.updateChannelHeader() in components to an action (#5098) --- webapp/actions/channel_actions.jsx | 22 ++++++++++++++++++++++ webapp/components/edit_channel_header_modal.jsx | 12 +++--------- 2 files changed, 25 insertions(+), 9 deletions(-) diff --git a/webapp/actions/channel_actions.jsx b/webapp/actions/channel_actions.jsx index 6cf8449f0..40e0cde30 100644 --- a/webapp/actions/channel_actions.jsx +++ b/webapp/actions/channel_actions.jsx @@ -321,3 +321,25 @@ export function createChannel(channel, success, error) { } ); } + +export function updateChannelHeader(channelId, header, success, error) { + Client.updateChannelHeader( + channelId, + header, + (channelData) => { + AppDispatcher.handleServerAction({ + type: ActionTypes.RECEIVED_CHANNEL, + channel: channelData + }); + + if (success) { + success(channelData); + } + }, + (err) => { + if (error) { + error(err); + } + } + ); +} diff --git a/webapp/components/edit_channel_header_modal.jsx b/webapp/components/edit_channel_header_modal.jsx index 490b9fb31..0d8eb8acb 100644 --- a/webapp/components/edit_channel_header_modal.jsx +++ b/webapp/components/edit_channel_header_modal.jsx @@ -2,13 +2,12 @@ // See License.txt for license information. import ReactDOM from 'react-dom'; -import AppDispatcher from '../dispatcher/app_dispatcher.jsx'; -import Client from 'client/web_client.jsx'; import Constants from 'utils/constants.jsx'; import * as Utils from 'utils/utils.jsx'; import PreferenceStore from 'stores/preference_store.jsx'; import {intlShape, injectIntl, defineMessages, FormattedMessage} from 'react-intl'; +import {updateChannelHeader} from 'actions/channel_actions.jsx'; import {Modal} from 'react-bootstrap'; @@ -64,17 +63,12 @@ class EditChannelHeaderModal extends React.Component { handleSubmit() { this.setState({submitted: true}); - Client.updateChannelHeader( + updateChannelHeader( this.props.channel.id, this.state.header, - (channel) => { + () => { this.setState({serverError: ''}); this.onHide(); - - AppDispatcher.handleServerAction({ - type: Constants.ActionTypes.RECEIVED_CHANNEL, - channel - }); }, (err) => { if (err.id === 'api.context.invalid_param.app_error') { -- cgit v1.2.3-1-g7c22