Skip to content

Commit c09f200

Browse files
committed
fixing merge conflict
1 parent 70c64e8 commit c09f200

File tree

2 files changed

+4
-43
lines changed

2 files changed

+4
-43
lines changed

src/client/views/settings-view/cluster-edit/cluster-edit.tsx

Lines changed: 3 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,7 @@ require('./cluster-edit.css');
1818

1919
import * as React from 'react';
2020
import * as Q from 'q';
21-
import { AttributeInfo, Attributes } from 'plywood';
2221

23-
import { Ajax } from '../../../utils/ajax/ajax';
2422
import { Fn, pluralIfNeeded, makeTitle } from '../../../../common/utils/general/general';
2523
import { classNames } from '../../../utils/dom/dom';
2624
import { NUM_REGEX } from '../../../../common/utils/string/string';
@@ -41,8 +39,7 @@ import { CLUSTER as LABELS } from '../../../../common/models/labels';
4139
export interface ClusterEditProps extends React.Props<any> {
4240
cluster?: Cluster;
4341
sources?: string[];
44-
onSave: (newCluster: Cluster) => Q.Promise<void>;
45-
onAddDataCubes?: (dataCubes: DataCube[]) => Q.Promise<void>;
42+
onSave: (newCluster: Cluster, newDataCubes: DataCube[]) => Q.Promise<void>;
4643
isNewCluster?: boolean;
4744
onCancel?: () => void;
4845
getSuggestedCubes?: () => DataCube[];
@@ -91,33 +88,11 @@ export class ClusterEdit extends React.Component<ClusterEditProps, ClusterEditSt
9188
}
9289

9390
save() {
94-
if (this.props.onSave) this.props.onSave(this.state.newInstance);
91+
if (this.props.onSave) this.props.onSave(this.state.newInstance, null);
9592
}
9693

9794
saveAndAddCubes(dataCubes: DataCube[]) {
98-
if (this.props.onSave) {
99-
this.props.onSave(this.state.newInstance)
100-
.then(() => {
101-
Q.all(dataCubes.map(this.fetchSuggestions)).then((attributes: Attributes[]) => {
102-
let newDataCubes = dataCubes.map((dc, i) => dc.fillAllFromAttributes(attributes[i]));
103-
this.props.onAddDataCubes(newDataCubes);
104-
});
105-
});
106-
}
107-
}
108-
109-
fetchSuggestions(dataCube: DataCube): Q.Promise<Attributes> {
110-
return Ajax.query({
111-
method: "POST",
112-
url: 'settings/attributes',
113-
data: {
114-
clusterName: dataCube.clusterName,
115-
source: dataCube.source
116-
}
117-
}).then(
118-
(resp) => AttributeInfo.fromJSs(resp.attributes)
119-
);
120-
;
95+
if (this.props.onSave) this.props.onSave(this.state.newInstance, dataCubes);
12196
}
12297

12398
goBack() {

src/client/views/settings-view/settings-view.tsx

Lines changed: 1 addition & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -215,19 +215,6 @@ export class SettingsView extends React.Component<SettingsViewProps, SettingsVie
215215
).then(this.backToClustersView.bind(this));
216216
}
217217

218-
addDataCubes(dataCubes: DataCube[]) {
219-
if (!dataCubes || !dataCubes.length) {
220-
return;
221-
}
222-
223-
this.onSave(
224-
this.state.settings.appendDataCubes(dataCubes),
225-
'Data cubes created'
226-
)
227-
.then(this.backToClustersView.bind(this))
228-
;
229-
}
230-
231218
// !-- Cluster creation flow
232219

233220
// -- DataCubes creation flow
@@ -240,7 +227,7 @@ export class SettingsView extends React.Component<SettingsViewProps, SettingsVie
240227
addDataCube(newDataCube: DataCube) {
241228
this.onSave(
242229
ImmutableUtils.addInArray(this.state.settings, 'dataCubes', newDataCube),
243-
'Cube created'
230+
'Data cube created'
244231
).then(this.backToDataCubesView.bind(this));
245232
}
246233

@@ -330,7 +317,6 @@ export class SettingsView extends React.Component<SettingsViewProps, SettingsVie
330317
cluster={tempCluster}
331318
sources={tempClusterSources}
332319
onSave={this.addClusterAndDataCubes.bind(this)}
333-
onAddDataCubes={this.addDataCubes.bind(this)}
334320
onCancel={this.backToClustersView.bind(this)}
335321
/>
336322
: <ClusterSeedModal

0 commit comments

Comments
 (0)