diff --git a/packages/strapi-admin/files/admin/src/app.js b/packages/strapi-admin/files/admin/src/app.js index 1590943ca3..9027ef4050 100644 --- a/packages/strapi-admin/files/admin/src/app.js +++ b/packages/strapi-admin/files/admin/src/app.js @@ -110,7 +110,7 @@ const registerPlugin = (plugin) => { // Create a new prefixed route for each plugin routes if (formattedPlugin && formattedPlugin.routes) { - formattedPlugin.routes.forEach(route => { + formattedPlugin.routes.forEach((route) => { pluginsRoute.childRoutes.push({ path: `/plugins/${formattedPlugin.id}${route.path}`, name: `plugins_${formattedPlugin.id}_${route.name}`, diff --git a/packages/strapi-admin/files/admin/src/components/LeftMenuLink/index.js b/packages/strapi-admin/files/admin/src/components/LeftMenuLink/index.js index 8fa5767320..8036e178a5 100644 --- a/packages/strapi-admin/files/admin/src/components/LeftMenuLink/index.js +++ b/packages/strapi-admin/files/admin/src/components/LeftMenuLink/index.js @@ -29,9 +29,9 @@ class LeftMenuLink extends React.Component { // eslint-disable-line react/prefer } LeftMenuLink.propTypes = { - destination: React.PropTypes.string, - icon: React.PropTypes.string, - label: React.PropTypes.string, + destination: React.PropTypes.string.isRequired.isRequired, + icon: React.PropTypes.string.isRequired.isRequired, + label: React.PropTypes.string.isRequired.isRequired, }; export default LeftMenuLink; diff --git a/packages/strapi-admin/files/admin/src/components/LeftMenuLinkContainer/index.js b/packages/strapi-admin/files/admin/src/components/LeftMenuLinkContainer/index.js index 29ea2767a2..354130f24f 100644 --- a/packages/strapi-admin/files/admin/src/components/LeftMenuLinkContainer/index.js +++ b/packages/strapi-admin/files/admin/src/components/LeftMenuLinkContainer/index.js @@ -88,7 +88,7 @@ class LeftMenuLinkContainer extends React.Component { // eslint-disable-line rea } LeftMenuLinkContainer.propTypes = { - plugins: React.PropTypes.object, + plugins: React.PropTypes.object.isRequired.isRequired, }; export default LeftMenuLinkContainer; diff --git a/packages/strapi-admin/files/admin/src/components/Notification/index.js b/packages/strapi-admin/files/admin/src/components/Notification/index.js index 5632836082..df163cc740 100644 --- a/packages/strapi-admin/files/admin/src/components/Notification/index.js +++ b/packages/strapi-admin/files/admin/src/components/Notification/index.js @@ -54,8 +54,8 @@ class Notification extends React.Component { // eslint-disable-line react/prefer } Notification.propTypes = { - notification: React.PropTypes.object, - onHideNotification: React.PropTypes.func, + notification: React.PropTypes.object.isRequired.isRequired, + onHideNotification: React.PropTypes.func.isRequired.isRequired, }; export default Notification; diff --git a/packages/strapi-admin/files/admin/src/components/NotificationsContainer/index.js b/packages/strapi-admin/files/admin/src/components/NotificationsContainer/index.js index 77e70573f0..1b05584521 100644 --- a/packages/strapi-admin/files/admin/src/components/NotificationsContainer/index.js +++ b/packages/strapi-admin/files/admin/src/components/NotificationsContainer/index.js @@ -38,8 +38,8 @@ class NotificationsContainer extends React.Component { // eslint-disable-line re } NotificationsContainer.propTypes = { - notifications: React.PropTypes.object, - onHideNotification: React.PropTypes.func, + notifications: React.PropTypes.object.isRequired.isRequired, + onHideNotification: React.PropTypes.func.isRequired.isRequired, }; export default NotificationsContainer; diff --git a/packages/strapi-admin/files/admin/src/components/Plugin/index.js b/packages/strapi-admin/files/admin/src/components/Plugin/index.js index d722bc4622..b7fa0088e4 100644 --- a/packages/strapi-admin/files/admin/src/components/Plugin/index.js +++ b/packages/strapi-admin/files/admin/src/components/Plugin/index.js @@ -23,8 +23,8 @@ class Plugin extends React.Component { // eslint-disable-line react/prefer-state } Plugin.propTypes = { - description: React.PropTypes.object, - title: React.PropTypes.object, + description: React.PropTypes.object.isRequired.isRequired, + title: React.PropTypes.object.isRequired.isRequired, }; export default Plugin; diff --git a/packages/strapi-admin/files/admin/src/components/PluginHeader/index.js b/packages/strapi-admin/files/admin/src/components/PluginHeader/index.js index 01779c340b..66835b50b9 100644 --- a/packages/strapi-admin/files/admin/src/components/PluginHeader/index.js +++ b/packages/strapi-admin/files/admin/src/components/PluginHeader/index.js @@ -31,9 +31,9 @@ class PluginHeader extends React.Component { // eslint-disable-line react/prefer } PluginHeader.propTypes = { - actions: React.PropTypes.array, - description: React.PropTypes.object, - title: React.PropTypes.object, + actions: React.PropTypes.array.isRequired.isRequired, + description: React.PropTypes.object.isRequired.isRequired, + title: React.PropTypes.object.isRequired.isRequired, }; export default PluginHeader; diff --git a/packages/strapi-admin/files/admin/src/components/PluginHeaderActions/index.js b/packages/strapi-admin/files/admin/src/components/PluginHeaderActions/index.js index 5b85f6739c..a3ea2b07ef 100644 --- a/packages/strapi-admin/files/admin/src/components/PluginHeaderActions/index.js +++ b/packages/strapi-admin/files/admin/src/components/PluginHeaderActions/index.js @@ -11,9 +11,9 @@ import styles from './styles.scss'; class PluginHeaderActions extends React.Component { // eslint-disable-line react/prefer-stateless-function render() { - const actions = this.props.actions && this.props.actions.map((action, i) => ( + const actions = this.props.actions && this.props.actions.map((action) => ( )} - ; + + ); return (
  • @@ -84,12 +88,12 @@ class RowLanguage extends React.Component { // eslint-disable-line react/prefer- } RowLanguage.propTypes = { - active: React.PropTypes.bool, - changeDefaultLanguage: React.PropTypes.func.isRequired, - handleLanguageDelete: React.PropTypes.func.isRequired, - listLanguages: React.PropTypes.object.isRequired, - liStyles: React.PropTypes.object, - name: React.PropTypes.string.isRequired, + active: React.PropTypes.bool.isRequired, + changeDefaultLanguage: React.PropTypes.func.isRequired.isRequired, + handleLanguageDelete: React.PropTypes.func.isRequired.isRequired, + listLanguages: React.PropTypes.object.isRequired.isRequired, + liStyles: React.PropTypes.object.isRequired, + name: React.PropTypes.string.isRequired.isRequired, }; export default RowLanguage; diff --git a/packages/strapi-plugin-settings-manager/admin/src/components/SelectOptionLanguage/index.js b/packages/strapi-plugin-settings-manager/admin/src/components/SelectOptionLanguage/index.js index 93cbb9ef80..cbb5c2d17a 100644 --- a/packages/strapi-plugin-settings-manager/admin/src/components/SelectOptionLanguage/index.js +++ b/packages/strapi-plugin-settings-manager/admin/src/components/SelectOptionLanguage/index.js @@ -39,10 +39,10 @@ class SelectOptionLanguage extends React.Component { // eslint-disable-line reac } SelectOptionLanguage.propTypes = { - isFocused: React.PropTypes.bool, - onFocus: React.PropTypes.func, - onSelect: React.PropTypes.func, - option: React.PropTypes.object.isRequired, + isFocused: React.PropTypes.bool.isRequired, + onFocus: React.PropTypes.func.isRequired, + onSelect: React.PropTypes.func.isRequired, + option: React.PropTypes.object.isRequired.isRequired, }; export default SelectOptionLanguage; diff --git a/packages/strapi-plugin-settings-manager/admin/src/components/WithFormSection/index.js b/packages/strapi-plugin-settings-manager/admin/src/components/WithFormSection/index.js index 138c5c825f..79dc6604a6 100644 --- a/packages/strapi-plugin-settings-manager/admin/src/components/WithFormSection/index.js +++ b/packages/strapi-plugin-settings-manager/admin/src/components/WithFormSection/index.js @@ -19,15 +19,15 @@ import styles from './styles.scss'; const WithFormSection = (InnerComponent) => class extends React.Component { static propTypes = { - addRequiredInputDesign: React.PropTypes.bool, - cancelAction: React.PropTypes.bool, - formErrors: React.PropTypes.array, - handleChange: React.PropTypes.func.isRequired, + addRequiredInputDesign: React.PropTypes.bool.isRequired, + cancelAction: React.PropTypes.bool.isRequired, + formErrors: React.PropTypes.array.isRequired, + handleChange: React.PropTypes.func.isRequired.isRequired, section: React.PropTypes.oneOfType([ - React.PropTypes.object, - React.PropTypes.array, - ]), - values: React.PropTypes.object, + React.PropTypes.object.isRequired, + React.PropTypes.array.isRequired, + ]).isRequired, + values: React.PropTypes.object.isRequired, } constructor(props) { diff --git a/packages/strapi-plugin-settings-manager/admin/src/containers/App/index.js b/packages/strapi-plugin-settings-manager/admin/src/containers/App/index.js index 2878ab47f3..660124f3fb 100644 --- a/packages/strapi-plugin-settings-manager/admin/src/containers/App/index.js +++ b/packages/strapi-plugin-settings-manager/admin/src/containers/App/index.js @@ -50,18 +50,18 @@ class App extends React.Component { } App.contextTypes = { - router: React.PropTypes.object.isRequired, + router: React.PropTypes.object.isRequired.isRequired, }; App.propTypes = { - children: React.PropTypes.node.isRequired, - environments: React.PropTypes.array, - environmentsFetch: React.PropTypes.func, - exposedComponents: React.PropTypes.object.isRequired, - loading: React.PropTypes.bool, - menuFetch: React.PropTypes.func, - params: React.PropTypes.object, - sections: React.PropTypes.array.isRequired, + children: React.PropTypes.node.isRequired.isRequired, + environments: React.PropTypes.array.isRequired, + environmentsFetch: React.PropTypes.func.isRequired, + exposedComponents: React.PropTypes.object.isRequired.isRequired, + loading: React.PropTypes.bool.isRequired, + menuFetch: React.PropTypes.func.isRequired, + params: React.PropTypes.object.isRequired, + sections: React.PropTypes.array.isRequired.isRequired, }; export function mapDispatchToProps(dispatch) { @@ -72,7 +72,7 @@ export function mapDispatchToProps(dispatch) { }, dispatch ); -}; +} const mapStateToProps = createStructuredSelector({ sections: makeSelectSections(), diff --git a/packages/strapi-plugin-settings-manager/admin/src/containers/HomePage/index.js b/packages/strapi-plugin-settings-manager/admin/src/containers/HomePage/index.js index 79c057e6e7..61b2c068e1 100644 --- a/packages/strapi-plugin-settings-manager/admin/src/containers/HomePage/index.js +++ b/packages/strapi-plugin-settings-manager/admin/src/containers/HomePage/index.js @@ -430,9 +430,9 @@ export class HomePage extends React.Component { // eslint-disable-line react/pre } setDefaultConnectionDb = () => { - const value = this.state.toggleDefaultConnection ? - this.props.home.addDatabaseSection.sections[1].items[0].value - : this.props.home.modifiedData[this.props.home.dbNameTarget]; + const value = this.state.toggleDefaultConnection + ? this.props.home.addDatabaseSection.sections[1].items[0].value + : this.props.home.modifiedData[this.props.home.dbNameTarget]; const target = { name: 'database.defaultConnection', value }; this.handleChange({target}); this.setState({ toggleDefaultConnection: !this.state.toggleDefaultConnection }); @@ -507,26 +507,26 @@ function mapDispatchToProps(dispatch) { } HomePage.propTypes = { - cancelChanges: React.PropTypes.func, - changeDefaultLanguage: React.PropTypes.func, - changeInput: React.PropTypes.func, - configFetch: React.PropTypes.func.isRequired, - databaseDelete: React.PropTypes.func, - databaseEdit: React.PropTypes.func, - databasesFetch: React.PropTypes.func, - editSettings: React.PropTypes.func, - emptyDbModifiedData: React.PropTypes.func, - environments: React.PropTypes.array, - home: React.PropTypes.object, - languageDelete: React.PropTypes.func, - languagesFetch: React.PropTypes.func, - location: React.PropTypes.object, - menuSections: React.PropTypes.array, - newDatabasePost: React.PropTypes.func, - newLanguagePost: React.PropTypes.func, - params: React.PropTypes.object.isRequired, - setErrors: React.PropTypes.func, - specificDatabaseFetch: React.PropTypes.func, + cancelChanges: React.PropTypes.func.isRequired, + changeDefaultLanguage: React.PropTypes.func.isRequired, + changeInput: React.PropTypes.func.isRequired, + configFetch: React.PropTypes.func.isRequired.isRequired, + databaseDelete: React.PropTypes.func.isRequired, + databaseEdit: React.PropTypes.func.isRequired, + databasesFetch: React.PropTypes.func.isRequired, + editSettings: React.PropTypes.func.isRequired, + emptyDbModifiedData: React.PropTypes.func.isRequired, + environments: React.PropTypes.array.isRequired, + home: React.PropTypes.object.isRequired, + languageDelete: React.PropTypes.func.isRequired, + languagesFetch: React.PropTypes.func.isRequired, + location: React.PropTypes.object.isRequired, + menuSections: React.PropTypes.array.isRequired, + newDatabasePost: React.PropTypes.func.isRequired, + newLanguagePost: React.PropTypes.func.isRequired, + params: React.PropTypes.object.isRequired.isRequired, + setErrors: React.PropTypes.func.isRequired, + specificDatabaseFetch: React.PropTypes.func.isRequired, }; export default connect(mapStateToProps, mapDispatchToProps)(HomePage); diff --git a/packages/strapi-plugin-settings-manager/admin/src/containers/HomePage/reducer.js b/packages/strapi-plugin-settings-manager/admin/src/containers/HomePage/reducer.js index 47fc0a4c62..26e5ba4543 100644 --- a/packages/strapi-plugin-settings-manager/admin/src/containers/HomePage/reducer.js +++ b/packages/strapi-plugin-settings-manager/admin/src/containers/HomePage/reducer.js @@ -60,9 +60,9 @@ function homePageReducer(state = initialState, action) { return state.updateIn(['modifiedData', action.key], () => action.value); case CANCEL_CHANGES: return state - .set('modifiedData', state.get('initialData')) - .set('formErrors', []) - .set('cancelAction', !state.get('cancelAction')); + .set('modifiedData', state.get('initialData')) + .set('formErrors', []) + .set('cancelAction', !state.get('cancelAction')); case DATABASES_FETCH_SUCCEEDED: return state .set('configsDisplay', OrderedMap(action.configsDisplay)) @@ -123,10 +123,10 @@ function homePageReducer(state = initialState, action) { case EMPTY_DB_MODIFIED_DATA: const defaultDbConnection = state.getIn(['modifiedData', 'database.defaultConnection']); return state - .set('modifiedData', Map()) - .set('dbNameTarget', 'database.connections.${name}.name') // eslint-disable-line no-template-curly-in-string - .set('formErrors', []) - .setIn(['modifiedData', 'database.defaultConnection'], defaultDbConnection); + .set('modifiedData', Map()) + .set('dbNameTarget', 'database.connections.${name}.name') // eslint-disable-line no-template-curly-in-string + .set('formErrors', []) + .setIn(['modifiedData', 'database.defaultConnection'], defaultDbConnection); case NEW_LANGUAGE_POST: const sections = state.getIn(['configsDisplay', 'sections']); sections.push({ active: false, name: state.getIn(['modifiedData', 'language.defaultLocale']) }); diff --git a/packages/strapi-plugin-settings-manager/admin/src/utils/sendUpdatedParams.js b/packages/strapi-plugin-settings-manager/admin/src/utils/sendUpdatedParams.js index 5b6c7dced9..40bf1caab3 100644 --- a/packages/strapi-plugin-settings-manager/admin/src/utils/sendUpdatedParams.js +++ b/packages/strapi-plugin-settings-manager/admin/src/utils/sendUpdatedParams.js @@ -9,9 +9,9 @@ export default function sendUpdatedParams() { body[key] = value; } else if (key === 'security.xframe.value.nested' && prevSettings['security.xframe.value.nested'] !== this.props.home.modifiedData['security.xframe.value.nested'] && this.props.home.modifiedData['security.xframe.value'] === 'ALLOW-FROM') { - const xFrameValue = includes(this.props.home.modifiedData['security.xframe.value.nested'], 'ALLOW-FROM') ? - `ALLOW-FROM ${this.props.home.modifiedData['security.xframe.value.nested']}` - : `ALLOW-FROM.ALLOW-FROM ${this.props.home.modifiedData['security.xframe.value.nested']}`; + const xFrameValue = includes(this.props.home.modifiedData['security.xframe.value.nested'], 'ALLOW-FROM') + ? `ALLOW-FROM ${this.props.home.modifiedData['security.xframe.value.nested']}` + : `ALLOW-FROM.ALLOW-FROM ${this.props.home.modifiedData['security.xframe.value.nested']}`; body['security.xframe.value'] = xFrameValue; } diff --git a/packages/strapi-plugin-settings-manager/package.json b/packages/strapi-plugin-settings-manager/package.json index 628ba8ae8a..310b083a05 100644 --- a/packages/strapi-plugin-settings-manager/package.json +++ b/packages/strapi-plugin-settings-manager/package.json @@ -34,16 +34,16 @@ }, "devDependencies": { "cross-env": "^3.1.3", - "eslint": "^3.9.0", - "eslint-config-airbnb": "^12.0.0", - "eslint-config-airbnb-base": "^9.0.0", + "eslint": "^4.4.1", + "eslint-config-airbnb": "^15.1.0", + "eslint-config-airbnb-base": "^11.3.1", "eslint-config-prettier": "^2.3.0", - "eslint-import-resolver-webpack": "^0.6.0", + "eslint-import-resolver-webpack": "^0.8.3", "eslint-plugin-babel": "^4.1.2", - "eslint-plugin-import": "^2.0.1", - "eslint-plugin-jsx-a11y": "^2.2.3", - "eslint-plugin-react": "^6.4.1", - "eslint-plugin-redux-saga": "^0.1.5", + "eslint-plugin-import": "^2.7.0", + "eslint-plugin-jsx-a11y": "^6.0.2", + "eslint-plugin-react": "^7.2.1", + "eslint-plugin-redux-saga": "^0.4.0", "plop": "^1.5.0", "prettier": "^1.5.3", "rimraf": "^2.5.4",