From e57b27d8a1b14ef3d9b851959052e85ebb801bc6 Mon Sep 17 00:00:00 2001
From: Mael <mael.thomas@data.gouv.fr>
Date: Thu, 1 Mar 2018 15:32:14 +0100
Subject: [PATCH] Etape d'affinage des objectifs

---
 source/components/BlueButton.js       |   3 +-
 source/components/TargetSelection.css |   2 +
 source/components/TargetSelection.js  | 120 ++++++++++++++------------
 source/reducers.js                    |   1 +
 4 files changed, 70 insertions(+), 56 deletions(-)

diff --git a/source/components/BlueButton.js b/source/components/BlueButton.js
index d4a07c900..b89e804a2 100644
--- a/source/components/BlueButton.js
+++ b/source/components/BlueButton.js
@@ -4,9 +4,10 @@ import { connect } from 'react-redux'
 
 export default connect(state => ({
 	themeColours: state.themeColours
-}))(({ themeColours, children, disabled, style }) => (
+}))(({ themeColours, children, disabled, style, onClick }) => (
 	<button
 		disabled={disabled}
+		onClick={onClick}
 		className="blueButton"
 		style={{ ...style, background: themeColours.colour }}
 	>
diff --git a/source/components/TargetSelection.css b/source/components/TargetSelection.css
index efcda4903..d2f02e3a4 100644
--- a/source/components/TargetSelection.css
+++ b/source/components/TargetSelection.css
@@ -68,6 +68,8 @@
 
 #targetSelection #action p {
 	font-size: 80%;
+	border-bottom: #4b4b66 solid 2px;
+	color: #4b4b66;
 }
 
 #targetSelection input[type='text'] {
diff --git a/source/components/TargetSelection.js b/source/components/TargetSelection.js
index fcc01b72c..b67de3051 100644
--- a/source/components/TargetSelection.js
+++ b/source/components/TargetSelection.js
@@ -10,7 +10,11 @@ import {
 	equals,
 	filter,
 	contains,
-	length
+	length,
+	without,
+	append,
+	ifElse,
+	__
 } from 'ramda'
 import { Link } from 'react-router-dom'
 import './TargetSelection.css'
@@ -32,6 +36,7 @@ let popularTargetNames = [...salaries, 'aides employeur']
 			formValueSelector('conversation')(state, dottedName),
 		targets: state.analysis ? state.analysis.targets : [],
 		flatRules: state.flatRules
+		targetNames: state.targetNames
 	}),
 	dispatch => ({
 		startConversation: (targetNames, fromScratch = false) =>
@@ -40,19 +45,14 @@ let popularTargetNames = [...salaries, 'aides employeur']
 )
 export default class TargetSelection extends Component {
 	state = {
-		targets: [],
-		activeInput: null
+		activeInput: null,
+		affinage: false
 	}
 
 	componentWillMount() {
 		this.props.startConversation(popularTargetNames)
 	}
 	render() {
-		let { targets } = this.state,
-			ready = targets.length > 0
-
-		console.log('yayayay', this.props.targets)
-
 		if (this.props.targets.length == 0) return null
 
 		return (
@@ -60,14 +60,21 @@ export default class TargetSelection extends Component {
 				<h1>Entrez un salaire mensuel</h1>
 				{this.renderOutputList()}
 
-				{false && (
+				{this.state.activeInput && (
 					<div id="action">
-						<p style={{ color: this.props.colours.textColourOnWhite }}>
-							Vous pouvez faire plusieurs choix
-						</p>
-						<Link to={'/simu/' + targets.join('+')}>
-							<BlueButton disabled={!ready}>Valider</BlueButton>
-						</Link>
+						{this.state.affinage ? (
+							<p style={{ color: this.props.colours.textColourOnWhite }}>
+								Sélectionnez un ou plusieurs objectifs
+							</p>
+						) : (
+							<BlueButton
+								onClick={() =>
+									console.log('iozn') || this.setState({ affinage: true })
+								}
+							>
+								Affiner
+							</BlueButton>
+						)}
 					</div>
 				)}
 			</section>
@@ -75,21 +82,18 @@ export default class TargetSelection extends Component {
 	}
 
 	renderOutputList() {
-		let popularTargets = popularTargetNames.map(
-				curry(findRuleByName)(flatRules)
-			),
-			{ targets } = this.state,
-			textColourOnWhite = this.props.colours.textColourOnWhite,
-			// You can't select 3 salaries, as one must be an input in the next step
-			optionDisabled = name =>
-				contains('salaire', name) &&
-				pipe(
-					reject(equals(name)),
-					filter(contains('salaire')),
-					length,
-					equals(2)
-				)(targets),
-			optionIsChecked = s => targets.includes(s.name)
+		let popularTargets = popularTargetNames.map(curry(findRuleByName)(flatRules)),
+			{
+				targets,
+				targetNames,
+				textColourOnWhite,
+				startConversation
+			} = this.props,
+			optionIsChecked = s => targetNames.includes(s.name),
+			visibleCheckbox = s =>
+				this.state.affinage && s.dottedName !== this.state.activeInput,
+			toggleTarget = target =>
+				ifElse(contains(target), without(target), append(target))
 
 		return (
 			<div>
@@ -97,19 +101,22 @@ export default class TargetSelection extends Component {
 					{popularTargets.map(s => (
 						<div key={s.name}>
 							<div className="main">
-								<input
-									id={s.name}
-									type="checkbox"
-									disabled={optionDisabled(s.name)}
-									checked={optionIsChecked(s)}
-									onChange={() =>
-										this.setState({
-											targets: targets.find(t => t === s.name)
-												? reject(t => t === s.name, targets)
-												: [...targets, s.name]
-										})
-									}
-								/>
+								{visibleCheckbox(s) && (
+									<input
+										id={s.name}
+										type="checkbox"
+										checked={optionIsChecked(s)}
+										onChange={() =>
+											startConversation(
+												toggleTarget(s.name)(
+													targetNames.filter(
+														t => !this.state.activeInput.includes(t)
+													)
+												)
+											)
+										}
+									/>
+								)}
 								<label
 									htmlFor={s.name}
 									key={s.name}
@@ -119,18 +126,21 @@ export default class TargetSelection extends Component {
 													color: textColourOnWhite
 											  }
 											: {}
-									}>
-									{optionIsChecked(s) ? (
-										<i
-											className="fa fa-check-square-o fa-2x"
-											style={{ color: textColourOnWhite }}
-										/>
-									) : (
-										<i
-											className="fa fa-square-o fa-2x"
-											style={{ color: '#4b4b66' }}
-										/>
-									)}
+									}
+								>
+									{visibleCheckbox(s) ? (
+										optionIsChecked(s) ? (
+											<i
+												className="fa fa-check-square-o fa-2x"
+												style={{ color: textColourOnWhite }}
+											/>
+										) : (
+											<i
+												className="fa fa-square-o fa-2x"
+												style={{ color: '#4b4b66' }}
+											/>
+										)
+									) : null}
 									<span className="optionTitle">{s.title || s.name}</span>
 								</label>
 								<span className="targetInputOrValue">
diff --git a/source/reducers.js b/source/reducers.js
index 7cea8029a..568ce3fe1 100644
--- a/source/reducers.js
+++ b/source/reducers.js
@@ -99,6 +99,7 @@ export let reduceSteps = (tracker, flatRules, answerSource) => (
 		rulesDefaults = collectDefaults(flatRules),
 		situationWithDefaults = assume(intermediateSituation, rulesDefaults)
 
+	console.log('targnames', targetNames)
 	let analysis = analyseMany(state.parsedRules, targetNames)(
 		situationWithDefaults(state)
 	)