summaryrefslogtreecommitdiffstats
path: root/webapp
diff options
context:
space:
mode:
authorSaturnino Abril <saturnino.abril@gmail.com>2017-06-28 01:27:31 +0800
committerGeorge Goldberg <george@gberg.me>2017-06-27 18:27:31 +0100
commite2daa425fac3c976e21ed048b13424a2dc437a20 (patch)
treed0de5de0141577a13b6ee3edd925fa278786ebb5 /webapp
parent9659a6da06852ede9bf6b87c0e39a543e88a5034 (diff)
downloadchat-e2daa425fac3c976e21ed048b13424a2dc437a20.tar.gz
chat-e2daa425fac3c976e21ed048b13424a2dc437a20.tar.bz2
chat-e2daa425fac3c976e21ed048b13424a2dc437a20.zip
fix JS errors in saving preferences - for user's display (including custom theme vector) and advanced settings (#6764)
Diffstat (limited to 'webapp')
-rw-r--r--webapp/actions/user_actions.jsx13
-rw-r--r--webapp/components/user_settings/user_settings_advanced.jsx3
-rw-r--r--webapp/components/user_settings/user_settings_display.jsx3
3 files changed, 3 insertions, 16 deletions
diff --git a/webapp/actions/user_actions.jsx b/webapp/actions/user_actions.jsx
index b28bc71c0..5ebec3efb 100644
--- a/webapp/actions/user_actions.jsx
+++ b/webapp/actions/user_actions.jsx
@@ -391,7 +391,7 @@ export function saveTheme(teamId, theme, cb) {
value: JSON.stringify(theme)
}];
- savePreferences(currentUserId, preference)(dispatch, getState).then(
+ savePreferencesRedux(currentUserId, preference)(dispatch, getState).then(
() => {
onThemeSaved(teamId, theme, cb);
}
@@ -793,17 +793,10 @@ export function loadMyTeamMembers() {
);
}
-export function savePreferences(prefs, success, error) {
+export function savePreferences(prefs, callback) {
const currentUserId = UserStore.getCurrentId();
savePreferencesRedux(currentUserId, prefs)(dispatch, getState).then(
- (data) => {
- if (data && success) {
- success(data);
- } else if (data == null && error) {
- const serverError = getState().requests.preferences.savePreferences.error;
- error({id: serverError.server_error_id, ...serverError});
- }
- }
+ () => callback()
);
}
diff --git a/webapp/components/user_settings/user_settings_advanced.jsx b/webapp/components/user_settings/user_settings_advanced.jsx
index 1e2ae2386..cc5824589 100644
--- a/webapp/components/user_settings/user_settings_advanced.jsx
+++ b/webapp/components/user_settings/user_settings_advanced.jsx
@@ -137,9 +137,6 @@ export default class AdvancedSettingsDisplay extends React.Component {
preferences,
() => {
this.updateSection('');
- },
- (err) => {
- this.setState({serverError: err.message});
}
);
}
diff --git a/webapp/components/user_settings/user_settings_display.jsx b/webapp/components/user_settings/user_settings_display.jsx
index ccc70a108..9311f6b45 100644
--- a/webapp/components/user_settings/user_settings_display.jsx
+++ b/webapp/components/user_settings/user_settings_display.jsx
@@ -94,9 +94,6 @@ export default class UserSettingsDisplay extends React.Component {
savePreferences([timePreference, namePreference, fontPreference, channelDisplayModePreference, messageDisplayPreference, collapseDisplayPreference],
() => {
this.updateSection('');
- },
- (err) => {
- this.setState({serverError: err.message});
}
);
}