diff --git a/web/react/components/user_settings_notifications.jsx b/web/react/components/user_settings_notifications.jsx index 799e4d65e2..33ae01eaac 100644 --- a/web/react/components/user_settings_notifications.jsx +++ b/web/react/components/user_settings_notifications.jsx @@ -9,94 +9,114 @@ var assign = require('object-assign'); function getNotificationsStateFromStores() { var user = UserStore.getCurrentUser(); var soundNeeded = !utils.isBrowserFirefox(); - var sound = (!user.notify_props || user.notify_props.desktop_sound == undefined) ? "true" : user.notify_props.desktop_sound; - var desktop = (!user.notify_props || user.notify_props.desktop == undefined) ? "all" : user.notify_props.desktop; - var email = (!user.notify_props || user.notify_props.email == undefined) ? "true" : user.notify_props.email; - var username_key = false; - var mention_key = false; - var custom_keys = ""; - var first_name_key = false; - var all_key = false; - var channel_key = false; + var sound = 'true'; + if (user.notify_props && user.notify_props.desktop_sound) { + sound = user.notify_props.desktop_sound; + } + var desktop = 'all'; + if (user.notify_props && user.notify_props.desktop) { + desktop = user.notify_props.desktop; + } + var email = 'true'; + if (user.notify_props && user.notify_props.email) { + email = user.notify_props.email; + } + + var usernameKey = false; + var mentionKey = false; + var customKeys = ''; + var firstNameKey = false; + var allKey = false; + var channelKey = false; if (user.notify_props) { - if (user.notify_props.mention_keys !== undefined) { + if (user.notify_props.mention_keys) { var keys = user.notify_props.mention_keys.split(','); if (keys.indexOf(user.username) !== -1) { - username_key = true; + usernameKey = true; keys.splice(keys.indexOf(user.username), 1); } else { - username_key = false; + usernameKey = false; } - if (keys.indexOf('@'+user.username) !== -1) { - mention_key = true; - keys.splice(keys.indexOf('@'+user.username), 1); + if (keys.indexOf('@' + user.username) !== -1) { + mentionKey = true; + keys.splice(keys.indexOf('@' + user.username), 1); } else { - mention_key = false; + mentionKey = false; } - custom_keys = keys.join(','); + customKeys = keys.join(','); } - if (user.notify_props.first_name !== undefined) { - first_name_key = user.notify_props.first_name === "true"; + if (user.notify_props.first_name) { + firstNameKey = user.notify_props.first_name === 'true'; } - if (user.notify_props.all !== undefined) { - all_key = user.notify_props.all === "true"; + if (user.notify_props.all) { + allKey = user.notify_props.all === 'true'; } - if (user.notify_props.channel !== undefined) { - channel_key = user.notify_props.channel === "true"; + if (user.notify_props.channel) { + channelKey = user.notify_props.channel === 'true'; } } - return { notify_level: desktop, enable_email: email, soundNeeded: soundNeeded, enable_sound: sound, username_key: username_key, mention_key: mention_key, custom_keys: custom_keys, custom_keys_checked: custom_keys.length > 0, first_name_key: first_name_key, all_key: all_key, channel_key: channel_key }; + return {notifyLevel: desktop, enableEmail: email, soundNeeded: soundNeeded, enableSound: sound, usernameKey: usernameKey, mentionKey: mentionKey, customKeys: customKeys, customKeysChecked: customKeys.length > 0, firstNameKey: firstNameKey, allKey: allKey, channelKey: channelKey}; } - module.exports = React.createClass({ displayName: 'NotificationsTab', + propTypes: { + user: React.PropTypes.object, + updateSection: React.PropTypes.func, + updateTab: React.PropTypes.func, + activeSection: React.PropTypes.string, + activeTab: React.PropTypes.string + }, handleSubmit: function() { - data = {} - data["user_id"] = this.props.user.id; - data["email"] = this.state.enable_email; - data["desktop_sound"] = this.state.enable_sound; - data["desktop"] = this.state.notify_level; + var data = {}; + data.user_id = this.props.user.id; + data.email = this.state.enableEmail; + data.desktop_sound = this.state.enableSound; + data.desktop = this.state.notifyLevel; - var mention_keys = []; - if (this.state.username_key) mention_keys.push(this.props.user.username); - if (this.state.mention_key) mention_keys.push('@'+this.props.user.username); - - var string_keys = mention_keys.join(','); - if (this.state.custom_keys.length > 0 && this.state.custom_keys_checked) { - string_keys += ',' + this.state.custom_keys; + var mentionKeys = []; + if (this.state.usernameKey) { + mentionKeys.push(this.props.user.username); + } + if (this.state.mentionKey) { + mentionKeys.push('@' + this.props.user.username); } - data["mention_keys"] = string_keys; - data["first_name"] = this.state.first_name_key ? "true" : "false"; - data["all"] = this.state.all_key ? "true" : "false"; - data["channel"] = this.state.channel_key ? "true" : "false"; + var stringKeys = mentionKeys.join(','); + if (this.state.customKeys.length > 0 && this.state.customKeysChecked) { + stringKeys += ',' + this.state.customKeys; + } + + data.mention_keys = stringKeys; + data.first_name = this.state.firstNameKey.toString(); + data.all = this.state.allKey.toString(); + data.channel = this.state.channelKey.toString(); client.updateUserNotifyProps(data, - function(data) { - this.props.updateSection(""); + function success() { + this.props.updateSection(''); AsyncClient.getMe(); }.bind(this), - function(err) { - this.setState({ server_error: err.message }); + function failure(err) { + this.setState({serverError: err.message}); }.bind(this) ); }, handleClose: function() { - $(this.getDOMNode()).find(".form-control").each(function() { - this.value = ""; + $(this.getDOMNode()).find('.form-control').each(function clearField() { + this.value = ''; }); - this.setState(assign({},getNotificationsStateFromStores(),{server_error: null})); + this.setState(assign({}, getNotificationsStateFromStores(), {serverError: null})); this.props.updateTab('general'); }, @@ -105,15 +125,15 @@ module.exports = React.createClass({ this.props.updateSection(section); }, componentDidMount: function() { - UserStore.addChangeListener(this._onChange); + UserStore.addChangeListener(this.onListenerChange); $('#user_settings').on('hidden.bs.modal', this.handleClose); }, componentWillUnmount: function() { - UserStore.removeChangeListener(this._onChange); + UserStore.removeChangeListener(this.onListenerChange); $('#user_settings').off('hidden.bs.modal', this.handleClose); this.props.updateSection(''); }, - _onChange: function() { + onListenerChange: function() { var newState = getNotificationsStateFromStores(); if (!utils.areStatesEqual(newState, this.state)) { this.setState(newState); @@ -123,31 +143,31 @@ module.exports = React.createClass({ return getNotificationsStateFromStores(); }, handleNotifyRadio: function(notifyLevel) { - this.setState({ notify_level: notifyLevel }); + this.setState({notifyLevel: notifyLevel}); this.refs.wrapper.getDOMNode().focus(); }, handleEmailRadio: function(enableEmail) { - this.setState({ enable_email: enableEmail }); + this.setState({enableEmail: enableEmail}); this.refs.wrapper.getDOMNode().focus(); }, handleSoundRadio: function(enableSound) { - this.setState({ enable_sound: enableSound }); + this.setState({enableSound: enableSound}); this.refs.wrapper.getDOMNode().focus(); }, updateUsernameKey: function(val) { - this.setState({ username_key: val }); + this.setState({usernameKey: val}); }, updateMentionKey: function(val) { - this.setState({ mention_key: val }); + this.setState({mentionKey: val}); }, updateFirstNameKey: function(val) { - this.setState({ first_name_key: val }); + this.setState({firstNameKey: val}); }, updateAllKey: function(val) { - this.setState({ all_key: val }); + this.setState({allKey: val}); }, updateChannelKey: function(val) { - this.setState({ channel_key: val }); + this.setState({channelKey: val}); }, updateCustomMentionKeys: function() { var checked = this.refs.customcheck.getDOMNode().checked; @@ -156,9 +176,9 @@ module.exports = React.createClass({ var text = this.refs.custommentions.getDOMNode().value; // remove all spaces and split string into individual keys - this.setState({ custom_keys: text.replace(/ /g, ''), custom_keys_checked: true }); + this.setState({customKeys: text.replace(/ /g, ''), customKeysChecked: true}); } else { - this.setState({ custom_keys: "", custom_keys_checked: false }); + this.setState({customKeys: '', customKeysChecked: false}); } }, onCustomChange: function() { @@ -166,7 +186,10 @@ module.exports = React.createClass({ this.updateCustomMentionKeys(); }, render: function() { - var server_error = this.state.server_error ? this.state.server_error : null; + var serverError = null; + if (this.state.serverError) { + serverError = this.state.serverError; + } var self = this; @@ -175,9 +198,9 @@ module.exports = React.createClass({ var desktopSection; if (this.props.activeSection === 'desktop') { var notifyActive = [false, false, false]; - if (this.state.notify_level === "mention") { + if (this.state.notifyLevel === 'mention') { notifyActive[1] = true; - } else if (this.state.notify_level === "none") { + } else if (this.state.notifyLevel === 'none') { notifyActive[2] = true; } else { notifyActive[0] = true; @@ -187,21 +210,21 @@ module.exports = React.createClass({ inputs.push(