diff --git a/public/app/app.js b/public/app/app.js index 0c59648a02..88b6d060bd 100644 --- a/public/app/app.js +++ b/public/app/app.js @@ -12,9 +12,9 @@ import '!file?name=[name].[ext]!./manifest.json'; import 'file?name=[name].[ext]!./.htaccess'; /* eslint-enable import/no-unresolved */ // Import all the third party stuff -import React from 'react'; -import ReactDOM from 'react-dom'; -import { Provider } from 'react-redux'; +// import React from 'react'; +// import ReactDOM from 'react-dom'; +// import { Provider } from 'react-redux'; import { browserHistory } from 'react-router'; // import { syncHistoryWithStore } from 'react-router-redux'; // import useScroll from 'react-router-scroll'; @@ -52,7 +52,7 @@ const rootRoute = { export { store, -} +}; // import SettingsManagerApp from 'containers/App/index'; @@ -74,7 +74,6 @@ export { // console.log('this', this); // ReactDOM.render( // return ( - // // ); // // ); @@ -106,13 +105,15 @@ export { // install(); // Register the plugin -window.Strapi.registerPlugin({ - name: 'Settings Manager', - id: 'settings-manager', - leftMenuLink: { - label: 'Settings Manager', - to: '/settings-manager', - }, - routes: rootRoute, - mainComponent: App, -}); +if (window.Strapi) { + window.Strapi.registerPlugin({ + name: 'Settings Manager', + id: 'settings-manager', + leftMenuLink: { + label: 'Settings Manager', + to: '/settings-manager', + }, + routes: rootRoute, + mainComponent: App, + }); +} diff --git a/public/app/components/PluginHeaderActions/index.js b/public/app/components/PluginHeaderActions/index.js index c2e0748e09..94f004ac8f 100644 --- a/public/app/components/PluginHeaderActions/index.js +++ b/public/app/components/PluginHeaderActions/index.js @@ -1,8 +1,8 @@ /** -* -* PluginHeaderActions -* -*/ + * + * PluginHeaderActions + * + */ import React from 'react'; @@ -12,16 +12,18 @@ class PluginHeaderActions extends React.Component { // eslint-disable-line react render() { return (
- - diff --git a/public/app/containers/HomePage/actions.js b/public/app/containers/HomePage/actions.js index 1340cbd25d..1a87f382db 100644 --- a/public/app/containers/HomePage/actions.js +++ b/public/app/containers/HomePage/actions.js @@ -71,47 +71,47 @@ export function generalSettingsLoadingError(error) { export function changeName(name) { return { type: CHANGE_NAME, - name: name + name, }; } export function changeDescription(description) { return { type: CHANGE_DESCRIPTION, - description: description + description, }; } export function changeVersion(version) { return { type: CHANGE_VERSION, - version: version + version, }; } export function updateGeneralSettings(data) { return { type: UPDATE_GENERAL_SETTINGS, - data: data - } + data, + }; } export function generalSettingsUpdated(data) { return { type: UPDATE_GENERAL_SETTINGS_SUCCESS, - data: data - } + data, + }; } export function generalSettingsUpdatedError(error) { return { type: UPDATE_GENERAL_SETTINGS_ERROR, - error: error - } + error, + }; } export function cancelGeneralSettings() { return { type: CANCEL_GENERAL_SETTINGS, - } + }; } diff --git a/public/app/containers/HomePage/index.js b/public/app/containers/HomePage/index.js index 06614582b6..c0465dce36 100644 --- a/public/app/containers/HomePage/index.js +++ b/public/app/containers/HomePage/index.js @@ -10,8 +10,8 @@ */ import React from 'react'; -import {connect} from 'react-redux'; -import {createStructuredSelector} from 'reselect'; +import { connect } from 'react-redux'; +import { createStructuredSelector } from 'reselect'; import PluginHeader from 'components/PluginHeader'; import RightContentSectionTitle from 'components/RightContentSectionTitle'; @@ -50,8 +50,7 @@ export class HomePage extends React.Component { - +
@@ -63,7 +62,7 @@ export class HomePage extends React.Component { id="applicationName" value={this.props.name || ''} onChange={this.props.onChangeName} - autoFocus={true} + autoFocus />
@@ -104,34 +103,35 @@ export class HomePage extends React.Component { HomePage.propTypes = { changeRoute: React.PropTypes.func, - loading: React.PropTypes.bool, - error: React.PropTypes.oneOfType([ - React.PropTypes.object, - React.PropTypes.bool, - ]), - name: React.PropTypes.oneOfType([ - React.PropTypes.string, - React.PropTypes.bool, - ]), description: React.PropTypes.oneOfType([ React.PropTypes.string, React.PropTypes.bool, ]), + error: React.PropTypes.oneOfType([ + React.PropTypes.object, + React.PropTypes.bool, + ]), + loading: React.PropTypes.bool, + name: React.PropTypes.oneOfType([ + React.PropTypes.string, + React.PropTypes.bool, + ]), + onCancel: React.PropTypes.func, + onChangeName: React.PropTypes.func, + onChangeDescription: React.PropTypes.func, + onChangeVersion: React.PropTypes.func, + onFormSubmit: React.PropTypes.func, + onPageLoad: React.PropTypes.func, version: React.PropTypes.oneOfType([ React.PropTypes.string, React.PropTypes.bool, ]), - onPageLoad: React.PropTypes.func, - onCancel: React.PropTypes.func, - onFormSubmit: React.PropTypes.func, - onChangeName: React.PropTypes.func, - onChangeDescription: React.PropTypes.func, - onChangeVersion: React.PropTypes.func, + }; export function mapDispatchToProps(dispatch) { return { - onCancel: (evt) => dispatch(cancelGeneralSettings()), + onCancel: () => dispatch(cancelGeneralSettings()), onChangeName: (evt) => dispatch(changeName(evt.target.value)), onChangeDescription: (evt) => dispatch(changeDescription(evt.target.value)), onChangeVersion: (evt) => dispatch(changeVersion(evt.target.value)), @@ -139,12 +139,10 @@ export function mapDispatchToProps(dispatch) { if (evt !== undefined && evt.preventDefault) evt.preventDefault(); dispatch(updateGeneralSettings()); }, - changeRoute: (url) => dispatch(push(url)), onPageLoad: (evt) => { if (evt !== undefined && evt.preventDefault) evt.preventDefault(); dispatch(loadGeneralSettings()); }, - dispatch, }; } @@ -152,9 +150,9 @@ export function mapDispatchToProps(dispatch) { const mapStateToProps = createStructuredSelector({ name: selectName(), description: selectDescription(), - version: selectVersion(), - loading: selectLoading(), error: selectError(), + loading: selectLoading(), + version: selectVersion(), }); // Wrap the component to inject dispatch and state into it diff --git a/public/app/containers/HomePage/reducer.js b/public/app/containers/HomePage/reducer.js index e7439c0818..fec5485bac 100644 --- a/public/app/containers/HomePage/reducer.js +++ b/public/app/containers/HomePage/reducer.js @@ -22,7 +22,7 @@ import { UPDATE_GENERAL_SETTINGS_ERROR, CANCEL_GENERAL_SETTINGS, } from './constants'; -import {fromJS} from 'immutable'; +import { fromJS } from 'immutable'; // The initial state of the App const initialState = fromJS({ @@ -36,7 +36,7 @@ const initialState = fromJS({ name: false, description: false, version: false, - }) + }), }); function appReducer(state = initialState, action) { @@ -85,11 +85,10 @@ function appReducer(state = initialState, action) { .set('error', action.error) .set('loading', false); case CANCEL_GENERAL_SETTINGS: - const backup = state.get('backup'); return state - .set('name', backup.get('name')) - .set('description', backup.get('description')) - .set('version', backup.get('version')); + .set('name', state.get('backup').get('name')) + .set('description', state.get('backup').get('description')) + .set('version', state.get('backup').get('version')); default: return state; } diff --git a/public/app/containers/HomePage/sagas.js b/public/app/containers/HomePage/sagas.js index 448a72bac8..42b6262208 100644 --- a/public/app/containers/HomePage/sagas.js +++ b/public/app/containers/HomePage/sagas.js @@ -7,7 +7,7 @@ import { LOCATION_CHANGE } from 'react-router-redux'; import { LOAD_GENERAL_SETTINGS, UPDATE_GENERAL_SETTINGS, -} from 'containers/HomePage/constants'; +} from 'containers/HomePage/constants'; import { generalSettingsLoaded, @@ -28,7 +28,7 @@ import request from 'utils/request'; * Github generalSettings request/response handler */ export function* getGeneralSettings() { - const requestURL = `http://localhost:1337/settingsmanager/settings/general`; + const requestURL = 'http://localhost:1337/settingsmanager/settings/general'; // Call our request helper (see 'utils/request') const generalSettings = yield call(request, requestURL); @@ -36,7 +36,7 @@ export function* getGeneralSettings() { if (!generalSettings.err) { yield put(generalSettingsLoaded(generalSettings.data)); } else { - yield put(repoLoadingError(generalSettings.err)); + yield put(generalSettingsLoadingError(generalSettings.err)); } } @@ -50,7 +50,7 @@ export function* updateGeneralSettings() { type: 'general', }; - const requestURL = `http://localhost:1337/settingsmanager/settings`; + const requestURL = 'http://localhost:1337/settingsmanager/settings'; // Call our request helper (see 'utils/request') const generalSettings = yield call( @@ -58,8 +58,8 @@ export function* updateGeneralSettings() { requestURL, { method: 'PUT', headers: { - 'Accept': 'application/json', - 'Content-Type': 'application/json' + Accept: 'application/json', + 'Content-Type': 'application/json', }, body: JSON.stringify(data), } @@ -96,7 +96,7 @@ export function* generalSettingsData() { // Fork watcher so we can continue execution const watcher = yield fork(getGeneralSettingsWatcher); - const updateWatcher = yield fork(updateGeneralSettingsWatcher); + yield fork(updateGeneralSettingsWatcher); // Suspend execution until location changes yield take(LOCATION_CHANGE); diff --git a/public/app/containers/NotFoundPage/index.js b/public/app/containers/NotFoundPage/index.js index c55ff5aca8..562c72754e 100644 --- a/public/app/containers/NotFoundPage/index.js +++ b/public/app/containers/NotFoundPage/index.js @@ -12,7 +12,6 @@ import React from 'react'; import { FormattedMessage } from 'react-intl'; import messages from './messages'; -import Container from 'components/Container'; export default class NotFound extends React.Component { // eslint-disable-line react/prefer-stateless-function diff --git a/public/app/routes.js b/public/app/routes.js index 9f6db8e6fd..bfb9ec847e 100644 --- a/public/app/routes.js +++ b/public/app/routes.js @@ -5,7 +5,6 @@ import { getAsyncInjectors } from 'utils/asyncInjectors'; const errorLoading = (err) => { - console.log('errLoading', err) console.error('Dynamic page loading failed', err); // eslint-disable-line no-console }; diff --git a/public/app/utils/request.js b/public/app/utils/request.js index fbfb736087..0f0a899721 100644 --- a/public/app/utils/request.js +++ b/public/app/utils/request.js @@ -16,23 +16,20 @@ function parseJSON(response) { * * @param {object} response A response from a network request * - * @return {object|undefined} Returns either the response, or throws an error + * @return {Promise} Returns either the response, or throws an error */ function checkStatus(response) { - return new Promise((resolve, reject) => { + return new Promise(resolve => { if (response.status >= 200 && response.status < 300) { return resolve(response); } - parseJSON(response) + return parseJSON(response) .then(data => { const error = new Error(data.message || response.statusText); error.data = data; error.response = response; throw error; - }) - .catch(err => { - reject(err); }); }); } @@ -47,13 +44,14 @@ function checkStatus(response) { */ export default function request(url, options) { // Default headers - options = options || {}; - options.headers = options && options.headers || { - 'Accept': 'application/json', - 'Content-Type': 'application/json' + const params = options || { }; + const defaultHeaders = { + Accept: 'application/json', + 'Content-Type': 'application/json', }; + params.headers = params && params.headers ? params.headers : defaultHeaders; - return fetch(url, options) + return fetch(url, params) .then(checkStatus) .then(parseJSON) .then((data) => ({ data }))