Browse Source

switch enable

itizawa 6 years ago
parent
commit
ebab8bb2bf

+ 1 - 6
src/client/js/components/Admin/MarkdownSetting/XssForm.jsx

@@ -16,16 +16,11 @@ class XssForm extends React.Component {
   constructor(props) {
     super(props);
 
-    this.onChangeEnableXss = this.onChangeEnableXss.bind(this);
     this.onChangeTagWhiteList = this.onChangeTagWhiteList.bind(this);
     this.onChangeAttrWhiteList = this.onChangeAttrWhiteList.bind(this);
     this.onClickSubmit = this.onClickSubmit.bind(this);
   }
 
-  onChangeEnableXss() {
-    this.setState({ isEnabledXss: !this.state.isEnabledXss });
-  }
-
   onChangeTagWhiteList(value) {
     this.setState({ tagWhiteList: value });
   }
@@ -80,7 +75,7 @@ class XssForm extends React.Component {
         <form className="row">
           <div className="form-group">
             <div className="col-xs-4 text-right">
-              <div className="checkbox checkbox-success" onChange={this.onChangeEnableXss}>
+              <div className="checkbox checkbox-success" onChange={markDownSettingContainer.onChangeEnableXss}>
                 <input type="checkbox" id="XssEnable" className="form-check-input" name="isEnabledXss" checked={markDownSettingContainer.state.isEnabledXss} />
                 <label htmlFor="XssEnable">
                   { t('markdown_setting.Enable XSS prevention') }

+ 9 - 1
src/client/js/services/MarkDownSettingContainer.js

@@ -19,12 +19,13 @@ export default class MarkDownSettingContainer extends Container {
     this.appContainer = appContainer;
 
     this.state = {
-      isEnabledXss: appContainer.config.xssOption || false,
+      isEnabledXss: (appContainer.config.xssOption != null),
       xssOption: appContainer.config.xssOption,
       tagWhiteList: appContainer.config.tagWhiteList,
       attrWhiteList: '',
     };
 
+    this.onChangeEnableXss = this.onChangeEnableXss.bind(this);
     this.onChangeXssOption = this.onChangeXssOption.bind(this);
   }
 
@@ -35,6 +36,13 @@ export default class MarkDownSettingContainer extends Container {
     return 'MarkDownSettingContainer';
   }
 
+  /**
+   * switch enableXss
+   */
+  onChangeEnableXss() {
+    this.setState({ isEnabledXss: !this.state.isEnabledXss });
+  }
+
   /**
    * switch xssOption
    * @param {value} int