From 61a35066b64085d264fbd049e2a57f70125ca65a Mon Sep 17 00:00:00 2001
From: Florian Orben <florian.orben@gmail.com>
Date: Fri, 27 Nov 2015 23:16:56 +0100
Subject: rename 'featureToggle' to 'isFeatureEnable' and move definition of
 available prerelease-features to constants.jsx

---
 web/react/components/textbox.jsx                   |  3 +-
 .../user_settings/user_settings_advanced.jsx       | 35 ++++++++++------------
 2 files changed, 18 insertions(+), 20 deletions(-)

(limited to 'web/react/components')

diff --git a/web/react/components/textbox.jsx b/web/react/components/textbox.jsx
index 1e9e912a8..10b3c0069 100644
--- a/web/react/components/textbox.jsx
+++ b/web/react/components/textbox.jsx
@@ -11,6 +11,7 @@ import * as Utils from '../utils/utils.jsx';
 import Constants from '../utils/constants.jsx';
 const ActionTypes = Constants.ActionTypes;
 const KeyCodes = Constants.KeyCodes;
+const PreReleaseFeatures = Constants.PRE_RELEASE_FEATURES;
 
 export default class Textbox extends React.Component {
     constructor(props) {
@@ -304,7 +305,7 @@ export default class Textbox extends React.Component {
 
     render() {
         let previewLink = null;
-        if (Utils.featureToggle('#1389')) {
+        if (Utils.isFeatureEnabled(PreReleaseFeatures.MARKDOWN_PREVIEW)) {
             const previewLinkVisible = this.props.messageText.length > 0;
             previewLink = (
                 <a
diff --git a/web/react/components/user_settings/user_settings_advanced.jsx b/web/react/components/user_settings/user_settings_advanced.jsx
index cfb360152..b4d34c658 100644
--- a/web/react/components/user_settings/user_settings_advanced.jsx
+++ b/web/react/components/user_settings/user_settings_advanced.jsx
@@ -6,6 +6,7 @@ import SettingItemMin from '../setting_item_min.jsx';
 import SettingItemMax from '../setting_item_max.jsx';
 import Constants from '../../utils/constants.jsx';
 import PreferenceStore from '../../stores/preference_store.jsx';
+const PreReleaseFeatures = Constants.PRE_RELEASE_FEATURES;
 
 export default class AdvancedSettingsDisplay extends React.Component {
     constructor(props) {
@@ -15,13 +16,9 @@ export default class AdvancedSettingsDisplay extends React.Component {
         this.updateSetting = this.updateSetting.bind(this);
         this.toggleFeature = this.toggleFeature.bind(this);
         this.saveEnabledFeatures = this.saveEnabledFeatures.bind(this);
-        this.setupInitialState = this.setupInitialState.bind(this);
 
-        this.state = {settings: {}, preReleaseFeatures: []};
-        this.setupInitialState();
-    }
-
-    setupInitialState() {
+        const preReleaseFeaturesKeys = Object.keys(PreReleaseFeatures);
+        const advancedSettings = PreferenceStore.getPreferences(Constants.Preferences.CATEGORY_ADVANCED_SETTINGS);
         const settings = {
             send_on_ctrl_enter: PreferenceStore.getPreference(
                 Constants.Preferences.CATEGORY_ADVANCED_SETTINGS,
@@ -31,20 +28,19 @@ export default class AdvancedSettingsDisplay extends React.Component {
         };
 
         let enabledFeatures = 0;
-        PreferenceStore.getPreferences(Constants.Preferences.CATEGORY_ADVANCED_SETTINGS).forEach((setting) => {
-            if (setting.name.lastIndexOf(Constants.FeatureTogglePrefix) === 0) {
-                settings[setting.name] = setting.value;
-                if (setting.value === 'true') {
-                    enabledFeatures++;
+        advancedSettings.forEach((setting) => {
+            preReleaseFeaturesKeys.forEach((key) => {
+                const feature = PreReleaseFeatures[key];
+                if (setting.name === Constants.FeatureTogglePrefix + feature.label) {
+                    settings[setting.name] = setting.value;
+                    if (setting.value === 'true') {
+                        enabledFeatures++;
+                    }
                 }
-            }
+            });
         });
 
-        Client.getAvailablePreReleaseFeatures((data) => {
-            this.setState({settings, preReleaseFeatures: data || [], enabledFeatures});
-        }, () => {
-            this.setState({settings, preReleaseFeatures: [], enabledFeatures});
-        });
+        this.state = {preReleaseFeatures: PreReleaseFeatures, settings, preReleaseFeaturesKeys, enabledFeatures};
     }
 
     updateSetting(setting, value) {
@@ -168,7 +164,7 @@ export default class AdvancedSettingsDisplay extends React.Component {
 
         let previewFeaturesSection;
         let previewFeaturesSectionDivider;
-        if (this.state.preReleaseFeatures.length > 0) {
+        if (this.state.preReleaseFeaturesKeys.length > 0) {
             previewFeaturesSectionDivider = (
                 <div className='divider-light'/>
             );
@@ -176,7 +172,8 @@ export default class AdvancedSettingsDisplay extends React.Component {
             if (this.props.activeSection === 'advancedPreviewFeatures') {
                 const inputs = [];
 
-                this.state.preReleaseFeatures.forEach((feature) => {
+                this.state.preReleaseFeaturesKeys.forEach((key) => {
+                    const feature = this.state.preReleaseFeatures[key];
                     inputs.push(
                         <div key={'advancedPreviewFeatures_' + feature.label}>
                             <div className='checkbox'>
-- 
cgit v1.2.3-1-g7c22