diff options
author | Carlos Tadeu Panato Junior <ctadeu@gmail.com> | 2016-11-17 15:25:11 +0100 |
---|---|---|
committer | enahum <nahumhbl@gmail.com> | 2016-11-17 11:25:11 -0300 |
commit | 091e5b8e03d6e0f107a0f70e1d918b753c52ae20 (patch) | |
tree | e82e3669ffc37f3d53905669a8125144bbcd9c9f /webapp | |
parent | be74ec22414a9a34f458a0fb30ea14e2d02dcd3f (diff) | |
download | chat-091e5b8e03d6e0f107a0f70e1d918b753c52ae20.tar.gz chat-091e5b8e03d6e0f107a0f70e1d918b753c52ae20.tar.bz2 chat-091e5b8e03d6e0f107a0f70e1d918b753c52ae20.zip |
PLT-4572 - Move instances of Client.updateUser() in components to an action (#4571)
* Move instances of Client.updateUser() in components to an action
Diffstat (limited to 'webapp')
-rw-r--r-- | webapp/actions/user_actions.jsx | 18 | ||||
-rw-r--r-- | webapp/components/user_settings/manage_languages.jsx | 5 | ||||
-rw-r--r-- | webapp/components/user_settings/user_settings_general.jsx | 3 |
3 files changed, 22 insertions, 4 deletions
diff --git a/webapp/actions/user_actions.jsx b/webapp/actions/user_actions.jsx index 304d36a62..5d07bed14 100644 --- a/webapp/actions/user_actions.jsx +++ b/webapp/actions/user_actions.jsx @@ -317,6 +317,24 @@ export function autocompleteUsersInTeam(username, success, error) { ); } +export function updateUser(username, success, error) { + Client.updateUser( + username, + (data) => { + if (success) { + success(data); + } + }, + (err) => { + AsyncClient.dispatchError(err, 'updateUser'); + + if (error) { + error(err); + } + } + ); +} + export function generateMfaSecret(success, error) { Client.generateMfaSecret( (data) => { diff --git a/webapp/components/user_settings/manage_languages.jsx b/webapp/components/user_settings/manage_languages.jsx index f4ae79088..4f5eb223d 100644 --- a/webapp/components/user_settings/manage_languages.jsx +++ b/webapp/components/user_settings/manage_languages.jsx @@ -3,13 +3,12 @@ import SettingItemMax from '../setting_item_max.jsx'; -import Client from 'client/web_client.jsx'; import * as I18n from 'i18n/i18n.jsx'; import * as GlobalActions from 'actions/global_actions.jsx'; import Constants from 'utils/constants.jsx'; import {FormattedMessage, FormattedHTMLMessage} from 'react-intl'; - +import {updateUser} from 'actions/user_actions.jsx'; import React from 'react'; export default class ManageLanguage extends React.Component { @@ -42,7 +41,7 @@ export default class ManageLanguage extends React.Component { this.submitUser(user); } submitUser(user) { - Client.updateUser(user, Constants.UserUpdateEvents.LANGUAGE, + updateUser(user, Constants.UserUpdateEvents.LANGUAGE, () => { GlobalActions.newLocalizationSelected(user.locale); }, diff --git a/webapp/components/user_settings/user_settings_general.jsx b/webapp/components/user_settings/user_settings_general.jsx index e794c4d4b..b99ddd16e 100644 --- a/webapp/components/user_settings/user_settings_general.jsx +++ b/webapp/components/user_settings/user_settings_general.jsx @@ -15,6 +15,7 @@ import * as AsyncClient from 'utils/async_client.jsx'; import * as Utils from 'utils/utils.jsx'; import {intlShape, injectIntl, defineMessages, FormattedMessage, FormattedHTMLMessage, FormattedDate} from 'react-intl'; +import {updateUser} from 'actions/user_actions.jsx'; const holders = defineMessages({ usernameReserved: { @@ -187,7 +188,7 @@ class UserSettingsGeneralTab extends React.Component { } submitUser(user, type, emailUpdated) { - Client.updateUser(user, type, + updateUser(user, type, () => { this.updateSection(''); AsyncClient.getMe(); |