From f57b7dcf181415e0e5739d22b8c827109d66f98d Mon Sep 17 00:00:00 2001 From: ysandler Date: Mon, 20 Jul 2020 22:41:39 -0500 Subject: [PATCH] fix: actually adding join nodule --- src/Controllers/CreateNoduleController.js | 9 ++++----- src/views/CreateNodule/CreateJoinNoduleForm.js | 9 +++++++-- src/views/CreateNodule/CreateNodule.js | 12 +++++++++++- 3 files changed, 22 insertions(+), 8 deletions(-) diff --git a/src/Controllers/CreateNoduleController.js b/src/Controllers/CreateNoduleController.js index 2641106..968cc75 100644 --- a/src/Controllers/CreateNoduleController.js +++ b/src/Controllers/CreateNoduleController.js @@ -24,7 +24,7 @@ class CreateNoduleController { } addNewJoinNodule = props => { - const { label, tablesToImportByLabel, baseTableLabel, joinParams } = props + const { label, tablesToImportByLabel, joinBy} = props const tables = tablesToImportByLabel.map(label => { return this.tables.getTableByLabel(label) }) @@ -32,11 +32,10 @@ class CreateNoduleController { this.nodules.addNewJoinNodule({ label, tables, - joinBy: { - baseTableLabel, - joinParams - } + joinBy }) + + console.log(this.nodules) document.dispatchEvent(this.updatedNodulesEvent) } diff --git a/src/views/CreateNodule/CreateJoinNoduleForm.js b/src/views/CreateNodule/CreateJoinNoduleForm.js index df36f37..14110fa 100644 --- a/src/views/CreateNodule/CreateJoinNoduleForm.js +++ b/src/views/CreateNodule/CreateJoinNoduleForm.js @@ -10,7 +10,7 @@ class CreateJoinNoduleForm extends Component { this.tables = new Tables() this.state = { - bastTableLabel: '', + baseTableLabel: '', joinParams: [], tables: this.tables.getCollectionProps() } @@ -35,7 +35,7 @@ class CreateJoinNoduleForm extends Component { } handleChange = (e, value) => { - this.setState({ bastTableLabel: value.value }) + this.setState({ baseTableLabel: value.value }) } getBaseTableDropDownOptions = () => { @@ -48,6 +48,11 @@ class CreateJoinNoduleForm extends Component { return options } + getJoinProperties = () => { + const { baseTableLabel, joinParams } = this.state + return { baseTableLabel, joinParams } + } + updateTableList = () => { this.setState({tables: this.tables.getCollectionProps()}) } diff --git a/src/views/CreateNodule/CreateNodule.js b/src/views/CreateNodule/CreateNodule.js index df3f4a9..19ddbe5 100644 --- a/src/views/CreateNodule/CreateNodule.js +++ b/src/views/CreateNodule/CreateNodule.js @@ -5,8 +5,9 @@ import './CreateNodule.css' import Tables from '../../Collections/Tables' import CreateNoduleController from '../../Controllers/CreateNoduleController' -import TableSelect from './TableSelect' import CreateJoinNoduleForm from './CreateJoinNoduleForm' +import CreateTransformNoduleForm from './CreateTransformNoduleForm' +import TableSelect from './TableSelect' class CreateNodule extends Component { constructor () { @@ -47,6 +48,14 @@ class CreateNodule extends Component { filterParams: filterProperties.filterParams }) } + else if (noduleType === 'join') { + const joinProperties = this.joinNoduleForm.current.getJoinProperties() + this.controller.addNewJoinNodule({ + label: noduleLabel, + tablesToImportByLabel: selectedTableLabels, + joinBy: joinProperties + }) + } } updateTableList = () => { @@ -58,6 +67,7 @@ class CreateNodule extends Component { if (noduleType === 'filter') return else if (noduleType === 'join') return + else if (noduleType === 'transform') return else return '' }