From c1d955beb4c949802b0c6eacf305d4bf4f9c8df7 Mon Sep 17 00:00:00 2001
From: cyril lopez <cyril.lpz@gmail.com>
Date: Wed, 19 Jul 2017 09:51:23 +0200
Subject: [PATCH] add target props to inputs

---
 .../admin/src/components/InputNumber/index.js              | 6 ++++--
 .../admin/src/components/InputText/index.js                | 4 ++--
 .../admin/src/components/InputToggle/index.js              | 6 +++---
 .../admin/src/containers/Home/index.js                     | 7 ++++---
 4 files changed, 13 insertions(+), 10 deletions(-)

diff --git a/packages/strapi-plugin-setings-manager/admin/src/components/InputNumber/index.js b/packages/strapi-plugin-setings-manager/admin/src/components/InputNumber/index.js
index 58f8ba1939..48b2975343 100644
--- a/packages/strapi-plugin-setings-manager/admin/src/components/InputNumber/index.js
+++ b/packages/strapi-plugin-setings-manager/admin/src/components/InputNumber/index.js
@@ -9,11 +9,12 @@
 *   - handleBlur: function
 *     overrides the default input validations
 *   - errors : array
-*     prevent from siplaying errors messages
+*     prevent from displaying errors messages
 *
 * Required
 *  - name : string
 *  - handleChange : function
+*  - target : string
 *  - value : string
 *  - validations : object
 *
@@ -94,7 +95,7 @@ class InputNumber extends React.Component { // eslint-disable-line react/prefer-
         <label htmlFor={this.props.name}>{this.props.name}</label>
         <input
           type="number"
-          name={this.props.name}
+          name={this.props.target}
           id={this.props.name}
           value={inputValue}
           onBlur={handleBlur}
@@ -124,6 +125,7 @@ InputNumber.propTypes = {
   name: React.PropTypes.string.isRequired,
   noErrorsDescription: React.PropTypes.bool,
   placeholder: React.PropTypes.string,
+  target: React.PropTypes.string.isRequired,
   validations: React.PropTypes.object.isRequired,
   value: React.PropTypes.oneOfType([
     React.PropTypes.number.isRequired,
diff --git a/packages/strapi-plugin-setings-manager/admin/src/components/InputText/index.js b/packages/strapi-plugin-setings-manager/admin/src/components/InputText/index.js
index 9f57d01af0..c5282b9e19 100644
--- a/packages/strapi-plugin-setings-manager/admin/src/components/InputText/index.js
+++ b/packages/strapi-plugin-setings-manager/admin/src/components/InputText/index.js
@@ -10,14 +10,14 @@
 *     overrides the default input validations
 *   - errors : array
 *   - noErrorsDescription : bool
-*     prevent from siplaying errors messages
+*     prevent from displaying errors messages
 *
 * Required
 *  - name : string
 *  - handleChange : function
 *  - value : string
-*  - validations : object
 *  - target : string
+*  - validations : object
 *
 * Optionnal
 * - description : input description
diff --git a/packages/strapi-plugin-setings-manager/admin/src/components/InputToggle/index.js b/packages/strapi-plugin-setings-manager/admin/src/components/InputToggle/index.js
index fbf70385f5..f4643fdac7 100644
--- a/packages/strapi-plugin-setings-manager/admin/src/components/InputToggle/index.js
+++ b/packages/strapi-plugin-setings-manager/admin/src/components/InputToggle/index.js
@@ -7,7 +7,7 @@
 *
 * Required
 *  - handleChange: function
-*  - name: string
+*  - target: string
 *  - isChecked: bool
 */
 
@@ -37,7 +37,7 @@ class InputToggle extends React.Component { // eslint-disable-line react/prefer-
       isChecked  = false;
     }
     const target = {
-      name: this.props.name,
+      name: this.props.target,
       value: isChecked,
     };
     this.setState({ isChecked });
@@ -63,7 +63,7 @@ InputToggle.propTypes = {
   customBootstrapClass: React.PropTypes.string,
   handleChange: React.PropTypes.func.isRequired,
   isChecked: React.PropTypes.bool.isRequired,
-  name: React.PropTypes.string.isRequired,
+  target: React.PropTypes.string.isRequired,
 }
 
 export default InputToggle;
diff --git a/packages/strapi-plugin-setings-manager/admin/src/containers/Home/index.js b/packages/strapi-plugin-setings-manager/admin/src/containers/Home/index.js
index b29e9a75cc..3ba6fc8b3d 100644
--- a/packages/strapi-plugin-setings-manager/admin/src/containers/Home/index.js
+++ b/packages/strapi-plugin-setings-manager/admin/src/containers/Home/index.js
@@ -8,7 +8,7 @@ import React from 'react';
 import { connect } from 'react-redux';
 import { bindActionCreators } from 'redux';
 import { createStructuredSelector } from 'reselect';
-import { findKey, includes } from 'lodash';
+import { findKey, includes, get } from 'lodash';
 
 import Helmet from 'react-helmet';
 import { router } from 'app';
@@ -30,6 +30,7 @@ export class Home extends React.Component { // eslint-disable-line react/prefer-
     this.customComponents = config.customComponents;
     this.components = {
       editForm: EditForm,
+      div: EditForm, // TODO change to default
     };
   }
 
@@ -38,7 +39,7 @@ export class Home extends React.Component { // eslint-disable-line react/prefer-
       const apiUrl = this.props.params.env ? `${this.props.params.slug}/${this.props.params.env}` : this.props.params.slug;
       this.props.configFetch(apiUrl);
     } else {
-      router.push(`/plugins/settings-manager/${this.props.sections[0].items[0].slug}`);
+      router.push(`/plugins/settings-manager/${get(this.props.sections, ['0', 'items', '0', 'slug'])}`);
     }
   }
 
@@ -52,7 +53,7 @@ export class Home extends React.Component { // eslint-disable-line react/prefer-
         this.props.configFetch(apiUrl);
       } else {
         // redirect user if no params slug provided
-        router.push(`/plugins/settings-manager/${this.props.sections[0].items[0].slug}`);
+        router.push(`/plugins/settings-manager/${get(this.props.sections, ['0', 'items', '0', 'slug'])}`);
       }
     } else if (this.props.params.env !== nextProps.params.env && nextProps.params.env) {
       // get data if params env updated