github-mirrors/kibana
1
0
Fork 0
mirror of https://github.com/elastic/kibana.git synced 2025-04-24 09:48:58 -04:00
Code Issues Projects Releases Packages Wiki Activity

[ML] [6.x] Group selector duplicate validation (#24139)

Browse source
This commit is contained in:
James Gowdy 2018-10-17 18:06:25 +01:00 • committed by GitHub
parent df9c17a091
commit 4e14ccc8d7
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 15 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
x-pack/plugins/ml/public/jobs/jobs_list/components/multi_job_actions/group_selector/group_selector.js
View file

@ -231,6 +231,7 @@ export class GroupSelector extends Component {
<NewGroupInput
addNewGroup={this.addNewGroup}
allJobIds={this.props.allJobIds}
/>
<EuiHorizontalRule margin="m" />
@ -254,5 +255,6 @@ export class GroupSelector extends Component {
}
GroupSelector.propTypes = {
jobs: PropTypes.array.isRequired,
allJobIds: PropTypes.array.isRequired,
refreshJobs: PropTypes.func.isRequired,
};

12
x-pack/plugins/ml/public/jobs/jobs_list/components/multi_job_actions/group_selector/new_group_input/new_group_input.js
View file

@ -34,7 +34,16 @@ export class NewGroupInput extends Component {
changeTempNewGroup = (e) => {
const tempNewGroupName = e.target.value;
const groupsValidationError = (tempNewGroupName === '') ? '' : validateGroupNames([tempNewGroupName]).message;
let groupsValidationError = '';
if (tempNewGroupName === '') {
groupsValidationError = '';
} else if (this.props.allJobIds.includes(tempNewGroupName)) {
groupsValidationError = 'A job with this ID already exists. Groups and jobs cannot use the same ID.';
} else {
groupsValidationError = validateGroupNames([tempNewGroupName]).message;
}
this.setState({
tempNewGroupName,
groupsValidationError,
@ -101,4 +110,5 @@ export class NewGroupInput extends Component {
NewGroupInput.propTypes = {
addNewGroup: PropTypes.func.isRequired,
allJobIds: PropTypes.array.isRequired,
};

2
x-pack/plugins/ml/public/jobs/jobs_list/components/multi_job_actions/multi_job_actions.js
View file

@ -34,6 +34,7 @@ export class MultiJobActions extends Component {
<GroupSelector
jobs={this.props.selectedJobs}
allJobIds={this.props.allJobIds}
refreshJobs={this.props.refreshJobs}
/>
@ -51,6 +52,7 @@ export class MultiJobActions extends Component {
}
MultiJobActions.propTypes = {
selectedJobs: PropTypes.array.isRequired,
allJobIds: PropTypes.array.isRequired,
showStartDatafeedModal: PropTypes.func.isRequired,
showDeleteJobModal: PropTypes.func.isRequired,
refreshJobs: PropTypes.func.isRequired,